Re: [PATCH 2/4] tee: optee: Do not warn about TEE_ERROR_STORAGE_NOT_AVAILABLE

"Kathpalia, Tanmay" <[email protected]>
Newsgroups gmane.comp.boot-loaders.u-boot
Message-ID <5b98ee84-4a9a-45a7-927a-c2f98191d312__35908.1864518643$1787496006$gmane$org@altera.com>
Hi Jan,

One minor nit.


On 21-08-2026 14:58, Jan Kiszka wrote:
> From: Jan Kiszka<[email protected]>
>
> This is a transitional error that is resolved once an RPMB becomes
> available. Keep it as debug output only.
>
> Signed-off-by: Jan Kiszka<[email protected]>
> ---
>   drivers/tee/optee/core.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/tee/optee/core.c b/drivers/tee/optee/core.c
> index 5600d5a4c7d..4b1396b8652 100644
> --- a/drivers/tee/optee/core.c
> +++ b/drivers/tee/optee/core.c
> @@ -126,7 +126,10 @@ static int __enum_services(struct udevice *dev, struct tee_shm *shm, size_t *shm
>   
>   	ret = tee_invoke_func(dev, &arg, 1, &param);
>   	if (ret || (arg.ret && arg.ret != TEE_ERROR_SHORT_BUFFER)) {
> -		dev_err(dev, "Enumeration command 0x%x failed: 0x%x\n", pta_cmd, arg.ret);
> +		if (arg.ret != TEE_ERROR_STORAGE_NOT_AVAILABLE)
> +			dev_err(dev, "Enumeration command 0x%x failed: 0x%x\n", pta_cmd, arg.ret);
> +		else
> +			debug("Enumeration command 0x%x failed due to unavailable storage\n", pta_cmd);

Prefer dev_dbg() over debug() so the device name stays in the log, same 
as the dev_err path.

>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.