diff options
| author |   Vinay Belgaumkar <vinay.belgaumkar@intel.com> | 2021-11-11 23:10:16 -0800 |
|---|---|---|
| committer |   John Harrison <John.C.Harrison@Intel.com> | 2021-11-12 10:19:48 -0800 |
| commit | 5f1176b419f9468f05dabdc8352bfa9b15427377 (patch) | |
| tree | 2cd5f6e3427806ac6d36ba8bef58cee61d7aea30 /drivers/gpu/drm | |
| parent | drm/i915/dg2: Program recommended HW settings (diff) | |
| download | linux-dev-5f1176b419f9468f05dabdc8352bfa9b15427377.tar.xz linux-dev-5f1176b419f9468f05dabdc8352bfa9b15427377.zip | |
drm/i915/guc/slpc: Check GuC status before freq boost
It's possible that i915 might get wedged between a boost
and un-boost. Validate the i915-GuC connection before trying
to send a H2G to change the min frequency.
Bug: https://gitlab.freedesktop.org/drm/intel/-/issues/4464
Cc: Ashutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20211112071016.9640-1-vinay.belgaumkar@intel.com
Diffstat (limited to 'drivers/gpu/drm')
| -rw-r--r-- | drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c index 4e1d3cd29164..22c1c12369f2 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c @@ -183,11 +183,15 @@ static int slpc_unset_param(struct intel_guc_slpc *slpc, static int slpc_force_min_freq(struct intel_guc_slpc *slpc, u32 freq) { struct drm_i915_private *i915 = slpc_to_i915(slpc); + struct intel_guc *guc = slpc_to_guc(slpc); intel_wakeref_t wakeref; int ret = 0; lockdep_assert_held(&slpc->lock); + if (!intel_guc_is_ready(guc)) + return -ENODEV; + /* * This function is a little different as compared to * intel_guc_slpc_set_min_freq(). Softlimit will not be updated |