Em Sun, 24 Jun 2018 15:42:50 +0200 Daniel Scheller <d.scheller.oss@xxxxxxxxx> escreveu: > From: Daniel Scheller <d.scheller@xxxxxxx> > > Add a break statement in set_params() for the SYS_DVBT(2) case to silence > this sparse warning: > > drivers/media/dvb-frontends/tda18271c2dd.c:1144:3: warning: this statement may fall through [-Wimplicit-fallthrough=] > > as reported in Hans' daily media_tree builds. > > Signed-off-by: Daniel Scheller <d.scheller@xxxxxxx> > --- > drivers/media/dvb-frontends/tda18271c2dd.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/media/dvb-frontends/tda18271c2dd.c b/drivers/media/dvb-frontends/tda18271c2dd.c > index 2e1d36ae943b..fcffc7b4acf7 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; This actually seems to be a bug. Did you test this patch with an actual hardware? Regards, Mauro Thanks, Mauro
