Re: [PATCH 5/7] dmaengine: sh: rz-dmac: Add suspend to RAM support

[Claudiu Beznea <claudiu.beznea@tuxon.dev>]

Hi, Biju,

On 1/26/26 17:28, Biju Das wrote:
> Hi All,
> 
>> -----Original Message-----
>> From: Biju Das
>> Sent: 26 January 2026 13:12
>> Subject: RE: [PATCH 5/7] dmaengine: sh: rz-dmac: Add suspend to RAM support
>>

[ ... ]

>>
>> For s2idle issue on RZ/G3L is DMA device is in asserted state, not forwarding any IRQ to cpu for
>> wakeup.
>>
>> For S2RAM issue on RZ/G3L is during suspend hardware turns DMAACLK off/ Asserted state. Clock framwork
>> is not turning On DMAACLK as it critical clk.
>>
>> Can you please check your TF-A for the second case? First case, RZ/G3S may ok for reset assert state,
>> it can forward IRQs to CPU.
> 
> Just to summarize, currently there are 2 differences identified between RZ/G3S and RZ/G3L:
> 
> SoC differences for s2idle:
> 
> RZ/G3S: Can wake the system if the DMA device is in the assert state
> 
> RZ/G3L: Cannot wake the system if the DMA device is in the assert state.
> 
> 
> TF-A differences for s2ram:
> 
> RZ/G3S: TF_A turns on DMA_ACLK during boot/resume.
> 
> RZ/G3L: TF_A does not handle DMA_ACLK during boot/resume.

I'm seeing at [1] you are addressing these differences in the clock/reset 
drivers. With that, are you still considering this patch is breaking your system?

Thank you,
Claudiu

[1] https://lore.kernel.org/all/20260130143456.256813-1-biju.das.jz@bp.renesas.com