Devicetree
 help / color / mirror / Atom feed
* [PATCH v2] dt-bindings: arm: omap: Convert L4 interconnect to DT schema
@ 2026-07-23 10:05 Eduard Bostina
  2026-08-03 21:07 ` Rob Herring
  0 siblings, 1 reply; 2+ messages in thread
From: Eduard Bostina @ 2026-07-23 10:05 UTC (permalink / raw)
  To: Conor Dooley, devicetree, Eduard Bostina, Krzysztof Kozlowski,
	linux-kernel, Mark Brown, Rob Herring
  Cc: daniel.baluta, simona.toaca, goledhruva, m-chawdhry

Convert the Texas Instruments OMAP L4 interconnect bus bindings
to DT schema.

During the conversion, several updates were made:
- Defined the compatible property as an array to require
  the TI-specific string first, followed by either 'simple-bus' or
  'simple-pm-bus'.
- Added a 'select' block to ensure this schema is only
  applied to TI-specific nodes, preventing duplicate
  schema warnings against the core simple-pm-bus.yaml.
- Added the L4 compatible strings used by the boards but missing from
  the text binding, and corrected the 'ti,am3-l4-wkup' typo to
  'ti,am33xx-l4-wkup'.
- Made 'reg' and 'reg-names' optional.

Signed-off-by: Eduard Bostina <egbostina@gmail.com>
---
Changes in v2:
- Renamed the example bus node so it matches the simple-bus schema,
  fixing the 'make dt_binding_check' nodename error.
- Added the L4 compatible strings used by the boards but missing from
  the text binding, and corrected the 'ti,am3-l4-wkup' typo to
  'ti,am33xx-l4-wkup'.

Link to v1:
https://lore.kernel.org/all/20260722171840.299303-1-egbostina@gmail.com/

 .../devicetree/bindings/arm/omap/l4.txt       |  37 ------
 .../bindings/arm/omap/ti,omap-l4.yaml         | 121 ++++++++++++++++++
 2 files changed, 121 insertions(+), 37 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/arm/omap/l4.txt
 create mode 100644 Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml

diff --git a/Documentation/devicetree/bindings/arm/omap/l4.txt b/Documentation/devicetree/bindings/arm/omap/l4.txt
deleted file mode 100644
index 6816adcdc15f..000000000000
--- a/Documentation/devicetree/bindings/arm/omap/l4.txt
+++ /dev/null
@@ -1,37 +0,0 @@
-L4 interconnect bindings
-
-These bindings describe the OMAP SoCs L4 interconnect bus.
-
-Required properties:
-- compatible : Should be "ti,omap2-l4" for OMAP2 family l4 core bus
-	       Should be "ti,omap2-l4-wkup" for OMAP2 family l4 wkup bus
-	       Should be "ti,omap3-l4-core" for OMAP3 family l4 core bus
-	       Should be "ti,omap4-l4-cfg" for OMAP4 family l4 cfg bus
-	       Should be "ti,omap4-l4-per" for OMAP4 family l4 per bus
-	       Should be "ti,omap4-l4-wkup" for OMAP4 family l4 wkup bus
-	       Should be "ti,omap5-l4-cfg" for OMAP5 family l4 cfg bus
-	       Should be "ti,omap5-l4-wkup" for OMAP5 family l4 wkup bus
-	       Should be "ti,dra7-l4-cfg" for DRA7 family l4 cfg bus
-	       Should be "ti,dra7-l4-wkup" for DRA7 family l4 wkup bus
-	       Should be "ti,am3-l4-wkup" for AM33xx family l4 wkup bus
-	       Should be "ti,am4-l4-wkup" for AM43xx family l4 wkup bus
-- ranges : contains the IO map range for the bus
-- reg : registers link agent and interconnect agent and access protection
-- reg-names : "la" for link agent, "ia0" to "ia3" for one to three
-              interconnect agent instances, "ap" for access if it exists
-
-Examples:
-
-l4: interconnect@48000000 {
-	compatible "ti,omap4-l4-per", "simple-bus";
-	reg = <0x48000000 0x800>,
-	      <0x48000800 0x800>,
-	      <0x48001000 0x400>,
-	      <0x48001400 0x400>,
-	      <0x48001800 0x400>,
-	      <0x48001c00 0x400>;
-	reg-names = "ap", "la", "ia0", "ia1", "ia2", "ia3";
-	#address-cells = <1>;
-	#size-cells = <1>;
-	ranges = <0 0x48000000 0x100000>;
-};
diff --git a/Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml b/Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml
new file mode 100644
index 000000000000..de3f5f9da3fc
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml
@@ -0,0 +1,121 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/omap/ti,omap-l4.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TI OMAP SoCs L4 interconnect bus
+
+maintainers:
+  - Eduard Bostina <egbostina@gmail.com>
+
+description:
+  These bindings describe the OMAP SoCs L4 interconnect bus.
+
+select:
+  properties:
+    compatible:
+      contains:
+        enum:
+          - ti,omap2-l4
+          - ti,omap2-l4-wkup
+          - ti,omap3-l4-core
+          - ti,omap4-l4-abe
+          - ti,omap4-l4-cfg
+          - ti,omap4-l4-per
+          - ti,omap4-l4-wkup
+          - ti,omap5-l4-abe
+          - ti,omap5-l4-cfg
+          - ti,omap5-l4-per
+          - ti,omap5-l4-wkup
+          - ti,dra7-l4-cfg
+          - ti,dra7-l4-per1
+          - ti,dra7-l4-per2
+          - ti,dra7-l4-per3
+          - ti,dra7-l4-wkup
+          - ti,am33xx-l4-fast
+          - ti,am33xx-l4-fw
+          - ti,am33xx-l4-mpuss
+          - ti,am33xx-l4-per
+          - ti,am33xx-l4-wkup
+          - ti,am4-l4-fast
+          - ti,am4-l4-per
+          - ti,am4-l4-wkup
+          - ti,dm814-l4hs
+          - ti,dm814-l4ls
+  required:
+    - compatible
+
+properties:
+  compatible:
+    items:
+      - enum:
+          - ti,omap2-l4
+          - ti,omap2-l4-wkup
+          - ti,omap3-l4-core
+          - ti,omap4-l4-abe
+          - ti,omap4-l4-cfg
+          - ti,omap4-l4-per
+          - ti,omap4-l4-wkup
+          - ti,omap5-l4-abe
+          - ti,omap5-l4-cfg
+          - ti,omap5-l4-per
+          - ti,omap5-l4-wkup
+          - ti,dra7-l4-cfg
+          - ti,dra7-l4-per1
+          - ti,dra7-l4-per2
+          - ti,dra7-l4-per3
+          - ti,dra7-l4-wkup
+          - ti,am33xx-l4-fast
+          - ti,am33xx-l4-fw
+          - ti,am33xx-l4-mpuss
+          - ti,am33xx-l4-per
+          - ti,am33xx-l4-wkup
+          - ti,am4-l4-fast
+          - ti,am4-l4-per
+          - ti,am4-l4-wkup
+          - ti,dm814-l4hs
+          - ti,dm814-l4ls
+      - enum:
+          - simple-bus
+          - simple-pm-bus
+
+  reg:
+    minItems: 1
+    maxItems: 6
+
+  reg-names:
+    minItems: 1
+    maxItems: 6
+    items:
+      enum:
+        - ap
+        - la
+        - ia0
+        - ia1
+        - ia2
+        - ia3
+
+  ranges: true
+
+required:
+  - compatible
+  - ranges
+
+additionalProperties: true
+
+examples:
+  - |
+    bus@48000000 {
+        compatible = "ti,omap4-l4-per", "simple-bus";
+        reg = <0x48000000 0x800>,
+              <0x48000800 0x800>,
+              <0x48001000 0x400>,
+              <0x48001400 0x400>,
+              <0x48001800 0x400>,
+              <0x48001c00 0x400>;
+        reg-names = "ap", "la", "ia0", "ia1", "ia2", "ia3";
+        #address-cells = <1>;
+        #size-cells = <1>;
+        ranges = <0 0x48000000 0x100000>;
+    };
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] dt-bindings: arm: omap: Convert L4 interconnect to DT schema
  2026-07-23 10:05 [PATCH v2] dt-bindings: arm: omap: Convert L4 interconnect to DT schema Eduard Bostina
@ 2026-08-03 21:07 ` Rob Herring
  0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring @ 2026-08-03 21:07 UTC (permalink / raw)
  To: Eduard Bostina
  Cc: Conor Dooley, devicetree, Krzysztof Kozlowski, linux-kernel,
	Mark Brown, daniel.baluta, simona.toaca, goledhruva, m-chawdhry

