Re: [PATCH] ASoC: ti: davinci-mcasp: Add system suspend/resume support

[Moteen Shah]

On 12/02/26 03:40, Sen Wang wrote:
> The McASP driver supports runtime PM callbacks for register save/restore
> during device idle, but doesn't provide system suspend/resume callbacks.
> This causes audio to fail to resume after system suspend.
>
> Since the driver already handles runtime suspend & resume, we can reuse
> existing runtime PM logics.
>
> Signed-off-by: Sen Wang <sen@xxxxxx>
> ---
> Tested on SK-AM62Px with audio playback during system suspend/resume cycles
>
> Testlog:
>    https://gist.github.com/SenWang125/065c2840071f2569a79db9e99010603f
>
>   sound/soc/ti/davinci-mcasp.c | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/sound/soc/ti/davinci-mcasp.c b/sound/soc/ti/davinci-mcasp.c
> index 8a2977093df1..643661ebee83 100644
> --- a/sound/soc/ti/davinci-mcasp.c
> +++ b/sound/soc/ti/davinci-mcasp.c
> @@ -2811,6 +2811,8 @@ static int davinci_mcasp_runtime_resume(struct device *dev)
>   #endif
>   
>   static const struct dev_pm_ops davinci_mcasp_pm_ops = {
> +	SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
> +				pm_runtime_force_resume)
>   	SET_RUNTIME_PM_OPS(davinci_mcasp_runtime_suspend,
>   			   davinci_mcasp_runtime_resume,
>   			   NULL)

Tested on j784s4.
Logs: https://gist.github.com/Jamm02/8ed69ad1c9b3a4f0cda4977cc18e01b8

Tested-by: Moteen Shah <m-shah@xxxxxx>

Regards,
Moteen