dt-bindings: iio: adc: ad7192: Fix 'single-channel' constraints
Rob Herring (Arm)
maintainerPATCH 1/1 dt-bindings: iio: adc: ad7192: Fix 'single-channel' constraints
The 'single-channel' property is an uint32, not an array, so 'items' is an incorrect constraint. This didn't matter until dtschema recently changed how properties are decoded. This results in this warning:
Documentation/devicetree/bindings/iio/adc/adi,ad7192.example.dtb: adc@0: \
channel@1:single-channel: 1 is not of type 'array'
--- Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +2 -3
@@ -120,9 +120,8 @@ patternProperties: description: Positive input can be connected to pins AIN1 to AIN16 by choosing the appropriate value from 1 to 16. Negative input is connected to AINCOM. - items: - minimum: 1 - maximum: 16 + minimum: 1 + maximum: 16 oneOf: - required:
Signature · 1 line
2.43.0
Quoted 8 lines · Rob Herring (Arm)
The 'single-channel' property is an uint32, not an array, so 'items' is an incorrect constraint. This didn't matter until dtschema recently changed how properties are decoded. This results in this warning:
Documentation/devicetree/bindings/iio/adc/adi,ad7192.example.dtb: adc@0: \
channel@1:single-channel: 1 is not of type 'array'Fixes caf7b7632b8d ("dt-bindings: iio: adc: ad7192: Add AD7194 support")
Quoted 24 lines
Signed-off-by Rob Herring (Arm) <[email protected]>--- Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +2
-3@@ -120,9 +120,8 @@ patternProperties: description: Positive input can be connected to pins AIN1 to AIN16 by choosing the appropriate value from 1 to 16. Negative input is connected to AINCOM. - items: - minimum: 1 - maximum: 16 + minimum: 1 + maximum: 16 oneOf: - required:Signature · 1 line
2.43.0
Rob Herring (Arm)
maintainerQuoted 4 lines · Rob Herring (Arm)
The 'single-channel' property is an uint32, not an array, so 'items' is an incorrect constraint. This didn't matter until dtschema recently changed how properties are decoded. This results in this warning:
Quoted 9 lines
Documentation/devicetree/bindings/iio/adc/adi,ad7192.example.dtb: adc@0: \
channel@1:single-channel: 1 is not of type 'array'Fixes caf7b7632b8d ("dt-bindings: iio: adc: ad7192: Add AD7194 support")Signed-off-by Rob Herring (Arm) <[email protected]>--- Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
I have other fixes for rc1 already, so I've applied this one.
Rob
Quoted 15 lines · On Fri, 26 Jul 2024 13:52:58 -0500 "Rob Herring (Arm)" <[email protected]> wrote:
Quoted 4 lines · Rob Herring (Arm)
The 'single-channel' property is an uint32, not an array, so 'items' is an incorrect constraint. This didn't matter until dtschema recently changed how properties are decoded. This results in this warning:
Quoted 9 lines
Documentation/devicetree/bindings/iio/adc/adi,ad7192.example.dtb: adc@0: \
channel@1:single-channel: 1 is not of type 'array'Fixes caf7b7632b8d ("dt-bindings: iio: adc: ad7192: Add AD7194 support")Signed-off-by Rob Herring (Arm) <[email protected]>--- Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
Quoted 3 lines
I have other fixes for rc1 already, so I've applied this one.
Rob
Cool
For the record (no need to add to patch!)