[U-Boot] [PATCH 1/1] x86: #define CONFIG_LOADADDR 0x1100000

[Heinrich Schuchardt]

On 1/24/19 9:18 PM, Heinrich Schuchardt wrote:
> arch/x86/dts/qemu-x86_i440fx.dts reserves memory for PCI at 0x1000000.
> Loading via the `dhcp` command to this address leads to a crash on
> qemu-x86_64_defconfig. So let's define CONFIG_LOADADDR as 0x1100000.
> 
> Reported-by: Alexander Graf <agraf at suse.de>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
> compatible='pci-x86' is a U-Boot specific device tree binding.
> Unfortunately it is not documented. Simon, it would be helpful
> if you could provide some README.
> ---
>  include/configs/x86-common.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h
> index 4180b25f97..e936230157 100644
> --- a/include/configs/x86-common.h
> +++ b/include/configs/x86-common.h
> @@ -99,7 +99,7 @@
>  #define CONFIG_ROOTPATH		"/opt/nfsroot"
>  #define CONFIG_HOSTNAME		"x86"
>  #define CONFIG_BOOTFILE		"bzImage"
> -#define CONFIG_LOADADDR		0x1000000
> +#define CONFIG_LOADADDR		0x1100000
>  #define CONFIG_RAMDISK_ADDR	0x4000000
>  #if defined(CONFIG_GENERATE_ACPI_TABLE) || defined(CONFIG_EFI_STUB)
>  #define CONFIG_OTHBOOTARGS	"othbootargs=\0"
> 

Hello Bin, hello Simon,

this is one of the three patches missing in U-Boot master that are
putting the EFI patch queue on hold. What is your opinion on the patch?

Best regards

Heinrich