From: Krystian Pradzynski <krystian.pradzyn...@linux.intel.com>

Configure autosuspend values per HW generation and per platform.

For non silicon platforms disable autosuspend for now, for silicon
reduce it to 10 ms.

Signed-off-by: Krystian Pradzynski <krystian.pradzyn...@linux.intel.com>
Reviewed-by: Stanislaw Gruszka <stanislaw.grus...@linux.intel.com>
Reviewed-by: Jeffrey Hugo <quic_jh...@quicinc.com>
Signed-off-by: Stanislaw Gruszka <stanislaw.grus...@linux.intel.com>
---
 drivers/accel/ivpu/ivpu_drv.h     |  1 +
 drivers/accel/ivpu/ivpu_hw_37xx.c |  2 ++
 drivers/accel/ivpu/ivpu_hw_40xx.c |  3 +++
 drivers/accel/ivpu/ivpu_pm.c      | 14 ++++++++------
 4 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/drivers/accel/ivpu/ivpu_drv.h b/drivers/accel/ivpu/ivpu_drv.h
index 9e8c075fe9ef..44d857094bbc 100644
--- a/drivers/accel/ivpu/ivpu_drv.h
+++ b/drivers/accel/ivpu/ivpu_drv.h
@@ -117,6 +117,7 @@ struct ivpu_device {
                int jsm;
                int tdr;
                int reschedule_suspend;
+               int autosuspend;
        } timeout;
 };
 
diff --git a/drivers/accel/ivpu/ivpu_hw_37xx.c 
b/drivers/accel/ivpu/ivpu_hw_37xx.c
index 9eae1c241bc0..1090c91e1ba3 100644
--- a/drivers/accel/ivpu/ivpu_hw_37xx.c
+++ b/drivers/accel/ivpu/ivpu_hw_37xx.c
@@ -113,11 +113,13 @@ static void ivpu_hw_timeouts_init(struct ivpu_device 
*vdev)
                vdev->timeout.jsm = 50000;
                vdev->timeout.tdr = 2000000;
                vdev->timeout.reschedule_suspend = 1000;
+               vdev->timeout.autosuspend = -1;
        } else {
                vdev->timeout.boot = 1000;
                vdev->timeout.jsm = 500;
                vdev->timeout.tdr = 2000;
                vdev->timeout.reschedule_suspend = 10;
+               vdev->timeout.autosuspend = 10;
        }
 }
 
diff --git a/drivers/accel/ivpu/ivpu_hw_40xx.c 
b/drivers/accel/ivpu/ivpu_hw_40xx.c
index 34626d66fa10..09ba07443396 100644
--- a/drivers/accel/ivpu/ivpu_hw_40xx.c
+++ b/drivers/accel/ivpu/ivpu_hw_40xx.c
@@ -135,16 +135,19 @@ static void ivpu_hw_timeouts_init(struct ivpu_device 
*vdev)
                vdev->timeout.jsm = 50000;
                vdev->timeout.tdr = 2000000;
                vdev->timeout.reschedule_suspend = 1000;
+               vdev->timeout.autosuspend = -1;
        } else if (ivpu_is_simics(vdev)) {
                vdev->timeout.boot = 50;
                vdev->timeout.jsm = 500;
                vdev->timeout.tdr = 10000;
                vdev->timeout.reschedule_suspend = 10;
+               vdev->timeout.autosuspend = -1;
        } else {
                vdev->timeout.boot = 1000;
                vdev->timeout.jsm = 500;
                vdev->timeout.tdr = 2000;
                vdev->timeout.reschedule_suspend = 10;
+               vdev->timeout.autosuspend = 10;
        }
 }
 
diff --git a/drivers/accel/ivpu/ivpu_pm.c b/drivers/accel/ivpu/ivpu_pm.c
index e6f27daf5560..954c9ecd3e14 100644
--- a/drivers/accel/ivpu/ivpu_pm.c
+++ b/drivers/accel/ivpu/ivpu_pm.c
@@ -286,6 +286,7 @@ int ivpu_pm_init(struct ivpu_device *vdev)
 {
        struct device *dev = vdev->drm.dev;
        struct ivpu_pm_info *pm = vdev->pm;
+       int delay;
 
        pm->vdev = vdev;
        pm->suspend_reschedule_counter = PM_RESCHEDULE_LIMIT;
@@ -293,14 +294,15 @@ int ivpu_pm_init(struct ivpu_device *vdev)
        atomic_set(&pm->in_reset, 0);
        INIT_WORK(&pm->recovery_work, ivpu_pm_recovery_work);
 
-       pm_runtime_use_autosuspend(dev);
-
        if (ivpu_disable_recovery)
-               pm_runtime_set_autosuspend_delay(dev, -1);
-       else if (ivpu_is_silicon(vdev))
-               pm_runtime_set_autosuspend_delay(dev, 100);
+               delay = -1;
        else
-               pm_runtime_set_autosuspend_delay(dev, 60000);
+               delay = vdev->timeout.autosuspend;
+
+       pm_runtime_use_autosuspend(dev);
+       pm_runtime_set_autosuspend_delay(dev, delay);
+
+       ivpu_dbg(vdev, PM, "Autosuspend delay = %d\n", delay);
 
        return 0;
 }
-- 
2.25.1

Reply via email to