[U-Boot] [PATCH 1/2] x86: x86-common.h: Add CONFIG_BOOTDELAY

Bin Meng bmeng.cn at gmail.com
Tue Jan 26 07:48:02 CET 2016


Hi Stefan,

On Tue, Jan 19, 2016 at 2:19 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
> On Mon, Jan 18, 2016 at 9:49 PM, Stefan Roese <sr at denx.de> wrote:
>> Without this CONFIG_BOOTDELAY, autobooting does not work at all. As
>> autoboot_command() from common/* will not get called. So lets define
>> CONFIG_BOOTDELAY, so that auto-booting works on x86.
>>
>> Signed-off-by: Stefan Roese <sr at denx.de>
>> Cc: Miao Yan <yanmiaobest at gmail.com>
>> Cc: Bin Meng <bmeng.cn at gmail.com>
>> Cc: Simon Glass <sjg at chromium.org>
>> ---
>>  include/configs/x86-common.h | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>
> Acked-by: Bin Meng <bmeng.cn at gmail.com>

Sorry, this patch does not build for efi-x86.

       x86:  +   efi-x86
+../common/autoboot.c: In function 'process_fdt_options':
+../common/autoboot.c:296:36: error: 'CONFIG_SYS_TEXT_BASE' undeclared
(first use in this function)
+   setenv_addr("kernaddr", (void *)(CONFIG_SYS_TEXT_BASE + addr));
+                                    ^
+../common/autoboot.c:296:36: note: each undeclared identifier is
reported only once for each function it appears in
+make[2]: *** [common/autoboot.o] Error 1
+make[1]: *** [common] Error 2
+make: *** [sub-make] Error 2

Could you please fix this? Sorry I did not run buildman earlier.

Regards,
Bin


More information about the U-Boot mailing list