Re: [PATCH v2 01/20] dt-bindings:iio:adc:fsl,vf610-adc conversion to yaml.

Jonathan Cameron
2020-09-19T13:33:55Z

On Thu, 17 Sep 2020 02:24:36 +0000
Andy Duan <fugang.duan@nxp.com> wrote:

> From: Jonathan Cameron <Jonathan.Cameron@Huawei.com> Sent: Wednesday, September 16, 2020 6:34 PM
> > On Tue, 15 Sep 2020 14:45:58 -0600
> > Rob Herring <robh@kernel.org> wrote:
> >
> > > On Wed, Sep 9, 2020 at 12:02 PM Jonathan Cameron <jic23@kernel.org>
> > wrote:
> > > >
> > > > From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > > >
> > > > A simple conversion of this freescale ADC binding from txt to yaml.
> > > > For maintainer I went with Fugang Duan as the original author of the
> > > > binding. Would be great to have confirmation of this.
> > > >
> > > > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > > > Cc: Fugang Duan <B38611@freescale.com>
> > > > Cc: Shawn Guo <shawnguo@kernel.org>
> > > > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > > > ---
> > > > .../bindings/iio/adc/fsl,vf610-adc.yaml | 81
> > +++++++++++++++++++
> > > > .../devicetree/bindings/iio/adc/vf610-adc.txt | 36 ---------
> > > > 2 files changed, 81 insertions(+), 36 deletions(-)
> > > >
> > > > diff --git
> > > > a/Documentation/devicetree/bindings/iio/adc/fsl,vf610-adc.yaml
> > > > b/Documentation/devicetree/bindings/iio/adc/fsl,vf610-adc.yaml
> > > > new file mode 100644
> > > > index 000000000000..99b6b55fd0a3
> > > > --- /dev/null
> > > > +++ b/Documentation/devicetree/bindings/iio/adc/fsl,vf610-adc.yaml
> > > > @@ -0,0 +1,81 @@
> > > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) %YAML 1.2
> > > > +---
> > > > +$id:
> > > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fde
> > > >
> > +vicetree.org%2Fschemas%2Fiio%2Fadc%2Ffsl%2Cvf610-adc.yaml%23&amp;da
> > > >
> > +ta=02%7C01%7Cfugang.duan%40nxp.com%7C750fcd8ebdf34af6655308d85a2
> > c41
> > > >
> > +8d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63735849344455
> > 0580&
> > > >
> > +amp;sdata=SwRIWqN5caXEUZBTmnjKKHJKAZ1FQ03s1Gdpf%2FTyXzE%3D&am
> > p;rese
> > > > +rved=0
> > > > +$schema:
> > > > +https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fde
> > > >
> > +vicetree.org%2Fmeta-schemas%2Fcore.yaml%23&amp;data=02%7C01%7Cfug
> > an
> > > >
> > +g.duan%40nxp.com%7C750fcd8ebdf34af6655308d85a2c418d%7C686ea1d3bc
> > 2b4
> > > >
> > +c6fa92cd99c5c301635%7C0%7C0%7C637358493444550580&amp;sdata=c4To
> > ZCOG
> > > > +iPM0XbxCnLGJEEXvnMvFKqSgg3gtLeDYACI%3D&amp;reserved=0
> > > > +
> > > > +title: ADC found on Freescale vf610 and similar SoCs
> > > > +
> > > > +maintainers:
> > > > + - Fugang Duan <B38611@freescale.com>
> > >
> > > I assume you got a bunch of bounces on this series, too? Looks like 1,
> > > 4, 7, 9, 13, 16, 19, and 20 bounced. Please fix those addresses before
> > > applying.
> >
> > Yup. I missed them on v1, but have saved the set for v2 to be able to fix those I
> > can.
> >
> > for some I can make a reasonable stab at a suitable maintainer as covered by a
> > catch all. For the others if I can't find a newer address I'll cover them myself
> > until someone else steps up.
> > I think that's just 9 and 13. For 4 I didn't get a bounce but can add that to my
> > list!
> >
> > For this particular one looks like fugang duan is still active but now with an NXP
> > address so I'll update that. I've changed the cc on this mail.
> >
> > Thanks,
> >
> > Jonathan
>
> Thanks, Jonathan.
>
> For the patch, it looks good.
>
> Reviewed-by: Fugang Duan <fugang.duan@nxp.com>

I've updated the maintainer address to the one you have for this tag
and applied to the togreg branch of iio.git, pushing out as testing
for the autobuilders to perhaps poke at it.

Thanks,

Jonathan

> >
> >
> > >
> > > Rob
> >
>