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

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@nxp.com Reviewed-by: Tom Rini trini@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.

York