To calm down static analysers define a default case and break
out of the test.

Cc: Chris Wilson <ch...@chris-wilson.co.uk>
Signed-off-by: Krzysztof Karas <krzysztof.ka...@intel.com>
---
 drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c 
b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
index 78734c404a6d..6d87356e928b 100644
--- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
+++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
@@ -498,13 +498,15 @@ static int igt_smoke_tiling(void *arg)
                        tile.stride = 0;
                        tile.swizzle = I915_BIT_6_SWIZZLE_NONE;
                        break;
-
                case I915_TILING_X:
                        tile.swizzle = to_gt(i915)->ggtt->bit_6_swizzle_x;
                        break;
                case I915_TILING_Y:
                        tile.swizzle = to_gt(i915)->ggtt->bit_6_swizzle_y;
                        break;
+               default:
+                       err = -EINVAL;
+                       break;
                }
 
                if (tile.swizzle == I915_BIT_6_SWIZZLE_9_17 ||
-- 
2.34.1


-- 
Best Regards,
Krzysztof

Reply via email to