[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