[PATCH v4 2/2] mtd/uclinux: add a comment about why uclinux_ram_map must not be static

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Wed Jan 16 09:36:56 EST 2013


I was (at least) the second person trying to fix a warning by sparse, so
document in the code why this is a bad idea and add an extern declaration to
make sparse happy.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
Acked-by: Mike Frysinger <vapier at gentoo.org>
Acked-by: Greg Ungerer <gerg at uclinux.org>
---
No changes between v3 and v4.

Changes since v2,
id:1350027693-19528-2-git-send-email-u.kleine-koenig at pengutronix.de:

 - note in the commit log that an extern declaration was added for sparse

Changes since v1:

 - new patch
---
 drivers/mtd/maps/uclinux.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/mtd/maps/uclinux.c b/drivers/mtd/maps/uclinux.c
index f56d0aa..c1af83d 100644
--- a/drivers/mtd/maps/uclinux.c
+++ b/drivers/mtd/maps/uclinux.c
@@ -29,6 +29,12 @@
 #define MAP_NAME "ram"
 #endif
 
+/*
+ * Blackfin uses uclinux_ram_map during startup, so it must not be static.
+ * Provide a dummy declaration to make sparse happy.
+ */
+extern struct map_info uclinux_ram_map;
+
 struct map_info uclinux_ram_map = {
 	.name = MAP_NAME,
 	.size = 0,
-- 
1.7.10.4




More information about the linux-mtd mailing list