[alsa-devel] [PATCH] ASoC: CS4271: Move Chip Select control out of the CODEC code.
From: Alexander Sverdlin subaparts@yandex.ru
Move Chip Select control out of the CODEC code for CS4271.
Signed-off-by: Alexander Sverdlin subaparts@yandex.ru --- include/sound/cs4271.h | 1 - sound/soc/codecs/cs4271.c | 22 +++------------------- 2 files changed, 3 insertions(+), 20 deletions(-)
diff --git a/include/sound/cs4271.h b/include/sound/cs4271.h index 16f8d32..50a059e 100644 --- a/include/sound/cs4271.h +++ b/include/sound/cs4271.h @@ -19,7 +19,6 @@
struct cs4271_platform_data { int gpio_nreset; /* GPIO driving Reset pin, if any */ - int gpio_disable; /* GPIO that disable serial bus, if any */ };
#endif /* __CS4271_H */ diff --git a/sound/soc/codecs/cs4271.c b/sound/soc/codecs/cs4271.c index 9c5b7db..1791796 100644 --- a/sound/soc/codecs/cs4271.c +++ b/sound/soc/codecs/cs4271.c @@ -441,22 +441,11 @@ static int cs4271_probe(struct snd_soc_codec *codec) struct cs4271_platform_data *cs4271plat = codec->dev->platform_data; int ret; int gpio_nreset = -EINVAL; - int gpio_disable = -EINVAL;
codec->control_data = cs4271->control_data;
- if (cs4271plat) { - if (gpio_is_valid(cs4271plat->gpio_nreset)) - gpio_nreset = cs4271plat->gpio_nreset; - if (gpio_is_valid(cs4271plat->gpio_disable)) - gpio_disable = cs4271plat->gpio_disable; - } - - if (gpio_disable >= 0) - if (gpio_request(gpio_disable, "CS4271 Disable")) - gpio_disable = -EINVAL; - if (gpio_disable >= 0) - gpio_direction_output(gpio_disable, 0); + if (cs4271plat && gpio_is_valid(cs4271plat->gpio_nreset)) + gpio_nreset = cs4271plat->gpio_nreset;
if (gpio_nreset >= 0) if (gpio_request(gpio_nreset, "CS4271 Reset")) @@ -471,7 +460,6 @@ static int cs4271_probe(struct snd_soc_codec *codec) }
cs4271->gpio_nreset = gpio_nreset; - cs4271->gpio_disable = gpio_disable;
/* * In case of I2C, chip address specified in board data. @@ -509,10 +497,9 @@ static int cs4271_probe(struct snd_soc_codec *codec) static int cs4271_remove(struct snd_soc_codec *codec) { struct cs4271_private *cs4271 = snd_soc_codec_get_drvdata(codec); - int gpio_nreset, gpio_disable; + int gpio_nreset;
gpio_nreset = cs4271->gpio_nreset; - gpio_disable = cs4271->gpio_disable;
if (gpio_is_valid(gpio_nreset)) { /* Set codec to the reset state */ @@ -520,9 +507,6 @@ static int cs4271_remove(struct snd_soc_codec *codec) gpio_free(gpio_nreset); }
- if (gpio_is_valid(gpio_disable)) - gpio_free(gpio_disable); - return 0; };
On Wednesday, February 02, 2011 5:12 PM, Alexander Sverdlin wrote:
Move Chip Select control out of the CODEC code for CS4271.
Signed-off-by: Alexander Sverdlin subaparts@yandex.ru
FWIW...
Reviewed-by: H Hartley Sweeten hsweeten@visionengravers.com
On Thu, 2011-02-03 at 03:11 +0300, Alexander Sverdlin wrote:
From: Alexander Sverdlin subaparts@yandex.ru
Move Chip Select control out of the CODEC code for CS4271.
Signed-off-by: Alexander Sverdlin subaparts@yandex.ru
Acked-by: Liam Girdwood lrg@slimlogic.co.uk
On Thu, Feb 03, 2011 at 03:11:45AM +0300, Alexander Sverdlin wrote:
From: Alexander Sverdlin subaparts@yandex.ru
Move Chip Select control out of the CODEC code for CS4271.
Applied, thanks.
participants (4)
-
Alexander Sverdlin
-
H Hartley Sweeten
-
Liam Girdwood
-
Mark Brown