[PATCH] arm64: dts: imx8mn: specify #sound-dai-cells for SAI nodes

[Marek Vasut]

On 2/27/23 21:00, Marco Felsch wrote:
> On 23-02-27, Marek Vasut wrote:
>> On 2/27/23 20:01, Marco Felsch wrote:
>>> Hi Marek,
>>>
>>> On 23-02-27, Marek Vasut wrote:
>>>> Add #sound-dai-cells properties to SAI nodes.
>>>>
>>>> Signed-off-by: Marek Vasut <marex at denx.de>
>>>> ---
>>>> Cc: Abel Vesa <abelvesa at kernel.org>
>>>> Cc: Alexander Stein <alexander.stein at ew.tq-group.com>
>>>> Cc: Fabio Estevam <festevam at gmail.com>
>>>> Cc: Jacky Bai <ping.bai at nxp.com>
>>>> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt at linaro.org>
>>>> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>>>> Cc: Luca Ceresoli <luca.ceresoli at bootlin.com>
>>>> Cc: Lucas Stach <l.stach at pengutronix.de>
>>>> Cc: Marco Felsch <m.felsch at pengutronix.de>
>>>> Cc: Michael Turquette <mturquette at baylibre.com>
>>>> Cc: NXP Linux Team <linux-imx at nxp.com>
>>>> Cc: Peng Fan <peng.fan at nxp.com>
>>>> Cc: Pengutronix Kernel Team <kernel at pengutronix.de>
>>>> Cc: Richard Cochran <richardcochran at gmail.com>
>>>> Cc: Rob Herring <robh+dt at kernel.org>
>>>> Cc: Sascha Hauer <s.hauer at pengutronix.de>
>>>> Cc: Shawn Guo <shawnguo at kernel.org>
>>>> Cc: Stephen Boyd <sboyd at kernel.org>
>>>> Cc: devicetree at vger.kernel.org
>>>> Cc: linux-arm-kernel at lists.infradead.org
>>>> Cc: linux-clk at vger.kernel.org
>>>> ---
>>>>    arch/arm64/boot/dts/freescale/imx8mn.dtsi | 5 +++++
>>>>    1 file changed, 5 insertions(+)
>>>>
>>>> diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
>>>> index ed9ac6c5047c0..bbec860ef8548 100644
>>>> --- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
>>>> +++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
>>>> @@ -294,6 +294,7 @@ spba2: spba-bus at 30000000 {
>>>>    				ranges;
>>>>    				sai2: sai at 30020000 {
>>>> +					#sound-dai-cells = <0>;
>>>
>>> Please don't add it in front of the compatible and the reg property.
>>
>> The #address-cells and #size-cells are also always on top, why should the
>> #sound-dai-cells be any different ? Where should they be ?
> 
> As of now my understanding of specifying a devicetree node was:
> 
> node-name at reg-nr {
> 	compatible = "";
> 	reg = <>;
> 	// all pending properties below
> 	...
> };
> 
> @Rob, @Krzysztof:
> Is this a (unwritten) rule/policy?
> 
> Marekt please ignore my comment if this is not the case.

Getting input from Rob/Krysztof is a good idea, thanks.