[U-Boot] [PATCH 2/2] ARM: Add support for IGEP COM AQUILA/CYGNUS
Tom Rini
trini at ti.com
Thu Apr 4 22:15:43 CEST 2013
On Thu, Apr 04, 2013 at 09:51:05AM +0200, Enric Balletbo Serra wrote:
[snip]
> >> > +/*
> >> > + * memtest works on 8 MB in DRAM after skipping 32MB from
> >> > + * start addr 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))
> >>
> >> The comment is wrong, and you can do any of:
> >> - Opt-out of mtest (and see Wolfgang's readme on why that's probably a
> >> good idea)
> >>
> >
> > Readed and convinced. Thanks for this point.
> >
>
> As explained in Wolfgang's readme shouldn't remove CONFIG_CMD_MEMTEST from
> config_cmd_default.h ?
When the time comes, as stated in doc/feature-removal-schedule.txt :)
For now, #undef it.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130404/810dd067/attachment.pgp>
More information about the U-Boot
mailing list