[linux-3.16/drivers/media/dvb-frontends/stv0900_sw.c:1738]: (style) Statements following return, break, continue, goto or throw will never be executed. switch (intp->srch_standard[demod]) { case STV0900_SEARCH_DVBS1: dprintk("Search Standard = DVBS1\n"); break; case STV0900_SEARCH_DSS: dprintk("Search Standard = DSS\n"); case STV0900_SEARCH_DVBS2: break; dprintk("Search Standard = DVBS2\n"); case STV0900_AUTO_SEARCH: default: dprintk("Search Standard = AUTO\n"); break; } Maybe something like switch (intp->srch_standard[demod]) { case STV0900_SEARCH_DVBS1: dprintk("Search Standard = DVBS1\n"); break; case STV0900_SEARCH_DSS: dprintk("Search Standard = DSS\n"); break; case STV0900_SEARCH_DVBS2: dprintk("Search Standard = DVBS2\n"); break; case STV0900_AUTO_SEARCH: default: dprintk("Search Standard = AUTO\n"); break; } would be better.
Thanks. Patch sent, see "[PATCH] drivers/media/dvb-frontends/stv0900_sw.c: Fix break placement"