On Fri, Aug 18, 2017 at 4:28 PM, Vikas MANOCHA vikas.manocha@st.com wrote:
Hi Bo,
-----Original Message----- From: Bo Shen [mailto:voice.shen@gmail.com] Sent: Thursday, August 17, 2017 10:07 PM To: Robert Nelson robertcnelson@gmail.com; Vikas MANOCHA vikas.manocha@st.com Cc: U-Boot-Denx u-boot@lists.denx.de; Christophe PRIOUZEAU christophe.priouzeau@st.com; Alexandre TORGUE alexandre.torgue@st.com Subject: Re: [U-Boot] [PATCH v2 3/7] stm32: stm32f7: add spl build support
Hi Vikas, Try to remove this magic (press 'c' on the keyboard when at the boot time),
It helps to select between U-Boot or OS booting.
I'd suggest to add a follow-up patch to move all the selected SPL related configuration from "arch/arm/mach-stm32/Kconfig" under STM32F7 to default configuration file (stm32f746- disco_defconfig). Then if the people want just boot up u-boot itself, what they need to do is just run "make menuconfig" and then de- select "Activate Falcon Mode" (SPL_OS_BOOT). Then everything will be fine.
Yes, Agree. How about using "imply" instead of "select" for SPL_OS_BOOT to make it configurable.
Well, we really need a /doc/README.stm32 that explains this "different" default.. It's definitely different then omap/imx/tegra/atmel in this tree. But the processor is also more handicapped, so not wasting time in normal u-boot just booting the kernel via spl/falcon more makes sense.
Regards,