[PATCH 1/2] ASoC: codecs: aw88261: use dvdd-supply regulator

Bharadwaj Raju via B4 Relay posted 2 patches 1 month ago
There is a newer version of this series
[PATCH 1/2] ASoC: codecs: aw88261: use dvdd-supply regulator
Posted by Bharadwaj Raju via B4 Relay 1 month ago
From: Bharadwaj Raju <bharadwaj.raju@machinesoul.in>

Enable the dvdd regulator on startup.

Signed-off-by: Bharadwaj Raju <bharadwaj.raju@machinesoul.in>
---
 sound/soc/codecs/aw88261.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/sound/soc/codecs/aw88261.c b/sound/soc/codecs/aw88261.c
index 6bd6e7ffd1aa..9a2d167cf809 100644
--- a/sound/soc/codecs/aw88261.c
+++ b/sound/soc/codecs/aw88261.c
@@ -11,6 +11,7 @@
 #include <linux/i2c.h>
 #include <linux/firmware.h>
 #include <linux/regmap.h>
+#include <linux/regulator/consumer.h>
 #include <sound/soc.h>
 #include "aw88261.h"
 #include "aw88395/aw88395_data_type.h"
@@ -1295,6 +1296,10 @@ static int aw88261_init(struct aw88261 **aw88261, struct i2c_client *i2c, struct
 	unsigned int chip_id;
 	int ret;
 
+	ret = devm_regulator_get_enable(&i2c->dev, "dvdd");
+	if (ret)
+		return dev_err_probe(&i2c->dev, ret, "Failed to enable dvdd supply\n");
+
 	/* read chip id */
 	ret = regmap_read(regmap, AW88261_ID_REG, &chip_id);
 	if (ret) {

-- 
2.43.0
Re: [PATCH 1/2] ASoC: codecs: aw88261: use dvdd-supply regulator
Posted by Krzysztof Kozlowski 1 month ago
On 11/01/2026 11:10, Bharadwaj Raju via B4 Relay wrote:
> From: Bharadwaj Raju <bharadwaj.raju@machinesoul.in>
> 
> Enable the dvdd regulator on startup.
> 
> Signed-off-by: Bharadwaj Raju <bharadwaj.raju@machinesoul.in>
> ---
>  sound/soc/codecs/aw88261.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/sound/soc/codecs/aw88261.c b/sound/soc/codecs/aw88261.c
> index 6bd6e7ffd1aa..9a2d167cf809 100644
> --- a/sound/soc/codecs/aw88261.c
> +++ b/sound/soc/codecs/aw88261.c
> @@ -11,6 +11,7 @@
>  #include <linux/i2c.h>
>  #include <linux/firmware.h>
>  #include <linux/regmap.h>
> +#include <linux/regulator/consumer.h>
>  #include <sound/soc.h>
>  #include "aw88261.h"
>  #include "aw88395/aw88395_data_type.h"
> @@ -1295,6 +1296,10 @@ static int aw88261_init(struct aw88261 **aw88261, struct i2c_client *i2c, struct
>  	unsigned int chip_id;
>  	int ret;
>  
> +	ret = devm_regulator_get_enable(&i2c->dev, "dvdd");

Do not use ABI before you document it. See submitting patches in DT dir.


Best regards,
Krzysztof