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.g...@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;
-- 
2.13.2

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to