diff mbox series

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

Message ID 20230227174535.87657-1-marex@denx.de (mailing list archive)
State Not Applicable, archived
Headers show
Series arm64: dts: imx8mn: specify #sound-dai-cells for SAI nodes | expand

Commit Message

Marek Vasut Feb. 27, 2023, 5:45 p.m. UTC
Add #sound-dai-cells properties to SAI nodes.

Signed-off-by: Marek Vasut <marex@denx.de>
---
Cc: Abel Vesa <abelvesa@kernel.org>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Jacky Bai <ping.bai@nxp.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Luca Ceresoli <luca.ceresoli@bootlin.com>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Marco Felsch <m.felsch@pengutronix.de>
Cc: Michael Turquette <mturquette@baylibre.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Richard Cochran <richardcochran@gmail.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Stephen Boyd <sboyd@kernel.org>
Cc: devicetree@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-clk@vger.kernel.org
---
 arch/arm64/boot/dts/freescale/imx8mn.dtsi | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Fabio Estevam Feb. 27, 2023, 5:49 p.m. UTC | #1
On Mon, Feb 27, 2023 at 2:45 PM Marek Vasut <marex@denx.de> wrote:
>
> Add #sound-dai-cells properties to SAI nodes.
>
> Signed-off-by: Marek Vasut <marex@denx.de>

Reviewed-by: Fabio Estevam <festevam@gmail.com>
Adam Ford Feb. 27, 2023, 6:05 p.m. UTC | #2
On Mon, Feb 27, 2023 at 11:58 AM Marek Vasut <marex@denx.de> wrote:
>
> Add #sound-dai-cells properties to SAI nodes.
>

Should this have a fixes tag?

Fixes: 9e9860069725 ("arm64: dts: imx8mn: Add SAI nodes")

I don't know when these became required, and/or if this should be
back-ported or not.

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

adam

> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Abel Vesa <abelvesa@kernel.org>
> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Jacky Bai <ping.bai@nxp.com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Luca Ceresoli <luca.ceresoli@bootlin.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marco Felsch <m.felsch@pengutronix.de>
> Cc: Michael Turquette <mturquette@baylibre.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Richard Cochran <richardcochran@gmail.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: devicetree@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-clk@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@30000000 {
>                                 ranges;
>
>                                 sai2: sai@30020000 {
> +                                       #sound-dai-cells = <0>;
>                                         compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>                                         reg = <0x30020000 0x10000>;
>                                         interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
> @@ -308,6 +309,7 @@ sai2: sai@30020000 {
>                                 };
>
>                                 sai3: sai@30030000 {
> +                                       #sound-dai-cells = <0>;
>                                         compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>                                         reg = <0x30030000 0x10000>;
>                                         interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
> @@ -322,6 +324,7 @@ sai3: sai@30030000 {
>                                 };
>
>                                 sai5: sai@30050000 {
> +                                       #sound-dai-cells = <0>;
>                                         compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>                                         reg = <0x30050000 0x10000>;
>                                         interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
> @@ -338,6 +341,7 @@ sai5: sai@30050000 {
>                                 };
>
>                                 sai6: sai@30060000 {
> +                                       #sound-dai-cells = <0>;
>                                         compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>                                         reg = <0x30060000  0x10000>;
>                                         interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
> @@ -395,6 +399,7 @@ spdif1: spdif@30090000 {
>                                 };
>
>                                 sai7: sai@300b0000 {
> +                                       #sound-dai-cells = <0>;
>                                         compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>                                         reg = <0x300b0000 0x10000>;
>                                         interrupts = <GIC_SPI 111 IRQ_TYPE_LEVEL_HIGH>;
> --
> 2.39.2
>
Marek Vasut Feb. 27, 2023, 6:59 p.m. UTC | #3
On 2/27/23 19:05, Adam Ford wrote:
> On Mon, Feb 27, 2023 at 11:58 AM Marek Vasut <marex@denx.de> wrote:
>>
>> Add #sound-dai-cells properties to SAI nodes.
>>
> 
> Should this have a fixes tag?
> 
> Fixes: 9e9860069725 ("arm64: dts: imx8mn: Add SAI nodes")
> 
> I don't know when these became required, and/or if this should be
> back-ported or not.

I am unsure myself, but this commit from 2020 does not have it:
ebfa8951e6cd2 ("arm64: dts: imx8mm: specify #sound-dai-cells for SAI nodes")
Marco Felsch Feb. 27, 2023, 7:01 p.m. UTC | #4
Hi Marek,

On 23-02-27, Marek Vasut wrote:
> Add #sound-dai-cells properties to SAI nodes.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> ---
> Cc: Abel Vesa <abelvesa@kernel.org>
> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Jacky Bai <ping.bai@nxp.com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Luca Ceresoli <luca.ceresoli@bootlin.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Marco Felsch <m.felsch@pengutronix.de>
> Cc: Michael Turquette <mturquette@baylibre.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Richard Cochran <richardcochran@gmail.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: devicetree@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-clk@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@30000000 {
>  				ranges;
>  
>  				sai2: sai@30020000 {
> +					#sound-dai-cells = <0>;

Please don't add it in front of the compatible and the reg property.

Regards,
  Marco

>  					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>  					reg = <0x30020000 0x10000>;
>  					interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
> @@ -308,6 +309,7 @@ sai2: sai@30020000 {
>  				};
>  
>  				sai3: sai@30030000 {
> +					#sound-dai-cells = <0>;
>  					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>  					reg = <0x30030000 0x10000>;
>  					interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
> @@ -322,6 +324,7 @@ sai3: sai@30030000 {
>  				};
>  
>  				sai5: sai@30050000 {
> +					#sound-dai-cells = <0>;
>  					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>  					reg = <0x30050000 0x10000>;
>  					interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
> @@ -338,6 +341,7 @@ sai5: sai@30050000 {
>  				};
>  
>  				sai6: sai@30060000 {
> +					#sound-dai-cells = <0>;
>  					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>  					reg = <0x30060000  0x10000>;
>  					interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
> @@ -395,6 +399,7 @@ spdif1: spdif@30090000 {
>  				};
>  
>  				sai7: sai@300b0000 {
> +					#sound-dai-cells = <0>;
>  					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
>  					reg = <0x300b0000 0x10000>;
>  					interrupts = <GIC_SPI 111 IRQ_TYPE_LEVEL_HIGH>;
> -- 
> 2.39.2
> 
>
Marek Vasut Feb. 27, 2023, 7:07 p.m. UTC | #5
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@denx.de>
>> ---
>> Cc: Abel Vesa <abelvesa@kernel.org>
>> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
>> Cc: Fabio Estevam <festevam@gmail.com>
>> Cc: Jacky Bai <ping.bai@nxp.com>
>> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
>> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>> Cc: Luca Ceresoli <luca.ceresoli@bootlin.com>
>> Cc: Lucas Stach <l.stach@pengutronix.de>
>> Cc: Marco Felsch <m.felsch@pengutronix.de>
>> Cc: Michael Turquette <mturquette@baylibre.com>
>> Cc: NXP Linux Team <linux-imx@nxp.com>
>> Cc: Peng Fan <peng.fan@nxp.com>
>> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
>> Cc: Richard Cochran <richardcochran@gmail.com>
>> Cc: Rob Herring <robh+dt@kernel.org>
>> Cc: Sascha Hauer <s.hauer@pengutronix.de>
>> Cc: Shawn Guo <shawnguo@kernel.org>
>> Cc: Stephen Boyd <sboyd@kernel.org>
>> Cc: devicetree@vger.kernel.org
>> Cc: linux-arm-kernel@lists.infradead.org
>> Cc: linux-clk@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@30000000 {
>>   				ranges;
>>   
>>   				sai2: sai@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 ?
Marco Felsch Feb. 27, 2023, 8 p.m. UTC | #6
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@denx.de>
> > > ---
> > > Cc: Abel Vesa <abelvesa@kernel.org>
> > > Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
> > > Cc: Fabio Estevam <festevam@gmail.com>
> > > Cc: Jacky Bai <ping.bai@nxp.com>
> > > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> > > Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > > Cc: Luca Ceresoli <luca.ceresoli@bootlin.com>
> > > Cc: Lucas Stach <l.stach@pengutronix.de>
> > > Cc: Marco Felsch <m.felsch@pengutronix.de>
> > > Cc: Michael Turquette <mturquette@baylibre.com>
> > > Cc: NXP Linux Team <linux-imx@nxp.com>
> > > Cc: Peng Fan <peng.fan@nxp.com>
> > > Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> > > Cc: Richard Cochran <richardcochran@gmail.com>
> > > Cc: Rob Herring <robh+dt@kernel.org>
> > > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > > Cc: Shawn Guo <shawnguo@kernel.org>
> > > Cc: Stephen Boyd <sboyd@kernel.org>
> > > Cc: devicetree@vger.kernel.org
> > > Cc: linux-arm-kernel@lists.infradead.org
> > > Cc: linux-clk@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@30000000 {
> > >   				ranges;
> > >   				sai2: sai@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@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.

Regards,
  Marco
Marek Vasut Feb. 27, 2023, 8:11 p.m. UTC | #7
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@denx.de>
>>>> ---
>>>> Cc: Abel Vesa <abelvesa@kernel.org>
>>>> Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
>>>> Cc: Fabio Estevam <festevam@gmail.com>
>>>> Cc: Jacky Bai <ping.bai@nxp.com>
>>>> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
>>>> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>>>> Cc: Luca Ceresoli <luca.ceresoli@bootlin.com>
>>>> Cc: Lucas Stach <l.stach@pengutronix.de>
>>>> Cc: Marco Felsch <m.felsch@pengutronix.de>
>>>> Cc: Michael Turquette <mturquette@baylibre.com>
>>>> Cc: NXP Linux Team <linux-imx@nxp.com>
>>>> Cc: Peng Fan <peng.fan@nxp.com>
>>>> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
>>>> Cc: Richard Cochran <richardcochran@gmail.com>
>>>> Cc: Rob Herring <robh+dt@kernel.org>
>>>> Cc: Sascha Hauer <s.hauer@pengutronix.de>
>>>> Cc: Shawn Guo <shawnguo@kernel.org>
>>>> Cc: Stephen Boyd <sboyd@kernel.org>
>>>> Cc: devicetree@vger.kernel.org
>>>> Cc: linux-arm-kernel@lists.infradead.org
>>>> Cc: linux-clk@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@30000000 {
>>>>    				ranges;
>>>>    				sai2: sai@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@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.
Krzysztof Kozlowski Feb. 28, 2023, 8:10 a.m. UTC | #8
On 27/02/2023 21:00, Marco Felsch wrote:
>>>> +					#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@reg-nr {
> 	compatible = "";
> 	reg = <>;
> 	// all pending properties below
> 	...
> };
> 
> @Rob, @Krzysztof:
> Is this a (unwritten) rule/policy?
> 

Each platform has its own coding style around this but I am not aware of
a coding style which puts address and size cells at the top. To me it is
really odd placement. First property is always "compatible", as the most
important. Then for most platforms second is "reg", as the one easiest
to compare with unit address. Some platforms put status as last property.

Best regards,
Krzysztof
Marek Vasut Feb. 28, 2023, 8:13 p.m. UTC | #9
On 2/28/23 09:10, Krzysztof Kozlowski wrote:
> On 27/02/2023 21:00, Marco Felsch wrote:
>>>>> +					#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@reg-nr {
>> 	compatible = "";
>> 	reg = <>;
>> 	// all pending properties below
>> 	...
>> };
>>
>> @Rob, @Krzysztof:
>> Is this a (unwritten) rule/policy?
>>
> 
> Each platform has its own coding style around this but I am not aware of
> a coding style which puts address and size cells at the top.

DTspec 0.3 and 0.4-rc agrees with the below.

Linux seems to be full of counter-examples though:
$ git grep -A 1 ' {$' arch/*/boot/dts/ | grep -B 1 cells

> To me it is
> really odd placement. First property is always "compatible", as the most
> important. Then for most platforms second is "reg", as the one easiest
> to compare with unit address. Some platforms put status as last property.

All right, so:

- compatible
- reg
- #whatever-cells
- properties
- status

Does that order look right ?

[...]
Marco Felsch March 1, 2023, 9:08 a.m. UTC | #10
On 23-02-28, Marek Vasut wrote:
> On 2/28/23 09:10, Krzysztof Kozlowski wrote:
> > On 27/02/2023 21:00, Marco Felsch wrote:
> > > > > > +					#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@reg-nr {
> > > 	compatible = "";
> > > 	reg = <>;
> > > 	// all pending properties below
> > > 	...
> > > };
> > > 
> > > @Rob, @Krzysztof:
> > > Is this a (unwritten) rule/policy?
> > > 
> > 
> > Each platform has its own coding style around this but I am not aware of
> > a coding style which puts address and size cells at the top.
> 
> DTspec 0.3 and 0.4-rc agrees with the below.
> 
> Linux seems to be full of counter-examples though:
> $ git grep -A 1 ' {$' arch/*/boot/dts/ | grep -B 1 cells
> 
> > To me it is
> > really odd placement. First property is always "compatible", as the most
> > important. Then for most platforms second is "reg", as the one easiest
> > to compare with unit address. Some platforms put status as last property.
> 
> All right, so:
> 
> - compatible
> - reg
> - #whatever-cells
> - properties
> - status
> 
> Does that order look right ?

I would swap the #whatever-cells with the properties, but that's just my
opinion. The rest looks good to me. Thanks for the research.

Regards,
  Marco

> 
> [...]
>
diff mbox series

Patch

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@30000000 {
 				ranges;
 
 				sai2: sai@30020000 {
+					#sound-dai-cells = <0>;
 					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
 					reg = <0x30020000 0x10000>;
 					interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
@@ -308,6 +309,7 @@  sai2: sai@30020000 {
 				};
 
 				sai3: sai@30030000 {
+					#sound-dai-cells = <0>;
 					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
 					reg = <0x30030000 0x10000>;
 					interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
@@ -322,6 +324,7 @@  sai3: sai@30030000 {
 				};
 
 				sai5: sai@30050000 {
+					#sound-dai-cells = <0>;
 					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
 					reg = <0x30050000 0x10000>;
 					interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
@@ -338,6 +341,7 @@  sai5: sai@30050000 {
 				};
 
 				sai6: sai@30060000 {
+					#sound-dai-cells = <0>;
 					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
 					reg = <0x30060000  0x10000>;
 					interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
@@ -395,6 +399,7 @@  spdif1: spdif@30090000 {
 				};
 
 				sai7: sai@300b0000 {
+					#sound-dai-cells = <0>;
 					compatible = "fsl,imx8mn-sai", "fsl,imx8mq-sai";
 					reg = <0x300b0000 0x10000>;
 					interrupts = <GIC_SPI 111 IRQ_TYPE_LEVEL_HIGH>;