On Fri, 21 Jun 2019 08:31:14 -0600 Rob Herring <[email protected]> wrote:
> On Fri, Jun 21, 2019 at 6:15 AM Mircea Caprioru > <[email protected]> wrote: > > > > Convert AD7124 bindings documentation to YAML format. > > > > Signed-off-by: Mircea Caprioru <[email protected]> > > --- > > > > 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 <[email protected]> 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
