From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rajesh Bhagat Date: Thu, 4 Oct 2018 11:30:03 +0000 Subject: [U-Boot] [PATCH 10/53] armv8: ls1043ardb: Add TFABOOT defconfig In-Reply-To: References: <20181003113736.14981-1-rajesh.bhagat@nxp.com> <20181003113736.14981-11-rajesh.bhagat@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de > -----Original Message----- > From: York Sun > Sent: Wednesday, October 3, 2018 9:47 PM > To: Rajesh Bhagat ; u-boot at lists.denx.de > Cc: Prabhakar Kushwaha ; Pankit Garg > > 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 > > Signed-off-by: Rajesh Bhagat > > --- > > 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