diff options
| author | Sakari Ailus <sakari.ailus@linux.intel.com> | 2016-09-05 04:09:42 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2016-09-09 11:22:02 -0300 |
| commit | 3933186aec80f66694cbf38ee21bb7011e935dcf (patch) | |
| tree | 5880c5e5e9f8a36c61890425b03afa608b305c02 | |
| parent | 68429f50ab60074e58b98010103fcc5bac4afd54 (diff) | |
| download | linux-3933186aec80f66694cbf38ee21bb7011e935dcf.tar.bz2 | |
[media] ad5820: Use bool for boolean values
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>
| -rw-r--r-- | drivers/media/i2c/ad5820.c | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/drivers/media/i2c/ad5820.c b/drivers/media/i2c/ad5820.c index d7ad5c1a1219..fd4c5f67163d 100644 --- a/drivers/media/i2c/ad5820.c +++ b/drivers/media/i2c/ad5820.c @@ -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 |