Re: [PATCH 6/9] tee: optee: implement shared mem alloc/free RPC commands

[Ahmad Fatoum]

Hello Sascha,

On 3/12/25 13:16, Sascha Hauer wrote:
>  void optee_rpc_cmd(struct tee_context *ctx, struct optee *optee,
>                  struct optee_msg_arg *arg)
>  {
> -     pr_notice_once("optee: No supplicant or RPC handler for command 
> 0x%x\n", arg->cmd);
> -     arg->ret = TEEC_ERROR_NOT_SUPPORTED;
> +     pr_debug("%s: receive RPC CMD: %d\n", __func__, arg->cmd);
> +
> +     switch (arg->cmd) {
> +     case OPTEE_RPC_CMD_SHM_ALLOC:
> +             cmd_shm_alloc(ctx, arg);
> +             break;
> +     case OPTEE_RPC_CMD_SHM_FREE:
> +             cmd_shm_free(arg);
> +             break;
> +     default:
> +             arg->ret = TEEC_ERROR_NOT_IMPLEMENTED;

Please move the pr_notice_once here, so barebox users know that
OP-TEE tried to do something not supported yet.

Thanks,
Ahmad

> +     }
> +
> +     arg->ret_origin = TEEC_ORIGIN_COMMS;