Commit b4e43e95 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[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: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent b1a88c71
...@@ -1125,11 +1125,10 @@ static enum lgdt3306a_modulation lgdt3306a_check_oper_mode(struct lgdt3306a_stat ...@@ -1125,11 +1125,10 @@ static enum lgdt3306a_modulation lgdt3306a_check_oper_mode(struct lgdt3306a_stat
if (val & 0x01) { if (val & 0x01) {
dbg_info("QAM256\n"); dbg_info("QAM256\n");
return LG3306_QAM256; return LG3306_QAM256;
} else { }
dbg_info("QAM64\n"); dbg_info("QAM64\n");
return LG3306_QAM64; return LG3306_QAM64;
} }
}
err: err:
pr_warn("UNKNOWN\n"); pr_warn("UNKNOWN\n");
return LG3306_UNKNOWN_MODE; return LG3306_UNKNOWN_MODE;
...@@ -1399,14 +1398,15 @@ static u32 log10_x1000(u32 x) ...@@ -1399,14 +1398,15 @@ static u32 log10_x1000(u32 x)
if (x <= 0) if (x <= 0)
return -1000000; /* signal error */ return -1000000; /* signal error */
if (x == 10)
return 0; /* log(1)=0 */
if (x < 10) { if (x < 10) {
while (x < 10) { while (x < 10) {
x = x * 10; x = x * 10;
log_val--; log_val--;
} }
} else if (x == 10) { } else { /* x > 10 */
return 0; /* log(1)=0 */
} else {
while (x >= 100) { while (x >= 100) {
x = x / 10; x = x / 10;
log_val++; log_val++;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment