Re: [PATCH 1/1] Media: radio: si470x: remove double mutex lock

2011-01-23 Thread Tobias Lorenz
Hi, good catch! Thanks. Acked-by: Tobias Lorenz Bye, Toby On Sun, 23 Jan 2011 12:16:54 +0300, Alexander Strakh wrote: > From: Alexander Strakh > > > 1. First mutex_lock on &radio->lock in line 441 > 2. Second in line 462 > > I think that mutex in line 462 is not needed. In kernel v2.6.36.

[PATCH 1/1] Media: radio: si470x: remove double mutex lock

2011-01-23 Thread Alexander Strakh
From: Alexander Strakh 1. First mutex_lock on &radio->lock in line 441 2. Second in line 462 I think that mutex in line 462 is not needed. In kernel v2.6.36.3 function si470 x_rds_on contains mutex_lock/mutex_unlock. Have no mutex_lock in line 441. In kernel v2.6.37 mutex_lock/mutex_unlock has