[PATCH] ls1028a: fix EFI boot

[Andy Tang]

Hi Yousaf,

Could you please try removing entire environment variable "scan_dev_for_boot" in the ls1028aqds/rdb header file?
In this way, the default value for scan_dev_for_boot could be used and meet your requirement.

BR,
Andy
> -----Original Message-----
> From: Poonam Aggrwal <poonam.aggrwal at nxp.com>
> Sent: 2020年7月22日 18:01
> To: Mian Yousaf Kaukab <ykaukab at suse.de>; u-boot at lists.denx.de; Rajesh
> Bhagat <rajesh.bhagat at nxp.com>; harninder.rai at nxp.com;
> sudhanshu.gupta at nxp.com
> Cc: prabhakar.kushwaha at nxp.com; Andy Tang <andy.tang at nxp.com>;
> Priyanka Jain <priyanka.jain at nxp.com>
> Subject: RE: [PATCH] ls1028a: fix EFI boot
> 
> Hello Yousaf
> 
> 
> > -----Original Message-----
> > From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Mian Yousaf
> > Kaukab
> > Sent: Tuesday, July 21, 2020 8:04 PM
> > To: u-boot at lists.denx.de; Rajesh Bhagat <rajesh.bhagat at nxp.com>;
> > harninder.rai at nxp.com; sudhanshu.gupta at nxp.com
> > Cc: prabhakar.kushwaha at nxp.com; Andy Tang <andy.tang at nxp.com>;
> > Priyanka Jain <priyanka.jain at nxp.com>; Mian Yousaf Kaukab
> > <ykaukab at suse.de>
> > Subject: [PATCH] ls1028a: fix EFI boot
> >
> > scan_dev_for_efi is supposed to be called from scan_dev_for_boot.
> > However this call is missing for ls1028ardb and ls1028aqds boards. As
> > a result EFI boot doesn’t work. Fix this issue by adding scan_dev_for_efi
> call.
> >
> Acked.
> 
> Andy, can you please also take a look.
> 
> Thanks .
> 
> Regards
> Poonam
> 
> > Signed-off-by: Mian Yousaf Kaukab <ykaukab at suse.de>
> > ---
> >  include/configs/ls1028aqds.h | 1 +
> >  include/configs/ls1028ardb.h | 1 +
> >  2 files changed, 2 insertions(+)
> >
> > diff --git a/include/configs/ls1028aqds.h
> > b/include/configs/ls1028aqds.h index 7bb740f48e36..3d622002ebca
> 100644
> > --- a/include/configs/ls1028aqds.h
> > +++ b/include/configs/ls1028aqds.h
> > @@ -135,6 +135,7 @@
> >  		"for prefix in ${boot_prefixes}; do " \
> >  			"run scan_dev_for_scripts; " \
> >  		"done;" \
> > +		SCAN_DEV_FOR_EFI \
> >  		"\0" \
> >  	"boot_a_script=" \
> >  		"load ${devtype} ${devnum}:${distro_bootpart} " \ diff --git
> > a/include/configs/ls1028ardb.h b/include/configs/ls1028ardb.h index
> > 07450113269c..0064c1dcc01a 100644
> > --- a/include/configs/ls1028ardb.h
> > +++ b/include/configs/ls1028ardb.h
> > @@ -118,6 +118,7 @@
> >  		"for prefix in ${boot_prefixes}; do "	  \
> >  			"run scan_dev_for_scripts; "	  \
> >  		"done;"					  \
> > +		SCAN_DEV_FOR_EFI			  \
> >  		"\0"					  \
> >  	"boot_a_script="				  \
> >  		"load ${devtype} ${devnum}:${distro_bootpart} "  \
> > --
> > 2.26.2