[U-Boot] [PATCH v4 7/8] x86: qemu: adjust ramdisk load address

[Simon Glass]

Hi Miao,

On 30 December 2015 at 19:55, Miao Yan <yanmiaobest at gmail.com> wrote:
> By default, ramdisk load address is defined to 02000000 in
> env string. When loading bzImage to 100000 (default address), there's
> a chance that the ramdisk header would be overwritten by
> the kernel. Thus increase the gap and make ramdisk load at 04000000
> by default.
>
> Signed-off-by: Miao Yan <yanmiaobest at gmail.com>
> ---
>  include/configs/qemu-x86.h | 13 +++++++++++++
>  1 file changed, 13 insertions(+)

Reviewed-by: Simon Glass <sjg at chromium.org>

But you add other env options here to. Can you mention more of these
in the commit message too?

>
> diff --git a/include/configs/qemu-x86.h b/include/configs/qemu-x86.h
> index 4258dcb..657b8af 100644
> --- a/include/configs/qemu-x86.h
> +++ b/include/configs/qemu-x86.h
> @@ -57,4 +57,17 @@
>  #undef CONFIG_ENV_IS_IN_SPI_FLASH
>  #define CONFIG_ENV_IS_NOWHERE
>
> +/* default ramdisk load address */
> +#define CONFIG_RAMDISK_ADDR    0x04000000
> +
> +#undef CONFIG_EXTRA_ENV_SETTINGS
> +#define CONFIG_EXTRA_ENV_SETTINGS                      \
> +       CONFIG_STD_DEVICES_SETTINGS                     \
> +       "pciconfighost=1\0"                             \
> +       "netdev=eth0\0"                                 \
> +       "consoledev=ttyS0\0"                            \
> +       "othbootargs=acpi=off\0"                        \
> +       "ramdiskaddr=0x4000000\0"                       \
> +       "ramdiskfile=initramfs.gz\0"
> +
>  #endif /* __CONFIG_H */
> --
> 1.9.1
>

Regards,
Simon