iris: Drop XXX about checking for swizzling
authorKenneth Graunke <kenneth@whitecape.org>
Tue, 12 Feb 2019 06:36:45 +0000 (22:36 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 21 Feb 2019 18:26:12 +0000 (10:26 -0800)
Caio noted that this is not necessary on Gen8+:

   "Before Gen8, there was a historical configuration control field to
    swizzle address bit[6] for in X/Y tiling modes.  This was set in
    three different places: TILECTL[1:0], ARB_MODE[5:4], and
    DISP_ARB_CTL[14:13].  For Gen8 and subsequent generations, the
    swizzle fields are all reserved, and the CPU's memory controller
    performs all address swizzling modifications."

Since we don't support earlier hardware, we can skip it entirely.

src/gallium/drivers/iris/iris_screen.c

index 728c2c23506b7aa3accf167c33d8ea2c122f3509..89f6f1bb85e915614b14cd8d99ce8ec3c9384ea6 100644 (file)
@@ -574,8 +574,7 @@ iris_screen_create(int fd)
 
    screen->precompile = env_var_as_boolean("shader_precompile", true);
 
-   bool hw_has_swizzling = false; // XXX: detect?
-   isl_device_init(&screen->isl_dev, &screen->devinfo, hw_has_swizzling);
+   isl_device_init(&screen->isl_dev, &screen->devinfo, false);
 
    screen->compiler = brw_compiler_create(screen, &screen->devinfo);
    screen->compiler->shader_debug_log = iris_shader_debug_log;