aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorChen Gang <gang.chen@asianux.com>2013-08-19 22:33:56 -0400
committerTomi Valkeinen <tomi.valkeinen@ti.com>2013-08-30 04:41:24 -0400
commit9bb40585c9d934db3585e1f422e2ffb1dd5e9631 (patch)
treeae644be218e1fd1ddeab3065ccb6bdef35d7c198 /drivers/video
parent51617d1bc8f37df703295cf3fd472da15b076ecb (diff)
drivers: video: fbcmap: remove the redundency and incorrect checkings
fb_set_cmap() already checks the parameters, so need remove the redundancy checking. This redundancy checking is also incorrect, the related warning: drivers/video/fbcmap.c:288:2: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits] Signed-off-by: Chen Gang <gang.chen@asianux.com> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r--drivers/video/fbcmap.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/video/fbcmap.c b/drivers/video/fbcmap.c
index 5c3960da755a..f89245b8ba8e 100644
--- a/drivers/video/fbcmap.c
+++ b/drivers/video/fbcmap.c
@@ -285,13 +285,8 @@ int fb_set_user_cmap(struct fb_cmap_user *cmap, struct fb_info *info)
285 rc = -ENODEV; 285 rc = -ENODEV;
286 goto out; 286 goto out;
287 } 287 }
288 if (cmap->start < 0 || (!info->fbops->fb_setcolreg && 288
289 !info->fbops->fb_setcmap)) {
290 rc = -EINVAL;
291 goto out1;
292 }
293 rc = fb_set_cmap(&umap, info); 289 rc = fb_set_cmap(&umap, info);
294out1:
295 unlock_fb_info(info); 290 unlock_fb_info(info);
296out: 291out:
297 fb_dealloc_cmap(&umap); 292 fb_dealloc_cmap(&umap);