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.

Reply via email to