On 2/21/23 16:05, Mukunda,Vijendar wrote: > On 21/02/23 21:29, Pierre-Louis Bossart wrote: >> >> >>> +static const struct snd_soc_dai_ops amd_sdw_dai_ops = { >>> + .hw_params = amd_sdw_hw_params, >> don't you need a .prepare to deal with suspend-resume? > DSP based solution is not implemented yet. > As per our understanding In .prepare () callback, there is > no extra logic needs to be implemented. Because of it, > we have dropped .prepare() callback. You don't need any logic IF the IP keeps its context while suspended. That wasn't the case for us, we had to re-program the registers when resuming.