On Fri, 2024-06-14 at 16:55 +0000, Manna, Animesh wrote:
>
>
> > -----Original Message-----
> > From: Hogander, Jouni <jouni.hogander@xxxxxxxxx>
> > Sent: Thursday, June 13, 2024 3:02 PM
> > To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> > Cc: Manna, Animesh <animesh.manna@xxxxxxxxx>; Kahola, Mika
> > <mika.kahola@xxxxxxxxx>; Hogander, Jouni <jouni.hogander@xxxxxxxxx>
> > Subject: [PATCH v8 06/20] drm/i915/psr: Disable PSR2 SU Region
> > Early
> > Transport if psr_enable is set
> >
> > Currently PSR2 SU Region Early Transport is enabled by default on
> > Lunarlake
> > if panel supports it despite psr_enable value. Prevent SU Region
> > Early
> > Transport if psr_enable is set to than -1 which is the default.
> >
> > Signed-off-by: Jouni Högander <jouni.hogander@xxxxxxxxx>
> > ---
> > drivers/gpu/drm/i915/display/intel_psr.c | 13 ++++++++++++-
> > 1 file changed, 12 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
> > b/drivers/gpu/drm/i915/display/intel_psr.c
> > index 0df557676e08..27cf330d13e2 100644
> > --- a/drivers/gpu/drm/i915/display/intel_psr.c
> > +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> > @@ -235,6 +235,16 @@ static bool psr2_global_enabled(struct
> > intel_dp
> > *intel_dp)
> > }
> > }
> >
> > +static bool psr2_su_region_et_global_enabled(struct intel_dp
> > *intel_dp)
> > +{
> > + struct drm_i915_private *i915 = dp_to_i915(intel_dp);
> > +
> > + if (i915->display.params.enable_psr != -1)
> > + return false;
> > +
> > + return true;
> > +}
> > +
> > static bool panel_replay_global_enabled(struct intel_dp
> > *intel_dp) {
> > struct drm_i915_private *i915 = dp_to_i915(intel_dp); @@ -
> > 683,7
> > +693,8 @@ static bool psr2_su_region_et_valid(struct intel_dp
> > *intel_dp,
> > bool panel_replay
> >
> > return panel_replay ?
> > intel_dp->pr_dpcd &
> > DP_PANEL_REPLAY_EARLY_TRANSPORT_SUPPORT :
> > - intel_dp->psr_dpcd[0] ==
> > DP_PSR2_WITH_Y_COORD_ET_SUPPORTED;
> > + intel_dp->psr_dpcd[0] ==
> > DP_PSR2_WITH_Y_COORD_ET_SUPPORTED &&
> > + psr2_su_region_et_global_enabled(intel_dp);
>
> How to enable early transport with psr/psr2? Is it not possible
> now/future?
It gets enabled if panel supports it. Unless disable bit is set
(I915_PSR_DEBUG_SU_REGION_ET_DISABLE) or enable_psr module parameter is
set.
BR,
Jouni Högander
>
> Regards,
> Animesh
>
> > }
> >
> > static void _panel_replay_enable_sink(struct intel_dp *intel_dp,
> > --
> > 2.34.1
>
