arm64: mvebu: incorrect check of fdt address cells
authorxypron.glpk@gmx.de <xypron.glpk@gmx.de>
Wed, 3 May 2017 21:31:58 +0000 (23:31 +0200)
committerTom Rini <trini@konsulko.com>
Fri, 5 May 2017 20:45:56 +0000 (16:45 -0400)
In dram_init_banksize there seems to be a typo concerning
a plausibility check of the fdt.
Testing sc > 2 twice does not make any sense.

The problem was indicated by cppcheck.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
arch/arm/mach-mvebu/arm64-common.c

index 1c0477a3cacd273d4b8e10f1f768a9aa17082c90..2ef5726905dedfa240e087eeb9c1006b178a8dea 100644 (file)
@@ -94,7 +94,7 @@ int dram_init_banksize(void)
 
        ac = fdt_address_cells(fdt, 0);
        sc = fdt_size_cells(fdt, 0);
-       if (ac < 1 || sc > 2 || sc < 1 || sc > 2) {
+       if (ac < 1 || ac > 2 || sc < 1 || sc > 2) {
                printf("invalid address/size cells\n");
                return -ENXIO;
        }