[PATCH] dt-bindings: firmware: fsl,scu: Mark multi-channel MU layouts as deprecated

Peng Fan (OSS) posted 1 patch 1 week, 5 days ago
There is a newer version of this series
.../devicetree/bindings/firmware/fsl,scu.yaml        | 20 +++++++++++---------
1 file changed, 11 insertions(+), 9 deletions(-)
[PATCH] dt-bindings: firmware: fsl,scu: Mark multi-channel MU layouts as deprecated
Posted by Peng Fan (OSS) 1 week, 5 days ago
From: Peng Fan <peng.fan@nxp.com>

The SCU MU driver has long supported the simple and efficient single-TX and
single-RX channel layout. The older multi-channel MU configurations
(tx0..tx3 and rx0..rx3) are less efficient in practice and not needed.

Mark these legacy mbox-names and mboxes tuple layouts as deprecated in the
binding schema. The driver continues to support them for backward
compatibility in case firmware publishes the legacy properties.

The example section is updated accordingly to demonstrate the recommended
layout.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 .../devicetree/bindings/firmware/fsl,scu.yaml        | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/Documentation/devicetree/bindings/firmware/fsl,scu.yaml b/Documentation/devicetree/bindings/firmware/fsl,scu.yaml
index f9ba18f06369215691c69cdc1538b53dfd369f37..307f1c6278532ace94db92e6e8b68312eb4c8a4f 100644
--- a/Documentation/devicetree/bindings/firmware/fsl,scu.yaml
+++ b/Documentation/devicetree/bindings/firmware/fsl,scu.yaml
@@ -76,7 +76,8 @@ properties:
           - description: TX0 MU channel
           - description: RX0 MU channel
           - description: optional MU channel for general interrupt
-      - items:
+      - deprecated: true
+        items:
           - description: TX0 MU channel
           - description: TX1 MU channel
           - description: TX2 MU channel
@@ -85,7 +86,8 @@ properties:
           - description: RX1 MU channel
           - description: RX2 MU channel
           - description: RX3 MU channel
-      - items:
+      - deprecated: true
+        items:
           - description: TX0 MU channel
           - description: TX1 MU channel
           - description: TX2 MU channel
@@ -105,7 +107,8 @@ properties:
           - const: tx0
           - const: rx0
           - const: gip3
-      - items:
+      - deprecated: true
+        items:
           - const: tx0
           - const: tx1
           - const: tx2
@@ -114,7 +117,8 @@ properties:
           - const: rx1
           - const: rx2
           - const: rx3
-      - items:
+      - deprecated: true
+        items:
           - const: tx0
           - const: tx1
           - const: tx2
@@ -167,11 +171,9 @@ examples:
     firmware {
         system-controller {
             compatible = "fsl,imx-scu";
-            mbox-names = "tx0", "tx1", "tx2", "tx3",
-                         "rx0", "rx1", "rx2", "rx3",
-                         "gip3";
-            mboxes = <&lsio_mu1 0 0 &lsio_mu1 0 1 &lsio_mu1 0 2 &lsio_mu1 0 3
-                      &lsio_mu1 1 0 &lsio_mu1 1 1 &lsio_mu1 1 2 &lsio_mu1 1 3
+            mbox-names = "tx0", "rx0", "gip3";
+            mboxes = <&lsio_mu1 0 0
+                      &lsio_mu1 1 0
                       &lsio_mu1 3 3>;
 
             clock-controller {

---
base-commit: ca3a02fda4da8e2c1cb6baee5d72352e9e2cfaea
change-id: 20260126-scu-c93d45e319f2

Best regards,
-- 
Peng Fan <peng.fan@nxp.com>
Re: [PATCH] dt-bindings: firmware: fsl,scu: Mark multi-channel MU layouts as deprecated
Posted by Frank Li 1 week, 4 days ago
On Mon, Jan 26, 2026 at 11:47:02AM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> The SCU MU driver has long supported the simple and efficient single-TX and

Does you means "long" is "already"

Frank
> single-RX channel layout. The older multi-channel MU configurations
> (tx0..tx3 and rx0..rx3) are less efficient in practice and not needed.
>
> Mark these legacy mbox-names and mboxes tuple layouts as deprecated in the
> binding schema. The driver continues to support them for backward
> compatibility in case firmware publishes the legacy properties.
>
> The example section is updated accordingly to demonstrate the recommended
> layout.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  .../devicetree/bindings/firmware/fsl,scu.yaml        | 20 +++++++++++---------
>  1 file changed, 11 insertions(+), 9 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/firmware/fsl,scu.yaml b/Documentation/devicetree/bindings/firmware/fsl,scu.yaml
> index f9ba18f06369215691c69cdc1538b53dfd369f37..307f1c6278532ace94db92e6e8b68312eb4c8a4f 100644
> --- a/Documentation/devicetree/bindings/firmware/fsl,scu.yaml
> +++ b/Documentation/devicetree/bindings/firmware/fsl,scu.yaml
> @@ -76,7 +76,8 @@ properties:
>            - description: TX0 MU channel
>            - description: RX0 MU channel
>            - description: optional MU channel for general interrupt
> -      - items:
> +      - deprecated: true
> +        items:
>            - description: TX0 MU channel
>            - description: TX1 MU channel
>            - description: TX2 MU channel
> @@ -85,7 +86,8 @@ properties:
>            - description: RX1 MU channel
>            - description: RX2 MU channel
>            - description: RX3 MU channel
> -      - items:
> +      - deprecated: true
> +        items:
>            - description: TX0 MU channel
>            - description: TX1 MU channel
>            - description: TX2 MU channel
> @@ -105,7 +107,8 @@ properties:
>            - const: tx0
>            - const: rx0
>            - const: gip3
> -      - items:
> +      - deprecated: true
> +        items:
>            - const: tx0
>            - const: tx1
>            - const: tx2
> @@ -114,7 +117,8 @@ properties:
>            - const: rx1
>            - const: rx2
>            - const: rx3
> -      - items:
> +      - deprecated: true
> +        items:
>            - const: tx0
>            - const: tx1
>            - const: tx2
> @@ -167,11 +171,9 @@ examples:
>      firmware {
>          system-controller {
>              compatible = "fsl,imx-scu";
> -            mbox-names = "tx0", "tx1", "tx2", "tx3",
> -                         "rx0", "rx1", "rx2", "rx3",
> -                         "gip3";
> -            mboxes = <&lsio_mu1 0 0 &lsio_mu1 0 1 &lsio_mu1 0 2 &lsio_mu1 0 3
> -                      &lsio_mu1 1 0 &lsio_mu1 1 1 &lsio_mu1 1 2 &lsio_mu1 1 3
> +            mbox-names = "tx0", "rx0", "gip3";
> +            mboxes = <&lsio_mu1 0 0
> +                      &lsio_mu1 1 0
>                        &lsio_mu1 3 3>;
>
>              clock-controller {
>
> ---
> base-commit: ca3a02fda4da8e2c1cb6baee5d72352e9e2cfaea
> change-id: 20260126-scu-c93d45e319f2
>
> Best regards,
> --
> Peng Fan <peng.fan@nxp.com>
>