* [PATCH v3] dt-bindings: Remove alt_ref from versal
@ 2023-11-28 10:43 Shubhrajyoti Datta
2023-11-28 16:44 ` Conor Dooley
2023-12-17 22:26 ` Stephen Boyd
0 siblings, 2 replies; 3+ messages in thread
From: Shubhrajyoti Datta @ 2023-11-28 10:43 UTC (permalink / raw)
To: linux-clk, devicetree
Cc: git, mturquette, sboyd, robh+dt, krzysztof.kozlowski+dt,
conor+dt, michal.simek
The alt_ref is present only in Versal-net devices.
Other versal devices do not have it. So remove alt_ref
for versal.
Fixes: 352546805a44 ("dt-bindings: clock: Add bindings for versal clock driver")
Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
---
Changes in v3:
squash the patch
Changes in v2:
- Have specific constraints for versal and versal net.
.../bindings/clock/xlnx,versal-clk.yaml | 31 +++++++++++++++----
.../firmware/xilinx/xlnx,zynqmp-firmware.yaml | 4 +--
2 files changed, 27 insertions(+), 8 deletions(-)
diff --git a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
index 1ba687d433b1..bef109d163a8 100644
--- a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
+++ b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
@@ -31,11 +31,11 @@ properties:
clocks:
description: List of clock specifiers which are external input
clocks to the given clock controller.
- minItems: 3
+ minItems: 2
maxItems: 8
clock-names:
- minItems: 3
+ minItems: 2
maxItems: 8
required:
@@ -59,15 +59,34 @@ allOf:
clocks:
items:
- description: reference clock
- - description: alternate reference clock
- description: alternate reference clock for programmable logic
clock-names:
items:
- const: ref
- - const: alt_ref
- const: pl_alt_ref
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - xlnx,versal-net-clk
+
+ then:
+ properties:
+ clocks:
+ items:
+ - description: reference clock
+ - description: alternate reference clock for programmable logic
+ - description: alternate reference clock
+
+ clock-names:
+ items:
+ - const: ref
+ - const: pl_alt_ref
+ - const: alt_ref
+
- if:
properties:
compatible:
@@ -110,8 +129,8 @@ examples:
versal_clk: clock-controller {
#clock-cells = <1>;
compatible = "xlnx,versal-clk";
- clocks = <&ref>, <&alt_ref>, <&pl_alt_ref>;
- clock-names = "ref", "alt_ref", "pl_alt_ref";
+ clocks = <&ref>, <&pl_alt_ref>;
+ clock-names = "ref", "pl_alt_ref";
};
};
};
diff --git a/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml b/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml
index 822864488dcb..8e584857ddd4 100644
--- a/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml
+++ b/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml
@@ -95,8 +95,8 @@ examples:
versal_clk: clock-controller {
#clock-cells = <1>;
compatible = "xlnx,versal-clk";
- clocks = <&ref>, <&alt_ref>, <&pl_alt_ref>;
- clock-names = "ref", "alt_ref", "pl_alt_ref";
+ clocks = <&ref>, <&pl_alt_ref>;
+ clock-names = "ref", "pl_alt_ref";
};
};
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3] dt-bindings: Remove alt_ref from versal
2023-11-28 10:43 [PATCH v3] dt-bindings: Remove alt_ref from versal Shubhrajyoti Datta
@ 2023-11-28 16:44 ` Conor Dooley
2023-12-17 22:26 ` Stephen Boyd
1 sibling, 0 replies; 3+ messages in thread
From: Conor Dooley @ 2023-11-28 16:44 UTC (permalink / raw)
To: Shubhrajyoti Datta
Cc: linux-clk, devicetree, git, mturquette, sboyd, robh+dt,
krzysztof.kozlowski+dt, conor+dt, michal.simek
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Tue, Nov 28, 2023 at 04:13:48PM +0530, Shubhrajyoti Datta wrote:
> The alt_ref is present only in Versal-net devices.
> Other versal devices do not have it. So remove alt_ref
> for versal.
>
> Fixes: 352546805a44 ("dt-bindings: clock: Add bindings for versal clock driver")
> Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
Acked-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] 3+ messages in thread
* Re: [PATCH v3] dt-bindings: Remove alt_ref from versal
2023-11-28 10:43 [PATCH v3] dt-bindings: Remove alt_ref from versal Shubhrajyoti Datta
2023-11-28 16:44 ` Conor Dooley
@ 2023-12-17 22:26 ` Stephen Boyd
1 sibling, 0 replies; 3+ messages in thread
From: Stephen Boyd @ 2023-12-17 22:26 UTC (permalink / raw)
To: Shubhrajyoti Datta, devicetree, linux-clk
Cc: git, mturquette, robh+dt, krzysztof.kozlowski+dt, conor+dt, michal.simek
Quoting Shubhrajyoti Datta (2023-11-28 02:43:48)
> The alt_ref is present only in Versal-net devices.
> Other versal devices do not have it. So remove alt_ref
> for versal.
>
> Fixes: 352546805a44 ("dt-bindings: clock: Add bindings for versal clock driver")
> Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-12-17 22:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-28 10:43 [PATCH v3] dt-bindings: Remove alt_ref from versal Shubhrajyoti Datta
2023-11-28 16:44 ` Conor Dooley
2023-12-17 22:26 ` Stephen Boyd
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.