Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ASoC: tlv320aic31xx: document that the regulators are mandatory
@ 2014-04-18  0:06 Alexandre Belloni
  2014-04-18 14:07 ` Mark Brown
  0 siblings, 1 reply; 2+ messages in thread
From: Alexandre Belloni @ 2014-04-18  0:06 UTC (permalink / raw)
  To: Mark Brown
  Cc: Liam Girdwood, Jyri Sarha, alsa-devel, linux-kernel,
	Alexandre Belloni

The driver will not probe if regulators are not provided. Move the regulator
properties to the required section.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 Documentation/devicetree/bindings/sound/tlv320aic31xx.txt | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/sound/tlv320aic31xx.txt b/Documentation/devicetree/bindings/sound/tlv320aic31xx.txt
index 74c66dee3e14..eff12be5e789 100644
--- a/Documentation/devicetree/bindings/sound/tlv320aic31xx.txt
+++ b/Documentation/devicetree/bindings/sound/tlv320aic31xx.txt
@@ -13,6 +13,9 @@ Required properties:
     "ti,tlv320aic3111" - TLV320AIC3111 (stereo speaker amp, MiniDSP)
 
 - reg - <int> -  I2C slave address
+- HPVDD-supply, SPRVDD-supply, SPLVDD-supply, AVDD-supply, IOVDD-supply,
+  DVDD-supply : power supplies for the device as covered in
+  Documentation/devicetree/bindings/regulator/regulator.txt
 
 
 Optional properties:
@@ -24,9 +27,6 @@ Optional properties:
         3 or MICBIAS_AVDD - MICBIAS output is connected to AVDD
 	If this node is not mentioned or if the value is unknown, then
 	micbias	is set to 2.0V.
-- HPVDD-supply, SPRVDD-supply, SPLVDD-supply, AVDD-supply, IOVDD-supply,
-  DVDD-supply : power supplies for the device as covered in
-  Documentation/devicetree/bindings/regulator/regulator.txt
 
 CODEC output pins:
   * HPL
-- 
1.8.3.2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ASoC: tlv320aic31xx: document that the regulators are mandatory
  2014-04-18  0:06 [PATCH] ASoC: tlv320aic31xx: document that the regulators are mandatory Alexandre Belloni
@ 2014-04-18 14:07 ` Mark Brown
  0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2014-04-18 14:07 UTC (permalink / raw)
  To: Alexandre Belloni; +Cc: Liam Girdwood, Jyri Sarha, alsa-devel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 202 bytes --]

On Fri, Apr 18, 2014 at 02:06:30AM +0200, Alexandre Belloni wrote:
> The driver will not probe if regulators are not provided. Move the regulator
> properties to the required section.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-04-18 14:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-18  0:06 [PATCH] ASoC: tlv320aic31xx: document that the regulators are mandatory Alexandre Belloni
2014-04-18 14:07 ` Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox