From mboxrd@z Thu Jan  1 00:00:00 1970
From: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
Date: Wed, 14 Aug 2019 05:29:45 +0000
Subject: [U-Boot] [PATCH v2] armv8: ls1028a: Add environment variables
 to facilitate the boot
In-Reply-To: <20190619071814.8691-1-andy.tang@nxp.com>
References: <20190619071814.8691-1-andy.tang@nxp.com>
Message-ID: <VI1PR0402MB3854329F73B85174456F68C297AD0@VI1PR0402MB3854.eurprd04.prod.outlook.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

Dear Andy,


> -----Original Message-----
> From: Yuantian Tang <andy.tang@nxp.com>
> Sent: Wednesday, June 19, 2019 12:48 PM
> To: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> Cc: u-boot at lists.denx.de; Andy Tang <andy.tang@nxp.com>
> Subject: [PATCH v2] armv8: ls1028a: Add environment variables to facilitate
> the boot
> 
> Add some environment variables to facilitate the auto boot.
> 
> Signed-off-by: Yuantian Tang <andy.tang@nxp.com>
> ---
> v2:
> 	- merge with its dependent patch
>  include/configs/ls1028a_common.h | 28 +++++++++++++++++++++++++---
>  1 file changed, 25 insertions(+), 3 deletions(-)
> 
> diff --git a/include/configs/ls1028a_common.h
> b/include/configs/ls1028a_common.h
> index 0db86396e9..022981bfa3 100644
> --- a/include/configs/ls1028a_common.h
> +++ b/include/configs/ls1028a_common.h
> @@ -67,6 +67,7 @@
> 
>  #define BOOT_TARGET_DEVICES(func) \
>  	func(MMC, mmc, 0) \
> +	func(MMC, mmc, 1) \
>  	func(USB, usb, 0)
>  #include <config_distro_bootcmd.h>
> 
> @@ -127,25 +128,46 @@
>  			"${scripthdraddr} ${prefix}${boot_script_hdr} " \
>  			"&& esbc_validate ${scripthdraddr};"    \
>  		"source ${scriptaddr}\0"	  \
> -	"sd_bootcmd=echo Trying load from SD ..;"	\
> +	"xspi_bootcmd=echo Trying load from FlexSPI flash ...;" \
> +		"sf probe 0:0 && sf read $load_addr " \
> +		"$kernel_start $kernel_size ; env exists secureboot &&" \
> +		"sf read $kernelheader_addr_r $kernelheader_start " \
> +		"$kernelheader_size && esbc_validate
> ${kernelheader_addr_r}; "\
> +		" bootm $load_addr#$board\0" \
> +	"xspi_hdploadcmd=echo Trying load HDP firmware from FlexSPI...;" \
> +		"sf probe 0:0 && sf read $load_addr 0x940000 0x30000 " \
> +		"&& hdp load $load_addr 0x2000\0"			\
> +	"sd_bootcmd=echo Trying load from SD ...;" \
>  		"mmcinfo; mmc read $load_addr "		\
>  		"$kernel_addr_sd $kernel_size_sd && "	\
>  		"env exists secureboot && mmc read $kernelheader_addr_r
> " \
>  		"$kernelhdr_addr_sd $kernelhdr_size_sd "		\
>  		" && esbc_validate ${kernelheader_addr_r};"	\
>  		"bootm $load_addr#$board\0"		\
> +	"sd_hdploadcmd=echo Trying load HDP firmware from SD..;"        \
> +		"mmcinfo;mmc read $load_addr 0x4a00 0x200 "             \
> +		"&& hdp load $load_addr 0x2000\0"	\
>  	"emmc_bootcmd=echo Trying load from EMMC ..;"	\
>  		"mmcinfo; mmc dev 1; mmc read $load_addr "		\
>  		"$kernel_addr_sd $kernel_size_sd && "	\
>  		"env exists secureboot && mmc read $kernelheader_addr_r
> " \
>  		"$kernelhdr_addr_sd $kernelhdr_size_sd "		\
>  		" && esbc_validate ${kernelheader_addr_r};"	\
> -		"bootm $load_addr#$board\0"
> +		"bootm $load_addr#$board\0"			\
> +	"emmc_hdploadcmd=echo Trying load HDP firmware from EMMC..;"
> \
> +		"mmc dev 1;mmcinfo;mmc read $load_addr 0x4a00 0x200 "
> 	\
> +		"&& hdp load $load_addr 0x2000\0"
> 
>  #undef CONFIG_BOOTCOMMAND
> 
> +#define XSPI_NOR_BOOTCOMMAND	\
> +	"run xspi_hdploadcmd; run distro_bootcmd; run xspi_bootcmd; " \
> +	"env exists secureboot && esbc_halt;;"
>  #define SD_BOOTCOMMAND	\
> -	"run distro_bootcmd;run sd_bootcmd; " \
> +	"run sd_hdploadcmd; run distro_bootcmd;run sd_bootcmd; " \
> +	"env exists secureboot && esbc_halt;"
> +#define SD2_BOOTCOMMAND	\
> +	"run emmc_hdploadcmd; run distro_bootcmd;run emmc_bootcmd;
> " \
>  	"env exists secureboot && esbc_halt;"
> 
>  /* Monitor Command Prompt */
> --
> 2.17.1

As there is no flexspi and hdp driver.  I will suggest to avoid these words in patch

--pk