Re: [PATCH v2 12/12] dt-bindings: iio: accel: Add ADIS16203 Inclinometer

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

 



On Mon, 30 Jan 2023 09:21:37 -0600
Rob Herring <robh@xxxxxxxxxx> wrote:

> On Mon, Jan 30, 2023 at 07:35:26AM -0600, Rob Herring wrote:
> > 
> > On Sun, 29 Jan 2023 15:56:02 +0000, Jonathan Cameron wrote:  
> > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> > > 
> > > There has been a driver in staging for quite a while.
> > > Given we are now moving it to the main tree, time to make sure it
> > > has binding documentation.
> > > 
> > > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> > > 
> > > --
> > > V2:
> > > - maxItems/minItems added where needed.
> > > - fix interrupt-names to allow multiple entries.
> > > ---
> > >  .../bindings/iio/accel/adi,adis16203.yaml     | 64 +++++++++++++++++++
> > >  1 file changed, 64 insertions(+)
> > >   
> > 
> > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
> > on your patch (DT_CHECKER_FLAGS is new in v5.13):
> > 
> > yamllint warnings/errors:
> > 
> > dtschema/dtc warnings/errors:
> > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/accel/adi,adis16203.example.dtb: accelerometer@0: Unevaluated properties are not allowed ('interrupt-names' was unexpected)
> > 	From schema: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/accel/adi,adis16201.yaml  
> 
> I was scratching my head on this one... Check the compatible string in 
> the example. You've got 201 instead of 203.
> 
> Are these parts that different?
> 
> Rob

Gah,  Not sure why I didn't see this error - definitely something odd going on when
I test at the moment. The parts are not particularly related.

I'll fix that and send a v3. 
 



[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