From: Daniel Scheller Date: Sun, 24 Jun 2018 13:42:50 +0000 (-0400) Subject: media: dvb-frontends/tda18271c2dd: fix handling of DVB-T parameters X-Git-Tag: v5.15~8227^2~99 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ca747d041adbf78d93bbea4d125c7ddc626effba;p=platform%2Fkernel%2Flinux-starfive.git media: dvb-frontends/tda18271c2dd: fix handling of DVB-T parameters Add a break statement in set_params() for the SYS_DVBT(2). As reported by gcc: drivers/media/dvb-frontends/tda18271c2dd.c:1144:3: warning: this statement may fall through [-Wimplicit-fallthrough=] There is a nested switch() inside the code with sets the tuner to the right standard. Without the break, the code will always set to DVB-C mode, with can be sub-optimal for DVB-T. Signed-off-by: Daniel Scheller Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/dvb-frontends/tda18271c2dd.c b/drivers/media/dvb-frontends/tda18271c2dd.c index 2e1d36a..fcffc7b 100644 --- a/drivers/media/dvb-frontends/tda18271c2dd.c +++ b/drivers/media/dvb-frontends/tda18271c2dd.c @@ -1154,6 +1154,7 @@ static int set_params(struct dvb_frontend *fe) default: return -EINVAL; } + break; case SYS_DVBC_ANNEX_A: case SYS_DVBC_ANNEX_C: if (bw <= 6000000)