All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: mailbox: Convert hisilicon,hi6220-mbox to DT schema
@ 2026-10-01 23:05 Rob Herring (Arm)
  2026-10-08 12:12 ` Wei Xu
  2026-10-12  1:34 ` Jassi Brar
  0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring (Arm) @ 2026-10-01 23:05 UTC (permalink / raw)
  To: Jassi Brar, Krzysztof Kozlowski, Conor Dooley, Wei Xu
  Cc: linux-kernel, devicetree

Convert the hisilicon,hi6220-mbox binding to DT schema format. It's a
straight forward conversion.

Assisted-by: LLM
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../mailbox/hisilicon,hi6220-mailbox.txt      | 74 -------------------
 .../mailbox/hisilicon,hi6220-mbox.yaml        | 63 ++++++++++++++++
 2 files changed, 63 insertions(+), 74 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt
 create mode 100644 Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml

diff --git a/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt b/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt
deleted file mode 100644
index de9fbf722bee..000000000000
--- a/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt
+++ /dev/null
@@ -1,74 +0,0 @@
-Hisilicon Hi6220 Mailbox Driver
-===============================
-
-Hisilicon Hi6220 mailbox supports up to 32 channels. Each channel
-is unidirectional with a maximum message size of 8 words. I/O is
-performed using register access (there is no DMA) and the cell
-raises an interrupt when messages are received.
-
-Mailbox Device Node:
-====================
-
-Required properties:
---------------------
-- compatible:		Shall be "hisilicon,hi6220-mbox"
-- reg:			Contains the mailbox register address range (base
-			address and length); the first item is for IPC
-			registers, the second item is shared buffer for
-			slots.
-- #mbox-cells:		Common mailbox binding property to identify the number
-			of cells required for the mailbox specifier. Must be 3.
-			<&phandle slot_id dst_irq ack_irq>
-			  phandle: Label name of mailbox controller
-			  slot_id: Slot id used either for TX or RX
-			  dst_irq: IRQ identifier index number which used by MCU
-			  ack_irq: IRQ identifier index number with generating a
-			           TX/RX interrupt to application processor,
-				   mailbox driver uses it to acknowledge interrupt
-- interrupts:		Contains the interrupt information for the mailbox
-			device. The format is dependent on which interrupt
-			controller the SoCs use.
-
-Optional Properties:
---------------------
-- hi6220,mbox-tx-noirq: Property of MCU firmware's feature, so mailbox driver
-			use this flag to ask MCU to enable "automatic idle
-			flag" mode or IRQ generated mode to acknowledge a TX
-			completion.
-
-Example:
---------
-
-	mailbox: mailbox@f7510000 {
-		compatible = "hisilicon,hi6220-mbox";
-		reg = <0x0 0xf7510000 0x0 0x1000>, /* IPC_S */
-		      <0x0 0x06dff800 0x0 0x0800>; /* Mailbox */
-		interrupt-parent = <&gic>;
-		interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
-		#mbox-cells = <3>;
-	};
-
-
-Mailbox client
-===============
-
-Required properties:
---------------------
-- compatible:		Many (See the client docs).
-- mboxes:		Standard property to specify a Mailbox
-			Cells must match 'mbox-cells' (See Mailbox Device Node above).
-
-Optional Properties:
---------------------
-- mbox-names:		Name given to channels seen in the 'mboxes' property.
-
-Example:
---------
-
-	stub_clock: stub_clock {
-		compatible = "hisilicon,hi6220-stub-clk";
-		hisilicon,hi6220-clk-sram = <&sram>;
-		#clock-cells = <1>;
-		mbox-names = "mbox-tx", "mbox-rx";
-		mboxes = <&mailbox 1 0 11>, <&mailbox 0 1 10>;
-	};
diff --git a/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml b/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml
new file mode 100644
index 000000000000..f4c8f1f29baa
--- /dev/null
+++ b/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml
@@ -0,0 +1,63 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mailbox/hisilicon,hi6220-mbox.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Hisilicon Hi6220 Mailbox Controller
+
+maintainers:
+  - Wei Xu <xuwei5@hisilicon.com>
+
+description:
+  Hisilicon Hi6220 mailbox supports up to 32 channels. Each channel is
+  unidirectional with a maximum message size of 8 words. I/O is performed using
+  register access (there is no DMA) and the cell raises an interrupt when
+  messages are received.
+
+properties:
+  compatible:
+    const: hisilicon,hi6220-mbox
+
+  reg:
+    items:
+      - description: IPC registers
+      - description: Shared buffer for mailbox slots
+
+  interrupts:
+    maxItems: 1
+
+  "#mbox-cells":
+    const: 3
+    description: >
+      The cells content are as follows:
+        - Slot number
+        - Remote destination interrupt vector
+        - Local ack interrupt vector
+
+  hi6220,mbox-tx-noirq:
+    description:
+      Flag to request MCU to enable automatic idle flag mode instead of IRQ for
+      TX completion.
+    type: boolean
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - "#mbox-cells"
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+    #include <dt-bindings/interrupt-controller/irq.h>
+
+    mailbox@f7510000 {
+        compatible = "hisilicon,hi6220-mbox";
+        reg = <0xf7510000 0x1000>,
+              <0x06dff800 0x0800>;
+        interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
+        #mbox-cells = <3>;
+    };
-- 
2.53.0


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

