diff options
author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-06-28 15:33:30 +1000 |
---|---|---|
committer | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-06-29 13:02:13 +0200 |
commit | 4d7539eefdedc4297e1529ee94df7165e2076fa1 (patch) | |
tree | 3e768b7c652ba58a4b653f19d5dedf9cb235f5f4 /src/cpu/x86/pae | |
parent | 2f9b3afc84de76c75dd783adc3876da3c600e6d8 (diff) |
cpu/x86/pae/pgtbl.c: Unsigned comparison < 0 always false
Comparison of unsigned expression < 0 is always false.
Change-Id: Idf4e7846b50f4376a5d33515681efbd773d1caca
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/6146
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Diffstat (limited to 'src/cpu/x86/pae')
-rw-r--r-- | src/cpu/x86/pae/pgtbl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cpu/x86/pae/pgtbl.c b/src/cpu/x86/pae/pgtbl.c index 935473828c..9be415d8de 100644 --- a/src/cpu/x86/pae/pgtbl.c +++ b/src/cpu/x86/pae/pgtbl.c @@ -61,7 +61,7 @@ void *map_2M_page(unsigned long page) void *result; int i; index = cpu_index(); - if ((index < 0) || (index >= CONFIG_MAX_CPUS)) { + if (index >= CONFIG_MAX_CPUS) { return MAPPING_ERROR; } window = page >> 10; |