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

[Stefan Roese <sr@denx.de>]

On 29.01.2016 09:27, Bin Meng wrote:
> 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.

Thanks.

> Do you plan to submit your new
> baytrail board patches for this release?

Not sure yet. I'm still waiting for a confirmation that my DDR
memory down parameters are "correct". It seems to be working,
but I would like to get an ack from the board manufacturer here.
So it might still be a few days until these board patches hit
the mailing list.

Thanks,
Stefan