Re: [PATCH 2/2] rockchip: dts: Enable UART DMA by adding default dma-names property

["Heiko Stübner" <heiko@sntech.de>]

Am Mittwoch, 17. September 2025, 14:22:52 Mitteleuropäische Sommerzeit schrieb Dragan Simic:
> Hello Ed,
> 
> On 2025-09-17 13:49, Ed Wildgoose wrote:
> > Kernel appears to need a dma-names set for DMA to actually enable. Set 
> > a
> > default dma-names property for all UARTs defined in the base 
> > rk356x-base
> > dtsi
> > 
> > This is tested on a Radxa Zero 3W (which has 5x UARTs) and removes the
> > warnings and enables DMA on this platform
> 
> Thanks for the patches.
> 
> We should (still) stay away from defining the "dma-names" property
> at the SoC level, because doing that causes serious issues in certain
> cases.  Thus, I'd suggest that this patch is dropped, and that the
> "dma-names" property is defined instead at the board level, where
> it's needed and tested to work as expected.
> 
> Please see commit bf6f26deb0e8 (arm64: dts: rockchip: Add dma-names
> to uart1 on quartz64-b, 2024-06-28) for further explanation.
> 
> If/when the underlying issues are debugged and resolved, we can get
> back to defining the "dma-names" property at the SoC level.

And apart from the whole stability thing, on a number of socs the pl330
dma controller has more possible sources for dma operations than it
has available channels ... and right now the driver does not support
handling this.

So you can (and if I remember correctly we did) end up with some uart
requesting dma channels (and not needing them) and then sound not
getting any.


Heiko