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

Bin Meng bmeng.cn at gmail.com
Fri Jan 29 09:27:48 CET 2016


Hi Stefan,

On Fri, Jan 29, 2016 at 4:26 PM, Stefan Roese <sr at denx.de> wrote:
> 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?
>

I will take this to the x86 branch. Do you plan to submit your new
baytrail board patches for this release?

Regards,
Bin


More information about the U-Boot mailing list