[PATCH] dt-bindings: usb: ux500: Convert to DT schema

[PATCH] dt-bindings: usb: ux500: Convert to DT schema

[Linus Walleij]

Convert the Ux500 MUSB text binding to DT schema. Describe the APB
clock and the existing ordered DMA endpoint channels, and reference
the common dual-role USB properties.

Signed-off-by: Linus Walleij <linusw@kernel.org>
---
Convert the Ux500 MUSB text binding to DT schema, describing the APB
clock, ordered DMA endpoint channels and common dual-role USB properties.

This patch is split from the Ux500 DeviceTree validation cleanup series.

Prepared with Codex.
---
 .../bindings/usb/stericsson,db8500-musb.yaml       | 79 ++++++++++++++++++++++
 .../devicetree/bindings/usb/ux500-usb.txt          | 50 --------------
 2 files changed, 79 insertions(+), 50 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/stericsson,db8500-musb.yaml b/Documentation/devicetree/bindings/usb/stericsson,db8500-musb.yaml
new file mode 100644
index 000000000000..9edf77aa43b6
--- /dev/null
+++ b/Documentation/devicetree/bindings/usb/stericsson,db8500-musb.yaml
@@ -0,0 +1,79 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/usb/stericsson,db8500-musb.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: ST-Ericsson DB8500 MUSB controller
+
+maintainers:
+  - Linus Walleij <linusw@kernel.org>
+
+allOf:
+  - $ref: usb-drd.yaml#
+
+properties:
+  compatible:
+    const: stericsson,db8500-musb
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1
+
+  interrupt-names:
+    items:
+      - const: mc
+
+  clocks:
+    maxItems: 1
+
+  phys:
+    maxItems: 1
+
+  phy-names:
+    items:
+      - const: usb
+
+  dmas:
+    minItems: 16
+    maxItems: 16
+
+  dma-names:
+    items:
+      - const: iep_1_9
+      - const: oep_1_9
+      - const: iep_2_10
+      - const: oep_2_10
+      - const: iep_3_11
+      - const: oep_3_11
+      - const: iep_4_12
+      - const: oep_4_12
+      - const: iep_5_13
+      - const: oep_5_13
+      - const: iep_6_14
+      - const: oep_6_14
+      - const: iep_7_15
+      - const: oep_7_15
+      - const: iep_8
+      - const: oep_8
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - dr_mode
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    usb@a03e0000 {
+        compatible = "stericsson,db8500-musb";
+        reg = <0xa03e0000 0x10000>;
+        interrupts = <0 23 4>;
+        interrupt-names = "mc";
+        clocks = <&prcc_pclk 5 0>;
+        dr_mode = "otg";
+    };
diff --git a/Documentation/devicetree/bindings/usb/ux500-usb.txt b/Documentation/devicetree/bindings/usb/ux500-usb.txt
deleted file mode 100644
index 439a41c79afa..000000000000
--- a/Documentation/devicetree/bindings/usb/ux500-usb.txt
+++ /dev/null
@@ -1,50 +0,0 @@
-Ux500 MUSB
-
-Required properties:
- - compatible : Should be "stericsson,db8500-musb"
- - reg        : Offset and length of registers
- - interrupts : Interrupt; mode, number and trigger
- - dr_mode    : Dual-role; either host mode "host", peripheral mode "peripheral"
-                or both "otg"
-
-Optional properties:
- - dmas       : A list of dma channels;
-                dma-controller, event-line, fixed-channel, flags
- - dma-names  : An ordered list of channel names affiliated to the above
-
-Example:
-
-usb_per5@a03e0000 {
-	compatible = "stericsson,db8500-musb";
-	reg = <0xa03e0000 0x10000>;
-	interrupts = <0 23 0x4>;
-	interrupt-names = "mc";
-
-	dr_mode = "otg";
-
-	dmas = <&dma 38 0 0x2>, /* Logical - DevToMem */
-	       <&dma 38 0 0x0>, /* Logical - MemToDev */
-	       <&dma 37 0 0x2>, /* Logical - DevToMem */
-	       <&dma 37 0 0x0>, /* Logical - MemToDev */
-	       <&dma 36 0 0x2>, /* Logical - DevToMem */
-	       <&dma 36 0 0x0>, /* Logical - MemToDev */
-	       <&dma 19 0 0x2>, /* Logical - DevToMem */
-	       <&dma 19 0 0x0>, /* Logical - MemToDev */
-	       <&dma 18 0 0x2>, /* Logical - DevToMem */
-	       <&dma 18 0 0x0>, /* Logical - MemToDev */
-	       <&dma 17 0 0x2>, /* Logical - DevToMem */
-	       <&dma 17 0 0x0>, /* Logical - MemToDev */
-	       <&dma 16 0 0x2>, /* Logical - DevToMem */
-	       <&dma 16 0 0x0>, /* Logical - MemToDev */
-	       <&dma 39 0 0x2>, /* Logical - DevToMem */
-	       <&dma 39 0 0x0>; /* Logical - MemToDev */
-
-	dma-names = "iep_1_9",  "oep_1_9",
-		    "iep_2_10", "oep_2_10",
-		    "iep_3_11", "oep_3_11",
-		    "iep_4_12", "oep_4_12",
-		    "iep_5_13", "oep_5_13",
-		    "iep_6_14", "oep_6_14",
-		    "iep_7_15", "oep_7_15",
-		    "iep_8",    "oep_8";
-};

---
base-commit: 6628e7d4c6b25c8d18f1345e3a20d21829aae7e4
change-id: 20261112-ux500-musb-bindings-c1d0e47c3e52

Best regards,
--  
Linus Walleij <linusw@kernel.org>

Re: [PATCH] dt-bindings: usb: ux500: Convert to DT schema

[sashiko-bot]

> Convert the Ux500 MUSB text binding to DT schema. Describe the APB
> clock and the existing ordered DMA endpoint channels, and reference
> the common dual-role USB properties.
> 
> Signed-off-by: Linus Walleij <linusw@kernel.org>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261113-ux500-musb-bindings-v1-1-553aa97c5c95@kernel.org?part=1