Hi Bin,
On 26.01.2016 07:48, Bin Meng wrote:
On Tue, Jan 19, 2016 at 2:19 PM, Bin Meng bmeng.cn@gmail.com wrote:
On Mon, Jan 18, 2016 at 9:49 PM, Stefan Roese sr@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@denx.de Cc: Miao Yan yanmiaobest@gmail.com Cc: Bin Meng bmeng.cn@gmail.com Cc: Simon Glass sjg@chromium.org
include/configs/x86-common.h | 2 ++ 1 file changed, 2 insertions(+)
Acked-by: Bin Meng bmeng.cn@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.
No need to be sorry for you. I should have spotted this problem before submitting. I'll fix this soon and will resubmit.
Thanks, Stefan