Re: [PATCH] drm/i915/psr: Take the selective fetch area from pipe_src

["Hogander, Jouni" <jouni.hogander@intel.com>]

On Wed, 2026-09-23 at 11:59 +0000, Mika Kahola wrote:
> The adjusted mode spans all the joined pipes, so it is the wrong
> reference for a per-pipe area. pipe_src is this pipe's own share.
> 
> No functional change, the two are the same without a joiner.
> 
> Assisted-by: Copilot:claude-opus-5
> Signed-off-by: Mika Kahola <mika.kahola@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_psr.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_psr.c
> b/drivers/gpu/drm/i915/display/intel_psr.c
> index 76260ceb49cb..c1994da38567 100644
> --- a/drivers/gpu/drm/i915/display/intel_psr.c
> +++ b/drivers/gpu/drm/i915/display/intel_psr.c
> @@ -2912,8 +2912,8 @@ int intel_psr2_sel_fetch_update(struct
> intel_atomic_state *state,
>  	struct drm_rect display_area = {
>  		.x1 = 0,
>  		.y1 = 0,
> -		.x2 = crtc_state->hw.adjusted_mode.crtc_hdisplay,
> -		.y2 = crtc_state->hw.adjusted_mode.crtc_vdisplay,
> +		.x2 = drm_rect_width(&crtc_state->pipe_src),
> +		.y2 = drm_rect_height(&crtc_state->pipe_src),

I think you should rather use crtc_state->hw.pipe_mode? Altought
selective update is not supported with pipe scaling.

BR,
Jouni Högander

>  	};
>  	bool full_update = false, su_area_changed;
>  	int i, ret;