diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-01-13 11:14:59 -0500 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2015-01-26 07:43:06 -0500 |
commit | 070a178331d056934fccf00bdc001eb9647ea029 (patch) | |
tree | 80168997e65bfe355ef42a2ed3756f7eef673ee6 | |
parent | 5b28f0fabbd745b2cc42a89f5b1d87757a68eed7 (diff) |
video: hgafb: remove unneeded comparison
var->yoffset is of the type __u32, hence the comparison will always
be false.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
-rw-r--r-- | drivers/video/fbdev/hgafb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/video/fbdev/hgafb.c b/drivers/video/fbdev/hgafb.c index 5ff9fe2116a4..15d3ccff2965 100644 --- a/drivers/video/fbdev/hgafb.c +++ b/drivers/video/fbdev/hgafb.c | |||
@@ -417,8 +417,7 @@ static int hgafb_pan_display(struct fb_var_screeninfo *var, | |||
417 | struct fb_info *info) | 417 | struct fb_info *info) |
418 | { | 418 | { |
419 | if (var->vmode & FB_VMODE_YWRAP) { | 419 | if (var->vmode & FB_VMODE_YWRAP) { |
420 | if (var->yoffset < 0 || | 420 | if (var->yoffset >= info->var.yres_virtual || |
421 | var->yoffset >= info->var.yres_virtual || | ||
422 | var->xoffset) | 421 | var->xoffset) |
423 | return -EINVAL; | 422 | return -EINVAL; |
424 | } else { | 423 | } else { |