diff options
author | Jani Nikula <ext-jani.1.nikula@nokia.com> | 2010-06-01 10:33:54 -0400 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@nokia.com> | 2010-08-05 09:52:09 -0400 |
commit | 7f8b183059edc6c74654301431d520a91539e45f (patch) | |
tree | 26d127d64813823894f00957144466d4c1c84d05 /drivers | |
parent | 86f2d7dd72e1ff4656107e42a12c999a7b9c26d4 (diff) |
OMAP: DSS2: OMAPFB: Remove redundant color register range check
Unsigned regno can never be less than zero. Found by Coverity.
Signed-off-by: Jani Nikula <ext-jani.1.nikula@nokia.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/video/omap2/omapfb/omapfb-main.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/video/omap2/omapfb/omapfb-main.c b/drivers/video/omap2/omapfb/omapfb-main.c index 8303776647f..4abb1d17231 100644 --- a/drivers/video/omap2/omapfb/omapfb-main.c +++ b/drivers/video/omap2/omapfb/omapfb-main.c | |||
@@ -1208,11 +1208,6 @@ static int _setcolreg(struct fb_info *fbi, u_int regno, u_int red, u_int green, | |||
1208 | if (r != 0) | 1208 | if (r != 0) |
1209 | break; | 1209 | break; |
1210 | 1210 | ||
1211 | if (regno < 0) { | ||
1212 | r = -EINVAL; | ||
1213 | break; | ||
1214 | } | ||
1215 | |||
1216 | if (regno < 16) { | 1211 | if (regno < 16) { |
1217 | u16 pal; | 1212 | u16 pal; |
1218 | pal = ((red >> (16 - var->red.length)) << | 1213 | pal = ((red >> (16 - var->red.length)) << |