On 29.03.2019, at 13:16, Kever Yang kever.yang@rock-chips.com wrote:
Philipp,
On 03/29/2019 07:25 PM, Philipp Tomsich wrote:
Kever,
On 29.03.2019, at 12:21, Kever Yang kever.yang@rock-chips.com wrote:
Rockchip provide tee binary release in 'rkbin' repository: https://github.com/rockchip-linux/rkbin For some historical reason, rockchip optee binary is using 'r1' instead of 'lr' as U-Boot entry.
Signed-off-by: Kever Yang kever.yang@rock-chips.com
common/spl/spl_optee.S | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/common/spl/spl_optee.S b/common/spl/spl_optee.S index 8bd1949ddf..092307b3cc 100644 --- a/common/spl/spl_optee.S +++ b/common/spl/spl_optee.S @@ -8,5 +8,8 @@
ENTRY(spl_optee_entry) ldr lr, =CONFIG_SYS_TEXT_BASE +#ifdef CONFIG_ARCH_ROCKCHIP
Can we make this selectable based on a dedicated config-option? We provide our own OPTEE port for some of our modules and I would like to have this as an opt-in or opt-out option in Kconfig.
I think you are using OPTEE for RK3368/RK3399, right? Then the use case is different, you are using OPTEE as bl32 for armv8, and this spl_optee is for armv7 only. I'm OK to add a Kconfig option if you really have different usage, and I think this patch does not break things because the no one use 'r1' now.
Thanks for the clarification. You’re right that I am only interested in ARMv8…
Thanks, Philipp.
Thanks,
- Kever
- ldr r1, =CONFIG_SYS_TEXT_BASE
+#endif mov pc, r3 ENDPROC(spl_optee_entry) -- 2.20.1