diff mbox series

dt-bindings: clock: drop unneeded quotes and use absolute /schemas path

Message ID 20230609140751.65129-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State Accepted, archived
Headers show
Series dt-bindings: clock: drop unneeded quotes and use absolute /schemas path | expand

Commit Message

Krzysztof Kozlowski June 9, 2023, 2:07 p.m. UTC
Cleanup bindings dropping unneeded quotes. Once all these are fixed,
checking for this can be enabled in yamllint.  Also absolute path
starting with /schemas is preferred.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/clock/ingenic,cgu.yaml      | 4 ++--
 .../devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

Comments

Stephen Boyd June 15, 2023, 12:35 a.m. UTC | #1
Quoting Krzysztof Kozlowski (2023-06-09 07:07:51)
> Cleanup bindings dropping unneeded quotes. Once all these are fixed,
> checking for this can be enabled in yamllint.  Also absolute path
> starting with /schemas is preferred.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---

It's not clear if I should pick this up. Do you want to take it through
some DT tree?
Krzysztof Kozlowski June 15, 2023, 7:32 a.m. UTC | #2
On 15/06/2023 02:35, Stephen Boyd wrote:
> Quoting Krzysztof Kozlowski (2023-06-09 07:07:51)
>> Cleanup bindings dropping unneeded quotes. Once all these are fixed,
>> checking for this can be enabled in yamllint.  Also absolute path
>> starting with /schemas is preferred.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
> 
> It's not clear if I should pick this up. Do you want to take it through
> some DT tree?

Please grab it.

Best regards,
Krzysztof
Stephen Boyd June 15, 2023, 5:35 p.m. UTC | #3
Quoting Krzysztof Kozlowski (2023-06-09 07:07:51)
> Cleanup bindings dropping unneeded quotes. Once all these are fixed,
> checking for this can be enabled in yamllint.  Also absolute path
> starting with /schemas is preferred.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---

Applied to clk-next
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/clock/ingenic,cgu.yaml b/Documentation/devicetree/bindings/clock/ingenic,cgu.yaml
index 9e733b10c392..509df06b9c9d 100644
--- a/Documentation/devicetree/bindings/clock/ingenic,cgu.yaml
+++ b/Documentation/devicetree/bindings/clock/ingenic,cgu.yaml
@@ -98,9 +98,9 @@  required:
 
 patternProperties:
   "^usb-phy@[a-f0-9]+$":
-    allOf: [ $ref: "../phy/ingenic,phy-usb.yaml#" ]
+    $ref: /schemas/phy/ingenic,phy-usb.yaml#
   "^mac-phy-ctrl@[a-f0-9]+$":
-    allOf: [ $ref: "../net/ingenic,mac.yaml#" ]
+    $ref: /schemas/net/ingenic,mac.yaml#
 
 additionalProperties: false
 
diff --git a/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml b/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
index 99686085f751..26d94cedc871 100644
--- a/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
+++ b/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
@@ -48,7 +48,7 @@  properties:
 patternProperties:
   "^dma-router@[a-f0-9]+$":
     type: object
-    $ref: "../dma/renesas,rzn1-dmamux.yaml#"
+    $ref: /schemas/dma/renesas,rzn1-dmamux.yaml#
 
 required:
   - compatible