diff --git a/src/intel/isl/isl_surface_state.c b/src/intel/isl/isl_surface_state.c
index 97f92d9d00f..8a5de4b3848 100644
--- a/src/intel/isl/isl_surface_state.c
+++ b/src/intel/isl/isl_surface_state.c
@@ -330,16 +330,13 @@ isl_genX(surf_fill_state_s)(const struct isl_device *dev, void *state,
        * assert-failures in the packing function below, we only set the field
        * when it's actually going to be used by the hardware.
        *
-       * Similaraly, the MinimumArrayElement field is ignored by all hardware
-       * prior to Sky Lake when texturing and we want it set to 0 anyway.
-       * Since it's already initialized to 0, we can just leave it alone for
-       * texture surfaces.
+       * The MinimumArrayElement field is ignored by all hardware
+       * prior to Sky Lake when texturing, and drivers are responsible
+       * for validating the correctness of this parameter.
+       * KHR_gl_texture_3D_image requires this functionality.
        */
-      if (info->view->usage & (ISL_SURF_USAGE_RENDER_TARGET_BIT |
-                               ISL_SURF_USAGE_STORAGE_BIT)) {
-         s.MinimumArrayElement = info->view->base_array_layer;
-         s.RenderTargetViewExtent = info->view->array_len - 1;
-      }
+      s.MinimumArrayElement = info->view->base_array_layer;
+      s.RenderTargetViewExtent = info->view->array_len - 1;
       break;
    default:
       unreachable("bad SurfaceType");