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