Handle the reset GPIO and reset the device every time we
start it.
Signed-off-by: Shengjiu Wang <[email protected]>
---
Changes in v2
- use devm_gpiod_get_optional instead of of_get_named_gpio
sound/soc/codecs/cs42xx8.c | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/sound/soc/codecs/cs42xx8.c b/sound/soc/codecs/cs42xx8.c
index 28a4ac36c4f8..b377cddaf2e6 100644
--- a/sound/soc/codecs/cs42xx8.c
+++ b/sound/soc/codecs/cs42xx8.c
@@ -14,6 +14,7 @@
#include <linux/delay.h>
#include <linux/module.h>
#include <linux/of_device.h>
+#include <linux/of_gpio.h>
#include <linux/pm_runtime.h>
#include <linux/regulator/consumer.h>
#include <sound/pcm_params.h>
@@ -45,6 +46,7 @@ struct cs42xx8_priv {
bool slave_mode;
unsigned long sysclk;
u32 tx_channels;
+ struct gpio_desc *gpiod_reset;
};
/* -127.5dB to 0dB with step of 0.5dB */
@@ -467,6 +469,13 @@ int cs42xx8_probe(struct device *dev, struct regmap *regmap)
return -EINVAL;
}
+ cs42xx8->gpiod_reset = devm_gpiod_get_optional(dev, "reset",
+ GPIOD_OUT_HIGH);
+ if (IS_ERR(cs42xx8->gpiod_reset))
+ return PTR_ERR(cs42xx8->gpiod_reset);
+
+ gpiod_set_value_cansleep(cs42xx8->gpiod_reset, 0);
+
cs42xx8->clk = devm_clk_get(dev, "mclk");
if (IS_ERR(cs42xx8->clk)) {
dev_err(dev, "failed to get the clock: %ld\n",
@@ -547,6 +556,8 @@ static int cs42xx8_runtime_resume(struct device *dev)
return ret;
}
+ gpiod_set_value_cansleep(cs42xx8->gpiod_reset, 0);
+
ret = regulator_bulk_enable(ARRAY_SIZE(cs42xx8->supplies),
cs42xx8->supplies);
if (ret) {
@@ -586,6 +597,8 @@ static int cs42xx8_runtime_suspend(struct device *dev)
regulator_bulk_disable(ARRAY_SIZE(cs42xx8->supplies),
cs42xx8->supplies);
+ gpiod_set_value_cansleep(cs42xx8->gpiod_reset, 1);
+
clk_disable_unprepare(cs42xx8->clk);
return 0;
--
2.21.0
On Thu, May 16, 2019 at 06:04:58AM +0000, S.j. Wang wrote:
> + cs42xx8->gpiod_reset = devm_gpiod_get_optional(dev, "reset",
> + GPIOD_OUT_HIGH);
> + if (IS_ERR(cs42xx8->gpiod_reset))
> + return PTR_ERR(cs42xx8->gpiod_reset);
You also need a binding document update for this.
Hi
>
> On Thu, May 16, 2019 at 06:04:58AM +0000, S.j. Wang wrote:
>
> > + cs42xx8->gpiod_reset = devm_gpiod_get_optional(dev, "reset",
> > + GPIOD_OUT_HIGH);
> > + if (IS_ERR(cs42xx8->gpiod_reset))
> > + return PTR_ERR(cs42xx8->gpiod_reset);
>
> You also need a binding document update for this.
ok, will send v3
best regards
wang shengjiu
On Thu, May 16, 2019 at 11:09:27AM +0000, S.j. Wang wrote:
> > You also need a binding document update for this.
> ok, will send v3
Separate patch please, I already applied this and binding docs should be
separate patches anyway.
Hi
>
> On Thu, May 16, 2019 at 11:09:27AM +0000, S.j. Wang wrote:
>
> > > You also need a binding document update for this.
> > ok, will send v3
>
> Separate patch please, I already applied this and binding docs should be
> separate patches anyway.
Ok, I already send v3. please forget the v3, I will resend v3 for binding doc only.
Best regards
Wang shengjiu