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

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk,

In message 1320458160-23136-3-git-send-email...@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 w...@denx.de
 Cc: Heiko Schocher h...@denx.de
 ---
  arch/powerpc/cpu/mpc5xxx/i2c.c |5 +
  1 files changed, 1 insertions(+), 4 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
A consultant is a person who borrows your watch, tells you what  time
it is, pockets the watch, and sends you a bill for it.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2011-11-04 Thread Wolfgang Denk
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 w...@denx.de
Cc: Heiko Schocher h...@denx.de
---
 arch/powerpc/cpu/mpc5xxx/i2c.c |5 +
 1 files changed, 1 insertions(+), 4 deletions(-)

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);
}
-- 
1.7.6.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot