[PATCH v4 01/10] ARM: armv7: add non-SPL enable for Cortex SMPEN

Marek Vasut marek.vasut at mailbox.org
Tue Apr 18 21:35:06 CEST 2023


On 4/18/23 16:32, Ralph Siemsen wrote:
> On Mon, Apr 17, 2023 at 4:21 PM Marek Vasut <marek.vasut at 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.siemsen@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"),

ACK


More information about the U-Boot mailing list