linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: imx8m-beacon-kit: Remove extra sound-sai entry
@ 2023-08-09 11:36 Fabio Estevam
  2023-08-09 12:11 ` Adam Ford
  2023-08-11 14:25 ` Shawn Guo
  0 siblings, 2 replies; 3+ messages in thread
From: Fabio Estevam @ 2023-08-09 11:36 UTC (permalink / raw)
  To: shawnguo; +Cc: aford173, linux-arm-kernel, Fabio Estevam

From: Fabio Estevam <festevam@denx.de>

'sound-dai' should contain only 'sai5'. Remove the extra entry to
fix the following schema warning:

imx8mm-beacon-kit.dtb: sound-hdmi: simple-audio-card,cpu:sound-dai: [[110], [0]] is too long
	from schema $id: http://devicetree.org/schemas/sound/simple-card.yaml#

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts | 2 +-
 arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts b/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
index 66d68ce98ca4..905c98cb080d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
@@ -47,7 +47,7 @@ sound-hdmi {
 		simple-audio-card,format = "i2s";
 
 		simple-audio-card,cpu {
-			sound-dai = <&sai5 0>;
+			sound-dai = <&sai5>;
 			system-clock-direction-out;
 		};
 
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
index 442ff2960677..35b8d2060cd9 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
@@ -47,7 +47,7 @@ sound-hdmi {
 		simple-audio-card,format = "i2s";
 
 		simple-audio-card,cpu {
-			sound-dai = <&sai5 0>;
+			sound-dai = <&sai5>;
 			system-clock-direction-out;
 		};
 
-- 
2.34.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] 3+ messages in thread

* Re: [PATCH] arm64: dts: imx8m-beacon-kit: Remove extra sound-sai entry
  2023-08-09 11:36 [PATCH] arm64: dts: imx8m-beacon-kit: Remove extra sound-sai entry Fabio Estevam
@ 2023-08-09 12:11 ` Adam Ford
  2023-08-11 14:25 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Adam Ford @ 2023-08-09 12:11 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: shawnguo, linux-arm-kernel, Fabio Estevam

On Wed, Aug 9, 2023 at 6:37 AM Fabio Estevam <festevam@gmail.com> wrote:
>
> From: Fabio Estevam <festevam@denx.de>
>
> 'sound-dai' should contain only 'sai5'. Remove the extra entry to
> fix the following schema warning:
>
> imx8mm-beacon-kit.dtb: sound-hdmi: simple-audio-card,cpu:sound-dai: [[110], [0]] is too long
>         from schema $id: http://devicetree.org/schemas/sound/simple-card.yaml#
>

Thanks for the fix.   Should it have a fixes tag?

Acked-by: Adam Ford <aford173@gmail.com>

> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>  arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts | 2 +-
>  arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts b/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
> index 66d68ce98ca4..905c98cb080d 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dts
> @@ -47,7 +47,7 @@ sound-hdmi {
>                 simple-audio-card,format = "i2s";
>
>                 simple-audio-card,cpu {
> -                       sound-dai = <&sai5 0>;
> +                       sound-dai = <&sai5>;
>                         system-clock-direction-out;
>                 };
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
> index 442ff2960677..35b8d2060cd9 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
> @@ -47,7 +47,7 @@ sound-hdmi {
>                 simple-audio-card,format = "i2s";
>
>                 simple-audio-card,cpu {
> -                       sound-dai = <&sai5 0>;
> +                       sound-dai = <&sai5>;
>                         system-clock-direction-out;
>                 };
>
> --
> 2.34.1
>

_______________________________________________
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] 3+ messages in thread

* Re: [PATCH] arm64: dts: imx8m-beacon-kit: Remove extra sound-sai entry
  2023-08-09 11:36 [PATCH] arm64: dts: imx8m-beacon-kit: Remove extra sound-sai entry Fabio Estevam
  2023-08-09 12:11 ` Adam Ford
@ 2023-08-11 14:25 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2023-08-11 14:25 UTC (permalink / raw)
  To: Fabio Estevam; +Cc: aford173, linux-arm-kernel, Fabio Estevam

On Wed, Aug 09, 2023 at 08:36:58AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> 'sound-dai' should contain only 'sai5'. Remove the extra entry to
> fix the following schema warning:
> 
> imx8mm-beacon-kit.dtb: sound-hdmi: simple-audio-card,cpu:sound-dai: [[110], [0]] is too long
> 	from schema $id: http://devicetree.org/schemas/sound/simple-card.yaml#
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

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] 3+ messages in thread

end of thread, other threads:[~2023-08-11 14:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-09 11:36 [PATCH] arm64: dts: imx8m-beacon-kit: Remove extra sound-sai entry Fabio Estevam
2023-08-09 12:11 ` Adam Ford
2023-08-11 14:25 ` Shawn Guo

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).