From: Nathan Chancellor Date: Sun, 15 Aug 2021 19:29:59 +0000 (-0700) Subject: drm/radeon: Add break to switch statement in radeonfb_create_pinned_object() X-Git-Tag: v5.15~303^2~10^2~12 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3a6e4106a8fdb87fd3ede0eb242b70b0c4c82bd5;p=platform%2Fkernel%2Flinux-starfive.git drm/radeon: Add break to switch statement in radeonfb_create_pinned_object() Clang + -Wimplicit-fallthrough warns: drivers/gpu/drm/radeon/radeon_fb.c:170:2: warning: unannotated fall-through between switch labels [-Wimplicit-fallthrough] default: ^ drivers/gpu/drm/radeon/radeon_fb.c:170:2: note: insert 'break;' to avoid fall-through default: ^ break; 1 warning generated. Clang's version of this warning is a little bit more pedantic than GCC's. Add the missing break to satisfy it to match what has been done all over the kernel tree. Reviewed-by: Christian König Signed-off-by: Nathan Chancellor Signed-off-by: Alex Deucher --- diff --git a/drivers/gpu/drm/radeon/radeon_fb.c b/drivers/gpu/drm/radeon/radeon_fb.c index 6640b7c..ca382fb 100644 --- a/drivers/gpu/drm/radeon/radeon_fb.c +++ b/drivers/gpu/drm/radeon/radeon_fb.c @@ -168,6 +168,7 @@ static int radeonfb_create_pinned_object(struct radeon_fbdev *rfbdev, break; case 2: tiling_flags |= RADEON_TILING_SWAP_16BIT; + break; default: break; }