From: "Hogander, Jouni" <jouni.hogander@intel.com>
To: "Deak, Imre" <imre.deak@intel.com>
Cc: "intel-xe@lists.freedesktop.org" <intel-xe@lists.freedesktop.org>,
"intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>
Subject: Re: [PATCH v2 7/8] drm/i915/psr: Move sink PSR and Panel Replay booleans to intel_connector
Date: Mon, 8 Dec 2025 08:37:18 +0000 [thread overview]
Message-ID: <76f856552de36a6ecfe47706e3c55bc1d2e52b99.camel@intel.com> (raw)
In-Reply-To: <aTBQCx60u-wsEtof@ideak-desk>
On Wed, 2025-12-03 at 16:58 +0200, Imre Deak wrote:
> On Wed, Dec 03, 2025 at 12:31:33PM +0200, Jouni Högander wrote:
> > As a preparation for MST Panel Replay we need to move Panel Replay
> > sink
> > related data into intel_connector. Move sink support booleans as
> > well
> > into intel_connector. Generally this is more correct place for this
> > data so
> > move PSR versions as well.
> >
> > Still sink_support and sink_panel_replay_support are kept to keep
> > CAN_PSR
> > and CAN_PANEL_REPLAY macros. Plan is to keep them that way as they
> > are
> > widely used from patch where connector is not available.
> >
> > Later we might want to clear intel_dp-
> > >psr.sink_panel_replay_support if any
> > of the devices in branch is not supporting Panel Replay (mst).
> >
> > v2:
> > - commit message updated
> > - Extra w/s removed
> >
> > Signed-off-by: Jouni Högander <jouni.hogander@intel.com>
>
> Reviewed-by: Imre Deak <imre.deak@intel.com>
Thank you Imre for your review. I will take care of those leftovers you
pointed out. All patches in this set now pushed to drm-intel-next.
BR,
Jouni Högander
>
> > ---
> > .../drm/i915/display/intel_display_types.h | 7 ++-
> > drivers/gpu/drm/i915/display/intel_dp.c | 3 +-
> > drivers/gpu/drm/i915/display/intel_psr.c | 44 +++++++++++----
> > ----
> > 3 files changed, 32 insertions(+), 22 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h
> > b/drivers/gpu/drm/i915/display/intel_display_types.h
> > index b5641698db695..fb1c8b2d3979f 100644
> > --- a/drivers/gpu/drm/i915/display/intel_display_types.h
> > +++ b/drivers/gpu/drm/i915/display/intel_display_types.h
> > @@ -572,6 +572,8 @@ struct intel_connector {
> > u8 dpcd[DP_PANEL_REPLAY_CAP_SIZE];
> > #define
> > INTEL_PR_DPCD_INDEX(pr_dpcd_register) ((pr_dpcd_register) - DP_PANEL_REPLAY_CAP_SUPPORT)
> >
> > + bool support;
> > + bool su_support;
> > enum intel_panel_replay_dsc_support
> > dsc_support;
> >
> > u16 su_w_granularity;
> > @@ -581,6 +583,9 @@ struct intel_connector {
> > struct {
> > u8 dpcd[EDP_PSR_RECEIVER_CAP_SIZE];
> >
> > + bool support;
> > + bool su_support;
> > +
> > u16 su_w_granularity;
> > u16 su_y_granularity;
> > } psr_caps;
> > @@ -1731,7 +1736,6 @@ struct intel_psr {
> > bool active;
> > struct work_struct work;
> > unsigned int busy_frontbuffer_bits;
> > - bool sink_psr2_support;
> > bool link_standby;
> > bool sel_update_enabled;
> > bool psr2_sel_fetch_enabled;
> > @@ -1747,7 +1751,6 @@ struct intel_psr {
> > u16 su_y_granularity;
> > bool source_panel_replay_support;
> > bool sink_panel_replay_support;
> > - bool sink_panel_replay_su_support;
> > bool panel_replay_enabled;
> > u32 dc3co_exitline;
> > u32 dc3co_exit_delay;
> > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c
> > b/drivers/gpu/drm/i915/display/intel_dp.c
> > index 8c5ddf0b09d25..4419954867455 100644
> > --- a/drivers/gpu/drm/i915/display/intel_dp.c
> > +++ b/drivers/gpu/drm/i915/display/intel_dp.c
> > @@ -6058,7 +6058,8 @@ intel_dp_detect(struct drm_connector
> > *_connector,
> > memset(connector->dp.panel_replay_caps.dpcd, 0,
> > sizeof(connector-
> > >dp.panel_replay_caps.dpcd));
> > intel_dp->psr.sink_panel_replay_support = false;
> > - intel_dp->psr.sink_panel_replay_su_support =
> > false;
> > + connector->dp.panel_replay_caps.support = false;
> > + connector->dp.panel_replay_caps.su_support =
> > false;
> > connector->dp.panel_replay_caps.dsc_support =
> > INTEL_DP_PANEL_REPLAY_DSC_NOT_SUPPORTED;
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
> > b/drivers/gpu/drm/i915/display/intel_psr.c
> > index 1d3864405c850..c01e06f9cf643 100644
> > --- a/drivers/gpu/drm/i915/display/intel_psr.c
> > +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> > @@ -634,11 +634,12 @@ static void _panel_replay_init_dpcd(struct
> > intel_dp *intel_dp, struct intel_conn
> > }
> > }
> >
> > + connector->dp.panel_replay_caps.support = true;
> > intel_dp->psr.sink_panel_replay_support = true;
> >
> > if (connector-
> > >dp.panel_replay_caps.dpcd[INTEL_PR_DPCD_INDEX(DP_PANEL_REPLAY_CAP_
> > SUPPORT)] &
> > DP_PANEL_REPLAY_SU_SUPPORT) {
> > - intel_dp->psr.sink_panel_replay_su_support = true;
> > + connector->dp.panel_replay_caps.su_support = true;
> >
> > _panel_replay_compute_su_granularity(connector);
> > }
> > @@ -647,7 +648,7 @@ static void _panel_replay_init_dpcd(struct
> > intel_dp *intel_dp, struct intel_conn
> >
> > drm_dbg_kms(display->drm,
> > "Panel replay %sis supported by panel (in DSC
> > mode: %s)\n",
> > - intel_dp->psr.sink_panel_replay_su_support ?
> > + connector->dp.panel_replay_caps.su_support ?
> > "selective_update " : "",
> > panel_replay_dsc_support_str(connector-
> > >dp.panel_replay_caps.dsc_support));
> > }
> > @@ -680,7 +681,9 @@ static void _psr_init_dpcd(struct intel_dp
> > *intel_dp, struct intel_connector *co
> > return;
> > }
> >
> > + connector->dp.psr_caps.support = true;
> > intel_dp->psr.sink_support = true;
> > +
> > intel_dp->psr.sink_sync_latency =
> > intel_dp_get_sink_sync_latency(intel_dp);
> >
> > @@ -700,13 +703,13 @@ static void _psr_init_dpcd(struct intel_dp
> > *intel_dp, struct intel_connector *co
> > * Y-coordinate requirement panels we would need
> > to enable
> > * GTC first.
> > */
> > - intel_dp->psr.sink_psr2_support = y_req &&
> > + connector->dp.psr_caps.su_support = y_req &&
> > intel_alpm_aux_wake_supported(intel_dp);
> > drm_dbg_kms(display->drm, "PSR2 %ssupported\n",
> > - intel_dp->psr.sink_psr2_support ? "" :
> > "not ");
> > + connector->dp.psr_caps.su_support ? ""
> > : "not ");
> > }
> >
> > - if (intel_dp->psr.sink_psr2_support)
> > + if (connector->dp.psr_caps.su_support)
> > _psr_compute_su_granularity(intel_dp, connector);
> > }
> >
> > @@ -1514,14 +1517,16 @@ static bool alpm_config_valid(struct
> > intel_dp *intel_dp,
> > }
> >
> > static bool intel_psr2_config_valid(struct intel_dp *intel_dp,
> > - struct intel_crtc_state
> > *crtc_state)
> > + struct intel_crtc_state
> > *crtc_state,
> > + struct drm_connector_state
> > *conn_state)
> > {
> > struct intel_display *display =
> > to_intel_display(intel_dp);
> > + struct intel_connector *connector =
> > to_intel_connector(conn_state->connector);
> > int crtc_hdisplay = crtc_state-
> > >hw.adjusted_mode.crtc_hdisplay;
> > int crtc_vdisplay = crtc_state-
> > >hw.adjusted_mode.crtc_vdisplay;
> > int psr_max_h = 0, psr_max_v = 0, max_bpp = 0;
> >
> > - if (!intel_dp->psr.sink_psr2_support || display-
> > >params.enable_psr == 1)
> > + if (!connector->dp.psr_caps.su_support || display-
> > >params.enable_psr == 1)
> > return false;
> >
> > /* JSL and EHL only supports eDP 1.3 */
> > @@ -1634,7 +1639,8 @@ static bool
> > intel_sel_update_config_valid(struct intel_crtc_state *crtc_state,
> > goto unsupported;
> > }
> >
> > - if (!crtc_state->has_panel_replay &&
> > !intel_psr2_config_valid(intel_dp, crtc_state))
> > + if (!crtc_state->has_panel_replay &&
> > !intel_psr2_config_valid(intel_dp, crtc_state,
> > +
> > conn_state))
> > goto unsupported;
> >
> > if (!_compute_psr2_sdp_prior_scanline_indication(intel_dp,
> > crtc_state)) {
> > @@ -1647,7 +1653,7 @@ static bool
> > intel_sel_update_config_valid(struct intel_crtc_state *crtc_state,
> > if (DISPLAY_VER(display) < 14)
> > goto unsupported;
> >
> > - if (!intel_dp->psr.sink_panel_replay_su_support)
> > + if (!connector->dp.panel_replay_caps.su_support)
> > goto unsupported;
> >
> > if (intel_dsc_enabled_on_link(crtc_state) &&
> > @@ -1736,6 +1742,9 @@ static bool
> > _panel_replay_compute_config(struct intel_crtc_state *crtc_state,
> > if (!CAN_PANEL_REPLAY(intel_dp))
> > return false;
> >
> > + if (!connector->dp.panel_replay_caps.support)
> > + return false;
> > +
> > if (!panel_replay_global_enabled(intel_dp)) {
> > drm_dbg_kms(display->drm, "Panel Replay disabled
> > by flag\n");
> > return false;
> > @@ -4117,22 +4126,19 @@ psr_source_status(struct intel_dp
> > *intel_dp, struct seq_file *m)
> > seq_printf(m, "Source PSR/PanelReplay status: %s
> > [0x%08x]\n", status, val);
> > }
> >
> > -static void intel_psr_sink_capability(struct intel_dp *intel_dp,
> > - struct intel_connector
> > *connector,
> > +static void intel_psr_sink_capability(struct intel_connector
> > *connector,
> > struct seq_file *m)
> > {
> > - struct intel_psr *psr = &intel_dp->psr;
> > -
> > seq_printf(m, "Sink support: PSR = %s",
> > - str_yes_no(psr->sink_support));
> > + str_yes_no(connector->dp.psr_caps.support));
> >
> > - if (psr->sink_support)
> > + if (connector->dp.psr_caps.support)
> > seq_printf(m, " [0x%02x]", connector-
> > >dp.psr_caps.dpcd[0]);
> > if (connector->dp.psr_caps.dpcd[0] ==
> > DP_PSR2_WITH_Y_COORD_ET_SUPPORTED)
> > seq_printf(m, " (Early Transport)");
> > - seq_printf(m, ", Panel Replay = %s", str_yes_no(psr-
> > >sink_panel_replay_support));
> > + seq_printf(m, ", Panel Replay = %s", str_yes_no(connector-
> > >dp.panel_replay_caps.support));
> > seq_printf(m, ", Panel Replay Selective Update = %s",
> > - str_yes_no(psr->sink_panel_replay_su_support));
> > + str_yes_no(connector-
> > >dp.panel_replay_caps.su_support));
> > seq_printf(m, ", Panel Replay DSC support = %s",
> > panel_replay_dsc_support_str(connector-
> > >dp.panel_replay_caps.dsc_support));
> > if (connector-
> > >dp.panel_replay_caps.dpcd[INTEL_PR_DPCD_INDEX(DP_PANEL_REPLAY_CAP_
> > SUPPORT)] &
> > @@ -4183,9 +4189,9 @@ static int intel_psr_status(struct seq_file
> > *m, struct intel_dp *intel_dp,
> > bool enabled;
> > u32 val, psr2_ctl;
> >
> > - intel_psr_sink_capability(intel_dp, connector, m);
> > + intel_psr_sink_capability(connector, m);
> >
> > - if (!(psr->sink_support || psr-
> > >sink_panel_replay_support))
> > + if (!(connector->dp.psr_caps.support || connector-
> > >dp.panel_replay_caps.support))
> > return 0;
> >
> > wakeref = intel_display_rpm_get(display);
> > --
> > 2.43.0
> >
next prev parent reply other threads:[~2025-12-08 8:37 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-03 10:31 [PATCH v2 0/8] Move PSR/Panel Replay sink data into intel_connector Jouni Högander
2025-12-03 10:31 ` [PATCH v2 1/8] drm/i915/psr: Add panel granularity information " Jouni Högander
2025-12-03 10:31 ` [PATCH v2 2/8] drm/i915/psr: Use SU granularity information available in intel_connector Jouni Högander
2025-12-03 14:49 ` Imre Deak
2025-12-03 15:23 ` [PATCH v3] " Jouni Högander
2025-12-03 10:31 ` [PATCH v2 3/8] drm/i915/psr: Compute Panel Replay/Adaptive Sync coexistence behavior Jouni Högander
2025-12-03 10:31 ` [PATCH v2 4/8] drm/i915/psr: Move pr_dpcd and psr_dpcd to intel_connector Jouni Högander
2025-12-03 10:31 ` [PATCH v2 5/8] drm/i915/psr: Clear pr_dpcd as well on disconnect Jouni Högander
2025-12-03 14:54 ` Imre Deak
2025-12-03 10:31 ` [PATCH v2 6/8] drm/i915/psr: Move Panel Replay DSC sink support data to intel_connector Jouni Högander
2025-12-03 10:31 ` [PATCH v2 7/8] drm/i915/psr: Move sink PSR and Panel Replay booleans " Jouni Högander
2025-12-03 14:58 ` Imre Deak
2025-12-08 8:37 ` Hogander, Jouni [this message]
2025-12-03 10:31 ` [PATCH v2 8/8] drm/i915/psr: Move sink_sync_latency " Jouni Högander
2025-12-03 11:29 ` ✓ i915.CI.BAT: success for Move PSR/Panel Replay sink data into intel_connector (rev2) Patchwork
2025-12-03 16:28 ` ✓ i915.CI.BAT: success for Move PSR/Panel Replay sink data into intel_connector (rev3) Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=76f856552de36a6ecfe47706e3c55bc1d2e52b99.camel@intel.com \
--to=jouni.hogander@intel.com \
--cc=imre.deak@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).