All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: imx6q-gw54xx: Do not use 'simple-audio-card, dai-link'
@ 2019-10-08 16:30 Fabio Estevam
  2019-10-25  6:02 ` [PATCH] ARM: dts: imx6q-gw54xx: Do not use 'simple-audio-card,dai-link' Shawn Guo
  0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2019-10-08 16:30 UTC (permalink / raw)
  To: shawnguo; +Cc: Tim Harvey, Fabio Estevam, linux-arm-kernel

According to Documentation/devicetree/bindings/sound/simple-card.txt
the 'simple-audio-card,dai-link' may be omitted when the card has
only one DAI link, which is the case here.
    
Get rid of 'simple-audio-card,dai-link' in order to fix the following
build warning with W=1:

arch/arm/boot/dts/imx6q-gw54xx.dts:19.32-31.5: Warning (unit_address_vs_reg): /sound-digital/simple-audio-card,dai-link@0: node has a unit name, but no reg property

Cc: Tim Harvey <tharvey@gateworks.com>
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/boot/dts/imx6q-gw54xx.dts | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/arch/arm/boot/dts/imx6q-gw54xx.dts b/arch/arm/boot/dts/imx6q-gw54xx.dts
index ecc3989f607b..d5d46908cf6e 100644
--- a/arch/arm/boot/dts/imx6q-gw54xx.dts
+++ b/arch/arm/boot/dts/imx6q-gw54xx.dts
@@ -15,19 +15,16 @@
 	sound-digital {
 		compatible = "simple-audio-card";
 		simple-audio-card,name = "tda1997x-audio";
+		simple-audio-card,format = "i2s";
+		simple-audio-card,bitclock-master = <&sound_codec>;
+		simple-audio-card,frame-master = <&sound_codec>;
 
-		simple-audio-card,dai-link@0 {
-			format = "i2s";
-
-			cpu {
-				sound-dai = <&ssi2>;
-			};
+		sound_cpu: simple-audio-card,cpu {
+			sound-dai = <&ssi2>;
+		};
 
-			codec {
-				bitclock-master;
-				frame-master;
-				sound-dai = <&hdmi_receiver>;
-			};
+		sound_codec: simple-audio-card,codec {
+			sound-dai = <&hdmi_receiver>;
 		};
 	};
 };
-- 
2.17.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: imx6q-gw54xx: Do not use 'simple-audio-card,dai-link'
  2019-10-08 16:30 [PATCH] ARM: dts: imx6q-gw54xx: Do not use 'simple-audio-card, dai-link' Fabio Estevam
@ 2019-10-25  6:02 ` Shawn Guo
  0 siblings, 0 replies; 2+ messages in thread
From: Shawn Guo @ 2019-10-25  6:02 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: Tim Harvey, linux-arm-kernel

On Tue, Oct 08, 2019 at 01:30:24PM -0300, Fabio Estevam wrote:
> According to Documentation/devicetree/bindings/sound/simple-card.txt
> the 'simple-audio-card,dai-link' may be omitted when the card has
> only one DAI link, which is the case here.
>     
> Get rid of 'simple-audio-card,dai-link' in order to fix the following
> build warning with W=1:
> 
> arch/arm/boot/dts/imx6q-gw54xx.dts:19.32-31.5: Warning (unit_address_vs_reg): /sound-digital/simple-audio-card,dai-link@0: node has a unit name, but no reg property
> 
> Cc: Tim Harvey <tharvey@gateworks.com>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

Applied, thanks.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, other threads:[~2019-10-25  6:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-08 16:30 [PATCH] ARM: dts: imx6q-gw54xx: Do not use 'simple-audio-card, dai-link' Fabio Estevam
2019-10-25  6:02 ` [PATCH] ARM: dts: imx6q-gw54xx: Do not use 'simple-audio-card,dai-link' Shawn Guo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.