Hi Tom,

> -----Original Message-----
> From: Tom Rini [mailto:tr...@konsulko.com]
> Sent: Saturday, June 10, 2017 6:46 AM
> To: Vikas MANOCHA <vikas.mano...@st.com>
> Cc: u-boot@lists.denx.de; Christophe KERELLO <christophe.kere...@st.com>; 
> Alexandre TORGUE <alexandre.tor...@st.com>;
> Christophe PRIOUZEAU <christophe.priouz...@st.com>
> Subject: Re: [U-Boot,v2,3/7] stm32: stm32f7: add spl build support
> 
> On Sun, May 28, 2017 at 12:55:10PM -0700, Vikas Manocha wrote:
> 
> > This commit supports booting from stm32 internal nor flash. spl U-Boot
> > initializes the sdram memory, copies next image (e.g. standard U-Boot)
> > to sdram & then jumps to entry point.
> >
> > Here are the flash memory addresses for U-Boot-spl & standard U-Boot:
> >     - spl U-Boot            : 0x0800_0000
> >     - standard U-Boot       : 0x0800_8000
> >
> > To compile u-boot without spl: Remove SUPPORT_SPL configuration
> > (arch/arm/mach-stm32/Kconfig)
> >
> > Signed-off-by: Vikas Manocha <vikas.mano...@st.com>
> 
> I had to rework the Kconfig locations here slightly (and in follow-up
> patches) to have the main select's be under STM32F7 in 
> arch/arm/mach-stm32/Kconfig as it's not right to introduce a second 'config
> SUPPORT_SPL\nselect ...' stanza inside of arch/arm/mach-stm32/stm32f7/Kconfig.

Thanks for the kconfig rework, appreciate it.

Cheers,
Vikas

> 
> Reviewed-by: Tom Rini <tr...@konsulko.com>
> 
> --
> Tom
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to