Re: [PATCH v2 1/7] dt-bindings: mailbox: add Allwinner sun55i msgbox schema

[Krzysztof Kozlowski]

On Sat, Sep 26, 2026 at 07:20:10PM -0500, Tim Michals wrote:
> Add Device Tree binding schema for the Allwinner 4-port hardware
> Message Box controller found on sun55i (A523, A527, T527) and
> sun60i (A733) SoCs.
> 
> The message box connects the ARM Cortex-A55 host cluster to the
> HiFi4 Audio DSP, Power Management Unit (CPUS), and XuanTie RISC-V
> co-processor across 12 logical channels with 8-entry hardware FIFOs.
> 
> Signed-off-by: Tim Michals <tcmichals@xxxxxxxxx>
> ---
>  .../mailbox/allwinner,sun55i-a523-msgbox.yaml | 102 ++++++++++++++++++
>  1 file changed, 102 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mailbox/allwinner,sun55i-a523-msgbox.yaml
>

Do not attach (thread) your patchsets to some other threads (unrelated
or older versions). This buries them deep in the mailbox and might
interfere with applying entire sets. See also:
https://elixir.bootlin.com/linux/v6.16-rc2/source/Documentation/process/submitting-patches.rst#L830

...

> +  interrupts:
> +    minItems: 1
> +    maxItems: 4
> +    description:
> +      One interrupt per processor port in port order (arm, dsp, cpus, rv).
> +      The ARM host port interrupt is required; remote port interrupts are
> +      optional. Use interrupt-names to identify which ports are present when
> +      providing a partial list.

No, this has to be constrained/specific. See writing bindings and any
existing examples. Also, just list the interrupts with minItems, instead
of free form text.


Best regards,
Krzysztof