On 10/21/2025 12:20 AM, Ville Syrjala wrote:
From: Ville Syrjälä <[email protected]>

Now that we always populate crtc_state->vrr.guardband even on
ICL/TGL intel_vrr_vblank_exit_length() has become rather pointless.
Get rid of it.

Signed-off-by: Ville Syrjälä <[email protected]>

Reviewed-by: Ankit Nautiyal <[email protected]>


---
  drivers/gpu/drm/i915/display/intel_vrr.c | 8 ++------
  1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c 
b/drivers/gpu/drm/i915/display/intel_vrr.c
index 71c5d8bf7557..ba92e0a76855 100644
--- a/drivers/gpu/drm/i915/display/intel_vrr.c
+++ b/drivers/gpu/drm/i915/display/intel_vrr.c
@@ -143,10 +143,6 @@ static int intel_vrr_pipeline_full_to_guardband(const 
struct intel_crtc_state *c
   *
   * framestart_delay is programmable 1-4.
   */
-static int intel_vrr_vblank_exit_length(const struct intel_crtc_state 
*crtc_state)
-{
-       return crtc_state->vrr.guardband;
-}
int intel_vrr_vmin_vtotal(const struct intel_crtc_state *crtc_state)
  {
@@ -161,12 +157,12 @@ int intel_vrr_vmax_vtotal(const struct intel_crtc_state 
*crtc_state)
int intel_vrr_vmin_vblank_start(const struct intel_crtc_state *crtc_state)
  {
-       return intel_vrr_vmin_vtotal(crtc_state) - 
intel_vrr_vblank_exit_length(crtc_state);
+       return intel_vrr_vmin_vtotal(crtc_state) - crtc_state->vrr.guardband;
  }
int intel_vrr_vmax_vblank_start(const struct intel_crtc_state *crtc_state)
  {
-       return intel_vrr_vmax_vtotal(crtc_state) - 
intel_vrr_vblank_exit_length(crtc_state);
+       return intel_vrr_vmax_vtotal(crtc_state) - crtc_state->vrr.guardband;
  }
static bool

Reply via email to