[U-Boot] [PATCH] autoboot.c: Fill env vars in process_fdt_options() only if TEXT_BASE is set

Stefan Roese sr at denx.de
Fri Jan 29 09:26:50 CET 2016


On 29.01.2016 09:24, Bin Meng wrote:
> On Fri, Jan 29, 2016 at 12:34 AM, Stefan Roese <sr at 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 at denx.de>
>> Cc: Simon Glass <sjg at chromium.org>
>> Cc: Bin Meng <bmeng.cn at gmail.com>
>> Cc: Tom Rini <trini at konsulko.com>
>> ---
>>   common/autoboot.c | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>
> Reviewed-by: Bin Meng <bmeng.cn at 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?

Thanks,
Stefan



More information about the U-Boot mailing list