2019-12-12 13:42:00

by Benjamin GAIGNARD

[permalink] [raw]
Subject: [PATCH] dt-bindings: iio: ad7192: Fix DTC error in reg schema

Add minimum field to be compliant to json-schema

Fixes: d898f9ac542f9 ("dt-bindings: iio: adc: Add dt-schema for AD7292")
Signed-off-by: Benjamin Gaignard <[email protected]>
---
Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 1 +
1 file changed, 1 insertion(+)

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

diff-channels:
--
2.15.0


2019-12-12 14:07:20

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: iio: ad7192: Fix DTC error in reg schema

On Thu, Dec 12, 2019 at 7:40 AM Benjamin Gaignard
<[email protected]> wrote:
>
> Add minimum field to be compliant to json-schema
>
> Fixes: d898f9ac542f9 ("dt-bindings: iio: adc: Add dt-schema for AD7292")
> Signed-off-by: Benjamin Gaignard <[email protected]>
> ---
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 1 +
> 1 file changed, 1 insertion(+)

Thanks, but there's already a fix pending in -next.

Rob

2019-12-20 14:02:49

by Marcelo Schmitt

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: iio: ad7192: Fix DTC error in reg schema

On 12/12, Benjamin Gaignard wrote:
> Add minimum field to be compliant to json-schema
>
> Fixes: d898f9ac542f9 ("dt-bindings: iio: adc: Add dt-schema for AD7292")
> Signed-off-by: Benjamin Gaignard <[email protected]>
> ---
> Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index b68be3aaf587..d804a8a78bea 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -53,6 +53,7 @@ patternProperties:
> description: |
> The channel number. It can have up to 8 channels numbered from 0 to 7.
> items:
> + minimum: 0
> maximum: 7
>
> diff-channels:
> --
> 2.15.0
>

Hi Benjamin,

Sorry for the late reply.

This issue has already been tackled down. I think it should be in
mainline now.
Anyway, thanks for your contribution. Nice holiday and a happy new year.


Thanks,

Marcelo