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

[Westergreen, Dalon]

On Mon, 2017-01-30 at 16:07 +0100, Alexander Graf wrote:
> 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?
> 
Bootfile was never used in the environment, neither have any of the
kits by default been using fitimages. When the environment was moved
to a common one i setting the bootfile def to match what was actually
used.  moving to distro boot is in the works and i will look at
SCAN_DEV_FOR_EFI for legacy support.

thanks,
dalon
> 
> Alex
>