Patchwork [2/2,v3] mtd/uclinux: add a comment about why uclinux_ram_map must not be static

login
register
mail settings
Submitter Uwe Kleine-König
Date Jan. 10, 2013, 4:31 p.m.
Message ID <1357835498-23904-2-git-send-email-u.kleine-koenig@pengutronix.de>
Download mbox | patch
Permalink /patch/211082/
State New
Headers show

Comments

Uwe Kleine-König - Jan. 10, 2013, 4:31 p.m.
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@pengutronix.de>
Acked-by: Mike Frysinger <vapier@gentoo.org>
Acked-by: Greg Ungerer <gerg@uclinux.org>
---
Changes since v2, sent with
Message-Id: 1350027693-19528-2-git-send-email-u.kleine-koenig@pengutronix.de:

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

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

Patch

diff --git a/drivers/mtd/maps/uclinux.c b/drivers/mtd/maps/uclinux.c
index b3a9c54..fe1c83c 100644
--- a/drivers/mtd/maps/uclinux.c
+++ b/drivers/mtd/maps/uclinux.c
@@ -23,6 +23,12 @@ 
 
 /****************************************************************************/
 
+/*
+ * 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 = {
 	.size = 0,
 };