diff options
| author | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2025-01-28 17:32:48 -0500 |
|---|---|---|
| committer | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2025-01-29 09:10:59 -0500 |
| commit | 55d4b69861e853ac987f6d593b44a3c18b468576 (patch) | |
| tree | 868142459187c80f957d7550e088dfd224dcb178 | |
| parent | 16016ade13f691da315fac7b23ebf1ab7b28b7ab (diff) | |
Revert "drm/xe/lnl: Enable GuC SLPC DCC task"
This reverts commit 50554bf3e56dd0c78ef1eedb685d0ab36c9c9987.
DCC in LNL should be disabled. It was a mistake to decide
to go against GuC platform defaults in this case and this
could lead to regressions in some TDP limited scenarios
instead of helping.
Cc: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
Cc: Jonathan Cavitt <jonathan.cavitt@intel.com>
Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250128223248.660748-1-rodrigo.vivi@intel.com
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
| -rw-r--r-- | drivers/gpu/drm/xe/xe_guc_pc.c | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/drivers/gpu/drm/xe/xe_guc_pc.c b/drivers/gpu/drm/xe/xe_guc_pc.c index 44b5211066ef..43f9617baba2 100644 --- a/drivers/gpu/drm/xe/xe_guc_pc.c +++ b/drivers/gpu/drm/xe/xe_guc_pc.c @@ -993,27 +993,6 @@ out: return ret; } -static int slpc_enable_dcc(struct xe_guc_pc *pc) -{ - int ret; - - ret = pc_action_set_param(pc, SLPC_PARAM_TASK_ENABLE_DCC, 1); - if (ret) - return ret; - - return pc_action_set_param(pc, SLPC_PARAM_TASK_DISABLE_DCC, 0); -} - -static int slpc_set_policies(struct xe_guc_pc *pc) -{ - struct xe_device *xe = pc_to_xe(pc); - - if (xe->info.platform == XE_LUNARLAKE) - return slpc_enable_dcc(pc); - - return 0; -} - /** * xe_guc_pc_start - Start GuC's Power Conservation component * @pc: Xe_GuC_PC instance @@ -1058,10 +1037,6 @@ int xe_guc_pc_start(struct xe_guc_pc *pc) goto out; } - ret = slpc_set_policies(pc); - if (ret) - goto out; - ret = pc_init_freqs(pc); if (ret) goto out; |