video: fbdev: pvr2fb: remove unnecessary comparison of unsigned integer with < 0
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Mon, 22 Jul 2019 20:33:58 +0000 (15:33 -0500)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Mon, 19 Aug 2019 13:52:23 +0000 (15:52 +0200)
There is no need to compare *var->xoffset* or *var->yoffset* with < 0
because such variables are of type unsigned, making it impossible to
hold a negative value.

Fix this by removing such comparisons.

Addresses-Coverity-ID: 1451964 ("Unsigned compared against 0")
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190722203358.GA29111@embeddedor
drivers/video/fbdev/pvr2fb.c

index 7ff4b6b842824666556310a92da657eee6520721..0a3b2b7c789125a63855354fb2200f427e39416e 100644 (file)
@@ -458,13 +458,11 @@ static int pvr2fb_check_var(struct fb_var_screeninfo *var, struct fb_info *info)
        set_color_bitfields(var);
 
        if (var->vmode & FB_VMODE_YWRAP) {
-               if (var->xoffset || var->yoffset < 0 ||
-                   var->yoffset >= var->yres_virtual) {
+               if (var->xoffset || var->yoffset >= var->yres_virtual) {
                        var->xoffset = var->yoffset = 0;
                } else {
                        if (var->xoffset > var->xres_virtual - var->xres ||
-                           var->yoffset > var->yres_virtual - var->yres ||
-                           var->xoffset < 0 || var->yoffset < 0)
+                           var->yoffset > var->yres_virtual - var->yres)
                                var->xoffset = var->yoffset = 0;
                }
        } else {