CMTG provides the clock required in DC3CO state. Gate the DC3CO enable on successful CMTG enable.
Signed-off-by: Dibin Moolakadan Subrahmanian <[email protected]> --- drivers/gpu/drm/i915/display/intel_display.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c index 9b0191d02e20..87c318e52890 100644 --- a/drivers/gpu/drm/i915/display/intel_display.c +++ b/drivers/gpu/drm/i915/display/intel_display.c @@ -7512,6 +7512,9 @@ static void intel_atomic_commit_tail(struct intel_atomic_state *state) struct ref_tracker *wakeref = NULL; int i; u32 target_dc_state; + bool cmtg_status = false; + bool dc3co_allowed = intel_display_power_dc3co_allowed(display); + /* * Delay re-enabling DC states by 17 ms to avoid the off->on->off * toggling overhead at and above 60 FPS. @@ -7629,7 +7632,8 @@ static void intel_atomic_commit_tail(struct intel_atomic_state *state) /* FIXME probably need to sequence this properly */ intel_program_dpkgc_latency(state); - intel_cmtg_program(state); + if (dc3co_allowed) + cmtg_status = intel_cmtg_program(state); intel_wait_for_vblank_workers(state); @@ -7727,7 +7731,7 @@ static void intel_atomic_commit_tail(struct intel_atomic_state *state) } if (intel_display_power_dc3co_supported(display)) { - if (intel_display_power_dc3co_allowed(display)) { + if (dc3co_allowed && cmtg_status) { /* * Use minimal re-enable delay to allow DC3CO entry on * the next idle frame, unlike the 17ms guard needed to -- 2.43.0
