Hi Bin,

On 26.01.2016 07:48, Bin Meng wrote:
On Tue, Jan 19, 2016 at 2:19 PM, Bin Meng <bmeng...@gmail.com> wrote:
On Mon, Jan 18, 2016 at 9:49 PM, Stefan Roese <s...@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 <s...@denx.de>
Cc: Miao Yan <yanmiaob...@gmail.com>
Cc: Bin Meng <bmeng...@gmail.com>
Cc: Simon Glass <s...@chromium.org>
---
  include/configs/x86-common.h | 2 ++
  1 file changed, 2 insertions(+)


Acked-by: Bin Meng <bmeng...@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

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to