Patchwork [v2,13/15] media/video/s5p-tv: mark const init data with __initconst instead of __initdata

mail settings
Submitter Uwe Kleine-König
Date March 30, 2012, 8:05 p.m.
Message ID <>
Download mbox | patch
Permalink /patch/149743/
State New
Headers show


Uwe Kleine-König - March 30, 2012, 8:05 p.m.
As long as there is no other non-const variable marked __initdata in the
same compilation unit it doesn't hurt. If there were one however
compilation would fail with

	error: $variablename causes a section type conflict

because a section containing const variables is marked read only and so
cannot contain non-const variables.

Signed-off-by: Uwe Kleine-König <>
Cc: Kyungmin Park <>
Cc: Tomasz Stanislawski <>
Cc: Mauro Carvalho Chehab <>
no changes since (implicit) v1

 drivers/media/video/s5p-tv/mixer_drv.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


diff --git a/drivers/media/video/s5p-tv/mixer_drv.c b/drivers/media/video/s5p-tv/mixer_drv.c
index a2c0c25..edca065 100644
--- a/drivers/media/video/s5p-tv/mixer_drv.c
+++ b/drivers/media/video/s5p-tv/mixer_drv.c
@@ -461,7 +461,7 @@  static struct platform_driver mxr_driver __refdata = {
 static int __init mxr_init(void)
 	int i, ret;
-	static const char banner[] __initdata = KERN_INFO
+	static const char banner[] __initconst = KERN_INFO
 		"Samsung TV Mixer driver, "
 		"(c) 2010-2011 Samsung Electronics Co., Ltd.\n";