[U-Boot] [PATCH 12/32] common/cmd_*.c: sparse fixes

Tom Rini trini at ti.com
Thu Oct 25 01:49:19 CEST 2012


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

[snip]
> cmd_mtdparts.c:157:4: warning: symbol 'current_mtd_partnum' was not declared. Should it be static?
[snip]
>  /* current active device and partition number */
> -struct mtd_device *current_mtd_dev = NULL;
> -u8 current_mtd_partnum = 0;
> +static struct mtd_device *current_mtd_dev = NULL;
> +static u8 current_mtd_partnum = 0;

This is wrong, cmd_jffs.c also uses current_mtd_partnum and has an
extern in the C file for it :(

-- 
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/20121024/567dbbce/attachment.pgp>


More information about the U-Boot mailing list