Message ID | 64a4483b3c2e3864dfdc0029497c9e4188a88887.1409775488.git.m.chehab@samsung.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Wed, Sep 03, 2014 at 05:32:48PM -0300, Mauro Carvalho Chehab wrote: > Instead of using 0 or 1 for boolean, use the true/false > defines. > > Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com> Thanks! Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> Would you like me to pick this patch to my tree, or would you like to apply it directly? I'm fine with either.
Em Thu, 04 Sep 2014 10:03:40 +0300 Sakari Ailus <sakari.ailus@iki.fi> escreveu: > On Wed, Sep 03, 2014 at 05:32:48PM -0300, Mauro Carvalho Chehab wrote: > > Instead of using 0 or 1 for boolean, use the true/false > > defines. > > > > Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com> > > Thanks! > > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> > > Would you like me to pick this patch to my tree, or would you like to apply > it directly? I'm fine with either. I'll pick it myself. Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/media/i2c/smiapp/smiapp-core.c b/drivers/media/i2c/smiapp/smiapp-core.c index c4cc5de3ae59..d312932bc56e 100644 --- a/drivers/media/i2c/smiapp/smiapp-core.c +++ b/drivers/media/i2c/smiapp/smiapp-core.c @@ -1312,7 +1312,7 @@ static void smiapp_power_off(struct smiapp_sensor *sensor) clk_disable_unprepare(sensor->ext_clk); usleep_range(5000, 5000); regulator_disable(sensor->vana); - sensor->streaming = 0; + sensor->streaming = false; } static int smiapp_set_power(struct v4l2_subdev *subdev, int on) @@ -1509,13 +1509,13 @@ static int smiapp_set_stream(struct v4l2_subdev *subdev, int enable) return 0; if (enable) { - sensor->streaming = 1; + sensor->streaming = true; rval = smiapp_start_streaming(sensor); if (rval < 0) - sensor->streaming = 0; + sensor->streaming = false; } else { rval = smiapp_stop_streaming(sensor); - sensor->streaming = 0; + sensor->streaming = false; } return rval;
Instead of using 0 or 1 for boolean, use the true/false defines. Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>