From: Tvrtko Ursulin Date: Thu, 30 Apr 2026 15:46:01 +0000 (+0300) Subject: drm/xe: Use fb cached min alignment X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=53bfcc08aff9a6d6354b0306d3060ac9507133ad;p=thirdparty%2Fkernel%2Flinux.git drm/xe: Use fb cached min alignment Instead of just looking at the first plane use the fb cached overall minimum alignment. This aligns with how the i915 version of intel_plane_pin_fb works. Signed-off-by: Tvrtko Ursulin [vsyrjala: Rebase due to pin_params] Signed-off-by: Ville Syrjälä Link: https://patch.msgid.link/20260430154602.11393-1-ville.syrjala@linux.intel.com Reviewed-by: Juha-Pekka Heikkila --- diff --git a/drivers/gpu/drm/xe/display/xe_fb_pin.c b/drivers/gpu/drm/xe/display/xe_fb_pin.c index 5d7b30f629308..9dc8d66d0faa1 100644 --- a/drivers/gpu/drm/xe/display/xe_fb_pin.c +++ b/drivers/gpu/drm/xe/display/xe_fb_pin.c @@ -458,6 +458,14 @@ found: return true; } +static unsigned int +intel_plane_fb_min_alignment(const struct intel_plane_state *plane_state) +{ + const struct intel_framebuffer *fb = to_intel_framebuffer(plane_state->hw.fb); + + return fb->min_alignment; +} + int intel_plane_pin_fb(struct intel_plane_state *new_plane_state, const struct intel_plane_state *old_plane_state) { @@ -468,7 +476,7 @@ int intel_plane_pin_fb(struct intel_plane_state *new_plane_state, struct intel_plane *plane = to_intel_plane(new_plane_state->uapi.plane); struct intel_fb_pin_params pin_params = { .view = &new_plane_state->view.gtt, - .alignment = plane->min_alignment(plane, fb, 0), + .alignment = intel_plane_fb_min_alignment(new_plane_state), .needs_cpu_lmem_access = intel_fb_needs_cpu_access(fb), };