Re: [PATCH V2 4/4] dt-bindings: iio: adc: Convert ad7124 documentation to YAML

From: Jonathan Cameron
Date: Sat Jun 22 2019 - 04:33:53 EST


On Fri, 21 Jun 2019 08:31:14 -0600
Rob Herring <robh+dt@xxxxxxxxxx> wrote:

> On Fri, Jun 21, 2019 at 6:15 AM Mircea Caprioru
> <mircea.caprioru@xxxxxxxxxx> wrote:
> >
> > Convert AD7124 bindings documentation to YAML format.
> >
> > Signed-off-by: Mircea Caprioru <mircea.caprioru@xxxxxxxxxx>
> > ---
> >
> > Changelog v2:
> > - modified SPDX license to GPL-2.0 OR BSD-2-Clause
> > - added regex for a range from 0 to 15
> > - added minimum and maximum constraints for reg property
> > - set type and range of values for adi,reference-select property
> > - used items for diff-channels property
> > - set bipolar, adi,buffered-positive and negative to type: boolean
> >
> > .../bindings/iio/adc/adi,ad7124.yaml | 155 ++++++++++++++++++
> > 1 file changed, 155 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
>
> Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
The result is presumably good but the structure of the patch is not
as straight forward as it should be. As the changelog makes clear we
aren't just doing a conversion here. It is a mixture of conversion
and new properties. Please break the patch in two to reflect that.
First convert the existing binding, then add the new properties.

That will make it easier for people to see what is new.

Thanks,

Jonathan