[U-Boot,02/56] arch/powerpc/cpu/mpc5xxx/i2c.c: fix GC 4.6 build warnings

Submitted by Wolfgang Denk on Nov. 5, 2011, 1:55 a.m.

Details

Message ID 1320458160-23136-3-git-send-email-wd@denx.de
State Accepted
Commit 93067497527a2828c0c2b0c8dd7c498241d75d79
Headers show

Commit Message

Wolfgang Denk Nov. 5, 2011, 1:55 a.m.
Fix:
i2c.c: In function 'wait_for_bb':
i2c.c:104:16: warning: variable 'temp' set but not used
[-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk <wd@denx.de>
Cc: Heiko Schocher <hs@denx.de>
---
 arch/powerpc/cpu/mpc5xxx/i2c.c |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

Comments

Wolfgang Denk Nov. 7, 2011, 9:09 p.m.
Dear Wolfgang Denk,

In message <1320458160-23136-3-git-send-email-wd@denx.de> you wrote:
> Fix:
> i2c.c: In function 'wait_for_bb':
> i2c.c:104:16: warning: variable 'temp' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> Cc: Heiko Schocher <hs@denx.de>
> ---
>  arch/powerpc/cpu/mpc5xxx/i2c.c |    5 +----
>  1 files changed, 1 insertions(+), 4 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

Patch hide | download patch | download mbox

diff --git a/arch/powerpc/cpu/mpc5xxx/i2c.c b/arch/powerpc/cpu/mpc5xxx/i2c.c
index f9d293b..b423d2f 100644
--- a/arch/powerpc/cpu/mpc5xxx/i2c.c
+++ b/arch/powerpc/cpu/mpc5xxx/i2c.c
@@ -100,14 +100,11 @@  static int wait_for_bb(void)
 	status = mpc_reg_in(&regs->msr);
 
 	while (timeout-- && (status & I2C_BB)) {
-#if 1
-		volatile int temp;
 		mpc_reg_out(&regs->mcr, I2C_STA, I2C_STA);
-		temp = mpc_reg_in(&regs->mdr);
+		(void)mpc_reg_in(&regs->mdr);
 		mpc_reg_out(&regs->mcr, 0, I2C_STA);
 		mpc_reg_out(&regs->mcr, 0, 0);
 		mpc_reg_out(&regs->mcr, I2C_EN, 0);
-#endif
 		udelay(15);
 		status = mpc_reg_in(&regs->msr);
 	}