Skip to content

Commit

Permalink
ARM: appropriate __init annotation for const data
Browse files Browse the repository at this point in the history
Init data marked const should be annotated with __initconst for
correctness and not __initdata.  In some cases the array gathering
references to that data has to be marked const as well. This fixes
LTO builds that otherwise fail with section mismatch errors.

Signed-off-by: Nicolas Pitre <[email protected]>
Signed-off-by: Olof Johansson <[email protected]>
  • Loading branch information
Nicolas Pitre authored and olofj committed Jul 28, 2015
1 parent a115681 commit 19c233b
Show file tree
Hide file tree
Showing 40 changed files with 50 additions and 50 deletions.
2 changes: 1 addition & 1 deletion arch/arm/kernel/psci.c
Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,7 @@ static int psci_0_1_init(struct device_node *np)
return err;
}

static const struct of_device_id psci_of_match[] __initconst = {
static const struct of_device_id const psci_of_match[] __initconst = {
{ .compatible = "arm,psci", .data = psci_0_1_init},
{ .compatible = "arm,psci-0.2", .data = psci_0_2_init},
{},
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-at91/at91rm9200.c
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ static void __init at91rm9200_dt_device_init(void)
at91rm9200_pm_init();
}

static const char *at91rm9200_dt_board_compat[] __initconst = {
static const char *const at91rm9200_dt_board_compat[] __initconst = {
"atmel,at91rm9200",
NULL
};
Expand Down
6 changes: 3 additions & 3 deletions arch/arm/mach-at91/at91sam9.c
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ static void __init at91sam9_dt_device_init(void)
at91sam9260_pm_init();
}

static const char *at91_dt_board_compat[] __initconst = {
static const char *const at91_dt_board_compat[] __initconst = {
"atmel,at91sam9",
NULL
};
Expand All @@ -89,7 +89,7 @@ static void __init at91sam9g45_dt_device_init(void)
at91sam9g45_pm_init();
}

static const char *at91sam9g45_board_compat[] __initconst = {
static const char *const at91sam9g45_board_compat[] __initconst = {
"atmel,at91sam9g45",
NULL
};
Expand All @@ -106,7 +106,7 @@ static void __init at91sam9x5_dt_device_init(void)
at91sam9x5_pm_init();
}

static const char *at91sam9x5_board_compat[] __initconst = {
static const char *const at91sam9x5_board_compat[] __initconst = {
"atmel,at91sam9x5",
"atmel,at91sam9n12",
NULL
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-at91/pm.c
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ static void at91sam9_sdram_standby(void)
at91_ramc_write(1, AT91_SDRAMC_LPR, saved_lpr1);
}

static const struct of_device_id ramc_ids[] __initconst = {
static const struct of_device_id const ramc_ids[] __initconst = {
{ .compatible = "atmel,at91rm9200-sdramc", .data = at91rm9200_standby },
{ .compatible = "atmel,at91sam9260-sdramc", .data = at91sam9_sdram_standby },
{ .compatible = "atmel,at91sam9g45-ddramc", .data = at91_ddr_standby },
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-at91/sama5.c
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ static void __init sama5_dt_device_init(void)
at91sam9x5_pm_init();
}

static const char *sama5_dt_board_compat[] __initconst = {
static const char *const sama5_dt_board_compat[] __initconst = {
"atmel,sama5",
NULL
};
Expand All @@ -63,7 +63,7 @@ DT_MACHINE_START(sama5_dt, "Atmel SAMA5")
.dt_compat = sama5_dt_board_compat,
MACHINE_END

static const char *sama5_alt_dt_board_compat[] __initconst = {
static const char *const sama5_alt_dt_board_compat[] __initconst = {
"atmel,sama5d4",
NULL
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-bcm/bcm_5301x.c
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ static void __init bcm5301x_init_early(void)
"imprecise external abort");
}

static const char __initconst *bcm5301x_dt_compat[] = {
static const char *const bcm5301x_dt_compat[] __initconst = {
"brcm,bcm4708",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-bcm/bcm_kona_smc.c
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ struct bcm_kona_smc_data {
unsigned result;
};

static const struct of_device_id bcm_kona_smc_ids[] __initconst = {
static const struct of_device_id const bcm_kona_smc_ids[] __initconst = {
{.compatible = "brcm,kona-smc"},
{.compatible = "bcm,kona-smc"}, /* deprecated name */
{},
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-clps711x/board-autcpu12.c
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ static struct platform_device autcpu12_mmgpio_pdev __initdata = {
},
};

static const struct gpio autcpu12_gpios[] __initconst = {
static const struct gpio const autcpu12_gpios[] __initconst = {
{ AUTCPU12_DPOT_CS, GPIOF_OUT_INIT_HIGH, "DPOT CS" },
{ AUTCPU12_DPOT_CLK, GPIOF_OUT_INIT_LOW, "DPOT CLK" },
{ AUTCPU12_DPOT_UD, GPIOF_OUT_INIT_LOW, "DPOT UD" },
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-cns3xxx/core.c
Original file line number Diff line number Diff line change
Expand Up @@ -346,7 +346,7 @@ static struct usb_ohci_pdata cns3xxx_usb_ohci_pdata = {
.power_off = csn3xxx_usb_power_off,
};

static struct of_dev_auxdata cns3xxx_auxdata[] __initconst = {
static const struct of_dev_auxdata const cns3xxx_auxdata[] __initconst = {
{ "intel,usb-ehci", CNS3XXX_USB_BASE, "ehci-platform", &cns3xxx_usb_ehci_pdata },
{ "intel,usb-ohci", CNS3XXX_USB_OHCI_BASE, "ohci-platform", &cns3xxx_usb_ohci_pdata },
{ "cavium,cns3420-ahci", CNS3XXX_SATA2_BASE, "ahci", NULL },
Expand Down Expand Up @@ -399,7 +399,7 @@ static void __init cns3xxx_init(void)
cns3xxx_auxdata, NULL);
}

static const char *cns3xxx_dt_compat[] __initdata = {
static const char *const cns3xxx_dt_compat[] __initconst = {
"cavium,cns3410",
"cavium,cns3420",
NULL,
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-davinci/da850.c
Original file line number Diff line number Diff line change
Expand Up @@ -715,7 +715,7 @@ const short da850_lcdcntl_pins[] __initconst = {
-1
};

const short da850_vpif_capture_pins[] __initdata = {
const short da850_vpif_capture_pins[] __initconst = {
DA850_VPIF_DIN0, DA850_VPIF_DIN1, DA850_VPIF_DIN2, DA850_VPIF_DIN3,
DA850_VPIF_DIN4, DA850_VPIF_DIN5, DA850_VPIF_DIN6, DA850_VPIF_DIN7,
DA850_VPIF_DIN8, DA850_VPIF_DIN9, DA850_VPIF_DIN10, DA850_VPIF_DIN11,
Expand All @@ -725,7 +725,7 @@ const short da850_vpif_capture_pins[] __initdata = {
-1
};

const short da850_vpif_display_pins[] __initdata = {
const short da850_vpif_display_pins[] __initconst = {
DA850_VPIF_DOUT0, DA850_VPIF_DOUT1, DA850_VPIF_DOUT2, DA850_VPIF_DOUT3,
DA850_VPIF_DOUT4, DA850_VPIF_DOUT5, DA850_VPIF_DOUT6, DA850_VPIF_DOUT7,
DA850_VPIF_DOUT8, DA850_VPIF_DOUT9, DA850_VPIF_DOUT10,
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-davinci/da8xx-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

#define DA8XX_NUM_UARTS 3

static const struct of_device_id da8xx_irq_match[] __initconst = {
static const struct of_device_id const da8xx_irq_match[] __initconst = {
{ .compatible = "ti,cp-intc", .data = cp_intc_of_init, },
{ }
};
Expand Down Expand Up @@ -59,7 +59,7 @@ static void __init da850_init_machine(void)

}

static const char *da850_boards_compat[] __initdata = {
static const char *const da850_boards_compat[] __initconst = {
"enbw,cmc",
"ti,da850-evm",
"ti,da850",
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-digicolor/digicolor.c
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

#include <asm/mach/arch.h>

static const char *digicolor_dt_compat[] __initconst = {
static const char *const digicolor_dt_compat[] __initconst = {
"cnxt,cx92755",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-imx/mach-imx7d.c
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ static void __init imx7d_init_irq(void)
irqchip_init();
}

static const char *imx7d_dt_compat[] __initconst = {
static const char *const imx7d_dt_compat[] __initconst = {
"fsl,imx7d",
NULL,
};
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-lpc32xx/phy3250.c
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ static struct lpc32xx_mlc_platform_data lpc32xx_mlc_data = {
.dma_filter = pl08x_filter_id,
};

static const struct of_dev_auxdata lpc32xx_auxdata_lookup[] __initconst = {
static const struct of_dev_auxdata const lpc32xx_auxdata_lookup[] __initconst = {
OF_DEV_AUXDATA("arm,pl022", 0x20084000, "dev:ssp0", NULL),
OF_DEV_AUXDATA("arm,pl022", 0x2008C000, "dev:ssp1", NULL),
OF_DEV_AUXDATA("arm,pl110", 0x31040000, "dev:clcd", &lpc32xx_clcd_data),
Expand Down Expand Up @@ -248,7 +248,7 @@ static void __init lpc3250_machine_init(void)
lpc32xx_auxdata_lookup, NULL);
}

static char const *lpc32xx_dt_compat[] __initdata = {
static const char *const lpc32xx_dt_compat[] __initconst = {
"nxp,lpc3220",
"nxp,lpc3230",
"nxp,lpc3240",
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-mmp/mmp-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@

extern void __init mmp_dt_init_timer(void);

static const char *pxa168_dt_board_compat[] __initdata = {
static const char *const pxa168_dt_board_compat[] __initconst = {
"mrvl,pxa168-aspenite",
NULL,
};

static const char *pxa910_dt_board_compat[] __initdata = {
static const char *const pxa910_dt_board_compat[] __initconst = {
"mrvl,pxa910-dkb",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-mmp/mmp2-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ static void __init mmp_init_time(void)
of_clk_init(NULL);
}

static const char *mmp2_dt_board_compat[] __initdata = {
static const char *const mmp2_dt_board_compat[] __initconst = {
"mrvl,mmp2-brownstone",
NULL,
};
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-mxs/mach-mxs.c
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,7 @@ static void __init apx4devkit_init(void)
#define TX28_FEC_PHY_RESET MXS_GPIO_NR(4, 13)
#define TX28_FEC_nINT MXS_GPIO_NR(4, 5)

static const struct gpio tx28_gpios[] __initconst = {
static const struct gpio const tx28_gpios[] __initconst = {
{ ENET0_MDC__GPIO_4_0, GPIOF_OUT_INIT_LOW, "GPIO_4_0" },
{ ENET0_MDIO__GPIO_4_1, GPIOF_OUT_INIT_LOW, "GPIO_4_1" },
{ ENET0_RX_EN__GPIO_4_2, GPIOF_OUT_INIT_LOW, "GPIO_4_2" },
Expand Down Expand Up @@ -528,7 +528,7 @@ static void mxs_restart(enum reboot_mode mode, const char *cmd)
soft_restart(0);
}

static const char *mxs_dt_compat[] __initdata = {
static const char *const mxs_dt_compat[] __initconst = {
"fsl,imx28",
"fsl,imx23",
NULL,
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-omap2/prm_common.c
Original file line number Diff line number Diff line change
Expand Up @@ -705,7 +705,7 @@ static struct omap_prcm_init_data scrm_data __initdata = {
};
#endif

static const struct of_device_id omap_prcm_dt_match_table[] __initconst = {
static const struct of_device_id const omap_prcm_dt_match_table[] __initconst = {
#ifdef CONFIG_SOC_AM33XX
{ .compatible = "ti,am3-prcm", .data = &am3_prm_data },
#endif
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-omap2/vc.c
Original file line number Diff line number Diff line change
Expand Up @@ -563,7 +563,7 @@ struct i2c_init_data {
u8 hsscll_12;
};

static const __initdata struct i2c_init_data omap4_i2c_timing_data[] = {
static const struct i2c_init_data const omap4_i2c_timing_data[] __initconst = {
{
.load = 50,
.loadbits = 0x3,
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-omap2/voltagedomains3xxx_data.c
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ static struct voltagedomain *voltagedomains_am35xx[] __initdata = {
};


static const char *sys_clk_name __initdata = "sys_ck";
static const char *const sys_clk_name __initconst = "sys_ck";

void __init omap3xxx_voltagedomains_init(void)
{
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-omap2/voltagedomains44xx_data.c
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ static struct voltagedomain *voltagedomains_omap4[] __initdata = {
NULL,
};

static const char *sys_clk_name __initdata = "sys_clkin_ck";
static const char *const sys_clk_name __initconst = "sys_clkin_ck";

void __init omap44xx_voltagedomains_init(void)
{
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-omap2/voltagedomains54xx_data.c
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ static struct voltagedomain *voltagedomains_omap5[] __initdata = {
NULL,
};

static const char *sys_clk_name __initdata = "sys_clkin";
static const char *const sys_clk_name __initconst = "sys_clkin";

void __init omap54xx_voltagedomains_init(void)
{
Expand Down
4 changes: 2 additions & 2 deletions arch/arm/mach-pxa/pxa-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
#include "generic.h"

#ifdef CONFIG_PXA3xx
static const struct of_dev_auxdata pxa3xx_auxdata_lookup[] __initconst = {
static const struct of_dev_auxdata const pxa3xx_auxdata_lookup[] __initconst = {
OF_DEV_AUXDATA("mrvl,pxa-uart", 0x40100000, "pxa2xx-uart.0", NULL),
OF_DEV_AUXDATA("mrvl,pxa-uart", 0x40200000, "pxa2xx-uart.1", NULL),
OF_DEV_AUXDATA("mrvl,pxa-uart", 0x40700000, "pxa2xx-uart.2", NULL),
Expand All @@ -39,7 +39,7 @@ static void __init pxa3xx_dt_init(void)
pxa3xx_auxdata_lookup, NULL);
}

static const char *pxa3xx_dt_board_compat[] __initdata = {
static const char *const pxa3xx_dt_board_compat[] __initconst = {
"marvell,pxa300",
"marvell,pxa310",
"marvell,pxa320",
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-realview/realview-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
#include <asm/hardware/cache-l2x0.h>
#include "core.h"

static const char *realview_dt_platform_compat[] __initconst = {
static const char *const realview_dt_platform_compat[] __initconst = {
"arm,realview-eb",
"arm,realview-pb1176",
"arm,realview-pb11mp",
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-s3c24xx/mach-s3c2416-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ static void __init s3c2416_dt_machine_init(void)
s3c_pm_init();
}

static char const *s3c2416_dt_compat[] __initdata = {
static const char *const s3c2416_dt_compat[] __initconst = {
"samsung,s3c2416",
"samsung,s3c2450",
NULL
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-s3c64xx/mach-s3c64xx-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ static void s3c64xx_dt_restart(enum reboot_mode mode, const char *cmd)
soft_restart(0);
}

static char const *s3c64xx_dt_compat[] __initdata = {
static const char *const s3c64xx_dt_compat[] __initconst = {
"samsung,s3c6400",
"samsung,s3c6410",
NULL
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/board-bockw-reference.c
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ static void __init bockw_init(void)
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char *bockw_boards_compat_dt[] __initdata = {
static const char *const bockw_boards_compat_dt[] __initconst = {
"renesas,bockw-reference",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/board-bockw.c
Original file line number Diff line number Diff line change
Expand Up @@ -723,7 +723,7 @@ static void __init bockw_init_late(void)
ADD_USB_FUNC_DEVICE_IF_POSSIBLE();
}

static const char *bockw_boards_compat_dt[] __initdata = {
static const char *const bockw_boards_compat_dt[] __initconst = {
"renesas,bockw",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/board-marzen-reference.c
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ static void __init marzen_init(void)
r8a7779_init_irq_extpin_dt(1); /* IRQ1 as individual interrupt */
}

static const char *marzen_boards_compat_dt[] __initdata = {
static const char *const marzen_boards_compat_dt[] __initconst = {
"renesas,marzen",
"renesas,marzen-reference",
NULL,
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/board-marzen.c
Original file line number Diff line number Diff line change
Expand Up @@ -330,7 +330,7 @@ static void __init marzen_init(void)
platform_add_devices(marzen_devices, ARRAY_SIZE(marzen_devices));
}

static const char *marzen_boards_compat_dt[] __initdata = {
static const char *const marzen_boards_compat_dt[] __initconst = {
"renesas,marzen",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/setup-r7s72100.c
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

#include "common.h"

static const char *r7s72100_boards_compat_dt[] __initdata = {
static const char *const r7s72100_boards_compat_dt[] __initconst = {
"renesas,r7s72100",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/setup-r8a73a4.c
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

#include "common.h"

static const char *r8a73a4_boards_compat_dt[] __initdata = {
static const char *const r8a73a4_boards_compat_dt[] __initconst = {
"renesas,r8a73a4",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/setup-r8a7740.c
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ static void __init r8a7740_generic_init(void)
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char *r8a7740_boards_compat_dt[] __initdata = {
static const char *const r8a7740_boards_compat_dt[] __initconst = {
"renesas,r8a7740",
NULL,
};
Expand Down
2 changes: 1 addition & 1 deletion arch/arm/mach-shmobile/setup-r8a7778.c
Original file line number Diff line number Diff line change
Expand Up @@ -615,7 +615,7 @@ void __init r8a7778_init_irq_dt(void)
iounmap(base);
}

static const char *r8a7778_compat_dt[] __initdata = {
static const char *const r8a7778_compat_dt[] __initconst = {
"renesas,r8a7778",
NULL,
};
Expand Down
Loading

0 comments on commit 19c233b

Please sign in to comment.