[PATCH v1] Makefile: socfpga: Generate spl/u-boot-splx4.sfp with 4 SPL images

[Tan, Ley Foon]

> -----Original Message-----
> From: Westergreen, Dalon <dalon.westergreen at intel.com>
> Sent: Saturday, August 15, 2020 3:24 AM
> To: u-boot at lists.denx.de; Ang, Chee Hong <chee.hong.ang at intel.com>
> Cc: See, Chin Liang <chin.liang.see at intel.com>; Tan, Ley Foon
> <ley.foon.tan at intel.com>; Chee, Tien Fong <tien.fong.chee at intel.com>;
> Lim, Elly Siew Chin <elly.siew.chin.lim at intel.com>
> Subject: Re: [PATCH v1] Makefile: socfpga: Generate spl/u-boot-splx4.sfp
> with 4 SPL images
> 
> Can you explain why this x4 image is needed?  the top level u-boot-with-
> spl.sfp or whatever it is called already creates four spl entries.  what are you
> generating the x4 image for?
If we put u-boot.img in FAT partition, then we don't need u-boot-with- spl.sfp. Just 4 x SPL images.



Regards
Ley Foon

> 
> On Wed, 2020-08-05 at 16:15 +0800, Chee Hong Ang wrote:
> > Generate spl/u-boot-splx4.sfp which consist of 4 SPL images required
> > for booting up Cyclone5/Arria10.
> >
> > Signed-off-by: Chee Hong Ang <chee.hong.ang at intel.com>
> > ---
> >  Makefile | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/Makefile b/Makefile
> > index 2629a74..13429a0 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -1578,8 +1578,9 @@ u-boot.spr: spl/u-boot-spl.img u-boot.img
> FORCE
> > ifneq ($(CONFIG_ARCH_SOCFPGA),)  quiet_cmd_socboot = SOCBOOT $@
> >  cmd_socboot = cat	spl/u-boot-spl.sfp spl/u-boot-spl.sfp	\
> > -			spl/u-boot-spl.sfp spl/u-boot-spl.sfp	\
> > -			u-boot.img > $@ || rm -f $@
> > +			spl/u-boot-spl.sfp \
> > +			spl/u-boot-spl.sfp > spl/u-boot-splx4.sfp ; \
> > +	      cat	spl/u-boot-splx4.sfp u-boot.img > $@ || rm -f $@
> >  u-boot-with-spl.sfp: spl/u-boot-spl.sfp u-boot.img FORCE
> >  	$(call if_changed,socboot)
> >