* [PATCH v2] video: fbdev: Mark expected switch fall-through @ 2019-07-30 15:25 ` Anders Roxell 2019-08-19 14:04 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 2+ messages in thread From: Anders Roxell @ 2019-07-30 15:25 UTC (permalink / raw) To: b.zolnierkie; +Cc: dri-devel, linux-fbdev, linux-kernel, gustavo, Anders Roxell Now that -Wimplicit-fallthrough is passed to GCC by default, the following warnings shows up: ../drivers/video/fbdev/sh_mobile_lcdcfb.c: In function ‘sh_mobile_lcdc_channel_fb_init’: ../drivers/video/fbdev/sh_mobile_lcdcfb.c:2086:22: warning: this statement may fall through [-Wimplicit-fallthrough=] info->fix.ypanstep = 2; ~~~~~~~~~~~~~~~~~~~^~~ ../drivers/video/fbdev/sh_mobile_lcdcfb.c:2087:2: note: here case V4L2_PIX_FMT_NV16: ^~~~ ../drivers/video/fbdev/sh_mobile_lcdcfb.c: In function ‘sh_mobile_lcdc_overlay_fb_init’: ../drivers/video/fbdev/sh_mobile_lcdcfb.c:1596:22: warning: this statement may fall through [-Wimplicit-fallthrough=] info->fix.ypanstep = 2; ~~~~~~~~~~~~~~~~~~~^~~ ../drivers/video/fbdev/sh_mobile_lcdcfb.c:1597:2: note: here case V4L2_PIX_FMT_NV16: ^~~~ Rework to address a warnings due to the enablement of -Wimplicit-fallthrough. Signed-off-by: Anders Roxell <anders.roxell@linaro.org> --- drivers/video/fbdev/sh_mobile_lcdcfb.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/video/fbdev/sh_mobile_lcdcfb.c b/drivers/video/fbdev/sh_mobile_lcdcfb.c index ac0bcac9a865..c249763dbf0b 100644 --- a/drivers/video/fbdev/sh_mobile_lcdcfb.c +++ b/drivers/video/fbdev/sh_mobile_lcdcfb.c @@ -1594,6 +1594,7 @@ sh_mobile_lcdc_overlay_fb_init(struct sh_mobile_lcdc_overlay *ovl) case V4L2_PIX_FMT_NV12: case V4L2_PIX_FMT_NV21: info->fix.ypanstep = 2; + /* Fall through */ case V4L2_PIX_FMT_NV16: case V4L2_PIX_FMT_NV61: info->fix.xpanstep = 2; @@ -2084,6 +2085,7 @@ sh_mobile_lcdc_channel_fb_init(struct sh_mobile_lcdc_chan *ch, case V4L2_PIX_FMT_NV12: case V4L2_PIX_FMT_NV21: info->fix.ypanstep = 2; + /* Fall through */ case V4L2_PIX_FMT_NV16: case V4L2_PIX_FMT_NV61: info->fix.xpanstep = 2; -- 2.20.1 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] video: fbdev: Mark expected switch fall-through 2019-07-30 15:25 ` [PATCH v2] video: fbdev: Mark expected switch fall-through Anders Roxell @ 2019-08-19 14:04 ` Bartlomiej Zolnierkiewicz 0 siblings, 0 replies; 2+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2019-08-19 14:04 UTC (permalink / raw) To: Anders Roxell; +Cc: dri-devel, linux-fbdev, linux-kernel, gustavo On 7/30/19 5:25 PM, Anders Roxell wrote: > Now that -Wimplicit-fallthrough is passed to GCC by default, the > following warnings shows up: > > ../drivers/video/fbdev/sh_mobile_lcdcfb.c: In function ‘sh_mobile_lcdc_channel_fb_init’: > ../drivers/video/fbdev/sh_mobile_lcdcfb.c:2086:22: warning: this statement may fall > through [-Wimplicit-fallthrough=] > info->fix.ypanstep = 2; > ~~~~~~~~~~~~~~~~~~~^~~ > ../drivers/video/fbdev/sh_mobile_lcdcfb.c:2087:2: note: here > case V4L2_PIX_FMT_NV16: > ^~~~ > ../drivers/video/fbdev/sh_mobile_lcdcfb.c: In function ‘sh_mobile_lcdc_overlay_fb_init’: > ../drivers/video/fbdev/sh_mobile_lcdcfb.c:1596:22: warning: this statement may fall > through [-Wimplicit-fallthrough=] > info->fix.ypanstep = 2; > ~~~~~~~~~~~~~~~~~~~^~~ > ../drivers/video/fbdev/sh_mobile_lcdcfb.c:1597:2: note: here > case V4L2_PIX_FMT_NV16: > ^~~~ > > Rework to address a warnings due to the enablement of > -Wimplicit-fallthrough. > > Signed-off-by: Anders Roxell <anders.roxell@linaro.org> Patch queued for v5.4, thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-08-19 14:04 UTC | newest] Thread overview: 2+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <CGME20190730152542epcas2p46c9b1c2e97c0b33df072e4fa1328f8d0@epcas2p4.samsung.com> 2019-07-30 15:25 ` [PATCH v2] video: fbdev: Mark expected switch fall-through Anders Roxell 2019-08-19 14:04 ` Bartlomiej Zolnierkiewicz
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).