| Message ID | 20241004094123.113708-1-sakari.ailus@linux.intel.com |
|---|---|
| State | New |
| Headers | show |
| Series | treewide: Switch to __pm_runtime_put_autosuspend() | expand |
On Fri, Oct 04, 2024 at 12:41:23PM +0300, 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(). > > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> > --- > drivers/input/misc/cs40l50-vibra.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/input/misc/cs40l50-vibra.c b/drivers/input/misc/cs40l50-vibra.c > index 03bdb7c26ec0..8555f0d93d3f 100644 > --- a/drivers/input/misc/cs40l50-vibra.c > +++ b/drivers/input/misc/cs40l50-vibra.c > @@ -307,7 +307,7 @@ static void cs40l50_add_worker(struct work_struct *work) > } > err_pm: > pm_runtime_mark_last_busy(vib->dev); > - pm_runtime_put_autosuspend(vib->dev); > + __pm_runtime_put_autosuspend(vib->dev); Same comment as with omap4 keypad: prefer to simply remove pm_runtime_mark_last_busy() once pm_runtime_put_autosuspend() does this for us. Thanks.
diff --git a/drivers/input/misc/cs40l50-vibra.c b/drivers/input/misc/cs40l50-vibra.c index 03bdb7c26ec0..8555f0d93d3f 100644 --- a/drivers/input/misc/cs40l50-vibra.c +++ b/drivers/input/misc/cs40l50-vibra.c @@ -307,7 +307,7 @@ static void cs40l50_add_worker(struct work_struct *work) } err_pm: pm_runtime_mark_last_busy(vib->dev); - pm_runtime_put_autosuspend(vib->dev); + __pm_runtime_put_autosuspend(vib->dev); err_exit: work_data->error = error; } @@ -366,7 +366,7 @@ static void cs40l50_start_worker(struct work_struct *work) } pm_runtime_mark_last_busy(vib->dev); - pm_runtime_put_autosuspend(vib->dev); + __pm_runtime_put_autosuspend(vib->dev); err_free: kfree(work_data); } @@ -382,7 +382,7 @@ static void cs40l50_stop_worker(struct work_struct *work) vib->dsp.write(vib->dev, vib->regmap, vib->dsp.stop_cmd); pm_runtime_mark_last_busy(vib->dev); - pm_runtime_put_autosuspend(vib->dev); + __pm_runtime_put_autosuspend(vib->dev); kfree(work_data); } @@ -454,7 +454,7 @@ static void cs40l50_erase_worker(struct work_struct *work) kfree(erase_effect); err_pm: pm_runtime_mark_last_busy(vib->dev); - pm_runtime_put_autosuspend(vib->dev); + __pm_runtime_put_autosuspend(vib->dev); err_exit: work_data->error = error; }
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(). Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> --- drivers/input/misc/cs40l50-vibra.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)