diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-04-25 16:15:11 -0400 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2014-05-09 05:55:49 -0400 |
commit | 34bdb666f4b225471f1b57f8c5313feda8ff3c92 (patch) | |
tree | 4b65e685f57f8d49a5a19a5523534f58798977e7 /drivers/video/fbdev | |
parent | 7a966fbd78683fe4ff7f019fd80fc08d9753880b (diff) |
fbdev: fbmem: remove positive test on unsigned values
fb_image.dx, fb_image.dy and fbconf2bmap.framebuffer are __u32
Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video/fbdev')
-rw-r--r-- | drivers/video/fbdev/core/fbmem.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c index 479fe2a5269c..b5e85f6c1c26 100644 --- a/drivers/video/fbdev/core/fbmem.c +++ b/drivers/video/fbdev/core/fbmem.c | |||
@@ -433,7 +433,7 @@ static void fb_do_show_logo(struct fb_info *info, struct fb_image *image, | |||
433 | image->dx += image->width + 8; | 433 | image->dx += image->width + 8; |
434 | } | 434 | } |
435 | } else if (rotate == FB_ROTATE_UD) { | 435 | } else if (rotate == FB_ROTATE_UD) { |
436 | for (x = 0; x < num && image->dx >= 0; x++) { | 436 | for (x = 0; x < num; x++) { |
437 | info->fbops->fb_imageblit(info, image); | 437 | info->fbops->fb_imageblit(info, image); |
438 | image->dx -= image->width + 8; | 438 | image->dx -= image->width + 8; |
439 | } | 439 | } |
@@ -445,7 +445,7 @@ static void fb_do_show_logo(struct fb_info *info, struct fb_image *image, | |||
445 | image->dy += image->height + 8; | 445 | image->dy += image->height + 8; |
446 | } | 446 | } |
447 | } else if (rotate == FB_ROTATE_CCW) { | 447 | } else if (rotate == FB_ROTATE_CCW) { |
448 | for (x = 0; x < num && image->dy >= 0; x++) { | 448 | for (x = 0; x < num; x++) { |
449 | info->fbops->fb_imageblit(info, image); | 449 | info->fbops->fb_imageblit(info, image); |
450 | image->dy -= image->height + 8; | 450 | image->dy -= image->height + 8; |
451 | } | 451 | } |
@@ -1180,7 +1180,7 @@ static long do_fb_ioctl(struct fb_info *info, unsigned int cmd, | |||
1180 | return -EFAULT; | 1180 | return -EFAULT; |
1181 | if (con2fb.console < 1 || con2fb.console > MAX_NR_CONSOLES) | 1181 | if (con2fb.console < 1 || con2fb.console > MAX_NR_CONSOLES) |
1182 | return -EINVAL; | 1182 | return -EINVAL; |
1183 | if (con2fb.framebuffer < 0 || con2fb.framebuffer >= FB_MAX) | 1183 | if (con2fb.framebuffer >= FB_MAX) |
1184 | return -EINVAL; | 1184 | return -EINVAL; |
1185 | if (!registered_fb[con2fb.framebuffer]) | 1185 | if (!registered_fb[con2fb.framebuffer]) |
1186 | request_module("fb%d", con2fb.framebuffer); | 1186 | request_module("fb%d", con2fb.framebuffer); |