[U-Boot] [PATCH] omap3: add CM-T35 board

Peter Tyser ptyser at xes-inc.com
Wed Nov 11 15:18:06 CET 2009


<snip>

> >>>> --- a/Makefile
> >>>> +++ b/Makefile
> >>>> @@ -3168,6 +3168,9 @@ omap3_zoom1_config :    unconfig
> >>>>  omap3_zoom2_config : unconfig
> >>>>       @$(MKCONFIG) $(@:_config=) arm arm_cortexa8 zoom2 logicpd omap3
> >>>>
> >>>> +omap3_cm-t35_config :        unconfig
> >>>> +     @$(MKCONFIG) $(@:_config=) arm arm_cortexa8 cm-t35 NULL omap3
> >>>> +
> > 
> > I'd title your _config rule the same as your board name, ie drop the
> > omap3_ prefix above.  And the above list should be alphabetically
> > sorted.
> 
> I'd prefer to keep the 'omap3_' prefix to stay inline with other omap3 boards.

Just because the other omap3_ board are wrong doesn't mean your board
should be too:)  This same question was discussed for the devkit8000:
http://www.mail-archive.com/u-boot@lists.denx.de/msg21317.html

Best,
Peter



More information about the U-Boot mailing list