diff options
author | Jani Nikula <ext-jani.1.nikula@nokia.com> | 2010-06-01 17:33:54 +0300 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@nokia.com> | 2010-08-05 16:52:09 +0300 |
commit | 7f8b183059edc6c74654301431d520a91539e45f (patch) | |
tree | 26d127d64813823894f00957144466d4c1c84d05 /drivers/video | |
parent | 86f2d7dd72e1ff4656107e42a12c999a7b9c26d4 (diff) | |
download | op-kernel-dev-7f8b183059edc6c74654301431d520a91539e45f.zip op-kernel-dev-7f8b183059edc6c74654301431d520a91539e45f.tar.gz |
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')
-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 8303776..4abb1d1 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)) << |