From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 4 Apr 2013 16:15:43 -0400 Subject: [U-Boot] [PATCH 2/2] ARM: Add support for IGEP COM AQUILA/CYGNUS In-Reply-To: References: <1364994723-19945-1-git-send-email-eballetbo@gmail.com> <1364994723-19945-3-git-send-email-eballetbo@gmail.com> <20130403223828.GA20668@bill-the-cat> Message-ID: <20130404201543.GC32357@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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: