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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

> 
> [...]
> 



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux