RE: [PATCH 10/22] ASoC: rsnd: Add DMA support infrastructure for RZ/G3E

[John Madieu]

Hi Kuninori,

> -----Original Message-----
> From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> Sent: Monday, March 23, 2026 2:19 AM
> To: John Madieu <john.madieu.xa@xxxxxxxxxxxxxx>
> Subject: Re: [PATCH 10/22] ASoC: rsnd: Add DMA support infrastructure for
> RZ/G3E
> 
> 
> Hi John
> 
> > RZ/G3E has different DMA register base addresses and offset
> > calculations compared to R-Car platforms, and requires additional
> > audmac-pp clock and reset lines for Audio DMAC operation.
> >
> > Add RZ/G3E-specific DMA address macros and audmac-pp clock/reset
> > support using optional APIs to remain transparent to other platforms.
> >
> > Signed-off-by: John Madieu <john.madieu.xa@xxxxxxxxxxxxxx>
> > ---
> (snip)
> > -	ret = rsnd_mod_init(priv, *dma_mod, ops, NULL, NULL,
> > -			    type, dma_id);
> > +	/*
> > +	 * Pass NULL for clock/reset - audmac_pp is managed globally in
> > +	 * rsnd_dma_probe() and core.c suspend/resume, not per-DMA-module.
> > +	 * See detailed explanation in rsnd_dma_probe().
> > +	 */
> > +	ret = rsnd_mod_init(priv, *dma_mod, ops, NULL, NULL, type, dma_id);
> >  	if (ret < 0)
> >  		return ret;
> 
> This patch change rsnd_mod_init() parameters independently.
> Is this patch-set can keep compile comapatible ??
> 

The patch changing rsnd_mod_init()'s parameters is 08/22, and while
doing it, it updates the call from other module code. So yes, every
single patch in the series is compile-compatible (assuming you meant
compile-tested ?)

Regards,
John

> Thank you for your help !!
> 
> Best regards
> ---
> Kuninori Morimoto