On 20/06/2024 19:56, Piotr Wojtaszczyk wrote:
LPC32XX SoCs use pl080 dma controller which have few request signals multiplexed between peripherals. This binding describes how devices can use the multiplexed request signals.
Signed-off-by: Piotr Wojtaszczyk piotr.wojtaszczyk@timesys.com
+properties:
- "#dma-cells":
- const: 3
- description: |
First two cells same as for device pointed in dma-masters.Third cell represents mux value for the request.- compatible:
- const: nxp,lpc3220-dmamux
Please put compatible first in the list of properties (and follow the same order in "required"). It's the most important piece, so we want it to be the first to see. It also follows the convention of DTS, where compatible is expected to be first.
- dma-masters:
- description: phandle to a dma node compatible with arm,pl080
maxItems: 1
- reg:
- maxItems: 1
Keep reg after compatible.
+required:
- compatible
- reg
- dma-masters
+additionalProperties: false
+examples:
- |
- syscon@40004000 {
compatible = "nxp,lpc3220-creg", "syscon", "simple-mfd";reg = <0x40004000 0x114>;ranges = <0 0x40004000 0x114>;#address-cells = <1>;#size-cells = <1>;
Drop the node above (you will see Rob's warning). Alternatively, this schema could skip the example and the nxp,lpc3220-creg could have one complete example for entire device with children.
dma-router@7c {compatible = "nxp,lpc3220-dmamux";reg = <0x7c 0x8>;#dma-cells = <3>;dma-masters = <&dma>;};- };
+... diff --git a/MAINTAINERS b/MAINTAINERS index aacccb376c28..f7adf9f66dfa 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2396,6 +2396,15 @@ F: drivers/usb/host/ohci-nxp.c F: drivers/watchdog/pnx4008_wdt.c N: lpc32xx
+ARM/LPC32XX DMAMUX SUPPORT
This should be just "LPC32XX DMAMUX SUPPORT"
+M: J.M.B. Downing jonathan.downing@nautel.com +M: Piotr Wojtaszczyk piotr.wojtaszczyk@timesys.com +R: Vladimir Zapolskiy vz@mleia.com +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) +S: Maintained +F: Documentation/devicetree/bindings/dma/nxp,lpc3220-dmamux.yaml +N: lpc32xx
I think this entry is only foor DMAMUX so the last "N:" is not appropriate.
You are welcomed to help maintaining the platform. Add yourself to appropriate place of LPC32xx ARM.
Best regards, Krzysztof