On Mon, Jan 19, 2015 at 01:05:45PM -0800, Tony Lindgren wrote:
> * Felipe Balbi <ba...@ti.com> [150119 12:46]:
> > On Mon, Jan 19, 2015 at 11:18:34AM -0800, Tony Lindgren wrote:
> > > * Tony Lindgren <t...@atomide.com> [150114 16:14]:
> > > > * Sergei Shtylyov <sergei.shtyl...@cogentembedded.com> [150114 05:54]:
> > > > > Hello.
> > > > > 
> > > > > On 1/14/2015 2:37 AM, Tony Lindgren wrote:
> > > > > 
> > > > > >This allows booting ti81xx boards with with when a .dts
> > > > > 
> > > > >    So, with, with or when? :-)
> > > > 
> > > > Heh thanks will updated to:
> > > > 
> > > > This allows booting ti81xx boards when a .dts file
> > > > is in place.
> > > 
> > > This too needs to be in a separate ifdef CONFIG_SOC_TI81XX block
> > > to avoid make randconfig build errors. Updated patch below.
> > > 
> > > Regards,
> > > 
> > > Tony
> > > 
> > > 8< -----------------
> > > From: Tony Lindgren <t...@atomide.com>
> > > Date: Mon, 19 Jan 2015 10:38:07 -0800
> > > Subject: [PATCH] ARM: OMAP2+: Add board-generic.c entry for ti81xx
> > > 
> > > This allows booting ti81xx boards when a .dts file
> > > is in place.
> > > 
> > > Cc: Brian Hutchinson <b.hutch...@gmail.com>
> > > Signed-off-by: Tony Lindgren <t...@atomide.com>
> > > 
> > > --- a/arch/arm/mach-omap2/board-generic.c
> > > +++ b/arch/arm/mach-omap2/board-generic.c
> > > @@ -142,6 +142,42 @@ DT_MACHINE_START(AM3517_DT, "Generic AM3517 
> > > (Flattened Device Tree)")
> > >   .dt_compat      = am3517_boards_compat,
> > >   .restart        = omap3xxx_restart,
> > >  MACHINE_END
> > > +
> > > +static const char *const ti814x_boards_compat[] __initconst = {
> > 
> > should this definition be within that ifdef too ?
> 
> Oops yes totally thanks for catching that. Updated patch below.
> 
> Regards,
> 
> Tony
> 
> 8< -----------------
> From: Tony Lindgren <t...@atomide.com>
> Date: Mon, 19 Jan 2015 10:38:07 -0800
> Subject: [PATCH] ARM: OMAP2+: Add board-generic.c entry for ti81xx
> 
> This allows booting ti81xx boards when a .dts file
> is in place.
> 
> Cc: Brian Hutchinson <b.hutch...@gmail.com>

Reviewed-by: Felipe Balbi <ba...@ti.com>

> Signed-off-by: Tony Lindgren <t...@atomide.com>
> 
> --- a/arch/arm/mach-omap2/board-generic.c
> +++ b/arch/arm/mach-omap2/board-generic.c
> @@ -144,6 +144,42 @@ DT_MACHINE_START(AM3517_DT, "Generic AM3517 (Flattened 
> Device Tree)")
>  MACHINE_END
>  #endif
>  
> +#ifdef CONFIG_SOC_TI81XX
> +static const char *const ti814x_boards_compat[] __initconst = {
> +     "ti,dm8148",
> +     "ti,dm814",
> +     NULL,
> +};
> +
> +DT_MACHINE_START(TI81XX_DT, "Generic ti814x (Flattened Device Tree)")
> +     .reserve        = omap_reserve,
> +     .map_io         = ti81xx_map_io,
> +     .init_early     = ti814x_init_early,
> +     .init_machine   = omap_generic_init,
> +     .init_late      = ti81xx_init_late,
> +     .init_time      = omap3_gptimer_timer_init,
> +     .dt_compat      = ti814x_boards_compat,
> +     .restart        = ti81xx_restart,
> +MACHINE_END
> +
> +static const char *const ti816x_boards_compat[] __initconst = {
> +     "ti,dm8168",
> +     "ti,dm816",
> +     NULL,
> +};
> +
> +DT_MACHINE_START(TI816X_DT, "Generic ti816x (Flattened Device Tree)")
> +     .reserve        = omap_reserve,
> +     .map_io         = ti81xx_map_io,
> +     .init_early     = ti816x_init_early,
> +     .init_machine   = omap_generic_init,
> +     .init_late      = ti81xx_init_late,
> +     .init_time      = omap3_gptimer_timer_init,
> +     .dt_compat      = ti816x_boards_compat,
> +     .restart        = ti81xx_restart,
> +MACHINE_END
> +#endif
> +
>  #ifdef CONFIG_SOC_AM33XX
>  static const char *const am33xx_boards_compat[] __initconst = {
>       "ti,am33xx",

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to