Re: [PATCH] dt-bindings: mailbox: Delete old 'mailbox.txt' bindings file

[email protected]
Newsgroups org.kernel.vger.linux-devicetree
Message-ID <[email protected]>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Deleting `mailbox.txt` without updating references to it in other binding files leaves dangling, broken documentation references.
--

--- Patch [1]: [PATCH] dt-bindings: mailbox: Delete old 'mailbox.txt' bindings file ---
commit 1b70b3f250d6bcf306aca223e0e91c9fb8a20cc2
Author: Douglas Anderson <[email protected]>

dt-bindings: mailbox: Delete old 'mailbox.txt' bindings file

This commit deletes the old mailbox.txt device-tree bindings file. The
documentation contained in the file has been moved to the dt-schema
repository.

Link: https://lore.kernel.org/r/[email protected] [1]
Link: https://lore.kernel.org/r/[email protected] [2]
Link: https://lore.kernel.org/r/CAL_JsqKVjQ2m-dzMUENQry-f_YE9QrYSOKoH9CPj1gjj8XkPRA@mail.gmail.com [3]
Link: https://github.com/devicetree-org/dt-schema/pull/202 [4]

> diff --git a/Documentation/devicetree/bindings/mailbox/mailbox.txt b/Documentation/devicetree/bindings/mailbox/mailbox.txt
> deleted file mode 100644
> index af8ecee2ac687..0000000000000
> --- a/Documentation/devicetree/bindings/mailbox/mailbox.txt
> +++ /dev/null
> @@ -1,60 +0,0 @@
> -* Generic Mailbox Controller and client driver bindings

[Severity: Low]
Does deleting this file leave dangling documentation links in other bindings
that still reference it?

For example, there are explicit references to mailbox.txt in:
- Documentation/devicetree/bindings/mailbox/ti,omap-mailbox.yaml
- Documentation/devicetree/bindings/power/reset/xlnx,zynqmp-power.yaml
- Documentation/devicetree/bindings/soc/qcom/qcom,aoss-qmp.yaml
- Documentation/devicetree/bindings/firmware/nvidia,tegra186-bpmp.yaml
- Documentation/devicetree/bindings/mailbox/allwinner,sun6i-a31-msgbox.yaml

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260804133801.1.I8b3ff71c528133e7d6f81fe926fa39e6451d63fd@changeid?part=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.