[PATCH] drivers/reset: appropriate __init annotation for const data

Nicolas Pitre nicolas.pitre at linaro.org
Mon Nov 23 06:47:37 PST 2015


On Mon, 23 Nov 2015, Philipp Zabel wrote:

> Hi Nicolas,
> 
> Am Samstag, den 21.11.2015, 20:56 -0500 schrieb Nicolas Pitre:
> > Init data marked const should be annotated with __initconst for
> > correctness and not __initdata.  This also fixes LTO builds that
> > otherwise fail with section mismatch errors.
> > 
> > Signed-off-by: Nicolas Pitre <nico at linaro.org>
> > Acked-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> > 
> > diff --git a/drivers/reset/reset-sunxi.c b/drivers/reset/reset-sunxi.c
> > index 3d95c87160..9353b2dd15 100644
> > --- a/drivers/reset/reset-sunxi.c
> > +++ b/drivers/reset/reset-sunxi.c
> > @@ -122,7 +122,7 @@ err_alloc:
> >   * our system, before we can even think of using a regular device
> >   * driver for it.
> >   */
> > -static const struct of_device_id sunxi_early_reset_dt_ids[] __initdata = {
> > +static const struct of_device_id sunxi_early_reset_dt_ids[] __initconst = {
> >  	{ .compatible = "allwinner,sun6i-a31-ahb1-reset", },
> >  	{ /* sentinel */ },
> >  };
> 
> Thank you for the patch, this change is already contained in the latest
> pull request sent to the arm-soc maintainers.

OK.  I initially posted it in July but it didn't show up in v4.2 nor in 
v4.3. I concluded it fell into a crack.


Nicolas



More information about the linux-arm-kernel mailing list