diff mbox series

[v3,01/10] dt-bindings: iio: dac: adi-axi-adc: fix ad7606 pwm-names

Message ID 20250129-wip-bl-ad7606_add_backend_sw_mode-v3-1-c3aec77c0ab7@baylibre.com (mailing list archive)
State Accepted
Headers show
Series add support for Software mode on AD7606's iio backend driver | expand

Commit Message

Angelo Dureghello Jan. 29, 2025, 11:03 a.m. UTC
From: Angelo Dureghello <adureghello@baylibre.com>

Fix make dt_binding_check warning:

DTC [C] Documentation/devicetree/bindings/iio/adc/adi,axi-adc.example.dtb
.../adc/adi,axi-adc.example.dtb: adc@0: pwm-names: ['convst1'] is too short
    from schema $id: http://devicetree.org/schemas/iio/adc/adi,ad7606.yaml#

Add "minItems" to pwm-names, it allows to use one single pwm when
connected to both adc conversion inputs.

Fixes: 7c2357b10490 ("dt-bindings: iio: adc: ad7606: Add iio backend bindings")
Signed-off-by: Angelo Dureghello <adureghello@baylibre.com>
---
 Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Rob Herring (Arm) Jan. 29, 2025, 4:43 p.m. UTC | #1
On Wed, 29 Jan 2025 12:03:02 +0100, Angelo Dureghello wrote:
> From: Angelo Dureghello <adureghello@baylibre.com>
> 
> Fix make dt_binding_check warning:
> 
> DTC [C] Documentation/devicetree/bindings/iio/adc/adi,axi-adc.example.dtb
> .../adc/adi,axi-adc.example.dtb: adc@0: pwm-names: ['convst1'] is too short
>     from schema $id: http://devicetree.org/schemas/iio/adc/adi,ad7606.yaml#
> 
> Add "minItems" to pwm-names, it allows to use one single pwm when
> connected to both adc conversion inputs.
> 
> Fixes: 7c2357b10490 ("dt-bindings: iio: adc: ad7606: Add iio backend bindings")
> Signed-off-by: Angelo Dureghello <adureghello@baylibre.com>
> ---
>  Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>
Jonathan Cameron Feb. 1, 2025, 12:54 p.m. UTC | #2
On Wed, 29 Jan 2025 10:43:31 -0600
"Rob Herring (Arm)" <robh@kernel.org> wrote:

> On Wed, 29 Jan 2025 12:03:02 +0100, Angelo Dureghello wrote:
> > From: Angelo Dureghello <adureghello@baylibre.com>
> > 
> > Fix make dt_binding_check warning:
> > 
> > DTC [C] Documentation/devicetree/bindings/iio/adc/adi,axi-adc.example.dtb
> > .../adc/adi,axi-adc.example.dtb: adc@0: pwm-names: ['convst1'] is too short
> >     from schema $id: http://devicetree.org/schemas/iio/adc/adi,ad7606.yaml#
> > 
> > Add "minItems" to pwm-names, it allows to use one single pwm when
> > connected to both adc conversion inputs.
> > 
> > Fixes: 7c2357b10490 ("dt-bindings: iio: adc: ad7606: Add iio backend bindings")
> > Signed-off-by: Angelo Dureghello <adureghello@baylibre.com>
> > ---
> >  Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> >   
> 
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
> 

Applied to the fixes-togreg branch of iio.git.
Seems unlikely this will cause us any merge conflicts other than some trivial
line changes so that shouldn't delay the rest of the series.

Jonathan
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
index ab5881d0d017..52d3f1ce3367 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
@@ -146,6 +146,7 @@  properties:
     maxItems: 2
 
   pwm-names:
+    minItems: 1
     items:
       - const: convst1
       - const: convst2