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

Andy Duan
2020-09-17T02:24:44Z

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>
>
>
> >
> > Rob
>