Re: [PATCH v3 2/2] dt-bindings: iio: adc: Add MAX1241 device tree bindings in documentation

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

 



On Fri, 2020-03-20 at 10:33 +0200, Alexandru Lazar wrote:
> [External]
> 
> > > +  reg:
> > > +    description: SPI chip select number for this device
> > 
> > Rob will probably complain that you don't need to document[add description]
> > to
> > standard attributes [like SPI's 'reg' parameter]
> > Just listing it here, should be sufficient. So, you can drop the
> > description.
> 
> Makes sense -- should've thought of it, considering I dropped the doc
> for spi-frequency as well. My brain is still wired to the non-standard
> txt docs it seems, sorry!
> 

I think many people aren't yet super-comfortable with the new yaml format, which
probably makes Rob's life a bit difficult.

I guess it should become easier once version 5.4+ becomes more popular.

> I'm going to make a v4 for this (I think I need to reorder the patch
> series anyway -- the dt-bindings should come first) and Cc: the
> dt-bindings list with as well.

The order of the bindings vs code is a bit relaxed [atm]. But may become more
strict [later at some point]. I think the official version is bindings-first &
driver-next, but it's still accepted to do it the other way around.

> 
> Thanks again for all your help!
> 
> Best regards,
> Alex




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux