[U-Boot] [PATCH] spl: make image arg or fdt blob address reconfigurable

[Vikas MANOCHA]

Hi Tom,

> -----Original Message-----
> From: Tom Rini [mailto:trini at konsulko.com]
> Sent: Monday, April 10, 2017 5:29 AM
> To: Vikas MANOCHA <vikas.manocha at st.com>
> Cc: u-boot at lists.denx.de; Marek Vasut <marex at denx.de>; Stefan Agner <stefan.agner at toradex.com>; Jeremy Hunt
> <Jeremy.Hunt at DEShawResearch.com>
> Subject: Re: [U-Boot] [PATCH] spl: make image arg or fdt blob address reconfigurable
> 
> On Fri, Apr 07, 2017 at 03:38:13PM -0700, Vikas Manocha wrote:
> 
> > At present fdt blob or argument address being passed to kernel is
> > fixed at compile time using macro CONFIG_SYS_SPL_ARGS_ADDR. FDT blob
> > from different media like nand, nor flash are copied to the address
> > pointed by the macro.
> > The problem is, it makes args/fdt blob compulsory to copy which is not
> > required in cases like for NOR Flash. This patch removes this limitation.
> >
> > Signed-off-by: Vikas Manocha <vikas.manocha at st.com>
> > ---
> >  arch/arm/lib/spl.c        | 7 +++----
> >  arch/microblaze/cpu/spl.c | 6 +++---
> >  arch/powerpc/lib/spl.c    | 8 ++++----
> >  common/spl/spl.c          | 6 ++++--
> >  common/spl/spl_nor.c      | 8 +-------
> >  include/spl.h             | 5 ++---
> 
> I assume you've tested the spl_nor case afterwards, yes?  Did this result in some measurable boot time decrease?  Thanks!

Yes, I tested it's working on board. Not sure how to measure the impact on boot time.

Cheers,
Vikas

> 
> --
> Tom