From: Josh Boyer <jwboyer@linux.vnet.ibm.com>
Date: Tue, 25 Dec 2007 01:46:06 +0000 (-0600)
Subject: [POWERPC] 4xx: Minor coding style cleanups for 4xx bootwrapper
X-Git-Tag: v2.6.25-rc1~1131^2~144
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=67196d7275fcbc62c41c34400855d3d3c95d8d1d;p=linux-2.6

[POWERPC] 4xx: Minor coding style cleanups for 4xx bootwrapper

Remove some unneeded braces and make a busy loop more obvious.

Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
---

diff --git a/arch/powerpc/boot/4xx.c b/arch/powerpc/boot/4xx.c
index 1a83efe274..33f25b6713 100644
--- a/arch/powerpc/boot/4xx.c
+++ b/arch/powerpc/boot/4xx.c
@@ -243,7 +243,8 @@ void ibm4xx_quiesce_eth(u32 *emac0, u32 *emac1)
 		*emac1 = EMAC_RESET;
 
 	mtdcr(DCRN_MAL0_CFG, MAL_RESET);
-	while (mfdcr(DCRN_MAL0_CFG) & MAL_RESET) {};
+	while (mfdcr(DCRN_MAL0_CFG) & MAL_RESET)
+		; /* loop until reset takes effect */
 }
 
 /* Read 4xx EBC bus bridge registers to get mappings of the peripheral
@@ -515,19 +516,17 @@ void ibm405gp_fixup_clocks(unsigned int sys_clk, unsigned int ser_clk)
 	opb = plb / opdv;
 	ebc = plb / epdv;
 
-	if (cpc0_cr0 & 0x80) {
+	if (cpc0_cr0 & 0x80)
 		/* uart0 uses the external clock */
 		uart0 = ser_clk;
-	} else {
+	else
 		uart0 = cpu / udiv;
-	}
 
-	if (cpc0_cr0 & 0x40) {
+	if (cpc0_cr0 & 0x40)
 		/* uart1 uses the external clock */
 		uart1 = ser_clk;
-	} else {
+	else
 		uart1 = cpu / udiv;
-	}
 
 	/* setup the timebase clock to tick at the cpu frequency */
 	cpc0_cr1 = cpc0_cr1 & ~0x00800000;