* Re: [PATCH] dt-bindings: mailbox: Convert hisilicon,hi6220-mbox to DT schema
  2026-10-01 23:05 [PATCH] dt-bindings: mailbox: Convert hisilicon,hi6220-mbox to DT schema Rob Herring (Arm)
@ 2026-10-08 12:12 ` Wei Xu
  2026-10-12  1:34 ` Jassi Brar
  1 sibling, 0 replies; 3+ messages in thread
From: Wei Xu @ 2026-10-08 12:12 UTC (permalink / raw)
  To: Rob Herring (Arm), Jassi Brar, Krzysztof Kozlowski, Conor Dooley
  Cc: linux-kernel, devicetree

Hi Rob,

On 2026/10/2 7:05, Rob Herring (Arm) wrote:
> Convert the hisilicon,hi6220-mbox binding to DT schema format. It's a
> straight forward conversion.
> 
> Assisted-by: LLM
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Acked-by: Wei Xu <xuwei5@hisilicon.com>

Thanks!

Best Regards,
Wei

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

* Re: [PATCH] dt-bindings: mailbox: Convert hisilicon,hi6220-mbox to DT schema
  2026-10-01 23:05 [PATCH] dt-bindings: mailbox: Convert hisilicon,hi6220-mbox to DT schema Rob Herring (Arm)
  2026-10-08 12:12 ` Wei Xu
@ 2026-10-12  1:34 ` Jassi Brar
  1 sibling, 0 replies; 3+ messages in thread
From: Jassi Brar @ 2026-10-12  1:34 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Wei Xu, linux-kernel,
	devicetree

