On Tue, Apr 11, 2017 at 11:44:00PM +0000, Vikas MANOCHA wrote: > Hi Tom, > > > -----Original Message----- > > From: Tom Rini [mailto:[email protected]] > > Sent: Monday, April 10, 2017 5:29 AM > > To: Vikas MANOCHA <[email protected]> > > Cc: [email protected]; Marek Vasut <[email protected]>; Stefan Agner > > <[email protected]>; Jeremy Hunt > > <[email protected]> > > 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 <[email protected]> > > > --- > > > 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.
There's always good old grabserial. But if you didn't measure bootspeed, did this decrease the code size? Or fix some other issue? Thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
