[U-Boot] [PATCH v2 1/6] ARMv8: LS1043A: change macro CONFIG_ARMV8_PSCI definition

Hongbo Zhang macro.wave.z at gmail.com
Fri Nov 4 09:25:21 CET 2016


Talked to the author Zhiqiang Hou who introduced the previous
CONFIG_ARMV8_PSCI, although this config was introduced due to FSL PPA,
codes covered by it in the armv8/cpu-dt.c, bootm-dt.c, psci-dt.c also
work for the coming generic ARMv8 PSCI framework, what's more,
Zhiqiang is sending patches to clean up PPA related configs to
Kconfig, so I don't need to send this 1/6 patch any more, I'll send
the other updated 5 patches in v3.

On Thu, Nov 3, 2016 at 5:56 PM, Hongbo Zhang <macro.wave.z at gmail.com> wrote:
> On Wed, Nov 2, 2016 at 10:27 PM, Tom Rini <trini at konsulko.com> wrote:
>> On Tue, Nov 01, 2016 at 06:04:27PM +0800, macro.wave.z at gmail.com wrote:
>>
>>> From: Hongbo Zhang <hongbo.zhang at nxp.com>
>>>
>>> NXP/Freescale uses macro CONFIG_ARMV8_PSCI to enable their private PSCI
>>> implementation in PPA firmware, but this macro naming too generic, so this
>>> patch replaces it with a specic one CONFIG_FSL_PPA_ARMV8_PSCI.
>>> And this macro CONFIG_ARMV8_PSCI will be used for a generic PSCI for ARMv8
>>> which will be added in following patchs.
>>>
>>> Signed-off-by: Hongbo Zhang <hongbo.zhang at nxp.com>
>> [snip]
>>> diff --git a/include/configs/ls1043ardb.h b/include/configs/ls1043ardb.h
>>> index 1d0a1ac..d2f8fc0 100644
>>> --- a/include/configs/ls1043ardb.h
>>> +++ b/include/configs/ls1043ardb.h
>>> @@ -12,7 +12,7 @@
>>>  #if defined(CONFIG_FSL_LS_PPA)
>>>  #define CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT
>>>  #define SEC_FIRMWARE_ERET_ADDR_REVERT
>>> -#define CONFIG_ARMV8_PSCI
>>> +#define CONFIG_FSL_PPA_ARMV8_PSCI
>>>
>>>  #define CONFIG_SYS_LS_PPA_FW_IN_XIP
>>>  #ifdef CONFIG_SYS_LS_PPA_FW_IN_XIP
>>
>> This will trip-up the whitelist check, please move the new symbol to
>> Kconfig and select it from LS1043ARDB + FSL_LS_PPA.  Thanks!
>>
>
> Payed much attention to the config options I introduced in the common
> framework, but neglected this legacy one.
> Yes will update this.
>
>> --
>> Tom


More information about the U-Boot mailing list