From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vikas MANOCHA Date: Tue, 11 Apr 2017 23:44:00 +0000 Subject: [U-Boot] [PATCH] spl: make image arg or fdt blob address reconfigurable In-Reply-To: <20170410122919.GA19897@bill-the-cat> References: <1491604694-20521-1-git-send-email-vikas.manocha@st.com> <20170410122919.GA19897@bill-the-cat> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tom, > -----Original Message----- > From: Tom Rini [mailto:trini at konsulko.com] > Sent: Monday, April 10, 2017 5:29 AM > To: Vikas MANOCHA > Cc: u-boot at lists.denx.de; Marek Vasut ; Stefan Agner ; Jeremy Hunt > > 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 > > --- > > 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