Re: [PATCH 1/1] accel/ivpu: Remove redundant pm_runtime_mark_last_busy() calls

[Falkowski, Maciej]

Reviewed-by: Maciej Falkowski <maciej.falkow...@linux.intel.com>

You didn't mentioned it is resend so I was a bit surprised the patch has
rev-by already from Jacek but I confirm it was given in the old thread.

Thanks,
Maciej

On 10/27/2025 2:39 PM, 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.ai...@linux.intel.com>
Reviewed-by: Jacek Lawrynowicz <jacek.lawrynow...@linux.intel.com>
---
  drivers/accel/ivpu/ivpu_job.c | 1 -
  drivers/accel/ivpu/ivpu_pm.c  | 2 --
  2 files changed, 3 deletions(-)

diff --git a/drivers/accel/ivpu/ivpu_job.c b/drivers/accel/ivpu/ivpu_job.c
index ba4535a75aa7..4eb7287cd083 100644
--- a/drivers/accel/ivpu/ivpu_job.c
+++ b/drivers/accel/ivpu/ivpu_job.c
@@ -1114,6 +1114,5 @@ void ivpu_context_abort_work_fn(struct work_struct *work)
        mutex_unlock(&vdev->submitted_jobs_lock);

runtime_put:

-       pm_runtime_mark_last_busy(vdev->drm.dev);
        pm_runtime_put_autosuspend(vdev->drm.dev);
  }
diff --git a/drivers/accel/ivpu/ivpu_pm.c b/drivers/accel/ivpu/ivpu_pm.c
index 63c95307faa1..fe6f93af6ab0 100644
--- a/drivers/accel/ivpu/ivpu_pm.c
+++ b/drivers/accel/ivpu/ivpu_pm.c
@@ -359,7 +359,6 @@ int ivpu_rpm_get(struct ivpu_device *vdev)

void ivpu_rpm_put(struct ivpu_device *vdev)

  {
-       pm_runtime_mark_last_busy(vdev->drm.dev);
        pm_runtime_put_autosuspend(vdev->drm.dev);
  }

@@ -428,7 +427,6 @@ void ivpu_pm_enable(struct ivpu_device *vdev)

        struct device *dev = vdev->drm.dev;

pm_runtime_allow(dev);

-       pm_runtime_mark_last_busy(dev);
        pm_runtime_put_autosuspend(dev);
  }