On Mon, Apr 17, 2023 at 4:21 PM Marek Vasut <marek.va...@mailbox.org> wrote: > > On 4/17/23 20:26, Ralph Siemsen wrote: > > On Mon, Apr 17, 2023 at 07:04:15PM +0200, Marek Vasut wrote: > >> You could send this patch as a separate one. > > > > In fact I did that almost a year ago: > > https://lore.kernel.org/u-boot/20220421175202.2614945-1-ralph.siem...@linaro.org/T/#u > > Hmmm, Tom (+Cc) pointed out the patch does not do anything, but that is > not correct, is it ? If I look at 2564fce7eea3 ("sunxi: move Cortex > SMPEN setting into start.S") , this extra Kconfig symbol just makes that > functionality available outside of SPL (i.e. in normal U-Boot), while > before this change the functionality was only available in SPL. Right ?
Correct, my patch makes the symbol available outside of SPL. I think Tom's objection was to adding the symbol without any users. This is rectified by patch 07 ("ARM: rzn1: basic support"), Ralph