[U-Boot] [PATCH v8 19/30] efi: Relocate FDT to 127MB instead of 128MB

Alexander Graf agraf at suse.de
Fri Jun 22 12:13:33 UTC 2018


On 06/21/2018 09:45 PM, Simon Glass wrote:
> Hi Alex,
>
> On 21 June 2018 at 04:01, Alexander Graf <agraf at suse.de> wrote:
>> On 06/21/2018 04:01 AM, Simon Glass wrote:
>>> Hi Alex,
>>>
>>> On 18 June 2018 at 08:52, Alexander Graf <agraf at suse.de> wrote:
>>>> On 06/18/2018 04:08 PM, Simon Glass wrote:
>>>>> Sandbox only has 128MB of memory so we cannot relocate the device tree
>>>>> up
>>>>> to start at 128MB. Use 127MB instead, which should be safe.
>>>>>
>>>>> Signed-off-by: Simon Glass <sjg at chromium.org>
>>>>
>>>> We should just drop into the fallback case if allocation fails, no?
>>> I cannot see how the allocation could ever fail, now that we are
>>> definitely in sandbox RAM. So I'm OK with doing that if you are.
>>
>> I think that's perfectly reasonable, yes :)
> OK I will update the patch.

No worries. I pulled in most parts that I consider ready to efi-next 
already. I'll prepare a small patch set that takes patches from your set 
and my set and gets us a working sandbox efi_loader on top.


Alex



More information about the U-Boot mailing list