Thread (9 messages) 9 messages, 4 authors, 2025-04-15

Re: [PATCH v2 1/3] dt-bindings: nvmem: rockchip,otp: Add support for rk3562 and rk3568

From: Srinivas Kandagatla <hidden>
Date: 2025-03-07 17:22:26
Also in: linux-devicetree, linux-rockchip, lkml

Hi Kever,


On 27/02/2025 11:08, Kever Yang wrote:
Add compatible entry for the otp controller in rk3562 and rk3568, add schema
for different clock names for new entry.

Signed-off-by: Kever Yang <redacted>
---
Am unable to apply this patch, Could you rebase it on top of linux-next 
or 
https://web.git.kernel.org/pub/scm/linux/kernel/git/srini/nvmem.git/log/?h=for-next


--srini
quoted hunk ↗ jump to hunk
Changes in v2:
- Update the commit message and add maxItems in schema.

  .../bindings/nvmem/rockchip,otp.yaml          | 53 ++++++++++++++++---
  1 file changed, 46 insertions(+), 7 deletions(-)
diff --git a/Documentation/devicetree/bindings/nvmem/rockchip,otp.yaml b/Documentation/devicetree/bindings/nvmem/rockchip,otp.yaml
index a44d44b32809..7572f4a1d73b 100644
--- a/Documentation/devicetree/bindings/nvmem/rockchip,otp.yaml
+++ b/Documentation/devicetree/bindings/nvmem/rockchip,otp.yaml
@@ -14,6 +14,8 @@ properties:
      enum:
        - rockchip,px30-otp
        - rockchip,rk3308-otp
+      - rockchip,rk3562-otp
+      - rockchip,rk3568-otp
        - rockchip,rk3588-otp
  
    reg:
@@ -25,19 +27,15 @@ properties:
  
    clock-names:
      minItems: 3
-    items:
-      - const: otp
-      - const: apb_pclk
-      - const: phy
-      - const: arb
+    maxItems: 4
  
    resets:
      minItems: 1
-    maxItems: 3
+    maxItems: 4
  
    reset-names:
      minItems: 1
-    maxItems: 3
+    maxItems: 4
  
  required:
    - compatible
@@ -62,12 +60,45 @@ allOf:
        properties:
          clocks:
            maxItems: 3
+        clock-names:
+          items:
+            - const: otp
+            - const: apb_pclk
+            - const: phy
          resets:
            maxItems: 1
          reset-names:
            items:
              - const: phy
  
+  - if:
+      properties:
+        compatible:
+          contains:
+            enum:
+              - rockchip,rk3562-otp
+              - rockchip,rk3568-otp
+    then:
+      properties:
+        clocks:
+          minItems: 4
+          maxItems: 4
+        clock-names:
+          items:
+            - const: usr
+            - const: sbpi
+            - const: apb_pclk
+            - const: phy
+        resets:
+          minItems: 4
+          maxItems: 4
+        reset-names:
+          items:
+            - const: usr
+            - const: sbpi
+            - const: apb
+            - const: phy
+
    - if:
        properties:
          compatible:
@@ -78,8 +109,16 @@ allOf:
        properties:
          clocks:
            minItems: 4
+          maxItems: 4
+        clock-names:
+          items:
+            - const: otp
+            - const: apb_pclk
+            - const: phy
+            - const: arb
          resets:
            minItems: 3
+          maxItems: 3
          reset-names:
            items:
              - const: otp
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help