[v3] dt-bindings: clock: versal: Add versal-net compatible string

Commit Message

Datta, Shubhrajyoti June 8, 2023, 9:25 a.m. UTC

Add dt-binding documentation for Versal NET platforms.

Signed-off-by: Jay Buddhabhatti <[e-mail]>
Signed-off-by: Shubhrajyoti Datta <[e-mail]>
---
Change in v2:
Add the versal net enum

Changes in v3:
Add the compatible for versal net
the usage will new compatible string followed by old one

 .../devicetree/bindings/clock/xlnx,versal-clk.yaml         | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Comments

Krzysztof Kozlowski June 8, 2023, 9:46 a.m. UTC | #1

On 08/06/2023 11:25, Shubhrajyoti Datta wrote:
> Add dt-binding documentation for Versal NET platforms.

What are versal net platforms? It's different SoC? You have entire
commit msg to explain it...

Best regards,
Krzysztof

Michal Simek June 8, 2023, 9:58 a.m. UTC | #2

On 6/8/23 11:46, Krzysztof Kozlowski wrote:
> On 08/06/2023 11:25, Shubhrajyoti Datta wrote:
>> Add dt-binding documentation for Versal NET platforms.
> 
> What are versal net platforms? It's different SoC? You have entire
> commit msg to explain it...

It is another/new Xilinx SOC.

Thanks,
Michal

Patch

diff --git a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
index 229af98b1d30..b90aa064a6d3 100644
--- a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
+++ b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml
@@ -20,7 +20,12 @@  select: false
 
 properties:
   compatible:
-    const: xlnx,versal-clk
+    oneOf:
+      - const: xlnx,versal-clk
+      - items:
+          - enum:
+              - xlnx,versal-net-clk
+          - const: xlnx,versal-clk
 
   "#clock-cells":
     const: 1