]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/xe: Use fb cached min alignment
authorTvrtko Ursulin <tvrtko.ursulin@igalia.com>
Thu, 30 Apr 2026 15:46:01 +0000 (18:46 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 6 May 2026 19:29:30 +0000 (22:29 +0300)
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 <tvrtko.ursulin@igalia.com>
[vsyrjala: Rebase due to pin_params]
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patch.msgid.link/20260430154602.11393-1-ville.syrjala@linux.intel.com
Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
drivers/gpu/drm/xe/display/xe_fb_pin.c

index 5d7b30f629308da8854f5a0813c764c25b7ea6a1..9dc8d66d0faa1679f1cb2939d56e247b119c3956 100644 (file)
@@ -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),
        };