Re: [PATCH v2] tee: optee: sync cache on pre-reloc OP-TEE invocation

On Wed, May 19, 2021 at 4:28 PM Etienne Carriere etienne.carriere@linaro.org wrote:

This change ensures both U-Boot and OP-TEE see the same content from shared memory when OP-TEE is invoked prior U-Boot relocation.

This change is required since U-Boot may execute with data cahce off

cache

while OP-TEE always enables cache on memory shared with U-Boot.

Signed-off-by: Etienne Carriere etienne.carriere@linaro.org

Changes since v1:

• Fix invocation argument buffer size in optee/core.c, using OPTEE_MSG_GET_ARG_SIZE().
• add missing #include <asm/cache.h> in tee-uclass.c

---

drivers/tee/optee/core.c | 21 ++++++++++++++++++++- drivers/tee/tee-uclass.c | 19 ++++++++++++++++++- include/tee.h | 6 ++++++ 3 files changed, 44 insertions(+), 2 deletions(-)

With the spell error in the commit message fixed: Reviewed-by: Jens Wiklander jens.wiklander@linaro.org

Cheers, Jens