Hey, D'oh, serves me right for changing patch around at last moment. :) Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> Cheers, ~Maarten Den 2024-08-29 kl. 16:29, skrev Rodrigo Vivi: > On Thu, Aug 29, 2024 at 03:29:28PM +0300, Juha-Pekka Heikkila wrote: >> Let I915_FORMAT_MOD_4_TILED_BMG_CCS show up as supported modifier >> > > doh! I just did the pull-request with that... I guess this is one extra > pull request next week or the first patch of -next-fixes :/ > > Fixes: 97c6efb36497 ("drm/i915/display: Plane capability for 64k phys alignment") > >> Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@xxxxxxxxx> >> --- >> drivers/gpu/drm/i915/display/skl_universal_plane.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c b/drivers/gpu/drm/i915/display/skl_universal_plane.c >> index 9452cad41d07..d28b98e7cbd7 100644 >> --- a/drivers/gpu/drm/i915/display/skl_universal_plane.c >> +++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c >> @@ -2453,6 +2453,9 @@ static u8 skl_get_plane_caps(struct drm_i915_private *i915, >> if (gen12_plane_has_mc_ccs(i915, plane_id)) >> caps |= INTEL_PLANE_CAP_CCS_MC; >> >> + if (GRAPHICS_VER(i915) >= 20 && IS_DGFX(i915)) > > we should be using display version here, no?! > >> + caps |= INTEL_PLANE_CAP_NEED64K_PHYS; >> + >> return caps; >> } >> >> -- >> 2.45.2 >>