> -----Original Message-----
> From: Yuantian Tang <[email protected]>
> Sent: Wednesday, April 3, 2019 12:48 PM
> To: Prabhakar Kushwaha <[email protected]>
> Cc: [email protected]; Sudhanshu Gupta
> <[email protected]>; Harninder Rai <[email protected]>; Rajesh
> Bhagat <[email protected]>; Bhaskar Upadhaya
> <[email protected]>; [email protected]; Andy Tang
> <[email protected]>
> Subject: [PATCH 3/3 v2] armv8: ls1028aqds: Add support of LS1028AQDS
> 
> From: Tang Yuantian <[email protected]>
> 
> LS1028AQDS Development System is a high-performance computing, evaluation,
> and development platform that supports LS1028A QorIQ Architecture processor.
> 
> Signed-off-by: Sudhanshu Gupta <[email protected]>
> Signed-off-by: Rai Harninder <[email protected]>
> Signed-off-by: Rajesh Bhagat <[email protected]>
> Signed-off-by: Bhaskar Upadhaya <[email protected]>
> Signed-off-by: Tang yuantian <[email protected]>
> ---


General comment
 - avoid FSPI, XSPI related word, configs as no flexspi driver in upstream
 - avoid enetc related word as no enetc driver in upstream
- Avoid #else part of CONFIG_TFABOOT as TFA boot is only supported mechanism,
 - Remove DDR related code as TFA doing DDR init 
- generate defconfig using make savedefcofig


Nut shell: Take care of all review comments from RDB patch


--pk
_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to