aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPing Li <pingli@codeaurora.org>2015-02-25 18:39:36 -0800
committerSimon Shields <keepcalm444@gmail.com>2016-12-13 00:14:53 +1100
commitf78455200fe1885ee2a9b1a1ae7176a51ff77aa9 (patch)
treee81a70f6dc328adc78f8108c0c0fe4750b595e62
parentb4331df968b885b4df986753ae2361f47d09589c (diff)
downloadkernel_samsung_smdk4412-f78455200fe1885ee2a9b1a1ae7176a51ff77aa9.zip
kernel_samsung_smdk4412-f78455200fe1885ee2a9b1a1ae7176a51ff77aa9.tar.gz
kernel_samsung_smdk4412-f78455200fe1885ee2a9b1a1ae7176a51ff77aa9.tar.bz2
fbcmap: Remove unnecessary condition check
In fb_set_user_cmap function, cmap->start variable is an unsigned integer, it doesn't need a condition check with the sign. Change-Id: I355ddb7edcc085ee52e4054833b687640376eee3 Signed-off-by: Ping Li <pingli@codeaurora.org>
-rw-r--r--drivers/video/fbcmap.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/fbcmap.c b/drivers/video/fbcmap.c
index 31e93a5..f67cdcd 100644
--- a/drivers/video/fbcmap.c
+++ b/drivers/video/fbcmap.c
@@ -297,8 +297,8 @@ int fb_set_user_cmap(struct fb_cmap_user *cmap, struct fb_info *info)
rc = -ENODEV;
goto out;
}
- if (cmap->start < 0 || (!info->fbops->fb_setcolreg &&
- !info->fbops->fb_setcmap)) {
+ if (!info->fbops->fb_setcolreg &&
+ !info->fbops->fb_setcmap) {
rc = -EINVAL;
goto out1;
}