Commit 05513992 authored by Andrzej Hajda's avatar Andrzej Hajda Committed by Ralf Baechle

MIPS: Remove invalid check

Unsigned values cannot be lesser than zero.

The problem has been detected using proposed semantic patch
scripts/coccinelle/tests/unsigned_lesser_than_zero.cocci [1].

[ralf@linux-mips.org: Chris Dearman's original commit
9318c51a ([MIPS] MIPS32/MIPS64 secondary
cache management) introduced these less than zero checks in 2.6.18.]

[1]: http://permalink.gmane.org/gmane.linux.kernel/2038576Signed-off-by: default avatarAndrzej Hajda <a.hajda@samsung.com>
Cc: linux-kernel@vger.kernel.org
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: Markos Chandras <markos.chandras@imgtec.com>
Cc: Chris Dearman <chris.dearman@imgtec.com>
Cc: linux-mips@linux-mips.org
Patchwork: https://patchwork.linux-mips.org/patch/11165/Signed-off-by: default avatarRalf Baechle <ralf@linux-mips.org>
parent 32b88194
...@@ -162,13 +162,13 @@ static inline int __init mips_sc_probe(void) ...@@ -162,13 +162,13 @@ static inline int __init mips_sc_probe(void)
return 0; return 0;
tmp = (config2 >> 8) & 0x0f; tmp = (config2 >> 8) & 0x0f;
if (0 <= tmp && tmp <= 7) if (tmp <= 7)
c->scache.sets = 64 << tmp; c->scache.sets = 64 << tmp;
else else
return 0; return 0;
tmp = (config2 >> 0) & 0x0f; tmp = (config2 >> 0) & 0x0f;
if (0 <= tmp && tmp <= 7) if (tmp <= 7)
c->scache.ways = tmp + 1; c->scache.ways = tmp + 1;
else else
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment