2023-07-12 08:14:17

by Krzysztof Kozlowski

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

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 <[email protected]>
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
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



2023-07-12 08:58:37

by Conor Dooley

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

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 <[email protected]>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Heh, was just about to go and do this myself - good thing I checked first.
Reviewed-by: Conor Dooley <[email protected]>

Cheers,
Conor.


Attachments:
(No filename) (592.00 B)
signature.asc (235.00 B)
Download all attachments

2023-07-12 15:27:54

by Rob Herring

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

On Wed, Jul 12, 2023 at 8:57 AM Rob Herring <[email protected]> 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 <[email protected]>
> > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> > ---
> > Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml | 3 ---
> > 1 file changed, 3 deletions(-)
> >
>
> Acked-by: Rob Herring <[email protected]>

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

Rob

2023-07-12 15:28:36

by Rob Herring (Arm)

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


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 <[email protected]>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml | 3 ---
> 1 file changed, 3 deletions(-)
>

Acked-by: Rob Herring <[email protected]>


2023-07-15 17:57:48

by Jonathan Cameron

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

On Wed, 12 Jul 2023 08:59:15 -0600
Rob Herring <[email protected]> wrote:

> On Wed, Jul 12, 2023 at 8:57 AM Rob Herring <[email protected]> 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 <[email protected]>
> > > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> > > ---
> > > Documentation/devicetree/bindings/iio/addac/adi,ad74115.yaml | 3 ---
> > > 1 file changed, 3 deletions(-)
> > >
> >
> > Acked-by: Rob Herring <[email protected]>
>
> 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