[U-Boot] [PATCH 13/32] common/misc: sparse fixes

Tom Rini trini at ti.com
Thu Oct 25 19:32:12 CEST 2012


On Tue, Oct 16, 2012 at 07:28:29PM -0500, Kim Phillips wrote:

> command.c:44:38: error: bad constant expression
> console.c:537:18: warning: symbol 'search_device' was not declared. Should it be static?
> dlmalloc.c:1468:2: warning: Using plain integer as NULL pointer
> dlmalloc.c:1468:5: warning: Using plain integer as NULL pointer
> dlmalloc.c:2176:12: warning: Using plain integer as NULL pointer
> dlmalloc.c:2179:31: warning: Using plain integer as NULL pointer
> dlmalloc.c:2382:14: warning: Using plain integer as NULL pointer
> dlmalloc.c:2436:14: warning: Using plain integer as NULL pointer
> dlmalloc.c:2582:31: warning: Using plain integer as NULL pointer
> dlmalloc.c:2585:17: warning: Using plain integer as NULL pointer
> dlmalloc.c:2646:14: warning: Using plain integer as NULL pointer
> dlmalloc.c:2659:19: warning: Using plain integer as NULL pointer
> dlmalloc.c:2692:19: warning: Using plain integer as NULL pointer
> dlmalloc.c:2707:19: warning: Using plain integer as NULL pointer
> dlmalloc.c:2708:14: warning: Using plain integer as NULL pointer
> dlmalloc.c:2786:31: warning: Using plain integer as NULL pointer
> dlmalloc.c:2801:12: warning: Using plain integer as NULL pointer
> dlmalloc.c:2801:22: warning: Using plain integer as NULL pointer
> dlmalloc.c:2926:27: warning: Using plain integer as NULL pointer
> dlmalloc.c:2928:14: warning: Using plain integer as NULL pointer
> dlmalloc.c:2929:12: warning: Using plain integer as NULL pointer
> dlmalloc.c:3075:14: warning: Using plain integer as NULL pointer
> hush.c:292:14: warning: symbol 'last_return_code' was not declared. Should it be static?
> hush.c:293:5: warning: symbol 'nesting_level' was not declared. Should it be static?
> hush.c:2175:20: warning: Using plain integer as NULL pointer
> hush.c:2175:34: warning: Using plain integer as NULL pointer
> hush.c:2210:41: warning: Using plain integer as NULL pointer
> hush.c:2216:45: warning: Using plain integer as NULL pointer
> hush.c:2249:25: warning: Using plain integer as NULL pointer
> hush.c:2332:13: warning: symbol 'new_pipe' was not declared. Should it be static?
> hush.c:2390:5: warning: symbol 'reserved_word' was not declared. Should it be static?
> hush.c:2927:5: warning: symbol 'parse_stream' was not declared. Should it be static?
> hush.c:3127:6: warning: symbol 'mapset' was not declared. Should it be static?
> hush.c:3133:6: warning: symbol 'update_ifs_map' was not declared. Should it be static?
> hush.c:3161:5: warning: symbol 'parse_stream_outer' was not declared. Should it be static?
> hush.c:3295:34: warning: Using plain integer as NULL pointer
> hush.c:3631:5: warning: symbol 'do_showvar' was not declared. Should it be static
> image.c:1282:29: warning: Using plain integer as NULL pointer
> image.c:1315:41: warning: Using plain integer as NULL pointer
> image.c:1330:25: warning: Using plain integer as NULL pointer
> image.c:1706:25: warning: Using plain integer as NULL pointer
> main.c:510:10: warning: symbol 'hist_num' was not declared. Should it be static?
> main.c:512:5: warning: symbol 'hist_list' was not declared. Should it be static?
> main.c:513:6: warning: symbol 'hist_lines' was not declared. Should it be static?
> usb_storage.c:195:6: warning: symbol 'usb_show_progress' was not declared. Should it be static?
> usb_storage.c:440:48: warning: Using plain integer as NULL pointer
> usb_storage.c:503:5: warning: symbol 'usb_stor_BBB_comdat' was not declared. Should it be static?
> usb_storage.c:551:5: warning: symbol 'usb_stor_CB_comdat' was not declared. Should it be static?
> usb_storage.c:629:55: warning: Using plain integer as NULL pointer
> usb_storage.c:620:5: warning: symbol 'usb_stor_CBI_get_status' was not declared. Should it be static?
> usb_storage.c:675:43: warning: Using plain integer as NULL pointer
> usb_storage.c:668:5: warning: symbol 'usb_stor_BBB_clear_endpt_stall' was not declared. Should it be static?
> usb_storage.c:679:5: warning: symbol 'usb_stor_BBB_transport' was not declared. Should it be static?
> usb_storage.c:801:5: warning: symbol 'usb_stor_CB_transport' was not declared. Sh
> xyzModem.c:104:1: warning: symbol 'CYGACC_COMM_IF_GETC_TIMEOUT' was not declared. Should it be static?
> xyzModem.c:122:1: warning: symbol 'CYGACC_COMM_IF_PUTC' was not declared. Should it be static?
> xyzModem.c:169:1: warning: symbol 'parse_num' was not declared. Should it be stat
> 
> note: hush.c's nesting_level deleted because not used.
> 
> Signed-off-by: Kim Phillips <kim.phillips at freescale.com>

This causes problems such as:
$ uboot-build.sh netspace_lite_v2
Testing netspace_lite_v2 on -00399-gef44c78
Thu Oct 25 10:31:26 MST 2012
Configuring for netspace_lite_v2 - Board: lacie_kw, Options:NETSPACE_LITE_V2
console.c:537:26: error: static declaration of 'search_device' follows non-static declaration

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20121025/4c589c9e/attachment.pgp>


More information about the U-Boot mailing list