[PATCH u-boot v2016.07-aspeed-openbmc] Add ast-g5 ncsi 2nd boot config.
c_mykolak at mellanox.com
Tue Jun 27 00:17:00 AEST 2017
Best regards. Mykola Kostenok.
> -----Original Message-----
> From: joel.stan at gmail.com [mailto:joel.stan at gmail.com] On Behalf Of Joel
> Sent: Monday, June 26, 2017 1:13 PM
> To: Mykola Kostenok <c_mykolak at mellanox.com>
> Cc: OpenBMC Maillist <openbmc at lists.ozlabs.org>
> Subject: Re: [PATCH u-boot v2016.07-aspeed-openbmc] Add ast-g5 ncsi 2nd
> boot config.
> On Fri, Jun 2, 2017 at 6:48 PM, Mykola Kostenok
> <c_mykolak at mellanox.com> wrote:
> > Add ncsi 2nd boot aspeed g5 configuration header.
> > It based on ast-g5-ncsi.h extended with defines for memtest.
> I was hoping to merge these changes into the normal configurations, instead
> of having a separate config.
> The only difference is the presence of the SYS_MEMTEST defines.
> > +/* platform.S settings */
> > +#define CONFIG_DRAM_ECC_SIZE 0x10000000
> > +
> > +#define CONFIG_SYS_MEMTEST_SIZE 128 * 1024 * 1024 /* 128MB */
> > +#define CONFIG_SYS_MEMTEST_START CONFIG_SYS_SDRAM_BASE
> > +#define CONFIG_SYS_MEMTEST_END (CONFIG_SYS_MEMTEST_START +
> > +CONFIG_SYS_MEMTEST_SIZE)
> I was looking at the u-boot tree, and it appears that these SYS_MEMTEST
> defines are used by the mtest command, and the mtest command is
> 2. The "mtest" command.
> This is probably the best known memory test utility in U-Boot.
> Unfortunately, it is also the most problematic, and the most
> useless one.
> Because of these issues, the "mtest" command is considered depre-
> cated. It should not be enabled in most normal ports of U-Boot,
> especially not in production. If you really need a memory test,
> then see 1. and 3. above resp. below.
> Are you sure that you want to enable the mtest command?
We added mtest, because it was the production requirement.
Following the reference from u-boot you pointed out:
+3. The most thorough memory test facility is available as part of the
+ POST (Power-On Self Test) sub-system, see "post/drivers/memory.c".
I see that in post/Makefile there is no entry for ARM
obj-$(CONFIG_ARM) += cpu/arm/
I’ll check it. If it’s work – would be good with us.
> > +
> > +#endif /* __AST_G5_NCSI_2BOOT_CONFIG_H */
> > --
> > 2.11.0
More information about the openbmc