[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CABS+qY00AkNJ2HbpK59Qjph_EPD7=caVyagTVE8okHrb74cnpA@mail.gmail.com>
Date: Mon, 24 Oct 2011 13:04:52 +0100
From: "Girdwood, Liam" <lrg@...com>
To: Axel Lin <axel.lin@...il.com>
Cc: linux-kernel@...r.kernel.org,
Mark Brown <broonie@...nsource.wolfsonmicro.com>,
Dimitris Papastamos <dp@...nsource.wolfsonmicro.com>,
alsa-devel@...a-project.org
Subject: Re: [PATCH] ASoC: wm8996: Avoid a redundant i2c_get_clientdata call
in wm8996_i2c_remove
On 24 October 2011 04:31, Axel Lin <axel.lin@...il.com> wrote:
> Signed-off-by: Axel Lin <axel.lin@...il.com>
> ---
> sound/soc/codecs/wm8996.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/sound/soc/codecs/wm8996.c b/sound/soc/codecs/wm8996.c
> index 645c980..32324c9 100644
> --- a/sound/soc/codecs/wm8996.c
> +++ b/sound/soc/codecs/wm8996.c
> @@ -3144,7 +3144,7 @@ static __devexit int wm8996_i2c_remove(struct i2c_client *client)
> snd_soc_unregister_codec(&client->dev);
> if (wm8996->pdata.ldo_ena > 0)
> gpio_free(wm8996->pdata.ldo_ena);
> - kfree(i2c_get_clientdata(client));
> + kfree(wm8996);
> return 0;
> }
>
Acked-by: Liam Girdwood <lrg@...com>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists