[U-Boot] [PATCH v4 3/6] ARMv8: Add secure sections for PSCI text and data

[Hongbo Zhang]

On Wed, Nov 30, 2016 at 4:49 AM, york sun <york.sun at nxp.com> wrote:
> On 11/25/2016 02:48 AM, Hongbo Zhang wrote:
>> This patch adds secure_text, secure_data and secure_stack sections for ARMv8 to
>> hold PSCI text and data, and it is based on the legacy implementation of ARMv7.
>>
>> Signed-off-by: Hongbo Zhang <hongbo.zhang at nxp.com>
>> Reviewed-by: Tom Rini <trini at konsulko.com>
>> ---
>>  arch/arm/config.mk            |  3 ++-
>>  arch/arm/cpu/armv8/Kconfig    | 31 +++++++++++++++++++++++
>>  arch/arm/cpu/armv8/u-boot.lds | 57 +++++++++++++++++++++++++++++++++++++++++++
>>  3 files changed, 90 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/config.mk b/arch/arm/config.mk
>> index 008da39..3a81f13 100644
>> --- a/arch/arm/config.mk
>> +++ b/arch/arm/config.mk
>> @@ -121,7 +121,8 @@ endif
>>
>>  # limit ourselves to the sections we want in the .bin.
>>  ifdef CONFIG_ARM64
>> -OBJCOPYFLAGS += -j .text -j .rodata -j .data -j .u_boot_list -j .rela.dyn
>> +OBJCOPYFLAGS += -j .text -j .secure_text -j .secure_data -j .rodata -j .data \
>> +             -j .u_boot_list -j .rela.dyn
>>  else
>>  OBJCOPYFLAGS += -j .text -j .secure_text -j .secure_data -j .rodata -j .hash \
>>               -j .data -j .got -j .got.plt -j .u_boot_list -j .rel.dyn
>> diff --git a/arch/arm/cpu/armv8/Kconfig b/arch/arm/cpu/armv8/Kconfig
>> index 965a8d1..edae43d 100644
>> --- a/arch/arm/cpu/armv8/Kconfig
>> +++ b/arch/arm/cpu/armv8/Kconfig
>> @@ -40,4 +40,35 @@ config PSCI_RESET
>>
>>         Select Y here to make use of PSCI calls for system reset
>>
>> +config ARMV8_PSCI
>> +     bool "Enable PSCI support" if EXPERT
>> +     default n
>> +     help
>> +       PSCI is Power State Coordination Interface defined by ARM.
>> +       The PSCI in U-boot provides a general framework and each platform
>> +       can implement their own specific PSCI functions.
>> +       Say Y here to enable PSCI support on ARMv8 platform.
>> +
>> +config ARMV8_PSCI_NR_CPUS
>> +     int "Maximum supported CPUs for PSCI"
>> +     depends on ARMV8_PSCI
>> +     default 4
>> +     help
>> +       The maximum number of CPUs supported in the PSCI firmware.
>> +       It is no problem to set a larger value than the number of CPUs in
>> +       the actual hardware implementation.
>> +
>> +if SYS_HAS_ARMV8_SECURE_BASE
>> +
>> +config ARMV8_SECURE_BASE
>> +     hex "Secure address for PSCI image"
>> +     depends on ARMV8_PSCI
>> +     help
>> +       Address for placing the PSCI text, data and stack sections.
>> +       If not defined, the PSCI sections are placed together with the u-boot
>> +       but platform can choose to place PSCI code image separately in other
>> +       places such as some secure RAM built-in SOC etc.
>> +
>> +endif
>> +
>
> These changes to Kconfig is not related to the secure sections you
> described in the commit message.
>

These configs were originally in the patch of introducing generic PSCI
framework, following Tom's suggest that configs should be introduced
in Kconfig when they are used first time. I'll mention this in commit
message to let it match.

> York
>