From: Mauro Carvalho Chehab Date: Wed, 3 Sep 2014 18:28:27 +0000 (-0300) Subject: [media] lm3560: simplify boolean tests X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=afb666d1e7b0af5ec8f8b35b6f9d813d538c95e3;p=deliverable%2Flinux.git [media] lm3560: simplify boolean tests Instead of using if (on == true), just use if (on). That allows a faster mental parsing when analyzing the code. Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/i2c/lm3560.c b/drivers/media/i2c/lm3560.c index c23de593c17d..d9ece4b2d047 100644 --- a/drivers/media/i2c/lm3560.c +++ b/drivers/media/i2c/lm3560.c @@ -100,14 +100,14 @@ static int lm3560_enable_ctrl(struct lm3560_flash *flash, int rval; if (led_no == LM3560_LED0) { - if (on == true) + if (on) rval = regmap_update_bits(flash->regmap, REG_ENABLE, 0x08, 0x08); else rval = regmap_update_bits(flash->regmap, REG_ENABLE, 0x08, 0x00); } else { - if (on == true) + if (on) rval = regmap_update_bits(flash->regmap, REG_ENABLE, 0x10, 0x10); else