From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Thu, 28 Jan 2016 16:04:25 +0100 Subject: [U-Boot] [PATCH 1/2] x86: x86-common.h: Add CONFIG_BOOTDELAY In-Reply-To: References: <1453124997-8927-1-git-send-email-sr@denx.de> Message-ID: <56AA2DF9.7020206@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Bin, On 26.01.2016 07:48, Bin Meng wrote: > On Tue, Jan 19, 2016 at 2:19 PM, Bin Meng wrote: >> On Mon, Jan 18, 2016 at 9:49 PM, Stefan Roese 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 >>> Cc: Miao Yan >>> Cc: Bin Meng >>> Cc: Simon Glass >>> --- >>> include/configs/x86-common.h | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >> >> Acked-by: Bin Meng > > 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