From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: Jihed Chaibi <jihed.chaibi.dev@gmail.com>, andrew@lunn.ch,
 sebastian.hesselbarth@gmail.com, robh@kernel.org
Cc: krzk+dt@kernel.org, conor+dt@kernel.org,
 linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
 linux-kernel@vger.kernel.org, shuah@kernel.org, jihed.chaibi.dev@gmail.com
Subject: Re: [PATCH] ARM: dts: kirkwood: Fix sound DAI cells for OpenRD clients
In-Reply-To: <20250830203750.239890-1-jihed.chaibi.dev@gmail.com>
References: <20250830203750.239890-1-jihed.chaibi.dev@gmail.com>
Date: Fri, 12 Sep 2025 07:24:27 +0200
Message-ID: <87348sp7jo.fsf@BLaptop.bootlin.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

Jihed Chaibi <jihed.chaibi.dev@gmail.com> writes:

> A previous commit changed the '#sound-dai-cells' property for the
> kirkwood audio controller from 1 to 0 in the kirkwood.dtsi file,
> but did not update the corresponding 'sound-dai' property in the
> kirkwood-openrd-client.dts file.
>
> This created a mismatch, causing a dtbs_check validation error where
> the dts provides one cell (<&audio0 0>) while the .dtsi expects zero.
>
> Remove the extraneous cell from the 'sound-dai' property to fix the
> schema validation warning and align with the updated binding.
>
> Fixes: e662e70fa419 ("arm: dts: kirkwood: fix error in #sound-dai-cells s=
ize")
> Signed-off-by: Jihed Chaibi <jihed.chaibi.dev@gmail.com>

Applied on mvebu/fixes

Thanks,

Gregory
> ---
>  arch/arm/boot/dts/marvell/kirkwood-openrd-client.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/marvell/kirkwood-openrd-client.dts b/arch/=
arm/boot/dts/marvell/kirkwood-openrd-client.dts
> index d4e0b8150a8..cf26e2ceaaa 100644
> --- a/arch/arm/boot/dts/marvell/kirkwood-openrd-client.dts
> +++ b/arch/arm/boot/dts/marvell/kirkwood-openrd-client.dts
> @@ -38,7 +38,7 @@ sound {
>  		simple-audio-card,mclk-fs =3D <256>;
>=20=20
>  		simple-audio-card,cpu {
> -			sound-dai =3D <&audio0 0>;
> +			sound-dai =3D <&audio0>;
>  		};
>=20=20
>  		simple-audio-card,codec {
> --=20
> 2.39.5
>

--=20
Gr=C3=A9gory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com