The if in the else branch is superfluous. We can use a simple if.
The problem was indicated by cppcheck.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de --- arch/arm/mach-stm32/stm32f1/flash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-stm32/stm32f1/flash.c b/arch/arm/mach-stm32/stm32f1/flash.c index 7d41f63733..9dc8e1198f 100644 --- a/arch/arm/mach-stm32/stm32f1/flash.c +++ b/arch/arm/mach-stm32/stm32f1/flash.c @@ -49,7 +49,7 @@ unsigned long flash_init(void) if (size <= STM32_MAX_BANK) { banks = 1; flash_info[0].sector_count = size >> 1; - } else if (size > STM32_MAX_BANK) { + } else { banks = 2; flash_info[0].sector_count = STM32_MAX_BANK >> 1; flash_info[1].sector_count = (size - STM32_MAX_BANK) >> 1;