On Wed, 12 Aug 2020 10:29:07 -0600 Rob Herring <robh@xxxxxxxxxx> wrote: > On Sun, 09 Aug 2020 12:17:48 +0100, Jonathan Cameron wrote: > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > > > Simple binding conversion. Only addition was #io-channel-cells to > > allow for potential consumers of the channels on this device. > > > > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > Cc: Akinobu Mita <akinobu.mita@xxxxxxxxx> > > --- > > .../bindings/iio/adc/ti,adc12138.yaml | 85 +++++++++++++++++++ > > .../bindings/iio/adc/ti-adc12138.txt | 37 -------- > > 2 files changed, 85 insertions(+), 37 deletions(-) > > > > > My bot found errors running 'make dt_binding_check' on your patch: > > Error: Documentation/devicetree/bindings/iio/adc/ti,adc12138.example.dts:26.34-35 syntax error > FATAL ERROR: Unable to parse input tree > make[1]: *** [scripts/Makefile.lib:330: Documentation/devicetree/bindings/iio/adc/ti,adc12138.example.dt.yaml] Error 1 > make[1]: *** Waiting for unfinished jobs.... > make: *** [Makefile:1334: dt_binding_check] Error 2 > > > See https://patchwork.ozlabs.org/patch/1342594 > > If you already ran 'make dt_binding_check' and didn't see the above > error(s), then make sure dt-schema is up to date: > > pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade > > Please check and re-submit. > Not sure how I missed this one in testing. It is a missing include. I don't propose to resend the series though until more comments are in. Thanks, Jonathan