[PATCH 09/13] [media] lgdt3306a: Don't use else were not needed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxx>

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++;
-- 
1.9.3

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux