> -----Original Message----- > From: Intel-gfx <[email protected]> On Behalf Of Jouni > Högander > Sent: Thursday, January 9, 2025 1:01 PM > To: [email protected]; [email protected] > Cc: Hogander, Jouni <[email protected]> > Subject: [PATCH v3 02/10] drm/i915/psr: Rename > psr_force_hw_tracking_exit as intel_psr_force_update > > psr_force_hw_tracking_exit is misleading name as it is used for PSR1, PSR2 > HW tracking and PSR2 selective fetch. Due to this rename it as > intel_psr_force_update. > > Signed-off-by: Jouni Högander <[email protected]>
LGTM. Reviewed-by: Animesh Manna <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_psr.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_psr.c > b/drivers/gpu/drm/i915/display/intel_psr.c > index 476305010e11..5411d0d6f362 100644 > --- a/drivers/gpu/drm/i915/display/intel_psr.c > +++ b/drivers/gpu/drm/i915/display/intel_psr.c > @@ -2310,7 +2310,7 @@ static u32 > man_trk_ctl_continuos_full_frame(struct intel_display *display) > PSR2_MAN_TRK_CTL_SF_CONTINUOS_FULL_FRAME; > } > > -static void psr_force_hw_tracking_exit(struct intel_dp *intel_dp) > +static void intel_psr_force_update(struct intel_dp *intel_dp) > { > struct intel_display *display = to_intel_display(intel_dp); > enum transcoder cpu_transcoder = intel_dp->psr.transcoder; @@ - > 2857,7 +2857,7 @@ void intel_psr_post_plane_update(struct > intel_atomic_state *state, > > /* Force a PSR exit when enabling CRC to avoid CRC timeouts > */ > if (crtc_state->crc_enabled && psr->enabled) > - psr_force_hw_tracking_exit(intel_dp); > + intel_psr_force_update(intel_dp); > > /* > * Clear possible busy bits in case we have @@ -3254,10 > +3254,10 @@ static void _psr_flush_handle(struct intel_dp *intel_dp) > * continuous full frame is disabled, only a single full > * frame is required > */ > - psr_force_hw_tracking_exit(intel_dp); > + intel_psr_force_update(intel_dp); > } > } else { > - psr_force_hw_tracking_exit(intel_dp); > + intel_psr_force_update(intel_dp); > > if (!intel_dp->psr.active && !intel_dp- > >psr.busy_frontbuffer_bits) > queue_work(dev_priv->unordered_wq, &intel_dp- > >psr.work); > -- > 2.43.0
