On 6/16/2026 8:12 PM, Mitul Golani wrote:
CMRR is subset of fix refresh rate, it is exclusive from
Adaptive mode, it does not need to set a separate mode flag during
state checker. Remove the pre-added TODO for the same.
Remove it at the same patch that actually renders the comment redundant.
Signed-off-by: Mitul Golani <[email protected]>
---
drivers/gpu/drm/i915/display/intel_vrr.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c
b/drivers/gpu/drm/i915/display/intel_vrr.c
index b2c5b148c487..92ded2e68b01 100644
--- a/drivers/gpu/drm/i915/display/intel_vrr.c
+++ b/drivers/gpu/drm/i915/display/intel_vrr.c
@@ -1202,11 +1202,6 @@ void intel_vrr_get_config(struct intel_crtc_state
*crtc_state)
intel_vrr_get_dc_balance_config(crtc_state);
- /*
- * #TODO: For Both VRR and CMRR the flag I915_MODE_FLAG_VRR is set for
mode_flags.
- * Since CMRR is currently disabled, set this flag for VRR for now.
- * Need to keep this in mind while re-enabling CMRR.
- */
if (crtc_state->vrr.enable)
crtc_state->mode_flags |= I915_MODE_FLAG_VRR;
