staging: iio: isl29028: remove enable flag from isl29028_enable_proximity()
authorBrian Masney <masneyb@onstation.org>
Tue, 17 Jan 2017 09:24:59 +0000 (04:24 -0500)
committerJonathan Cameron <jic23@kernel.org>
Sun, 22 Jan 2017 13:21:38 +0000 (13:21 +0000)
isl29028_enable_proximity() has a boolean argument named enable. This
function is only called once and the enable flag is set to true in that
call. This patch removes the enable parameter from that function.

Signed-off-by: Brian Masney <masneyb@onstation.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/light/isl29028.c

index bc9c01d05424b3d0f1e417981f2363912a3e5b8a..f1b3651c779b92a8be8e2c8f40e1416283d69dbb 100644 (file)
@@ -103,15 +103,13 @@ static int isl29028_set_proxim_sampling(struct isl29028_chip *chip,
        return ret;
 }
 
-static int isl29028_enable_proximity(struct isl29028_chip *chip, bool enable)
+static int isl29028_enable_proximity(struct isl29028_chip *chip)
 {
        int ret;
-       int val = 0;
 
-       if (enable)
-               val = ISL29028_CONF_PROX_EN;
        ret = regmap_update_bits(chip->regmap, ISL29028_REG_CONFIGURE,
-                                ISL29028_CONF_PROX_EN_MASK, val);
+                                ISL29028_CONF_PROX_EN_MASK,
+                                ISL29028_CONF_PROX_EN);
        if (ret < 0)
                return ret;
 
@@ -225,7 +223,7 @@ static int isl29028_read_proxim(struct isl29028_chip *chip, int *prox)
        int ret;
 
        if (!chip->enable_prox) {
-               ret = isl29028_enable_proximity(chip, true);
+               ret = isl29028_enable_proximity(chip);
                if (ret < 0)
                        return ret;