[adding Sylwester and removing my samsung.com e-mail which is no longer valid] On 11.09.2014 23:04, Uwe Kleine-König wrote: > The statement > > static const char *name[]; > > defines a modifiable array of pointers to constant chars. That is > > *name[0] = 'f'; > > is forbidden, but > > name[0] = "f"; > > is not. So marking an array that is defined as above with __initconst is > wrong. Either an additional const must be added such that the whole > definition reads: > > static const char *const name[] __initconst; > > or where this is not possible __initdata must be used. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > --- > drivers/clk/hisilicon/clk-hix5hd2.c | 6 ++-- > drivers/clk/mxs/clk-imx23.c | 12 ++++---- > drivers/clk/mxs/clk-imx28.c | 18 ++++++------ > drivers/clk/rockchip/clk.h | 2 +- > drivers/clk/samsung/clk-s5pv210.c | 56 ++++++++++++++++++------------------- For drivers/clk/samsung/* Acked-by: Tomasz Figa <tomasz.figa@xxxxxxxxx> Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html