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

[Ahmad Fatoum <a.fatoum@pengutronix.de>]

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;