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, ¶m); > 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. >