Hi Rajendra, On 04/24/2014 05:06 AM, Nayak, Rajendra wrote: > The only difference from the dra74x devices is the missing .smp entry. > > While at it, also fix the use of __initdata (across the file) and replace them > with __initconst as reported by checkpatch > > ERROR: Use of const init definition must use __initconst > +static const char *dra72x_boards_compat[] __initdata = { Since it looks like you are gonna do another respin, care to separate out the __initdata to __initconst change for existing ones into a different patch, and limit this patch to just what the patch subject says. regards Suman > > Signed-off-by: Rajendra Nayak <rnayak@xxxxxx> > --- > arch/arm/mach-omap2/board-generic.c | 45 ++++++++++++++++++++++++----------- > 1 file changed, 31 insertions(+), 14 deletions(-) > > diff --git a/arch/arm/mach-omap2/board-generic.c b/arch/arm/mach-omap2/board-generic.c > index b8920b6..dabb9f6 100644 > --- a/arch/arm/mach-omap2/board-generic.c > +++ b/arch/arm/mach-omap2/board-generic.c > @@ -43,7 +43,7 @@ static void __init omap_generic_init(void) > } > > #ifdef CONFIG_SOC_OMAP2420 > -static const char *omap242x_boards_compat[] __initdata = { > +static const char *omap242x_boards_compat[] __initconst = { > "ti,omap2420", > NULL, > }; > @@ -62,7 +62,7 @@ MACHINE_END > #endif > > #ifdef CONFIG_SOC_OMAP2430 > -static const char *omap243x_boards_compat[] __initdata = { > +static const char *omap243x_boards_compat[] __initconst = { > "ti,omap2430", > NULL, > }; > @@ -81,7 +81,7 @@ MACHINE_END > #endif > > #ifdef CONFIG_ARCH_OMAP3 > -static const char *omap3_boards_compat[] __initdata = { > +static const char *omap3_boards_compat[] __initconst = { > "ti,omap3430", > "ti,omap3", > NULL, > @@ -100,7 +100,7 @@ DT_MACHINE_START(OMAP3_DT, "Generic OMAP3 (Flattened Device Tree)") > .restart = omap3xxx_restart, > MACHINE_END > > -static const char *omap36xx_boards_compat[] __initdata = { > +static const char *omap36xx_boards_compat[] __initconst = { > "ti,omap36xx", > NULL, > }; > @@ -118,7 +118,7 @@ DT_MACHINE_START(OMAP36XX_DT, "Generic OMAP36xx (Flattened Device Tree)") > .restart = omap3xxx_restart, > MACHINE_END > > -static const char *omap3_gp_boards_compat[] __initdata = { > +static const char *omap3_gp_boards_compat[] __initconst = { > "ti,omap3-beagle", > "timll,omap3-devkit8000", > NULL, > @@ -137,7 +137,7 @@ DT_MACHINE_START(OMAP3_GP_DT, "Generic OMAP3-GP (Flattened Device Tree)") > .restart = omap3xxx_restart, > MACHINE_END > > -static const char *am3517_boards_compat[] __initdata = { > +static const char *am3517_boards_compat[] __initconst = { > "ti,am3517", > NULL, > }; > @@ -157,7 +157,7 @@ MACHINE_END > #endif > > #ifdef CONFIG_SOC_AM33XX > -static const char *am33xx_boards_compat[] __initdata = { > +static const char *am33xx_boards_compat[] __initconst = { > "ti,am33xx", > NULL, > }; > @@ -177,7 +177,7 @@ MACHINE_END > #endif > > #ifdef CONFIG_ARCH_OMAP4 > -static const char *omap4_boards_compat[] __initdata = { > +static const char *omap4_boards_compat[] __initconst = { > "ti,omap4460", > "ti,omap4430", > "ti,omap4", > @@ -199,7 +199,7 @@ MACHINE_END > #endif > > #ifdef CONFIG_SOC_OMAP5 > -static const char *omap5_boards_compat[] __initdata = { > +static const char *omap5_boards_compat[] __initconst = { > "ti,omap5432", > "ti,omap5430", > "ti,omap5", > @@ -221,7 +221,7 @@ MACHINE_END > #endif > > #ifdef CONFIG_SOC_AM43XX > -static const char *am43_boards_compat[] __initdata = { > +static const char *am43_boards_compat[] __initconst = { > "ti,am4372", > "ti,am43", > NULL, > @@ -240,13 +240,13 @@ MACHINE_END > #endif > > #ifdef CONFIG_SOC_DRA7XX > -static const char *dra7xx_boards_compat[] __initdata = { > - "ti,dra7xx", > +static const char *dra74x_boards_compat[] __initconst = { > + "ti,dra74", > "ti,dra7", > NULL, > }; > > -DT_MACHINE_START(DRA7XX_DT, "Generic DRA7XX (Flattened Device Tree)") > +DT_MACHINE_START(DRA74X_DT, "Generic DRA74X (Flattened Device Tree)") > .reserve = omap_reserve, > .smp = smp_ops(omap4_smp_ops), > .map_io = omap5_map_io, > @@ -255,7 +255,24 @@ DT_MACHINE_START(DRA7XX_DT, "Generic DRA7XX (Flattened Device Tree)") > .init_irq = omap_gic_of_init, > .init_machine = omap_generic_init, > .init_time = omap5_realtime_timer_init, > - .dt_compat = dra7xx_boards_compat, > + .dt_compat = dra74x_boards_compat, > + .restart = omap44xx_restart, > +MACHINE_END > + > +static const char *dra72x_boards_compat[] __initconst = { > + "ti,dra72", > + NULL, > +}; > + > +DT_MACHINE_START(DRA72_DT, "Generic DRA72X (Flattened Device Tree)") > + .reserve = omap_reserve, > + .map_io = omap5_map_io, > + .init_early = dra7xx_init_early, > + .init_late = dra7xx_init_late, > + .init_irq = omap_gic_of_init, > + .init_machine = omap_generic_init, > + .init_time = omap5_realtime_timer_init, > + .dt_compat = dra72x_boards_compat, > .restart = omap44xx_restart, > MACHINE_END > #endif > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html