[media] ad5820: Use bool for boolean values
authorSakari Ailus <sakari.ailus@linux.intel.com>
Mon, 5 Sep 2016 07:09:42 +0000 (04:09 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Fri, 9 Sep 2016 14:22:02 +0000 (11:22 -0300)
The driver used integers for what boolean would have been a better fit.
Use boolean instead.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/i2c/ad5820.c

index d7ad5c1..fd4c5f6 100644 (file)
@@ -58,7 +58,7 @@ struct ad5820_device {
        struct mutex power_lock;
        int power_count;
 
-       unsigned int standby : 1;
+       bool standby;
 };
 
 static int ad5820_write(struct ad5820_device *coil, u16 data)
@@ -108,7 +108,7 @@ static int ad5820_update_hw(struct ad5820_device *coil)
 /*
  * Power handling
  */
-static int ad5820_power_off(struct ad5820_device *coil, int standby)
+static int ad5820_power_off(struct ad5820_device *coil, bool standby)
 {
        int ret = 0, ret2;
 
@@ -117,7 +117,7 @@ static int ad5820_power_off(struct ad5820_device *coil, int standby)
         * (single power line control for both coil and sensor).
         */
        if (standby) {
-               coil->standby = 1;
+               coil->standby = true;
                ret = ad5820_update_hw(coil);
        }
 
@@ -127,7 +127,7 @@ static int ad5820_power_off(struct ad5820_device *coil, int standby)
        return ret2;
 }
 
-static int ad5820_power_on(struct ad5820_device *coil, int restore)
+static int ad5820_power_on(struct ad5820_device *coil, bool restore)
 {
        int ret;
 
@@ -137,7 +137,7 @@ static int ad5820_power_on(struct ad5820_device *coil, int restore)
 
        if (restore) {
                /* Restore the hardware settings. */
-               coil->standby = 0;
+               coil->standby = false;
                ret = ad5820_update_hw(coil);
                if (ret)
                        goto fail;
@@ -145,7 +145,7 @@ static int ad5820_power_on(struct ad5820_device *coil, int restore)
        return 0;
 
 fail:
-       coil->standby = 1;
+       coil->standby = true;
        regulator_disable(coil->vana);
 
        return ret;
@@ -227,7 +227,8 @@ ad5820_set_power(struct v4l2_subdev *subdev, int on)
         * update the power state.
         */
        if (coil->power_count == !on) {
-               ret = on ? ad5820_power_on(coil, 1) : ad5820_power_off(coil, 1);
+               ret = on ? ad5820_power_on(coil, true) :
+                       ad5820_power_off(coil, true);
                if (ret < 0)
                        goto done;
        }
@@ -279,7 +280,7 @@ static int ad5820_suspend(struct device *dev)
        if (!coil->power_count)
                return 0;
 
-       return ad5820_power_off(coil, 0);
+       return ad5820_power_off(coil, false);
 }
 
 static int ad5820_resume(struct device *dev)
@@ -291,7 +292,7 @@ static int ad5820_resume(struct device *dev)
        if (!coil->power_count)
                return 0;
 
-       return ad5820_power_on(coil, 1);
+       return ad5820_power_on(coil, true);
 }
 
 #else