On Fri, 04 Jul 2025 09:55:00 +0200,
Sakari Ailus wrote:
>
> pm_runtime_put_autosuspend(), pm_runtime_put_sync_autosuspend(),
> pm_runtime_autosuspend() and pm_request_autosuspend() now include a call
> to pm_runtime_mark_last_busy(). Remove the now-reduntant explicit call to
> pm_runtime_mark_last_busy().
>
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Takashi Iwai <tiwai@suse.de>
thanks,
Takashi
> ---
> The cover letter of the set can be found here
> <URL:https://lore.kernel.org/linux-pm/20250704075225.3212486-1-sakari.ailus@linux.intel.com>.
>
> In brief, this patch depends on PM runtime patches adding marking the last
> busy timestamp in autosuspend related functions. The patches are here, on
> rc2:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git \
> pm-runtime-6.17-rc1
>
> sound/x86/intel_hdmi_audio.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/sound/x86/intel_hdmi_audio.c b/sound/x86/intel_hdmi_audio.c
> index fe5cb4139088..cc54539c6030 100644
> --- a/sound/x86/intel_hdmi_audio.c
> +++ b/sound/x86/intel_hdmi_audio.c
> @@ -1102,7 +1102,6 @@ static int had_pcm_open(struct snd_pcm_substream *substream)
>
> return retval;
> error:
> - pm_runtime_mark_last_busy(intelhaddata->dev);
> pm_runtime_put_autosuspend(intelhaddata->dev);
> return retval;
> }
> @@ -1127,7 +1126,6 @@ static int had_pcm_close(struct snd_pcm_substream *substream)
> }
> spin_unlock_irq(&intelhaddata->had_spinlock);
>
> - pm_runtime_mark_last_busy(intelhaddata->dev);
> pm_runtime_put_autosuspend(intelhaddata->dev);
> return 0;
> }
> @@ -1589,7 +1587,6 @@ static void had_audio_wq(struct work_struct *work)
> }
>
> mutex_unlock(&ctx->mutex);
> - pm_runtime_mark_last_busy(ctx->dev);
> pm_runtime_put_autosuspend(ctx->dev);
> }
>
> --
> 2.39.5
>