intel/isl: Remove inconsistency when choosing Tile64

We don't check the gfx version when choosing the tiling except when
choosing Tile64. Drop the version check for consistency and to remove
doubts about the order of operations occuring as expected within the
CHOOSE macro.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Rohan Garg <rohan.garg@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28284>
This commit is contained in:
Nanley Chery 2024-03-12 16:07:30 -04:00 committed by Marge Bot
parent f8e48b561e
commit 81d8c071ac
1 changed files with 4 additions and 2 deletions

View File

@ -1114,7 +1114,8 @@ isl_surf_choose_tiling(const struct isl_device *dev,
* shapes.
*/
if (info->usage & ISL_SURF_USAGE_SPARSE_BIT) {
CHOOSE(ISL_GFX_VER(dev) >= 20 ? ISL_TILING_64_XE2 : ISL_TILING_64);
CHOOSE(ISL_TILING_64_XE2);
CHOOSE(ISL_TILING_64);
CHOOSE(ISL_TILING_ICL_Ys);
CHOOSE(ISL_TILING_SKL_Ys);
}
@ -1141,7 +1142,8 @@ isl_surf_choose_tiling(const struct isl_device *dev,
CHOOSE(ISL_TILING_ICL_Yf);
CHOOSE(ISL_TILING_SKL_Ys);
CHOOSE(ISL_TILING_ICL_Ys);
CHOOSE(ISL_GFX_VER(dev) >= 20 ? ISL_TILING_64_XE2 : ISL_TILING_64);
CHOOSE(ISL_TILING_64);
CHOOSE(ISL_TILING_64_XE2);
CHOOSE(ISL_TILING_X);
CHOOSE(ISL_TILING_W);