summaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorAndrzej Hajda <a.hajda@samsung.com>2015-09-21 15:34:00 +0200
committerTomi Valkeinen <tomi.valkeinen@ti.com>2015-09-24 13:46:15 +0300
commitc50d5cd235210abffa1e88affbead1adbd5f294a (patch)
tree6447c26b4de4cc3095d94c9ddaa3bd00ce6decd2 /drivers/video
parent8a9d4626db312fe0a682c35ef43fe8d9160c2ecd (diff)
downloadop-kernel-dev-c50d5cd235210abffa1e88affbead1adbd5f294a.zip
op-kernel-dev-c50d5cd235210abffa1e88affbead1adbd5f294a.tar.gz
video/omap: remove invalid check
regno is unsigned so it cannot be negative. The problem has been detected using proposed semantic patch scripts/coccinelle/tests/unsigned_lesser_than_zero.cocci [1]. [1]: http://permalink.gmane.org/gmane.linux.kernel/2038576 Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r--drivers/video/fbdev/omap/omapfb_main.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/video/fbdev/omap/omapfb_main.c b/drivers/video/fbdev/omap/omapfb_main.c
index 1fb3ea3..393ae1b 100644
--- a/drivers/video/fbdev/omap/omapfb_main.c
+++ b/drivers/video/fbdev/omap/omapfb_main.c
@@ -276,11 +276,6 @@ static int _setcolreg(struct fb_info *info, u_int regno, u_int red, u_int green,
if (r != 0)
break;
- if (regno < 0) {
- r = -EINVAL;
- break;
- }
-
if (regno < 16) {
u16 pal;
pal = ((red >> (16 - var->red.length)) <<
OpenPOWER on IntegriCloud