> -----Original Message----- > From: York Sun > Sent: Wednesday, October 3, 2018 9:47 PM > To: Rajesh Bhagat <[email protected]>; [email protected] > Cc: Prabhakar Kushwaha <[email protected]>; Pankit Garg > <[email protected]> > 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 <[email protected]> > > Signed-off-by: Rajesh Bhagat <[email protected]> > > --- > > 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 _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
