On Mon, Oct 13, 2025 at 06:05:53PM +0530, Ankit Nautiyal wrote: > Move intel_dpll_crtc_compute_clock in the beginning of the function so that > clocks are set before other things. > > This will help in subsequent changes when the vrr guardband computation > is moved to intel_crtc_compute_config(). > > Signed-off-by: Ankit Nautiyal <[email protected]>
Reviewed-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_display.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c > b/drivers/gpu/drm/i915/display/intel_display.c > index d5b2612d4ec2..3f725553599e 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -2410,11 +2410,11 @@ static int intel_crtc_compute_config(struct > intel_atomic_state *state, > intel_atomic_get_new_crtc_state(state, crtc); > int ret; > > - ret = intel_crtc_compute_set_context_latency(state, crtc); > + ret = intel_dpll_crtc_compute_clock(state, crtc); > if (ret) > return ret; > > - ret = intel_dpll_crtc_compute_clock(state, crtc); > + ret = intel_crtc_compute_set_context_latency(state, crtc); > if (ret) > return ret; > > -- > 2.45.2 -- Ville Syrjälä Intel
