video: fbdev: fsl-diu-fb: remove impossible condition
authorSachin Shukla <sachin.s5@samsung.com>
Wed, 8 Feb 2017 15:44:00 +0000 (16:44 +0100)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Wed, 8 Feb 2017 15:44:00 +0000 (16:44 +0100)
xoffset and yoffset of struct fb_var_screeninfo are unsigned and so
they can never be less than 0.

Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: sachiniiitm@gmail.com
Cc: ravikant.s2@samsung.com
Cc: p.shailesh@samsung.com
Cc: ashish.kalra@samsung.com
Cc: vidushi.koul@samsung.com
Signed-off-by: Sachin Shukla <sachin.s5@samsung.com>
Acked-by: Timur Tabi <timur@tabi.org>
[b.zolnierkie: split from the bigger patch]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
drivers/video/fbdev/fsl-diu-fb.c

index fe00a07c122e69f0735baf8832fdd112677b4dd6..c48a59e35faeeaebd05d398e76fb048c99bc0376 100644 (file)
@@ -703,12 +703,6 @@ static int fsl_diu_check_var(struct fb_var_screeninfo *var,
        if (var->yres_virtual < var->yres)
                var->yres_virtual = var->yres;
 
-       if (var->xoffset < 0)
-               var->xoffset = 0;
-
-       if (var->yoffset < 0)
-               var->yoffset = 0;
-
        if (var->xoffset + info->var.xres > info->var.xres_virtual)
                var->xoffset = info->var.xres_virtual - info->var.xres;
 
@@ -1254,8 +1248,7 @@ static int fsl_diu_pan_display(struct fb_var_screeninfo *var,
            (info->var.yoffset == var->yoffset))
                return 0;       /* No change, do nothing */
 
-       if (var->xoffset < 0 || var->yoffset < 0
-           || var->xoffset + info->var.xres > info->var.xres_virtual
+       if (var->xoffset + info->var.xres > info->var.xres_virtual
            || var->yoffset + info->var.yres > info->var.yres_virtual)
                return -EINVAL;