On Thu, Oct 1, 2026 at 6:05 PM Rob Herring (Arm) <robh@kernel.org> wrote:
>
> Convert the hisilicon,hi6220-mbox binding to DT schema format. It's a
> straight forward conversion.
>
> Assisted-by: LLM
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../mailbox/hisilicon,hi6220-mailbox.txt      | 74 -------------------
>  .../mailbox/hisilicon,hi6220-mbox.yaml        | 63 ++++++++++++++++
>  2 files changed, 63 insertions(+), 74 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt
>  create mode 100644 Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml
>
> diff --git a/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt b/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt
> deleted file mode 100644
> index de9fbf722bee..000000000000
> --- a/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mailbox.txt
> +++ /dev/null
> @@ -1,74 +0,0 @@
> -Hisilicon Hi6220 Mailbox Driver
> -===============================
> -
> -Hisilicon Hi6220 mailbox supports up to 32 channels. Each channel
> -is unidirectional with a maximum message size of 8 words. I/O is
> -performed using register access (there is no DMA) and the cell
> -raises an interrupt when messages are received.
> -
> -Mailbox Device Node:
> -====================
> -
> -Required properties:
> ---------------------
> -- compatible:          Shall be "hisilicon,hi6220-mbox"
> -- reg:                 Contains the mailbox register address range (base
> -                       address and length); the first item is for IPC
> -                       registers, the second item is shared buffer for
> -                       slots.
> -- #mbox-cells:         Common mailbox binding property to identify the number
> -                       of cells required for the mailbox specifier. Must be 3.
> -                       <&phandle slot_id dst_irq ack_irq>
> -                         phandle: Label name of mailbox controller
> -                         slot_id: Slot id used either for TX or RX
> -                         dst_irq: IRQ identifier index number which used by MCU
> -                         ack_irq: IRQ identifier index number with generating a
> -                                  TX/RX interrupt to application processor,
> -                                  mailbox driver uses it to acknowledge interrupt
> -- interrupts:          Contains the interrupt information for the mailbox
> -                       device. The format is dependent on which interrupt
> -                       controller the SoCs use.
> -
> -Optional Properties:
> ---------------------
> -- hi6220,mbox-tx-noirq: Property of MCU firmware's feature, so mailbox driver
> -                       use this flag to ask MCU to enable "automatic idle
> -                       flag" mode or IRQ generated mode to acknowledge a TX
> -                       completion.
> -
> -Example:
> ---------
> -
> -       mailbox: mailbox@f7510000 {
> -               compatible = "hisilicon,hi6220-mbox";
> -               reg = <0x0 0xf7510000 0x0 0x1000>, /* IPC_S */
> -                     <0x0 0x06dff800 0x0 0x0800>; /* Mailbox */
> -               interrupt-parent = <&gic>;
> -               interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> -               #mbox-cells = <3>;
> -       };
> -
> -
> -Mailbox client
> -===============
> -
> -Required properties:
> ---------------------
> -- compatible:          Many (See the client docs).
> -- mboxes:              Standard property to specify a Mailbox
> -                       Cells must match 'mbox-cells' (See Mailbox Device Node above).
> -
> -Optional Properties:
> ---------------------
> -- mbox-names:          Name given to channels seen in the 'mboxes' property.
> -
> -Example:
> ---------
> -
> -       stub_clock: stub_clock {
> -               compatible = "hisilicon,hi6220-stub-clk";
> -               hisilicon,hi6220-clk-sram = <&sram>;
> -               #clock-cells = <1>;
> -               mbox-names = "mbox-tx", "mbox-rx";
> -               mboxes = <&mailbox 1 0 11>, <&mailbox 0 1 10>;
> -       };
> diff --git a/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml b/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml
> new file mode 100644
> index 000000000000..f4c8f1f29baa
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mailbox/hisilicon,hi6220-mbox.yaml
> @@ -0,0 +1,63 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/mailbox/hisilicon,hi6220-mbox.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Hisilicon Hi6220 Mailbox Controller
> +
> +maintainers:
> +  - Wei Xu <xuwei5@hisilicon.com>
> +
> +description:
> +  Hisilicon Hi6220 mailbox supports up to 32 channels. Each channel is
> +  unidirectional with a maximum message size of 8 words. I/O is performed using
> +  register access (there is no DMA) and the cell raises an interrupt when
> +  messages are received.
> +
> +properties:
> +  compatible:
> +    const: hisilicon,hi6220-mbox
> +
> +  reg:
> +    items:
> +      - description: IPC registers
> +      - description: Shared buffer for mailbox slots
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  "#mbox-cells":
> +    const: 3
> +    description: >
> +      The cells content are as follows:
> +        - Slot number
> +        - Remote destination interrupt vector
> +        - Local ack interrupt vector
> +
> +  hi6220,mbox-tx-noirq:
> +    description:
> +      Flag to request MCU to enable automatic idle flag mode instead of IRQ for
> +      TX completion.
> +    type: boolean
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +  - "#mbox-cells"
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +
> +    mailbox@f7510000 {
> +        compatible = "hisilicon,hi6220-mbox";
> +        reg = <0xf7510000 0x1000>,
> +              <0x06dff800 0x0800>;
> +        interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
> +        #mbox-cells = <3>;
> +    };
> --
> 2.53.0
>
Applied to mailbox/for-next

Thanks
Jassi

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

end of thread, other threads:[~2026-10-12  1:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-01 23:05 [PATCH] dt-bindings: mailbox: Convert hisilicon,hi6220-mbox to DT schema Rob Herring (Arm)
2026-10-08 12:12 ` Wei Xu
2026-10-12  1:34 ` Jassi Brar

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.