From 15dec3087700492cd6ab538250ba9de8a50d4f57 Mon Sep 17 00:00:00 2001 From: Nanley Chery Date: Thu, 4 May 2023 13:36:21 -0700 Subject: [PATCH] intel/isl: Move the Tile4 modifier score case down Group modifiers by platform first, then the score. I find it easier to read this way. Reviewed-by: Lionel Landwerlin Part-of: --- src/intel/isl/isl_drm.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/intel/isl/isl_drm.c b/src/intel/isl/isl_drm.c index b8855db..0520c0d 100644 --- a/src/intel/isl/isl_drm.c +++ b/src/intel/isl/isl_drm.c @@ -202,12 +202,6 @@ isl_drm_modifier_get_score(const struct intel_device_info *devinfo, return 0; return 3; - case I915_FORMAT_MOD_4_TILED: - /* Gfx12.5 introduces Tile4. */ - if (devinfo->verx10 < 125) - return 0; - - return 3; case I915_FORMAT_MOD_Y_TILED_CCS: /* Not supported before Gfx9 and also Gfx12's CCS layout differs from * Gfx9-11. @@ -219,6 +213,12 @@ isl_drm_modifier_get_score(const struct intel_device_info *devinfo, return 0; return 4; + case I915_FORMAT_MOD_4_TILED: + /* Gfx12.5 introduces Tile4. */ + if (devinfo->verx10 < 125) + return 0; + + return 3; } } -- 2.7.4