[U-Boot] [PATCH 07/10] ti814x_evm: add ti814x evm board support
Matt Porter
mporter at ti.com
Fri Feb 15 22:19:54 CET 2013
On Fri, Feb 15, 2013 at 04:38:22PM +0000, Tom Rini wrote:
> On Wed, Feb 13, 2013 at 09:44:01AM -0500, Matt Porter wrote:
>
> > Add TI814X EVM board directory and config file.
> >
> > Signed-off-by: Matt Porter <mporter at ti.com>
> [snip]
> > +++ b/board/ti/ti814x/evm.h
> > @@ -0,0 +1,7 @@
> > +#ifndef _EVM_H
> [snip]
> > +++ b/include/configs/ti814x_evm.h
> > @@ -0,0 +1,213 @@
> > +#define __CONFIG_TI814X_EVM_H
>
> Needs GPLv2 or later boilerplate.
Ok.
>
> [snip]
> > +#define CONFIG_CMDLINE_TAG 1 /* enable passing of ATAGs */
> > +#define CONFIG_SETUP_MEMORY_TAGS 1
> > +#define CONFIG_INITRD_TAG 1 /* for ramdisk support */
>
> Just #define CONFIG_FOO in these cases (and some others that follow)
Ok.
>
> [snip]
> > + "findfdt="\
> > + "if test $board_name = A335BONE; then " \
> > + "setenv fdtfile am335x-bone.dtb; fi; " \
> > + "if test $board_name = A33515BB; then " \
> > + "setenv fdtfile am335x-evm.dtb; fi; " \
> > + "if test $board_name = A335X_SK; then " \
> > + "setenv fdtfile am335x-evmsk.dtb; fi\0" \
>
> That's not right :)
LOL, indeed..forgot about that am335x specific stuff. Will fix.
> [snip]
> > +/* memtest works on 8 MB in DRAM offset 32MB from start of ram disk*/
> > +#define CONFIG_SYS_MEMTEST_START (PHYS_DRAM_1 + (64 * 1024 * 1024))
> > +#define CONFIG_SYS_MEMTEST_END (CONFIG_SYS_MEMTEST_START \
> > + + (8 * 1024 * 1024))
>
> This isn't your mistake initially, but this should just be start of
> memory to smallest possible config the evm can come with - 4MB.
Ok.
>
> [snip]
> > +#undef CONFIG_NAND_OMAP_GPMC
>
> Just leave it out.
Ok.
> > +#define CONFIG_SYS_BAUDRATE_TABLE { 110, 300, 600, 1200, 2400, \
> > +4800, 9600, 14400, 19200, 28800, 38400, 56000, 57600, 115200 }
>
> Leave that out and get the generic table. am335x needs to be updated
> for that.
Ok, will do. I even saw something about this and forgot to update this
config.
> [snip]
> > +/* Since SPL did pll and ddr initialization for us,
> > + * we don't need to do it twice.
> > + */
>
> /*
> * Must be like
> * this.
> */
Ok.
More information about the U-Boot
mailing list