[PATCH] arm64: dts: exynosautov920: add DMA nodes

[Faraz Ata]

Hi Tudor

> -----Original Message-----
> From: Tudor Ambarus <tudor.ambarus at linaro.org>
> Sent: Monday, December 9, 2024 1:13 PM
> To: Faraz Ata <faraz.ata at samsung.com>; devicetree at vger.kernel.org; linux-
> samsung-soc at vger.kernel.org; krzk+dt at kernel.org; robh at kernel.org
> Cc: linux-arm-kernel at lists.infradead.org; alim.akhtar at samsung.com;
> rosa.pila at samsung.com; Peter Griffin <peter.griffin at linaro.org>; André
> Draszik <andre.draszik at linaro.org>; William McVicker
> <willmcvicker at google.com>; kernel-team at android.com
> Subject: Re: [PATCH] arm64: dts: exynosautov920: add DMA nodes
> 
> 
> 
> On 12/4/24 12:23 PM, Faraz Ata wrote:
> > ExynosAutov920 SoC has 7 DMA controllers. Two secure DMAC
> > (SPDMA0 & SPDMA1) and five non-secure DMAC (PDMA0 to PDMA4).
> > Adds the required dt node for the same.
> 
> typos: s/adds/add, s/node/nodes
> 
Thanks for your review, will correct in the v2

> Hi, Faraz!
> 
> Have you tested DMA on ExynosAutov920, did it work with the upstream
> driver code?
> 
Yes, I tested with dmatest for M2M for all channel, it is working as expected with mainline PL330 driver. 

> I tried enabling it for GS101 and I had some questions that I couldn't answer,
> and I moved forward in the meantime. Would you mind taking a look at:
> 
> https://lore.kernel.org/linux-arm-kernel/e4089855-aca9-4993-bac6-
> e59370cf7954 at linaro.org/
> 
I am not sure about GS101 architecture, will try to reachout internally to people who might be aware of this.

> Thanks a ton!
> ta