[U-Boot] [PATCH 13/13] board/esd/cpci750/sdram_init.c: Fix GCC 4.6 build warnings

Wolfgang Denk wd at denx.de
Wed Nov 16 21:12:01 CET 2011


Dear Wolfgang Denk,

In message <1320866946-28235-13-git-send-email-wd at denx.de> you wrote:
> Fix:
> sdram_init.c: In function 'check_dimm':
> sdram_init.c:267:50: warning: variable 'trrd_clocks' set but not used
> [-Wunused-but-set-variable]
> sdram_init.c:267:37: warning: variable 'tras_clocks' set but not used
> [-Wunused-but-set-variable]
> sdram_init.c:267:24: warning: variable 'trcd_clocks' set but not used
> [-Wunused-but-set-variable]
> sdram_init.c:267:8: warning: variable 'trp_clocks' set but not used
> [-Wunused-but-set-variable]
> sdram_init.c:253:16: warning: variable 'spd_checksum' set but not used
> [-Wunused-but-set-variable]
> sdram_init.c: In function 'initdram':
> sdram_init.c:1693:14: warning: variable 's1' set but not used
> [-Wunused-but-set-variable]
> sdram_init.c:1693:6: warning: variable 's0' set but not used
> [-Wunused-but-set-variable]
> 
> Biggest part o the fix is converting the custom debug code to standard
> debug().
> 
> No attempts were made to cleanup the code.
> 
> Signed-off-by: Wolfgang Denk <wd at denx.de>
> Cc: Reinhard Arlt <reinhard.arlt at esd-electronics.com>
> ---
>  board/esd/cpci750/sdram_init.c |  508 +++++++++++++++-------------------------
>  1 files changed, 195 insertions(+), 313 deletions(-)

Applied, thanks. [Also removed the useless "#if 0" / "#endif" part.]

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: wd at denx.de
"What if" is a trademark of Hewlett Packard, so stop using it in your
sentences without permission, or risk being sued.


More information about the U-Boot mailing list