2019-12-04 16:00:49

by Marcelo Schmitt

[permalink] [raw]
Subject: [PATCH v2 1/2] dt-bindings: iio: adc: ad7292: fix channel constraint

Change items property of AD7292 channels to correctly constrain their
quantity.

Signed-off-by: Marcelo Schmitt <[email protected]>
---
Changelog V2
- Shortened the message to make it closer to 50 columns.

Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
index b68be3aaf587..18f1032b86f3 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
@@ -53,7 +53,8 @@ patternProperties:
description: |
The channel number. It can have up to 8 channels numbered from 0 to 7.
items:
- maximum: 7
+ - minimum: 0
+ maximum: 7

diff-channels:
description: see Documentation/devicetree/bindings/iio/adc/adc.txt
--
2.23.0


2019-12-04 19:28:51

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] dt-bindings: iio: adc: ad7292: fix channel constraint

On Wed, Dec 04, 2019 at 12:59:23PM -0300, Marcelo Schmitt wrote:
> Change items property of AD7292 channels to correctly constrain their
> quantity.
>
> Signed-off-by: Marcelo Schmitt <[email protected]>
> ---
> Changelog V2
> - Shortened the message to make it closer to 50 columns.
>
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)

I think Jonathan already applied this series.

>
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index b68be3aaf587..18f1032b86f3 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -53,7 +53,8 @@ patternProperties:
> description: |
> The channel number. It can have up to 8 channels numbered from 0 to 7.
> items:
> - maximum: 7
> + - minimum: 0
> + maximum: 7
>
> diff-channels:
> description: see Documentation/devicetree/bindings/iio/adc/adc.txt
> --
> 2.23.0
>

2019-12-04 22:00:55

by Marcelo Schmitt

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] dt-bindings: iio: adc: ad7292: fix channel constraint

I see now that I had misunderstood part of the last thread.
Sorry for the unnecessary spamming. Please, ignore this V2 series.

On 12/04, Rob Herring wrote:
> On Wed, Dec 04, 2019 at 12:59:23PM -0300, Marcelo Schmitt wrote:
> > Change items property of AD7292 channels to correctly constrain their
> > quantity.
> >
> > Signed-off-by: Marcelo Schmitt <[email protected]>
> > ---
> > Changelog V2
> > - Shortened the message to make it closer to 50 columns.
> >
> > Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
>
> I think Jonathan already applied this series.
>
> >
> > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> > index b68be3aaf587..18f1032b86f3 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> > @@ -53,7 +53,8 @@ patternProperties:
> > description: |
> > The channel number. It can have up to 8 channels numbered from 0 to 7.
> > items:
> > - maximum: 7
> > + - minimum: 0
> > + maximum: 7
> >
> > diff-channels:
> > description: see Documentation/devicetree/bindings/iio/adc/adc.txt
> > --
> > 2.23.0
> >