Re: [PATCH] dt-bindings: iio: adc: Add imx6ul & imx6sx compatibles

[Jonathan Cameron <jic23@kernel.org>]

On Fri, 15 Jul 2022 09:56:45 +0200
Alexander Stein <alexander.stein@ew.tq-group.com> wrote:

> Am Dienstag, 28. Juni 2022, 23:09:19 CEST schrieb Rob Herring:
> > On Mon, Jun 20, 2022 at 10:12:44AM +0200, Alexander Stein wrote:  
> > > Hello,
> > > 
> > > Am Samstag, 18. Juni 2022, 19:01:29 CEST schrieb Jonathan Cameron:  
> > > > On Fri, 17 Jun 2022 16:44:48 -0600
> > > > 
> > > > Rob Herring <robh@kernel.org> wrote:  
> > > > > On Mon, Jun 13, 2022 at 11:34:46AM -0600, Rob Herring wrote:  
> > > > > > On Mon, 13 Jun 2022 14:35:29 +0200, Alexander Stein wrote:  
> > > > > > > Both are already using the vf610 compatible.
> > > > > > > 
> > > > > > > Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> > > > > > > ---
> > > > > > > 
> > > > > > >  .../devicetree/bindings/iio/adc/fsl,vf610-adc.yaml       | 9
> > > > > > >  ++++++++-
> > > > > > >  1 file changed, 8 insertions(+), 1 deletion(-)  
> > > > > > 
> > > > > > Running 'make dtbs_check' with the schema in this patch gives the
> > > > > > following warnings. Consider if they are expected or the schema is
> > > > > > incorrect. These may not be new warnings.
> > > > > > 
> > > > > > Note that it is not yet a requirement to have 0 warnings for
> > > > > > dtbs_check.
> > > > > > This will change in the future.
> > > > > > 
> > > > > > Full log is available here: https://patchwork.ozlabs.org/patch/
> > > > > > 
> > > > > > 
> > > > > > adc@2198000: 'num-channels' does not match any of the regexes:
> > > > > > 'pinctrl-[0-9]+'>  
> > > > > 
> > > > > Looks like you need to add 'num-channels'?  
> > > > 
> > > > or a lot of wrong dtbs :)
> > > > 
> > > > By which I mean ones providing a property that may or may not be
> > > > actually
> > > > used by any drivers...  
> > > 
> > > This got already fixed by Baruch's patch which is currently in Shawn's
> > > imx-
> > > fixes-5.19 branch at [1]  
> > 
> > Great!
> > 
> > Reviewed-by: Rob Herring <robh@kernel.org>  
> 
> Did this got missed? Or is it applied somwhere I' not aware of?
Missed it.  Thanks for pointing it out!

Applied to the togreg branch of iio.git and pushed out initially as testing
for 0-day to poke at it.  Note this may well not make this cycle now unfortunately
given the timing.

Thanks,

Jonathan

> 
> Best regards,
> Alexander
> 
> 
>