On Fri, Oct 04, 2024 at 12:56:17PM +0200, Takashi Iwai wrote: > Sakari Ailus wrote: > > pm_runtime_put_autosuspend() will soon be changed to include a call to > > pm_runtime_mark_last_busy(). This patch switches the current users to > > __pm_runtime_put_autosuspend() which will continue to have the > > functionality of old pm_runtime_put_autosuspend(). > Many of those calls are with pm_runtime_mark_last_busy(). > (Actually the only one without it can be an oversight, it's better to > be with *_mark_last_busy(), too). > Wouldn't it be better to keep without converting and rather drop the > superfluous *_mark_last_busy()? Indeed, the same is true for the majority of cases in other patches. These changes seem counterproductive.
Attachment:
signature.asc
Description: PGP signature
- References:
- [PATCH 00/51] treewide: Switch to __pm_runtime_put_autosuspend()
- From: Sakari Ailus
- [PATCH 48/51] ALSA: hda: Switch to __pm_runtime_put_autosuspend()
- From: Sakari Ailus
- Re: [PATCH 48/51] ALSA: hda: Switch to __pm_runtime_put_autosuspend()
- From: Takashi Iwai
- [PATCH 00/51] treewide: Switch to __pm_runtime_put_autosuspend()
- Prev by Date: Re: [GIT PULL] sound fixes for 6.12-rc2
- Next by Date: Re: [PATCH] ASoC: sh: rz-ssi: Use SSIFCR_FIFO_RST macro
- Previous by thread: Re: [PATCH 48/51] ALSA: hda: Switch to __pm_runtime_put_autosuspend()
- Next by thread: [PATCH 50/51] ALSA: intel_hdmi: Switch to __pm_runtime_put_autosuspend()
- Index(es):