[U-Boot] [PATCH 01/12] ls2080aqds: eth: add missing braces

Andre Przywara andre.przywara at arm.com
Wed Nov 16 01:50:05 CET 2016


The whole error message should be within the else clause, not just
the first part.
Pointed out by GCC 6.2's -Wmisleading-indentation warning.

Signed-off-by: Andre Przywara <andre.przywara at arm.com>
---
 board/freescale/ls2080aqds/eth.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/board/freescale/ls2080aqds/eth.c b/board/freescale/ls2080aqds/eth.c
index 95ff68b..7bf7a5b 100644
--- a/board/freescale/ls2080aqds/eth.c
+++ b/board/freescale/ls2080aqds/eth.c
@@ -196,12 +196,13 @@ static void sgmii_configure_repeater(int serdes_port)
 					       value);
 					i = 5;
 					j = 5;
-				} else
+				} else {
 					printf("DPMAC %d :PHY is failed to ",
 					       dpmac_id);
 					printf("configure the repeater 0x%x\n",
 					       value);
 				}
+			}
 		}
 	}
 error:
-- 
2.8.2



More information about the U-Boot mailing list