* [PATCH] ASoC: tlv320aic3x: Add platform data and reset gpio handling
@ 2010-05-05 8:13 Jarkko Nikula
2010-05-05 9:35 ` Peter Ujfalusi
0 siblings, 1 reply; 3+ messages in thread
From: Jarkko Nikula @ 2010-05-05 8:13 UTC (permalink / raw)
To: alsa-devel; +Cc: Mark Brown, Peter Ujfalusi, Liam Girdwood
Signed-off-by: Jarkko Nikula <jhnikula@gmail.com>
---
include/sound/tlv320aic3x.h | 17 +++++++++++++++++
sound/soc/codecs/tlv320aic3x.c | 26 ++++++++++++++++++++++++++
2 files changed, 43 insertions(+), 0 deletions(-)
create mode 100644 include/sound/tlv320aic3x.h
diff --git a/include/sound/tlv320aic3x.h b/include/sound/tlv320aic3x.h
new file mode 100644
index 0000000..e493a14
--- /dev/null
+++ b/include/sound/tlv320aic3x.h
@@ -0,0 +1,17 @@
+/*
+ * Platform data for Texas Instruments TLV320AIC3x codec
+ *
+ * Author: Jarkko Nikula <jhnikula@gmail.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+#ifndef __TLV320AIC3x_H__
+#define __TLV320AIC3x_H__
+
+struct aic3x_pdata {
+ int gpio_reset;
+};
+
+#endif
\ No newline at end of file
diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c
index b2c2794..cdcfb6b 100644
--- a/sound/soc/codecs/tlv320aic3x.c
+++ b/sound/soc/codecs/tlv320aic3x.c
@@ -38,6 +38,7 @@
#include <linux/delay.h>
#include <linux/pm.h>
#include <linux/i2c.h>
+#include <linux/gpio.h>
#include <linux/regulator/consumer.h>
#include <linux/platform_device.h>
#include <linux/slab.h>
@@ -48,6 +49,7 @@
#include <sound/soc-dapm.h>
#include <sound/initval.h>
#include <sound/tlv.h>
+#include <sound/tlv320aic3x.h>
#include "tlv320aic3x.h"
@@ -65,6 +67,7 @@ struct aic3x_priv {
struct regulator_bulk_data supplies[AIC3X_NUM_SUPPLIES];
unsigned int sysclk;
int master;
+ int gpio_reset;
};
/*
@@ -1279,6 +1282,10 @@ static int aic3x_unregister(struct aic3x_priv *aic3x)
snd_soc_unregister_dai(&aic3x_dai);
snd_soc_unregister_codec(&aic3x->codec);
+ if (aic3x->gpio_reset >= 0) {
+ gpio_set_value(aic3x->gpio_reset, 0);
+ gpio_free(aic3x->gpio_reset);
+ }
regulator_bulk_disable(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
regulator_bulk_free(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
@@ -1303,6 +1310,7 @@ static int aic3x_i2c_probe(struct i2c_client *i2c,
{
struct snd_soc_codec *codec;
struct aic3x_priv *aic3x;
+ struct aic3x_pdata *pdata = i2c->dev.platform_data;
int ret, i;
aic3x = kzalloc(sizeof(struct aic3x_priv), GFP_KERNEL);
@@ -1319,6 +1327,15 @@ static int aic3x_i2c_probe(struct i2c_client *i2c,
i2c_set_clientdata(i2c, aic3x);
+ aic3x->gpio_reset = -1;
+ if (pdata && pdata->gpio_reset >= 0) {
+ ret = gpio_request(pdata->gpio_reset, "tlv320aic3x reset");
+ if (ret != 0)
+ goto err_gpio;
+ aic3x->gpio_reset = pdata->gpio_reset;
+ gpio_direction_output(aic3x->gpio_reset, 0);
+ }
+
for (i = 0; i < ARRAY_SIZE(aic3x->supplies); i++)
aic3x->supplies[i].supply = aic3x_supply_names[i];
@@ -1336,11 +1353,20 @@ static int aic3x_i2c_probe(struct i2c_client *i2c,
goto err_enable;
}
+ if (aic3x->gpio_reset >= 0) {
+ udelay(1);
+ gpio_set_value(aic3x->gpio_reset, 1);
+ }
+
return aic3x_register(codec);
err_enable:
regulator_bulk_free(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
err_get:
+ regulator_bulk_disable(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
+ if (aic3x->gpio_reset >= 0)
+ gpio_free(aic3x->gpio_reset);
+err_gpio:
kfree(aic3x);
return ret;
}
--
1.7.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ASoC: tlv320aic3x: Add platform data and reset gpio handling
2010-05-05 8:13 [PATCH] ASoC: tlv320aic3x: Add platform data and reset gpio handling Jarkko Nikula
@ 2010-05-05 9:35 ` Peter Ujfalusi
2010-05-05 9:50 ` Jarkko Nikula
0 siblings, 1 reply; 3+ messages in thread
From: Peter Ujfalusi @ 2010-05-05 9:35 UTC (permalink / raw)
To: ext Jarkko Nikula; +Cc: alsa-devel@alsa-project.org, Mark Brown, Liam Girdwood
Hi,
On Wednesday 05 May 2010 11:13:20 ext Jarkko Nikula wrote:
> Signed-off-by: Jarkko Nikula <jhnikula@gmail.com>
> ---
> include/sound/tlv320aic3x.h | 17 +++++++++++++++++
> sound/soc/codecs/tlv320aic3x.c | 26 ++++++++++++++++++++++++++
> 2 files changed, 43 insertions(+), 0 deletions(-)
> create mode 100644 include/sound/tlv320aic3x.h
>
> diff --git a/include/sound/tlv320aic3x.h b/include/sound/tlv320aic3x.h
> new file mode 100644
> index 0000000..e493a14
> --- /dev/null
> +++ b/include/sound/tlv320aic3x.h
> @@ -0,0 +1,17 @@
> +/*
> + * Platform data for Texas Instruments TLV320AIC3x codec
> + *
> + * Author: Jarkko Nikula <jhnikula@gmail.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +#ifndef __TLV320AIC3x_H__
> +#define __TLV320AIC3x_H__
> +
> +struct aic3x_pdata {
> + int gpio_reset;
> +};
> +
> +#endif
> \ No newline at end of file
> diff --git a/sound/soc/codecs/tlv320aic3x.c
> b/sound/soc/codecs/tlv320aic3x.c index b2c2794..cdcfb6b 100644
> --- a/sound/soc/codecs/tlv320aic3x.c
> +++ b/sound/soc/codecs/tlv320aic3x.c
> @@ -38,6 +38,7 @@
> #include <linux/delay.h>
> #include <linux/pm.h>
> #include <linux/i2c.h>
> +#include <linux/gpio.h>
> #include <linux/regulator/consumer.h>
> #include <linux/platform_device.h>
> #include <linux/slab.h>
> @@ -48,6 +49,7 @@
> #include <sound/soc-dapm.h>
> #include <sound/initval.h>
> #include <sound/tlv.h>
> +#include <sound/tlv320aic3x.h>
>
> #include "tlv320aic3x.h"
>
> @@ -65,6 +67,7 @@ struct aic3x_priv {
> struct regulator_bulk_data supplies[AIC3X_NUM_SUPPLIES];
> unsigned int sysclk;
> int master;
> + int gpio_reset;
> };
>
> /*
> @@ -1279,6 +1282,10 @@ static int aic3x_unregister(struct aic3x_priv
> *aic3x) snd_soc_unregister_dai(&aic3x_dai);
> snd_soc_unregister_codec(&aic3x->codec);
>
> + if (aic3x->gpio_reset >= 0) {
> + gpio_set_value(aic3x->gpio_reset, 0);
> + gpio_free(aic3x->gpio_reset);
> + }
> regulator_bulk_disable(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
> regulator_bulk_free(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
>
> @@ -1303,6 +1310,7 @@ static int aic3x_i2c_probe(struct i2c_client *i2c,
> {
> struct snd_soc_codec *codec;
> struct aic3x_priv *aic3x;
> + struct aic3x_pdata *pdata = i2c->dev.platform_data;
> int ret, i;
>
> aic3x = kzalloc(sizeof(struct aic3x_priv), GFP_KERNEL);
> @@ -1319,6 +1327,15 @@ static int aic3x_i2c_probe(struct i2c_client *i2c,
>
> i2c_set_clientdata(i2c, aic3x);
>
> + aic3x->gpio_reset = -1;
> + if (pdata && pdata->gpio_reset >= 0) {
> + ret = gpio_request(pdata->gpio_reset, "tlv320aic3x reset");
> + if (ret != 0)
> + goto err_gpio;
> + aic3x->gpio_reset = pdata->gpio_reset;
> + gpio_direction_output(aic3x->gpio_reset, 0);
> + }
> +
> for (i = 0; i < ARRAY_SIZE(aic3x->supplies); i++)
> aic3x->supplies[i].supply = aic3x_supply_names[i];
>
> @@ -1336,11 +1353,20 @@ static int aic3x_i2c_probe(struct i2c_client *i2c,
> goto err_enable;
> }
>
> + if (aic3x->gpio_reset >= 0) {
> + udelay(1);
> + gpio_set_value(aic3x->gpio_reset, 1);
> + }
> +
> return aic3x_register(codec);
>
> err_enable:
> regulator_bulk_free(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
> err_get:
> + regulator_bulk_disable(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
This looks suspicious (disable after free?)
On the other hand, is there a need to disable at all in error case?
> + if (aic3x->gpio_reset >= 0)
> + gpio_free(aic3x->gpio_reset);
> +err_gpio:
> kfree(aic3x);
> return ret;
> }
Would be better to order:
err_enable:
regulator_bulk_free(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
err_get:
if (aic3x->gpio_reset >= 0)
gpio_free(aic3x->gpio_reset);
err_gpio:
kfree(aic3x);
return ret;
--
Péter
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ASoC: tlv320aic3x: Add platform data and reset gpio handling
2010-05-05 9:35 ` Peter Ujfalusi
@ 2010-05-05 9:50 ` Jarkko Nikula
0 siblings, 0 replies; 3+ messages in thread
From: Jarkko Nikula @ 2010-05-05 9:50 UTC (permalink / raw)
To: Peter Ujfalusi; +Cc: alsa-devel@alsa-project.org, Liam, Mark Brown, Girdwood
On Wed, 5 May 2010 12:35:55 +0300
Peter Ujfalusi <peter.ujfalusi@nokia.com> wrote:
> > err_enable:
> > regulator_bulk_free(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
> > err_get:
> > + regulator_bulk_disable(ARRAY_SIZE(aic3x->supplies), aic3x->supplies);
>
> This looks suspicious (disable after free?)
> On the other hand, is there a need to disable at all in error case?
>
Oops. How that went there. Thanks for noticing!
--
Jarkko
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-05-05 9:49 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-05 8:13 [PATCH] ASoC: tlv320aic3x: Add platform data and reset gpio handling Jarkko Nikula
2010-05-05 9:35 ` Peter Ujfalusi
2010-05-05 9:50 ` Jarkko Nikula
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).