[U-Boot] [PATCH v8 5/7] arm: socfpga: Update DE1 environment

[Alexander Graf]

On 01/29/2017 12:05 AM, Dalon Westergreen wrote:
> From: Dalon Westergreen <dalon.westergreen at intel.com>
>
> Remove the default environment as it is now in a common
> header.
>
> Add the CONFIG_DEFAULT_DEVICE_TREE to the board's defconfig
> to set the linux devicetree name.
>
> Signed-off-by: Dalon Westergreen <dalon.westergreen at intel.com>
> Acked-by: Marek Vasut <marex at denx.de>
> Acked-by: Dinh Nguyen <dinguyen at kernel.org>
> ---
>   configs/socfpga_de1_soc_defconfig |  1 +
>   include/configs/socfpga_de1_soc.h | 19 +------------------
>   2 files changed, 2 insertions(+), 18 deletions(-)
>
> diff --git a/configs/socfpga_de1_soc_defconfig b/configs/socfpga_de1_soc_defconfig
> index 032deef..d78e8a1 100644
> --- a/configs/socfpga_de1_soc_defconfig
> +++ b/configs/socfpga_de1_soc_defconfig
> @@ -6,6 +6,7 @@ CONFIG_TARGET_SOCFPGA_TERASIC_DE1_SOC=y
>   CONFIG_SPL_STACK_R_ADDR=0x00800000
>   CONFIG_SPL_YMODEM_SUPPORT=y
>   CONFIG_DEFAULT_DEVICE_TREE="socfpga_cyclone5_de1_soc"
> +CONFIG_DEFAULT_FDT_FILE="socfpga_cyclone5_de1_soc.dtb"
>   CONFIG_FIT=y
>   CONFIG_SYS_CONSOLE_IS_IN_ENV=y
>   CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE=y
> diff --git a/include/configs/socfpga_de1_soc.h b/include/configs/socfpga_de1_soc.h
> index deec647..3142bd1 100644
> --- a/include/configs/socfpga_de1_soc.h
> +++ b/include/configs/socfpga_de1_soc.h
> @@ -18,7 +18,7 @@
>   #define PHYS_SDRAM_1_SIZE		0x40000000	/* 1GiB */
>   
>   /* Booting Linux */
> -#define CONFIG_BOOTFILE		"fitImage"
> +#define CONFIG_BOOTFILE		"zImage"

ok, here you're confusing me. I thought the point of having the crude, 
hacky mmcload/mmcboot in bootcmd was because you want to be legacy 
compatible. If you're changing the bootfile name, that point is moot.

So at this point, why not just switch to distro boot and extlinux.conf / 
efi altogether?


Alex