summaryrefslogtreecommitdiff
path: root/drivers/gpu/drm/msm/msm_gpu.c
diff options
context:
space:
mode:
authorAkhil P Oommen <quic_akhilpo@quicinc.com>2022-08-18 23:22:12 +0300
committerRob Clark <robdclark@chromium.org>2022-08-28 19:29:27 +0300
commitf350bfb92b5a89f8056b0e218e05522665e8ac9a (patch)
treec998b8d927c7a94d2c9e63ddedf3a86b7d4d5525 /drivers/gpu/drm/msm/msm_gpu.c
parent06097e372accb843db7174b337367c5a3f97af52 (diff)
downloadlinux-f350bfb92b5a89f8056b0e218e05522665e8ac9a.tar.xz
drm/msm: Fix cx collapse issue during recovery
There are some hardware logic under CX domain. For a successful recovery, we should ensure cx headswitch collapses to ensure all the stale states are cleard out. This is especially true to for a6xx family where we can GMU co-processor. Currently, cx doesn't collapse due to a devlink between gpu and its smmu. So the *struct gpu device* needs to be runtime suspended to ensure that the iommu driver removes its vote on cx gdsc. Signed-off-by: Akhil P Oommen <quic_akhilpo@quicinc.com> Patchwork: https://patchwork.freedesktop.org/patch/498398/ Link: https://lore.kernel.org/r/20220819015030.v5.4.I4ac27a0b34ea796ce0f938bb509e257516bc6f57@changeid Signed-off-by: Rob Clark <robdclark@chromium.org>
Diffstat (limited to 'drivers/gpu/drm/msm/msm_gpu.c')
-rw-r--r--drivers/gpu/drm/msm/msm_gpu.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c
index da6242cbf87d..9ec9a99ffe77 100644
--- a/drivers/gpu/drm/msm/msm_gpu.c
+++ b/drivers/gpu/drm/msm/msm_gpu.c
@@ -422,9 +422,7 @@ static void recover_worker(struct kthread_work *work)
/* retire completed submits, plus the one that hung: */
retire_submits(gpu);
- pm_runtime_get_sync(&gpu->pdev->dev);
gpu->funcs->recover(gpu);
- pm_runtime_put_sync(&gpu->pdev->dev);
/*
* Replay all remaining submits starting with highest priority
@@ -441,7 +439,7 @@ static void recover_worker(struct kthread_work *work)
}
}
- pm_runtime_put_sync(&gpu->pdev->dev);
+ pm_runtime_put(&gpu->pdev->dev);
mutex_unlock(&gpu->lock);