On Fri, Aug 18, 2017 at 4:28 PM, Vikas MANOCHA <[email protected]> wrote: > Hi Bo, > >> -----Original Message----- >> From: Bo Shen [mailto:[email protected]] >> Sent: Thursday, August 17, 2017 10:07 PM >> To: Robert Nelson <[email protected]>; Vikas MANOCHA >> <[email protected]> >> Cc: U-Boot-Denx <[email protected]>; Christophe PRIOUZEAU >> <[email protected]>; Alexandre TORGUE >> <[email protected]> >> 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, -- Robert Nelson https://rcn-ee.com/ _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
