[U-Boot] [PATCH 1/2] mx6slevk: Simplify the return value of setup_fec()

Fabio Estevam festevam at gmail.com
Mon Nov 17 02:44:41 CET 2014


From: Fabio Estevam <fabio.estevam at freescale.com>

We can simply the return the value from enable_fec_anatop_clock() to make the 
code smaller and simpler.

Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
---
 board/freescale/mx6slevk/mx6slevk.c | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/board/freescale/mx6slevk/mx6slevk.c b/board/freescale/mx6slevk/mx6slevk.c
index e76c343..8111edf 100644
--- a/board/freescale/mx6slevk/mx6slevk.c
+++ b/board/freescale/mx6slevk/mx6slevk.c
@@ -230,16 +230,11 @@ int board_eth_init(bd_t *bis)
 static int setup_fec(void)
 {
 	struct iomuxc *iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
-	int ret;
 
 	/* clear gpr1[14], gpr1[18:17] to select anatop clock */
 	clrsetbits_le32(&iomuxc_regs->gpr[1], IOMUX_GPR1_FEC_MASK, 0);
 
-	ret = enable_fec_anatop_clock(ENET_50MHz);
-	if (ret)
-		return ret;
-
-	return 0;
+	return enable_fec_anatop_clock(ENET_50MHz);
 }
 #endif
 
-- 
1.9.1



More information about the U-Boot mailing list