[U-Boot] [PATCH 05/14] DB64360: Fix GCC 4.6 build warnings

Wolfgang Denk wd at denx.de
Fri Dec 2 00:11:29 CET 2011


Dear Wolfgang Denk,

In message <1322641077-829-6-git-send-email-wd at denx.de> you wrote:
> Fix:
> db64360.c: In function 'debug_led':
> db64360.c:413:6: warning: variable 'dummy' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_real_open':
> mv_eth.c:424:6: warning: variable 'port_status' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_stop':
> mv_eth.c:642:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_xmit':
> mv_eth.c:718:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_receive':
> mv_eth.c:803:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_get_stats':
> mv_eth.c:902:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_update_stat':
> mv_eth.c:930:24: warning: variable 'dummy' set but not used [-Wunused-but-set-variable]
> mv_eth.c:929:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_print_stat':
> mv_eth.c:1011:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable]
> mv_eth.c: In function 'eth_clear_mib_counters':
> mv_eth.c:2068:15: warning: variable 'dummy' set but not used [-Wunused-but-set-variable]
> sdram_init.c: In function 'check_dimm':
> sdram_init.c:289:50: warning: variable 'trrd_clocks' set but not used [-Wunused-but-set-variable]
> sdram_init.c:289:37: warning: variable 'tras_clocks' set but not used [-Wunused-but-set-variable]
> sdram_init.c:289:24: warning: variable 'trcd_clocks' set but not used [-Wunused-but-set-variable]
> sdram_init.c:289:8: warning: variable 'trp_clocks' set but not used [-Wunused-but-set-variable]
> sdram_init.c:279:34: warning: variable 'devicesForErrCheck' set but not used [-Wunused-but-set-variable]
> sdram_init.c: In function 'setup_sdram':
> sdram_init.c:1256:13: warning: variable 'check' set but not used [-Wunused-but-set-variable]
> sdram_init.c: In function 'initdram':
> sdram_init.c:1735:25: warning: variable 'check' set but not used [-Wunused-but-set-variable]
> sdram_init.c:1733:14: warning: variable 's1' set but not used [-Wunused-but-set-variable]
> sdram_init.c:1733:6: warning: variable 's0' set but not used [-Wunused-but-set-variable]
> 
> Note: no attempt was make to clean up the mess coding style and
> other issues in sdram_init.c
> 
> Signed-off-by: Wolfgang Denk <wd at denx.de>
> ---
> WARNING: space prohibited between function name and open parenthesis '('
> WARNING: line over 80 characters
> WARNING: unnecessary whitespace before a quoted newline
> ...
> total: 0 errors, 101 warnings, 1327 lines checked
> 
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
> Warnings ignored, no further cleanup attempted.
> 
>  board/Marvell/db64360/db64360.c    |    3 +-
>  board/Marvell/db64360/mv_eth.c     |   33 +--
>  board/Marvell/db64360/sdram_init.c |  517 +++++++++++++++++-------------------
>  3 files changed, 254 insertions(+), 299 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: wd at denx.de
In general, if you think something isn't in Perl, try it out, because
it usually is :-) - Larry Wall in <1991Jul31.174523.9447 at netlabs.com>


More information about the U-Boot mailing list