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

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to