Hi Tudor > -----Original Message----- > From: Tudor Ambarus <tudor.ambarus@xxxxxxxxxx> > Sent: Monday, December 9, 2024 1:13 PM > To: Faraz Ata <faraz.ata@xxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; linux- > samsung-soc@xxxxxxxxxxxxxxx; krzk+dt@xxxxxxxxxx; robh@xxxxxxxxxx > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; alim.akhtar@xxxxxxxxxxx; > rosa.pila@xxxxxxxxxxx; Peter Griffin <peter.griffin@xxxxxxxxxx>; André > Draszik <andre.draszik@xxxxxxxxxx>; William McVicker > <willmcvicker@xxxxxxxxxx>; kernel-team@xxxxxxxxxxx > 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@xxxxxxxxxx/ > I am not sure about GS101 architecture, will try to reachout internally to people who might be aware of this. > Thanks a ton! > ta
