26 Jan
2019
26 Jan
'19
10:40 p.m.
On 1/26/19 9:33 PM, Stephen Warren wrote:
On 1/26/19 7:25 AM, Heinrich Schuchardt wrote:
On x86_64 the size of the file u-boot loaded by the tftp test has grown in size such that when loading the file to 0x200000 it overwrites a memory area reserved for PCI.
If no load address is specified for tftp do not use the ram base address (or if zero 0x200000) but the default address.
I assume that default is $loadaddr? If so, then this patch seems fine.
Thanks for reviewing.
Yes, the address `tftp filename` uses is environment variable loadaddr.
This is specified in cmd/net.c, function netboot_common():
188 /* pre-set load_addr */ 189 s = env_get("loadaddr");
Best regards
Heinrich Schuchardt