diff mbox series

ARM: dts: imx6dl-b1x5pv2: Fix simple-audio routing property

Message ID 20230809114431.2075965-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series ARM: dts: imx6dl-b1x5pv2: Fix simple-audio routing property | expand

Commit Message

Fabio Estevam Aug. 9, 2023, 11:44 a.m. UTC
From: Fabio Estevam <festevam@denx.de>

Per simple-card.yaml, 'simple-audio-card,audio-routing' is not a valid
property.

Change it to 'simple-audio-card,routing'.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sebastian Reichel Aug. 9, 2023, 7:36 p.m. UTC | #1
Hi,

On Wed, Aug 09, 2023 at 08:44:31AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Per simple-card.yaml, 'simple-audio-card,audio-routing' is not a valid
> property.
> 
> Change it to 'simple-audio-card,routing'.
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---

Fixes: 1fac5db35ee9 ("ARM: dts: imx6: Add GE B1x5v2")
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.com>

I no longer have that hardware to test the change.
+Cc Ian and Kitty for testing.

-- Sebastian

>  arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi b/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
> index 37697fac9dea..6e487ebf27a2 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
> +++ b/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
> @@ -257,7 +257,7 @@ sound {
>  		simple-audio-card,bitclock-master = <&dailink_master>;
>  		simple-audio-card,frame-master = <&dailink_master>;
>  		simple-audio-card,widgets = "Speaker", "Ext Spk";
> -		simple-audio-card,audio-routing = "Ext Spk", "LINE";
> +		simple-audio-card,routing = "Ext Spk", "LINE";
>  
>  		simple-audio-card,cpu {
>  			sound-dai = <&ssi1>;
> -- 
> 2.34.1
>
Shawn Guo Aug. 11, 2023, 2:27 p.m. UTC | #2
On Wed, Aug 09, 2023 at 08:44:31AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Per simple-card.yaml, 'simple-audio-card,audio-routing' is not a valid
> property.
> 
> Change it to 'simple-audio-card,routing'.
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Applied, thanks!
Wang, Huan Kitty (GE HealthCare) Aug. 14, 2023, 7:42 a.m. UTC | #3
> -----Original Message-----
> From: Sebastian Reichel <sebastian.reichel@collabora.com>
> Sent: Thursday, August 10, 2023 3:37 AM
> To: Fabio Estevam <festevam@gmail.com>
> Cc: shawnguo@kernel.org; linux-arm-kernel@lists.infradead.org; Fabio Estevam
> <festevam@denx.de>; Ray, Ian (GE HealthCare) <ian.ray@ge.com>; Wang, Huan
> Kitty (GE HealthCare) <HuanWang@ge.com>
> Subject: EXT: Re: [PATCH] ARM: dts: imx6dl-b1x5pv2: Fix simple-audio routing
> property
> 
> WARNING: This email originated from outside of GE. Please validate the
> sender's email address before clicking on links or attachments as they may not
> be safe.
> 
> Hi,
> 
> On Wed, Aug 09, 2023 at 08:44:31AM -0300, Fabio Estevam wrote:
> > From: Fabio Estevam <festevam@denx.de>
> >
> > Per simple-card.yaml, 'simple-audio-card,audio-routing' is not a valid
> > property.
> >
> > Change it to 'simple-audio-card,routing'.
> >
> > Signed-off-by: Fabio Estevam <festevam@denx.de>
> > ---
> 
> Fixes: 1fac5db35ee9 ("ARM: dts: imx6: Add GE B1x5v2")
> Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.com>
> 
> I no longer have that hardware to test the change.
> +Cc Ian and Kitty for testing.

Thanks for the patch.

Tested-by: Wang, Huan Kitty (GE HealthCare) <HuanWang@ge.com>

> 
> -- Sebastian
> 
> >  arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
> > b/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
> > index 37697fac9dea..6e487ebf27a2 100644
> > --- a/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
> > +++ b/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
> > @@ -257,7 +257,7 @@ sound {
> >  		simple-audio-card,bitclock-master = <&dailink_master>;
> >  		simple-audio-card,frame-master = <&dailink_master>;
> >  		simple-audio-card,widgets = "Speaker", "Ext Spk";
> > -		simple-audio-card,audio-routing = "Ext Spk", "LINE";
> > +		simple-audio-card,routing = "Ext Spk", "LINE";
> >
> >  		simple-audio-card,cpu {
> >  			sound-dai = <&ssi1>;
> > --
> > 2.34.1
> >
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi b/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
index 37697fac9dea..6e487ebf27a2 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6dl-b1x5pv2.dtsi
@@ -257,7 +257,7 @@  sound {
 		simple-audio-card,bitclock-master = <&dailink_master>;
 		simple-audio-card,frame-master = <&dailink_master>;
 		simple-audio-card,widgets = "Speaker", "Ext Spk";
-		simple-audio-card,audio-routing = "Ext Spk", "LINE";
+		simple-audio-card,routing = "Ext Spk", "LINE";
 
 		simple-audio-card,cpu {
 			sound-dai = <&ssi1>;