iris: Add missing break in switch in modifier_is_supported
authorJordan Justen <jordan.l.justen@intel.com>
Tue, 23 Jun 2020 02:46:23 +0000 (19:46 -0700)
committerJordan Justen <jordan.l.justen@intel.com>
Fri, 10 Jul 2020 00:20:14 +0000 (17:20 -0700)
The current fall-through doesn't cause a difference in code flow, but
I think we want a break here.

Fixes: 2305ab693820 ("iris: Refactor modifier_is_supported for gen12")
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Nanley Chery <nanley.g.chery@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5618>

src/gallium/drivers/iris/iris_resource.c

index 9fdd79aeffe487d6e8a7cf3f69a0828997b6674c..ffb0990bdedf88f6715d95f689ece58149f7da97 100644 (file)
@@ -108,6 +108,7 @@ modifier_is_supported(const struct gen_device_info *devinfo,
       if (rt_format == ISL_FORMAT_UNSUPPORTED ||
           !isl_format_supports_ccs_e(devinfo, rt_format))
          return false;
+      break;
    }
    default:
       break;