Skip to contentSkip to conversation

ASoC: adau1701: add regulator consumer support

PATCH Superseded [email protected] 2 participants 3 messages Apr 14, 2015 – Apr 20, 2015

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ASoC: adau1701: add regulator consumer support

From: Pascal Huerst <[email protected]>

The adau1701 has two power domains, DVDD and AVDD. Enable them both as long as the codec is in use.

---
 .../devicetree/bindings/sound/adi,adau1701.txt     |   4 +
 sound/soc/codecs/adau1701.c                        | 125 ++++++++++++++++++---
 2 files changed, 114 insertions(+), 15 deletions(-)
Documentation/devicetree/bindings/sound/adi,adau1701.txt +4 -0
index 547a49b..0d1128c 100644
@@ -20,6 +20,8 @@ Optional properties:
 			pin configurations as described in the datasheet,
 			table 53. Note that the value of this property has
 			to be prefixed with '/bits/ 8'.
+ - avdd-supply: 	Power supply for AVDD, providing 3.3V
+ - dvdd-supply: 	Power supply for DVDD, providing 3.3V
 
 Examples:
 
@@ -28,6 +30,8 @@ Examples:
 			compatible = "adi,adau1701";
 			reg = <0x34>;
 			reset-gpio = <&gpio 23 0>;
+			avdd-supply = <&vdd_3v3_reg>;
+			dvdd-supply = <&vdd_3v3_reg>;
 			adi,pll-mode-gpios = <&gpio 24 0 &gpio 25 0>;
 			adi,pin-config = /bits/ 8 <0x4 0x7 0x5 0x5 0x4 0x4
                                                    0x4 0x4 0x4 0x4 0x4 0x4>;
sound/soc/codecs/adau1701.c +110 -15

214 lines not on this page · raw message

Signature · 1 line
1.9.3
maintainer
#1 lore raw
Quoted 5 lines · On Tue, Apr 14, 2015 at 12:47:00PM +0200, [email protected] wrote:

From: Pascal Huerst <[email protected]>

The adau1701 has two power domains, DVDD and AVDD. Enable them both as long as the codec is in use.
---

This looks fine but I can't apply it because you've not signed it off - I need a signoff for anything that's going to mainline. Please see Documentation/SubmittingPatches.

#2 lore raw
Quoted 10 lines · Mark Brown
Quoted 5 lines · On Tue, Apr 14, 2015 at 12:47:00PM +0200, [email protected] wrote:

From: Pascal Huerst <[email protected]>

The adau1701 has two power domains, DVDD and AVDD. Enable them both as long as the codec is in use.
---

This looks fine but I can't apply it because you've not signed it off - I need a signoff for anything that's going to mainline. Please see Documentation/SubmittingPatches.

Oh, seems like I missed a '-s', sorry! I'll resend.