[U-Boot] [PATCH v4] OMAP3: add CM-T35 board

Tom Tom.Rix at windriver.com
Sun Nov 15 21:34:14 CET 2009


Paulraj, Sandeep wrote:
>> Add CM-T35 board support
>>

Please add more description of the board.

>> Signed-off-by: Mike Rapoport <mike at compulab.co.il>
>>
>> --
>> v2 changes:
>> - rename board config file from omap3_cm-t35.h to cm-t35.h
>> - remove SZ_xx references
>> - add MAKEALL/MAINTEINERS entries


Wolfgang had made this comment on V2 that has not been addressed.

>  > diff --git a/include/configs/cm-t35.h b/include/configs/cm-t35.h
> > new file mode 100755
> > > index 0000000..b881112
> > > --- /dev/null
> > > +++ b/include/configs/cm-t35.h
> ...
> > > +#ifndef __ASSEMBLY__
> > > +extern struct gpmc *gpmc_cfg;
> > > +extern unsigned int boot_flash_base;
> > > +extern volatile unsigned int boot_flash_env_addr;
> > > +extern unsigned int boot_flash_off;
> > > +extern unsigned int boot_flash_sec;
> > > +extern unsigned int boot_flash_type;
> > > +#endif
> 
> These should not be needed in your board config file. Please move to a
> more appropriate header.
> 
> Best regards,
> 
> Wolfgang Denk

Please do not push this change.

Tom



More information about the U-Boot mailing list