Instead of using 0 or 1 for boolean, use the true/false defines. Signed-off-by: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx> diff --git a/drivers/media/dvb-frontends/m88ds3103.c b/drivers/media/dvb-frontends/m88ds3103.c index 6eae2c619843..81657e94c5a4 100644 --- a/drivers/media/dvb-frontends/m88ds3103.c +++ b/drivers/media/dvb-frontends/m88ds3103.c @@ -1063,16 +1063,16 @@ static int m88ds3103_set_voltage(struct dvb_frontend *fe, switch (fe_sec_voltage) { case SEC_VOLTAGE_18: - voltage_sel = 1; - voltage_dis = 0; + voltage_sel = true; + voltage_dis = false; break; case SEC_VOLTAGE_13: - voltage_sel = 0; - voltage_dis = 0; + voltage_sel = false; + voltage_dis = false; break; case SEC_VOLTAGE_OFF: - voltage_sel = 0; - voltage_dis = 1; + voltage_sel = false; + voltage_dis = true; break; default: dev_dbg(&priv->i2c->dev, "%s: invalid fe_sec_voltage\n", -- 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