]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
OMAPDSS: DISPC: Fix assignment of 0/1 to bool variables
authorPeter Senna Tschudin <peter.senna@gmail.com>
Sun, 22 Sep 2013 18:44:11 +0000 (20:44 +0200)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 23 Sep 2013 07:34:30 +0000 (10:34 +0300)
Convert 0 to false and 1 to true when assigning values to bool
variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.

The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):

@@
bool b;
@@
(
-b = 0
+b = false
|
-b = 1
+b = true
)

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/video/omap2/dss/dispc.c

index 02a7340111dfbf4b856122aae670bdbff2c34150..8fbeb299fac391604b0077e325b3f09c234bd290 100644 (file)
@@ -2352,7 +2352,7 @@ int dispc_ovl_check(enum omap_plane plane, enum omap_channel channel,
 {
        enum omap_overlay_caps caps = dss_feat_get_overlay_caps(plane);
        bool five_taps = true;
-       bool fieldmode = 0;
+       bool fieldmode = false;
        u16 in_height = oi->height;
        u16 in_width = oi->width;
        bool ilace = timings->interlace;
@@ -2365,7 +2365,7 @@ int dispc_ovl_check(enum omap_plane plane, enum omap_channel channel,
        out_height = oi->out_height == 0 ? oi->height : oi->out_height;
 
        if (ilace && oi->height == out_height)
-               fieldmode = 1;
+               fieldmode = true;
 
        if (ilace) {
                if (fieldmode)
@@ -2396,7 +2396,7 @@ static int dispc_ovl_setup_common(enum omap_plane plane,
                bool mem_to_mem)
 {
        bool five_taps = true;
-       bool fieldmode = 0;
+       bool fieldmode = false;
        int r, cconv = 0;
        unsigned offset0, offset1;
        s32 row_inc;
@@ -2417,7 +2417,7 @@ static int dispc_ovl_setup_common(enum omap_plane plane,
        out_height = out_height == 0 ? height : out_height;
 
        if (ilace && height == out_height)
-               fieldmode = 1;
+               fieldmode = true;
 
        if (ilace) {
                if (fieldmode)