Hi Stefan,
On Fri, Jan 29, 2016 at 4:26 PM, Stefan Roese sr@denx.de wrote:
On 29.01.2016 09:24, Bin Meng wrote:
On Fri, Jan 29, 2016 at 12:34 AM, Stefan Roese sr@denx.de wrote:
The x86 build target "efi-x86" has no TEXT_BASE configured. And with the introduction of CONFIG_BOOTDELAY for x86, this function is now called for this board as well. Resulting in compile errors for this target.
Without TEXT_BASE it makes no sense to fill these values. So lets only configure the env variable if TEXT_BASE is defined.
Signed-off-by: Stefan Roese sr@denx.de Cc: Simon Glass sjg@chromium.org Cc: Bin Meng bmeng.cn@gmail.com Cc: Tom Rini trini@konsulko.com
common/autoboot.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Bin Meng bmeng.cn@gmail.com
Thanks Bin. Will you pick up the BOOTDELAY patch, once this patch arrives in mainline? Or will you include this into your x86 branch as well?
I will take this to the x86 branch. Do you plan to submit your new baytrail board patches for this release?
Regards, Bin