All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp
@ 2023-07-12  8:05 Krzysztof Kozlowski
  2023-07-12  8:24 ` Conor Dooley
  2023-07-12 14:57 ` Rob Herring
  0 siblings, 2 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-12  8:05 UTC (permalink / raw)
  To: Cosmin Tanislav, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	linux-iio, devicetree, linux-kernel
  Cc: Krzysztof Kozlowski, Cosmin Tanislav

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


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp
  2023-07-12  8:05 [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp Krzysztof Kozlowski
@ 2023-07-12  8:24 ` Conor Dooley
  2023-07-12 14:57 ` Rob Herring
  1 sibling, 0 replies; 5+ messages in thread
From: Conor Dooley @ 2023-07-12  8:24 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Cosmin Tanislav, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	linux-iio, devicetree, linux-kernel, Cosmin Tanislav

[-- Attachment #1: Type: text/plain, Size: 592 bytes --]

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.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp
  2023-07-12  8:05 [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp Krzysztof Kozlowski
  2023-07-12  8:24 ` Conor Dooley
@ 2023-07-12 14:57 ` Rob Herring
  2023-07-12 14:59   ` Rob Herring
  1 sibling, 1 reply; 5+ messages in thread
From: Rob Herring @ 2023-07-12 14:57 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-kernel, Michael Hennerich, Conor Dooley, Rob Herring,
	Krzysztof Kozlowski, Lars-Peter Clausen, devicetree,
	Cosmin Tanislav, linux-iio, Jonathan Cameron, Cosmin Tanislav


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>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp
  2023-07-12 14:57 ` Rob Herring
@ 2023-07-12 14:59   ` Rob Herring
  2023-07-15 17:12     ` Jonathan Cameron
  0 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2023-07-12 14:59 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Jonathan Cameron
  Cc: linux-kernel, Michael Hennerich, Conor Dooley,
	Krzysztof Kozlowski, Lars-Peter Clausen, devicetree,
	Cosmin Tanislav, linux-iio, Cosmin Tanislav

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp
  2023-07-12 14:59   ` Rob Herring
@ 2023-07-15 17:12     ` Jonathan Cameron
  0 siblings, 0 replies; 5+ messages in thread
From: Jonathan Cameron @ 2023-07-15 17:12 UTC (permalink / raw)
  To: Rob Herring
  Cc: Krzysztof Kozlowski, linux-kernel, Michael Hennerich,
	Conor Dooley, Krzysztof Kozlowski, Lars-Peter Clausen,
	devicetree, Cosmin Tanislav, linux-iio, Cosmin Tanislav

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-07-15 17:12 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-12  8:05 [PATCH] dt-bindings: iio: adi,ad74115: remove ref from -nanoamp Krzysztof Kozlowski
2023-07-12  8:24 ` Conor Dooley
2023-07-12 14:57 ` Rob Herring
2023-07-12 14:59   ` Rob Herring
2023-07-15 17:12     ` Jonathan Cameron

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.