On Thu, Jul 23, 2026 at 10:05:40AM +0000, Eduard Bostina wrote:
> Convert the Texas Instruments OMAP L4 interconnect bus bindings
> to DT schema.
> 
> During the conversion, several updates were made:
> - Defined the compatible property as an array to require
>   the TI-specific string first, followed by either 'simple-bus' or
>   'simple-pm-bus'.
> - Added a 'select' block to ensure this schema is only
>   applied to TI-specific nodes, preventing duplicate
>   schema warnings against the core simple-pm-bus.yaml.
> - Added the L4 compatible strings used by the boards but missing from
>   the text binding, and corrected the 'ti,am3-l4-wkup' typo to
>   'ti,am33xx-l4-wkup'.
> - Made 'reg' and 'reg-names' optional.
> 
> Signed-off-by: Eduard Bostina <egbostina@gmail.com>
> ---
> Changes in v2:
> - Renamed the example bus node so it matches the simple-bus schema,
>   fixing the 'make dt_binding_check' nodename error.
> - Added the L4 compatible strings used by the boards but missing from
>   the text binding, and corrected the 'ti,am3-l4-wkup' typo to
>   'ti,am33xx-l4-wkup'.
> 
> Link to v1:
> https://lore.kernel.org/all/20260722171840.299303-1-egbostina@gmail.com/
> 
>  .../devicetree/bindings/arm/omap/l4.txt       |  37 ------
>  .../bindings/arm/omap/ti,omap-l4.yaml         | 121 ++++++++++++++++++
>  2 files changed, 121 insertions(+), 37 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/arm/omap/l4.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml
> 
> diff --git a/Documentation/devicetree/bindings/arm/omap/l4.txt b/Documentation/devicetree/bindings/arm/omap/l4.txt
> deleted file mode 100644
> index 6816adcdc15f..000000000000
> --- a/Documentation/devicetree/bindings/arm/omap/l4.txt
> +++ /dev/null
> @@ -1,37 +0,0 @@
> -L4 interconnect bindings
> -
> -These bindings describe the OMAP SoCs L4 interconnect bus.
> -
> -Required properties:
> -- compatible : Should be "ti,omap2-l4" for OMAP2 family l4 core bus
> -	       Should be "ti,omap2-l4-wkup" for OMAP2 family l4 wkup bus
> -	       Should be "ti,omap3-l4-core" for OMAP3 family l4 core bus
> -	       Should be "ti,omap4-l4-cfg" for OMAP4 family l4 cfg bus
> -	       Should be "ti,omap4-l4-per" for OMAP4 family l4 per bus
> -	       Should be "ti,omap4-l4-wkup" for OMAP4 family l4 wkup bus
> -	       Should be "ti,omap5-l4-cfg" for OMAP5 family l4 cfg bus
> -	       Should be "ti,omap5-l4-wkup" for OMAP5 family l4 wkup bus
> -	       Should be "ti,dra7-l4-cfg" for DRA7 family l4 cfg bus
> -	       Should be "ti,dra7-l4-wkup" for DRA7 family l4 wkup bus
> -	       Should be "ti,am3-l4-wkup" for AM33xx family l4 wkup bus
> -	       Should be "ti,am4-l4-wkup" for AM43xx family l4 wkup bus
> -- ranges : contains the IO map range for the bus
> -- reg : registers link agent and interconnect agent and access protection
> -- reg-names : "la" for link agent, "ia0" to "ia3" for one to three
> -              interconnect agent instances, "ap" for access if it exists
> -
> -Examples:
> -
> -l4: interconnect@48000000 {
> -	compatible "ti,omap4-l4-per", "simple-bus";
> -	reg = <0x48000000 0x800>,
> -	      <0x48000800 0x800>,
> -	      <0x48001000 0x400>,
> -	      <0x48001400 0x400>,
> -	      <0x48001800 0x400>,
> -	      <0x48001c00 0x400>;
> -	reg-names = "ap", "la", "ia0", "ia1", "ia2", "ia3";
> -	#address-cells = <1>;
> -	#size-cells = <1>;
> -	ranges = <0 0x48000000 0x100000>;
> -};
> diff --git a/Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml b/Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml
> new file mode 100644
> index 000000000000..de3f5f9da3fc
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/omap/ti,omap-l4.yaml
> @@ -0,0 +1,121 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/omap/ti,omap-l4.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: TI OMAP SoCs L4 interconnect bus
> +
> +maintainers:
> +  - Eduard Bostina <egbostina@gmail.com>
> +
> +description:
> +  These bindings describe the OMAP SoCs L4 interconnect bus.
> +
> +select:
> +  properties:
> +    compatible:
> +      contains:
> +        enum:
> +          - ti,omap2-l4
> +          - ti,omap2-l4-wkup
> +          - ti,omap3-l4-core
> +          - ti,omap4-l4-abe
> +          - ti,omap4-l4-cfg
> +          - ti,omap4-l4-per
> +          - ti,omap4-l4-wkup
> +          - ti,omap5-l4-abe
> +          - ti,omap5-l4-cfg
> +          - ti,omap5-l4-per
> +          - ti,omap5-l4-wkup
> +          - ti,dra7-l4-cfg
> +          - ti,dra7-l4-per1
> +          - ti,dra7-l4-per2
> +          - ti,dra7-l4-per3
> +          - ti,dra7-l4-wkup
> +          - ti,am33xx-l4-fast
> +          - ti,am33xx-l4-fw
> +          - ti,am33xx-l4-mpuss
> +          - ti,am33xx-l4-per
> +          - ti,am33xx-l4-wkup
> +          - ti,am4-l4-fast
> +          - ti,am4-l4-per
> +          - ti,am4-l4-wkup
> +          - ti,dm814-l4hs
> +          - ti,dm814-l4ls
> +  required:
> +    - compatible
> +
> +properties:
> +  compatible:
> +    items:
> +      - enum:
> +          - ti,omap2-l4
> +          - ti,omap2-l4-wkup
> +          - ti,omap3-l4-core
> +          - ti,omap4-l4-abe
> +          - ti,omap4-l4-cfg
> +          - ti,omap4-l4-per
> +          - ti,omap4-l4-wkup
> +          - ti,omap5-l4-abe
> +          - ti,omap5-l4-cfg
> +          - ti,omap5-l4-per
> +          - ti,omap5-l4-wkup
> +          - ti,dra7-l4-cfg
> +          - ti,dra7-l4-per1
> +          - ti,dra7-l4-per2
> +          - ti,dra7-l4-per3
> +          - ti,dra7-l4-wkup
> +          - ti,am33xx-l4-fast
> +          - ti,am33xx-l4-fw
> +          - ti,am33xx-l4-mpuss
> +          - ti,am33xx-l4-per
> +          - ti,am33xx-l4-wkup
> +          - ti,am4-l4-fast
> +          - ti,am4-l4-per
> +          - ti,am4-l4-wkup
> +          - ti,dm814-l4hs
> +          - ti,dm814-l4ls
> +      - enum:
> +          - simple-bus
> +          - simple-pm-bus
> +
> +  reg:
> +    minItems: 1
> +    maxItems: 6
> +
> +  reg-names:
> +    minItems: 1
> +    maxItems: 6
> +    items:
> +      enum:
> +        - ap
> +        - la
> +        - ia0
> +        - ia1
> +        - ia2
> +        - ia3
> +
> +  ranges: true
> +
> +required:
> +  - compatible
> +  - ranges
> +
> +additionalProperties: true

Restrict anything else to nodes:

additionalProperties:
  type: object

You'll need to define #address-cells and #size-cells.


> +
> +examples:
> +  - |
> +    bus@48000000 {
> +        compatible = "ti,omap4-l4-per", "simple-bus";
> +        reg = <0x48000000 0x800>,
> +              <0x48000800 0x800>,
> +              <0x48001000 0x400>,
> +              <0x48001400 0x400>,
> +              <0x48001800 0x400>,
> +              <0x48001c00 0x400>;
> +        reg-names = "ap", "la", "ia0", "ia1", "ia2", "ia3";
> +        #address-cells = <1>;
> +        #size-cells = <1>;
> +        ranges = <0 0x48000000 0x100000>;
> +    };
> -- 
> 2.43.0
> 
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-08-03 21:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-23 10:05 [PATCH v2] dt-bindings: arm: omap: Convert L4 interconnect to DT schema Eduard Bostina
2026-08-03 21:07 ` Rob Herring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox