-----Original Message----- From: York Sun Sent: Wednesday, October 3, 2018 9:47 PM To: Rajesh Bhagat rajesh.bhagat@nxp.com; u-boot@lists.denx.de Cc: Prabhakar Kushwaha prabhakar.kushwaha@nxp.com; Pankit Garg pankit.garg@nxp.com Subject: Re: [PATCH 10/53] armv8: ls1043ardb: Add TFABOOT defconfig
On 10/03/2018 04:38 AM, Rajesh Bhagat wrote:
This defconfig is for TFABOOT, to be loaded by trusted firmware.
Signed-off-by: Pankit Garg pankit.garg@nxp.com Signed-off-by: Rajesh Bhagat rajesh.bhagat@nxp.com
board/freescale/ls1043ardb/ddr.c | 6 ++- board/freescale/ls1043ardb/ls1043ardb.c | 12 ++++++ configs/ls1043ardb_ram_defconfig | 49
+++++++++++++++++++++++++
In this and other patches, you said "add TFABOOT defconfig", but you use "ram_defconfig" in the file name. If this boot method is only for TFA boot, maybe you should consider to rename the file. If the ram boot can work without TFA, you need to rewrite your commit subject and message.
Will rename as below in v2. ls1043aqds_ram_defconfig => ls1043aqds_tfa_defconfig
York