From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dalon Westergreen <dwesterg@gmail.com>
Date: Sat, 18 Feb 2017 15:31:59 -0800
Subject: [U-Boot] [PATCH 8/8] arm: socfpga: sr1500 use environment in
 common header
In-Reply-To: <9ba15d4f-b79e-563e-1fd5-5820a57daf0f@denx.de>
References: <1487451108-4277-1-git-send-email-dwesterg@gmail.com>
 <1487451108-4277-9-git-send-email-dwesterg@gmail.com>
 <9ba15d4f-b79e-563e-1fd5-5820a57daf0f@denx.de>
Message-ID: <1487460719.6396.21.camel@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sat, 2017-02-18 at 21:56 +0100, Marek Vasut wrote:
> On 02/18/2017 09:51 PM, Dalon Westergreen wrote:
> > 
> > This removes the default environment from the sr1500 header
> > and instead uses the common environment provided in
> > socfpga_common.h which now uses distro boot.
> > 
> > This board has no upstream devicetree in the kernel source,
> > so leave the default devicetree name socfpga.dtb.
> > 
> > Signed-off-by: Dalon Westergreen <dwesterg@gmail.com>
> > ---
> > ?configs/socfpga_sr1500_defconfig |??1 +
> > ?include/configs/socfpga_sr1500.h | 27 ---------------------------
> > ?2 files changed, 1 insertion(+), 27 deletions(-)
> > 
> > diff --git a/configs/socfpga_sr1500_defconfig
> > b/configs/socfpga_sr1500_defconfig
> > index 981600b..b0925ab 100644
> > --- a/configs/socfpga_sr1500_defconfig
> > +++ b/configs/socfpga_sr1500_defconfig
> > @@ -4,6 +4,7 @@ CONFIG_SYS_MALLOC_F_LEN=0x2000
> > ?CONFIG_TARGET_SOCFPGA_SR1500=y
> > ?CONFIG_SPL_STACK_R_ADDR=0x00800000
> > ?CONFIG_DEFAULT_DEVICE_TREE="socfpga_cyclone5_sr1500"
> > +CONFIG_DEFAULT_FDT_FILE="socfpga.dtb"
> 
> Shouldn't this be socfpga_cyclone5_sr1500.dtb too ?
We discussed this before, my thought was to leave the dtb name
as it was before where there was no mainlined device tree in
the linux kernel source. ?CONFIG_DEFAULT_FDT_FILE is specifically
the kernel devicetree used for linux boot and is unrelated to the
dtb for uboot.

--dalon
> 
> > 
> > ?CONFIG_FIT=y
> > ?CONFIG_SYS_CONSOLE_IS_IN_ENV=y
> > ?CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE=y
> > diff --git a/include/configs/socfpga_sr1500.h
> > b/include/configs/socfpga_sr1500.h
> > index f67fafd..d94771c 100644
> > --- a/include/configs/socfpga_sr1500.h
> > +++ b/include/configs/socfpga_sr1500.h
> > @@ -17,8 +17,6 @@
> > ?
> > ?/* Booting Linux */
> > ?#define CONFIG_BOOTFILE		"uImage"
> > -#define CONFIG_BOOTARGS		"console=ttyS0,"
> > __stringify(CONFIG_BAUDRATE)
> > -#define CONFIG_BOOTCOMMAND	"run mmcload; run mmcboot"
> > ?#define CONFIG_LOADADDR		0x01000000
> > ?#define CONFIG_SYS_LOAD_ADDR	CONFIG_LOADADDR
> > ?
> > @@ -28,31 +26,6 @@
> > ?#define CONFIG_PHY_MARVELL
> > ?#define PHY_ANEG_TIMEOUT	8000
> > ?
> > -#define CONFIG_EXTRA_ENV_SETTINGS \
> > -	"verify=n\0" \
> > -	"loadaddr=" __stringify(CONFIG_SYS_LOAD_ADDR) "\0" \
> > -	"ramboot=setenv bootargs " CONFIG_BOOTARGS ";" \
> > -		"bootm ${loadaddr} - ${fdt_addr}\0" \
> > -	"bootimage=zImage\0" \
> > -	"fdt_addr=100\0" \
> > -	"fdtimage=socfpga.dtb\0" \
> > -		"fsloadcmd=ext2load\0" \
> > -	"bootm ${loadaddr} - ${fdt_addr}\0" \
> > -	"mmcroot=/dev/mmcblk0p2\0" \
> > -	"mmcboot=setenv bootargs " CONFIG_BOOTARGS \
> > -		" root=${mmcroot} rw rootwait;" \
> > -		"bootz ${loadaddr} - ${fdt_addr}\0" \
> > -	"mmcload=mmc rescan;" \
> > -		"load mmc 0:1 ${loadaddr} ${bootimage};" \
> > -		"load mmc 0:1 ${fdt_addr} ${fdtimage}\0" \
> > -	"qspiload=sf probe && mtdparts default && run ubiload\0" \
> > -	"qspiboot=setenv bootargs " CONFIG_BOOTARGS \
> > -		" ubi.mtd=1,64 root=ubi0:rootfs rw rootfstype=ubifs;"\
> > -		"bootz ${loadaddr} - ${fdt_addr}\0" \
> > -	"ubiload=ubi part UBI && ubifsmount ubi0 && " \
> > -		"ubifsload ${loadaddr} /boot/${bootimage} && " \
> > -		"ubifsload ${fdt_addr} /boot/${fdtimage}\0"
> > -
> > ?/* Environment */
> > ?#define CONFIG_ENV_IS_IN_SPI_FLASH
> > ?
> > 
> 
>