From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Thu, 04 Aug 2011 08:31:20 +0200 Subject: [U-Boot] [PATCH V2 2/4] Don't compile in large memory test function by default. In-Reply-To: <20110707143109.GC5438@harvey-pc.matrox.com> References: <20110707142144.GA5438@harvey-pc.matrox.com> <20110707143109.GC5438@harvey-pc.matrox.com> Message-ID: <4E3A3CB8.2030703@aribaud.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Christopher, Le 07/07/2011 16:31, Christopher Harvey a ?crit : > Signed-off-by: Christopher Harvey > --- > > V2: > I didn't receive comments on this one. By not compiling this by > default it removes a couple of #defines people need to think > about. Also, I'm guessing the memory test isn't used very much, so > this makes u-boot a bit smaller. > > common/cmd_mem.c | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/common/cmd_mem.c b/common/cmd_mem.c > index a5576aa..833af66 100644 > --- a/common/cmd_mem.c > +++ b/common/cmd_mem.c > @@ -610,6 +610,8 @@ int do_mem_loopw (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) > } > #endif /* CONFIG_LOOPW */ > > +#ifdef CONFIG_CMD_MTEST > + > /* > * Perform a memory test. A more complete alternative test can be > * configured using CONFIG_SYS_ALT_MEMTEST. The complete test loops until > @@ -965,6 +967,7 @@ int do_mem_mtest (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) > return 0; /* not reached */ > } > > +#endif /* CONFIG_CMD_MTEST */ > > /* Modify memory. > * > @@ -1245,11 +1248,13 @@ U_BOOT_CMD( > ); > #endif /* CONFIG_LOOPW */ > > +#ifdef CONFIG_CMD_MTEST > U_BOOT_CMD( > mtest, 5, 1, do_mem_mtest, > "simple RAM read/write test", > "[start [end [pattern [iterations]]]]" > ); > +#endif /* CONFIG_CMD_MTEST */ > > #ifdef CONFIG_MX_CYCLIC > U_BOOT_CMD( Applied to u-boot-arm/master, thanks! Amicalement, -- Albert.