devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: imx8mn-bsh-smm-s2pro: Remove invalid audio codec clock
@ 2024-08-21 14:34 Fabio Estevam
  2024-08-27 12:03 ` Michael Nazzareno Trimarchi
  0 siblings, 1 reply; 5+ messages in thread
From: Fabio Estevam @ 2024-08-21 14:34 UTC (permalink / raw)
  To: shawnguo
  Cc: robh, krzk+dt, conor+dt, devicetree, linux-arm-kernel,
	ariel.dalessandro, michael, Fabio Estevam

From: Fabio Estevam <festevam@denx.de>

According to ti,tlv320dac3100.yaml, 'clocks' is not a valid
property.

Remove it to fix the following dt-schema warning:

Unevaluated properties are not allowed ('clocks' was unexpected)

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
index c6ad65becc97..475cbf9e0d1e 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
@@ -64,7 +64,6 @@ tlv320dac3101: audio-codec@18 {
 		DVDD-supply = <&buck5_reg>;
 		reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
 		ai31xx-micbias-vg = <MICBIAS_AVDDV>;
-		clocks = <&clk IMX8MN_CLK_SAI3_ROOT>;
 	};
 };
 
-- 
2.34.1


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

* Re: [PATCH] arm64: dts: imx8mn-bsh-smm-s2pro: Remove invalid audio codec clock
  2024-08-21 14:34 [PATCH] arm64: dts: imx8mn-bsh-smm-s2pro: Remove invalid audio codec clock Fabio Estevam
@ 2024-08-27 12:03 ` Michael Nazzareno Trimarchi
  2024-10-17 11:43   ` Fabio Estevam
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Nazzareno Trimarchi @ 2024-08-27 12:03 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: shawnguo, robh, krzk+dt, conor+dt, devicetree, linux-arm-kernel,
	ariel.dalessandro, Fabio Estevam, Dario Binacchi

Hi Fabio

On Wed, Aug 21, 2024 at 4:34 PM Fabio Estevam <festevam@gmail.com> wrote:
>
> From: Fabio Estevam <festevam@denx.de>
>
> According to ti,tlv320dac3100.yaml, 'clocks' is not a valid
> property.
>
> Remove it to fix the following dt-schema warning:
>
> Unevaluated properties are not allowed ('clocks' was unexpected)
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> index c6ad65becc97..475cbf9e0d1e 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> @@ -64,7 +64,6 @@ tlv320dac3101: audio-codec@18 {
>                 DVDD-supply = <&buck5_reg>;
>                 reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
>                 ai31xx-micbias-vg = <MICBIAS_AVDDV>;
> -               clocks = <&clk IMX8MN_CLK_SAI3_ROOT>;
>         };
>  };

Let's just give a test to the board

Michael

>
> --
> 2.34.1
>


-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael@amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info@amarulasolutions.com
www.amarulasolutions.com

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

* Re: [PATCH] arm64: dts: imx8mn-bsh-smm-s2pro: Remove invalid audio codec clock
  2024-08-27 12:03 ` Michael Nazzareno Trimarchi
@ 2024-10-17 11:43   ` Fabio Estevam
  2024-10-17 11:50     ` Michael Nazzareno Trimarchi
  0 siblings, 1 reply; 5+ messages in thread
From: Fabio Estevam @ 2024-10-17 11:43 UTC (permalink / raw)
  To: Michael Nazzareno Trimarchi
  Cc: shawnguo, robh, krzk+dt, conor+dt, devicetree, linux-arm-kernel,
	ariel.dalessandro, Fabio Estevam, Dario Binacchi

Hi Michael,

On Tue, Aug 27, 2024 at 9:03 AM Michael Nazzareno Trimarchi
<michael@amarulasolutions.com> wrote:
>
> Hi Fabio
>
> On Wed, Aug 21, 2024 at 4:34 PM Fabio Estevam <festevam@gmail.com> wrote:
> >
> > From: Fabio Estevam <festevam@denx.de>
> >
> > According to ti,tlv320dac3100.yaml, 'clocks' is not a valid
> > property.
> >
> > Remove it to fix the following dt-schema warning:
> >
> > Unevaluated properties are not allowed ('clocks' was unexpected)
> >
> > Signed-off-by: Fabio Estevam <festevam@denx.de>
> > ---
> >  arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > index c6ad65becc97..475cbf9e0d1e 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > @@ -64,7 +64,6 @@ tlv320dac3101: audio-codec@18 {
> >                 DVDD-supply = <&buck5_reg>;
> >                 reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
> >                 ai31xx-micbias-vg = <MICBIAS_AVDDV>;
> > -               clocks = <&clk IMX8MN_CLK_SAI3_ROOT>;
> >         };
> >  };
>
> Let's just give a test to the board

Have you had a chance to test it?

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

* Re: [PATCH] arm64: dts: imx8mn-bsh-smm-s2pro: Remove invalid audio codec clock
  2024-10-17 11:43   ` Fabio Estevam
@ 2024-10-17 11:50     ` Michael Nazzareno Trimarchi
  2024-12-13 21:23       ` Frank Li
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Nazzareno Trimarchi @ 2024-10-17 11:50 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: shawnguo, robh, krzk+dt, conor+dt, devicetree, linux-arm-kernel,
	ariel.dalessandro, Fabio Estevam, Dario Binacchi

Hi Fabio

On Thu, Oct 17, 2024 at 1:43 PM Fabio Estevam <festevam@gmail.com> wrote:
>
> Hi Michael,
>
> On Tue, Aug 27, 2024 at 9:03 AM Michael Nazzareno Trimarchi
> <michael@amarulasolutions.com> wrote:
> >
> > Hi Fabio
> >
> > On Wed, Aug 21, 2024 at 4:34 PM Fabio Estevam <festevam@gmail.com> wrote:
> > >
> > > From: Fabio Estevam <festevam@denx.de>
> > >
> > > According to ti,tlv320dac3100.yaml, 'clocks' is not a valid
> > > property.
> > >
> > > Remove it to fix the following dt-schema warning:
> > >
> > > Unevaluated properties are not allowed ('clocks' was unexpected)
> > >
> > > Signed-off-by: Fabio Estevam <festevam@denx.de>
> > > ---
> > >  arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > > index c6ad65becc97..475cbf9e0d1e 100644
> > > --- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > > +++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > > @@ -64,7 +64,6 @@ tlv320dac3101: audio-codec@18 {
> > >                 DVDD-supply = <&buck5_reg>;
> > >                 reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
> > >                 ai31xx-micbias-vg = <MICBIAS_AVDDV>;
> > > -               clocks = <&clk IMX8MN_CLK_SAI3_ROOT>;
> > >         };
> > >  };
> >
> > Let's just give a test to the board
>
> Have you had a chance to test it?

I will test on monday ;). Is that ok?

Michael

-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael@amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info@amarulasolutions.com
www.amarulasolutions.com

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

* Re: [PATCH] arm64: dts: imx8mn-bsh-smm-s2pro: Remove invalid audio codec clock
  2024-10-17 11:50     ` Michael Nazzareno Trimarchi
@ 2024-12-13 21:23       ` Frank Li
  0 siblings, 0 replies; 5+ messages in thread
From: Frank Li @ 2024-12-13 21:23 UTC (permalink / raw)
  To: Michael Nazzareno Trimarchi, imx
  Cc: Fabio Estevam, shawnguo, robh, krzk+dt, conor+dt, devicetree,
	linux-arm-kernel, ariel.dalessandro, Fabio Estevam,
	Dario Binacchi

On Thu, Oct 17, 2024 at 01:50:13PM +0200, Michael Nazzareno Trimarchi wrote:
> Hi Fabio
>
> On Thu, Oct 17, 2024 at 1:43 PM Fabio Estevam <festevam@gmail.com> wrote:
> >
> > Hi Michael,
> >
> > On Tue, Aug 27, 2024 at 9:03 AM Michael Nazzareno Trimarchi
> > <michael@amarulasolutions.com> wrote:
> > >
> > > Hi Fabio
> > >
> > > On Wed, Aug 21, 2024 at 4:34 PM Fabio Estevam <festevam@gmail.com> wrote:
> > > >
> > > > From: Fabio Estevam <festevam@denx.de>
> > > >
> > > > According to ti,tlv320dac3100.yaml, 'clocks' is not a valid
> > > > property.
> > > >
> > > > Remove it to fix the following dt-schema warning:
> > > >
> > > > Unevaluated properties are not allowed ('clocks' was unexpected)
> > > >
> > > > Signed-off-by: Fabio Estevam <festevam@denx.de>
> > > > ---
> > > >  arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts | 1 -
> > > >  1 file changed, 1 deletion(-)
> > > >
> > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > > > index c6ad65becc97..475cbf9e0d1e 100644
> > > > --- a/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > > > +++ b/arch/arm64/boot/dts/freescale/imx8mn-bsh-smm-s2pro.dts
> > > > @@ -64,7 +64,6 @@ tlv320dac3101: audio-codec@18 {
> > > >                 DVDD-supply = <&buck5_reg>;
> > > >                 reset-gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
> > > >                 ai31xx-micbias-vg = <MICBIAS_AVDDV>;
> > > > -               clocks = <&clk IMX8MN_CLK_SAI3_ROOT>;
> > > >         };
> > > >  };
> > >
> > > Let's just give a test to the board
> >
> > Have you had a chance to test it?
>
> I will test on monday ;). Is that ok?

Any update ?

Frank

>
> Michael
>
> --
> Michael Nazzareno Trimarchi
> Co-Founder & Chief Executive Officer
> M. +39 347 913 2170
> michael@amarulasolutions.com
> __________________________________
>
> Amarula Solutions BV
> Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> T. +31 (0)85 111 9172
> info@amarulasolutions.com
> www.amarulasolutions.com

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

end of thread, other threads:[~2024-12-13 21:23 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-21 14:34 [PATCH] arm64: dts: imx8mn-bsh-smm-s2pro: Remove invalid audio codec clock Fabio Estevam
2024-08-27 12:03 ` Michael Nazzareno Trimarchi
2024-10-17 11:43   ` Fabio Estevam
2024-10-17 11:50     ` Michael Nazzareno Trimarchi
2024-12-13 21:23       ` Frank Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).