On Tue, 2025-07-15 at 15:09 -0400, Rodrigo Vivi wrote:
> On Tue, Jul 15, 2025 at 01:55:07PM +0300, Jouni Högander wrote:
> > Current approach is that Early Transport is disabled in case
> > enable_psr
> > module parameter is set. Let's ignore enable_psr parameter when
> > choosing if
> > Early Transport can be used.
> >
> > Signed-off-by: Jouni Högander <jouni.hogan...@intel.com>
>
> Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com>
Thank you Rodrigo for your review here. Whole set is now pushed to drm-
intel-next.
BR,
Jouni Högander
>
> > ---
> > drivers/gpu/drm/i915/display/intel_psr.c | 13 +------------
> > 1 file changed, 1 insertion(+), 12 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
> > b/drivers/gpu/drm/i915/display/intel_psr.c
> > index ae9053919211..a2b5688f0c82 100644
> > --- a/drivers/gpu/drm/i915/display/intel_psr.c
> > +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> > @@ -265,16 +265,6 @@ static bool psr2_global_enabled(struct
> > intel_dp *intel_dp)
> > }
> > }
> >
> > -static bool psr2_su_region_et_global_enabled(struct intel_dp
> > *intel_dp)
> > -{
> > - struct intel_display *display =
> > to_intel_display(intel_dp);
> > -
> > - if (display->params.enable_psr != -1)
> > - return false;
> > -
> > - return true;
> > -}
> > -
> > static bool panel_replay_global_enabled(struct intel_dp *intel_dp)
> > {
> > struct intel_display *display =
> > to_intel_display(intel_dp);
> > @@ -742,8 +732,7 @@ static bool psr2_su_region_et_valid(struct
> > intel_dp *intel_dp, bool panel_replay
> > return panel_replay ?
> > intel_dp-
> > >pr_dpcd[INTEL_PR_DPCD_INDEX(DP_PANEL_REPLAY_CAP_SUPPORT)] &
> > DP_PANEL_REPLAY_EARLY_TRANSPORT_SUPPORT :
> > - intel_dp->psr_dpcd[0] ==
> > DP_PSR2_WITH_Y_COORD_ET_SUPPORTED &&
> > - psr2_su_region_et_global_enabled(intel_dp);
> > + intel_dp->psr_dpcd[0] ==
> > DP_PSR2_WITH_Y_COORD_ET_SUPPORTED;
> > }
> >
> > static void _panel_replay_enable_sink(struct intel_dp *intel_dp,
> > --
> > 2.43.0
> >
