aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>2014-10-28 12:05:35 -0200
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2014-10-28 12:11:01 -0200
commitc69d7ca6f9f920567b3c9dfddd429f3ded29fae0 (patch)
tree24dd4c845c65dc8bd0408adff07187579bd85493
parent2cf3f13aa583cf9f7454b88ea9953cc3a5f297fe (diff)
[media] lgdt3306a: Don't use else were not needed
Get rid of the remaining checkpatch.pl warnings: WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r--drivers/media/dvb-frontends/lgdt3306a.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/media/dvb-frontends/lgdt3306a.c b/drivers/media/dvb-frontends/lgdt3306a.c
index 0356810da444..38b64b2c745c 100644
--- a/drivers/media/dvb-frontends/lgdt3306a.c
+++ b/drivers/media/dvb-frontends/lgdt3306a.c
@@ -1125,10 +1125,9 @@ static enum lgdt3306a_modulation lgdt3306a_check_oper_mode(struct lgdt3306a_stat
if (val & 0x01) {
dbg_info("QAM256\n");
return LG3306_QAM256;
- } else {
- dbg_info("QAM64\n");
- return LG3306_QAM64;
}
+ dbg_info("QAM64\n");
+ return LG3306_QAM64;
}
err:
pr_warn("UNKNOWN\n");
@@ -1399,14 +1398,15 @@ static u32 log10_x1000(u32 x)
if (x <= 0)
return -1000000; /* signal error */
+ if (x == 10)
+ return 0; /* log(1)=0 */
+
if (x < 10) {
while (x < 10) {
x = x * 10;
log_val--;
}
- } else if (x == 10) {
- return 0; /* log(1)=0 */
- } else {
+ } else { /* x > 10 */
while (x >= 100) {
x = x / 10;
log_val++;

Privacy Policy