[PATCH master] optee: don't warn about missing OP-TEE header
Ahmad Fatoum
a.fatoum at pengutronix.de
Wed Feb 28 10:03:20 PST 2024
OP-TEE header is checked once in PBL, saved into scratch area after
verification and then checked again in barebox proper.
The check in PBL fails silently, but the check in barebox proper that
should always follow, because the header isn't written to the scratch
area is printed with error log level.
Printing an error in either case is wrong though as using a raw OP-TEE
binary without header is a valid use case and the OP-TEE header may
also be missing when barebox is chainloaded from a running barebox.
Therefore reduce the message to debug log level.
Signed-off-by: Ahmad Fatoum <a.fatoum at pengutronix.de>
---
common/optee.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/common/optee.c b/common/optee.c
index 34667f1f51e0..a8a43554e757 100644
--- a/common/optee.c
+++ b/common/optee.c
@@ -14,8 +14,8 @@ int optee_verify_header(const struct optee_header *hdr)
return -EINVAL;
if (hdr->magic != OPTEE_MAGIC) {
- pr_err("Invalid header magic 0x%08x, expected 0x%08x\n",
- hdr->magic, OPTEE_MAGIC);
+ pr_debug("Invalid header magic 0x%08x, expected 0x%08x\n",
+ hdr->magic, OPTEE_MAGIC);
return -EINVAL;
}
--
2.39.2
More information about the barebox
mailing list