A switch statement fits better in this case,
specially considering we have a few extra
frequencies to use.

Signed-off-by: Felipe Balbi <[email protected]>
---
 board/ti/am43xx/board.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/board/ti/am43xx/board.c b/board/ti/am43xx/board.c
index 9874773..c418e68 100644
--- a/board/ti/am43xx/board.c
+++ b/board/ti/am43xx/board.c
@@ -381,11 +381,14 @@ void scale_vcores(void)
        if (i2c_probe(TPS65218_CHIP_PM))
                return;
 
-       if (mpu_params->m == 1000) {
+       switch (mpu_params->m) {
+       case 1000:
                mpu_vdd = TPS65218_DCDC_VOLT_SEL_1330MV;
-       } else if (mpu_params->m == 600) {
+               break;
+       case 600:
                mpu_vdd = TPS65218_DCDC_VOLT_SEL_1100MV;
-       } else {
+               break;
+       default:
                puts("Unknown MPU clock, not scaling\n");
                return;
        }
-- 
2.2.0

_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to