dt-bindings: iio: adi,ad74115: remove ref from -nanoamp

[PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp

Posted by Krzysztof Kozlowski 2 years, 7 months ago

dtschema v2023.06 comes with support for properties with -nanoamp
suffix, thus bindings should not have a ref for it:

  adi,ad74115.yaml: properties:adi,ext1-burnout-current-nanoamp: '$ref' should not be valid under {'const': '$ref'}

Cc: Cosmin Tanislav <demonsingur@gmail.com>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml b/Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml
index 72d2e910f206..2594fa192f93 100644
--- a/Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml
+++ b/Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml
@@ -216,7 +216,6 @@ properties:
     description: Whether to enable burnout current for EXT1.
 
   adi,ext1-burnout-current-nanoamp:
-    $ref: /schemas/types.yaml#/definitions/uint32
     description:
       Burnout current in nanoamps to be applied to EXT1.
     enum: [0, 50, 500, 1000, 10000]
@@ -233,7 +232,6 @@ properties:
     description: Whether to enable burnout current for EXT2.
 
   adi,ext2-burnout-current-nanoamp:
-    $ref: /schemas/types.yaml#/definitions/uint32
     description: Burnout current in nanoamps to be applied to EXT2.
     enum: [0, 50, 500, 1000, 10000]
     default: 0
@@ -249,7 +247,6 @@ properties:
     description: Whether to enable burnout current for VIOUT.
 
   adi,viout-burnout-current-nanoamp:
-    $ref: /schemas/types.yaml#/definitions/uint32
     description: Burnout current in nanoamps to be applied to VIOUT.
     enum: [0, 1000, 10000]
     default: 0
-- 
2.34.1

Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp

Posted by Conor Dooley 2 years, 7 months ago

On Wed, Jul 12, 2023 at 10:05:12AM +0200, Krzysztof Kozlowski wrote:
> dtschema v2023.06 comes with support for properties with -nanoamp
> suffix, thus bindings should not have a ref for it:
> 
>   adi,ad74115.yaml: properties:adi,ext1-burnout-current-nanoamp: '$ref' should not be valid under {'const': '$ref'}
> 
> Cc: Cosmin Tanislav <demonsingur@gmail.com>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Heh, was just about to go and do this myself - good thing I checked first.
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.

Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp

Posted by Rob Herring 2 years, 7 months ago

On Wed, 12 Jul 2023 10:05:12 +0200, Krzysztof Kozlowski wrote:
> dtschema v2023.06 comes with support for properties with -nanoamp
> suffix, thus bindings should not have a ref for it:
> 
>   adi,ad74115.yaml: properties:adi,ext1-burnout-current-nanoamp: '$ref' should not be valid under {'const': '$ref'}
> 
> Cc: Cosmin Tanislav <demonsingur@gmail.com>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml | 3 ---
>  1 file changed, 3 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp

Posted by Rob Herring 2 years, 7 months ago

On Wed, Jul 12, 2023 at 8:57 AM Rob Herring <robh@kernel.org> wrote:
>
>
> On Wed, 12 Jul 2023 10:05:12 +0200, Krzysztof Kozlowski wrote:
> > dtschema v2023.06 comes with support for properties with -nanoamp
> > suffix, thus bindings should not have a ref for it:
> >
> >   adi,ad74115.yaml: properties:adi,ext1-burnout-current-nanoamp: '$ref' should not be valid under {'const': '$ref'}
> >
> > Cc: Cosmin Tanislav <demonsingur@gmail.com>
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > ---
> >  Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml | 3 ---
> >  1 file changed, 3 deletions(-)
> >
>
> Acked-by: Rob Herring <robh@kernel.org>

Jonathan, Please pick this up for 6.5. Or I can if you prefer.

Rob

Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp

Posted by Jonathan Cameron 2 years, 6 months ago

On Wed, 12 Jul 2023 08:59:15 -0600
Rob Herring <robh+dt@kernel.org> wrote:

> On Wed, Jul 12, 2023 at 8:57 AM Rob Herring <robh@kernel.org> wrote:
> >
> >
> > On Wed, 12 Jul 2023 10:05:12 +0200, Krzysztof Kozlowski wrote:  
> > > dtschema v2023.06 comes with support for properties with -nanoamp
> > > suffix, thus bindings should not have a ref for it:
> > >
> > >   adi,ad74115.yaml: properties:adi,ext1-burnout-current-nanoamp: '$ref' should not be valid under {'const': '$ref'}
> > >
> > > Cc: Cosmin Tanislav <demonsingur@gmail.com>
> > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > > ---
> > >  Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml | 3 ---
> > >  1 file changed, 3 deletions(-)
> > >  
> >
> > Acked-by: Rob Herring <robh@kernel.org>  
> 
> Jonathan, Please pick this up for 6.5. Or I can if you prefer.
> 
> Rob

Ok. Applied to the fixes-togreg branch of iio.git given desire to get
it in 6.5

Thanks,

Jonathan