diff options
| author | Jani Nikula <ext-jani.1.nikula@nokia.com> | 2010-06-01 18:33:54 +0400 |
|---|---|---|
| committer | Tomi Valkeinen <tomi.valkeinen@nokia.com> | 2010-08-05 17:52:09 +0400 |
| commit | 7f8b183059edc6c74654301431d520a91539e45f (patch) | |
| tree | 26d127d64813823894f00957144466d4c1c84d05 /drivers/video/omap2/omapfb | |
| parent | 86f2d7dd72e1ff4656107e42a12c999a7b9c26d4 (diff) | |
| download | linux-7f8b183059edc6c74654301431d520a91539e45f.tar.xz | |
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/video/omap2/omapfb')
| -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 8303776647fc..4abb1d17231b 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, if (r != 0) break; - if (regno < 0) { - r = -EINVAL; - break; - } - if (regno < 16) { u16 pal; pal = ((red >> (16 - var->red.length)) << |