[PATCH] ARM: dts: imx28-tx28: Pass #sound-dai-cells

[PATCH] ARM: dts: imx28-tx28: Pass #sound-dai-cells

[Fabio Estevam]

From: Fabio Estevam <festevam@denx.de>

Per sgtl5000.yaml, '#sound-dai-cells' is a required property.

Pass it to fix the following dt-schema warning:

imx28-tx28.dtb: sgtl5000@a: '#sound-dai-cells' is a required property
	from schema $id: http://devicetree.org/schemas/sound/sgtl5000.yaml#

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts b/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
index 153e4017951d..5485fe118dc4 100644
--- a/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
+++ b/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
@@ -260,6 +260,7 @@ &i2c0 {
 	sgtl5000: sgtl5000@a {
 		compatible = "fsl,sgtl5000";
 		reg = <0x0a>;
+		#sound-dai-cells = <0>;
 		VDDA-supply = <&reg_2p5v>;
 		VDDIO-supply = <&reg_3p3v>;
 		clocks = <&mclk>;
-- 
2.34.1


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

Re: [PATCH] ARM: dts: imx28-tx28: Pass #sound-dai-cells

[Stefan Wahren]

Hi Fabio,

Am 05.12.23 um 15:53 schrieb Fabio Estevam:
> From: Fabio Estevam <festevam@denx.de>
>
> Per sgtl5000.yaml, '#sound-dai-cells' is a required property.
>
> Pass it to fix the following dt-schema warning:
>
> imx28-tx28.dtb: sgtl5000@a: '#sound-dai-cells' is a required property
> 	from schema $id: http://devicetree.org/schemas/sound/sgtl5000.yaml#
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
>   arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts b/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
> index 153e4017951d..5485fe118dc4 100644
> --- a/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
> +++ b/arch/arm/boot/dts/nxp/mxs/imx28-tx28.dts
> @@ -260,6 +260,7 @@ &i2c0 {
>   	sgtl5000: sgtl5000@a {
this patch is fine

Reviewed-by: Stefan Wahren <wahrenst@gmx.net>

but this node name looks wrong.

It should be sgtl5000: codec@a

Regards
>   		compatible = "fsl,sgtl5000";
>   		reg = <0x0a>;
> +		#sound-dai-cells = <0>;
>   		VDDA-supply = <&reg_2p5v>;
>   		VDDIO-supply = <&reg_3p3v>;
>   		clocks = <&mclk>;


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

Re: [PATCH] ARM: dts: imx28-tx28: Pass #sound-dai-cells

[Fabio Estevam]

Hi Stefan,

On Tue, Dec 5, 2023 at 3:32 PM Stefan Wahren <wahrenst@gmx.net> wrote:

> this patch is fine
>
> Reviewed-by: Stefan Wahren <wahrenst@gmx.net>

Thanks for your review.

> but this node name looks wrong.
>
> It should be sgtl5000: codec@a

Yes, I noticed that too.

I will address this on a separate patch. There are other i.MX DTs that use the
sgtl5000@a format and need to be fixed as well.

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

Re: [PATCH] ARM: dts: imx28-tx28: Pass #sound-dai-cells

[Shawn Guo]

On Tue, Dec 05, 2023 at 11:53:35AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> Per sgtl5000.yaml, '#sound-dai-cells' is a required property.
> 
> Pass it to fix the following dt-schema warning:
> 
> imx28-tx28.dtb: sgtl5000@a: '#sound-dai-cells' is a required property
> 	from schema $id: http://devicetree.org/schemas/sound/sgtl5000.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