[alsa-devel] [PATCH] ASoc: tpa6130a2: Remove unnecessary variable
Signed-off-by: Ilkka Koskinen ilkka.koskinen@nokia.com --- sound/soc/codecs/tpa6130a2.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/tpa6130a2.c b/sound/soc/codecs/tpa6130a2.c index 0eb33d4..8e98ccf 100644 --- a/sound/soc/codecs/tpa6130a2.c +++ b/sound/soc/codecs/tpa6130a2.c @@ -267,12 +267,8 @@ static const struct snd_kcontrol_new tpa6130a2_controls[] = { */ static void tpa6130a2_channel_enable(u8 channel, int enable) { - struct tpa6130a2_data *data; u8 val;
- BUG_ON(tpa6130a2_client == NULL); - data = i2c_get_clientdata(tpa6130a2_client); - if (enable) { /* Enable channel */ /* Enable amplifier */
On Monday 04 January 2010 14:30:03 Koskinen Ilkka (Nokia-D/Tampere) wrote:
Signed-off-by: Ilkka Koskinen ilkka.koskinen@nokia.com
sound/soc/codecs/tpa6130a2.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/tpa6130a2.c b/sound/soc/codecs/tpa6130a2.c index 0eb33d4..8e98ccf 100644 --- a/sound/soc/codecs/tpa6130a2.c +++ b/sound/soc/codecs/tpa6130a2.c @@ -267,12 +267,8 @@ static const struct snd_kcontrol_new tpa6130a2_controls[] = { */ static void tpa6130a2_channel_enable(u8 channel, int enable) {
struct tpa6130a2_data *data; u8 val;
BUG_ON(tpa6130a2_client == NULL);
data = i2c_get_clientdata(tpa6130a2_client);
if (enable) { /* Enable channel */ /* Enable amplifier */
Yes, I think this is a leftover from some earlier code that I had. Good catch!
Acked-by: Peter Ujfalusi peter.ujfalusi@nokia.com
On Mon, Jan 04, 2010 at 02:30:03PM +0200, Ilkka Koskinen wrote:
Signed-off-by: Ilkka Koskinen ilkka.koskinen@nokia.com
Applied, thanks.
participants (3)
-
Ilkka Koskinen
-
Mark Brown
-
Peter Ujfalusi