devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: leds: Fix JSON pointer in max-brightness
@ 2023-11-22 20:54 Rob Herring
  2023-11-23  7:42 ` Krzysztof Kozlowski
  2023-11-23  9:09 ` (subset) " Lee Jones
  0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring @ 2023-11-22 20:54 UTC (permalink / raw)
  To: Pavel Machek, Lee Jones, Krzysztof Kozlowski, Conor Dooley,
	Jacek Anaszewski
  Cc: linux-leds, devicetree, linux-kernel

A valid JSON pointer should begin with a '/'. The json-schema package is
lax on this allowing either form, but that's changing in new versions.

Signed-off-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/leds/common.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/leds/common.yaml b/Documentation/devicetree/bindings/leds/common.yaml
index c8d0ba5f2327..55a8d1385e21 100644
--- a/Documentation/devicetree/bindings/leds/common.yaml
+++ b/Documentation/devicetree/bindings/leds/common.yaml
@@ -167,7 +167,7 @@ properties:
       Note that this flag is mainly used for PWM-LEDs, where it is not possible
       to map brightness to current. Drivers for other controllers should use
       led-max-microamp.
-    $ref: /schemas/types.yaml#definitions/uint32
+    $ref: /schemas/types.yaml#/definitions/uint32
 
   panic-indicator:
     description:
-- 
2.42.0


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

* Re: [PATCH] dt-bindings: leds: Fix JSON pointer in max-brightness
  2023-11-22 20:54 [PATCH] dt-bindings: leds: Fix JSON pointer in max-brightness Rob Herring
@ 2023-11-23  7:42 ` Krzysztof Kozlowski
  2023-11-23  9:09 ` (subset) " Lee Jones
  1 sibling, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2023-11-23  7:42 UTC (permalink / raw)
  To: Rob Herring, Pavel Machek, Lee Jones, Krzysztof Kozlowski,
	Conor Dooley, Jacek Anaszewski
  Cc: linux-leds, devicetree, linux-kernel

On 22/11/2023 21:54, Rob Herring wrote:
> A valid JSON pointer should begin with a '/'. The json-schema package is
> lax on this allowing either form, but that's changing in new versions.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


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

* Re: (subset) [PATCH] dt-bindings: leds: Fix JSON pointer in max-brightness
  2023-11-22 20:54 [PATCH] dt-bindings: leds: Fix JSON pointer in max-brightness Rob Herring
  2023-11-23  7:42 ` Krzysztof Kozlowski
@ 2023-11-23  9:09 ` Lee Jones
  1 sibling, 0 replies; 3+ messages in thread
From: Lee Jones @ 2023-11-23  9:09 UTC (permalink / raw)
  To: Pavel Machek, Krzysztof Kozlowski, Conor Dooley, Jacek Anaszewski,
	Rob Herring
  Cc: linux-leds, devicetree, linux-kernel

On Wed, 22 Nov 2023 13:54:17 -0700, Rob Herring wrote:
> A valid JSON pointer should begin with a '/'. The json-schema package is
> lax on this allowing either form, but that's changing in new versions.
> 
> 

Applied, thanks!

[1/1] dt-bindings: leds: Fix JSON pointer in max-brightness
      commit: 2dc0a14846ecf85fb4de29be186581f008d197dd

--
Lee Jones [李琼斯]


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

end of thread, other threads:[~2023-11-23  9:09 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-22 20:54 [PATCH] dt-bindings: leds: Fix JSON pointer in max-brightness Rob Herring
2023-11-23  7:42 ` Krzysztof Kozlowski
2023-11-23  9:09 ` (subset) " Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).