diff options
| author | Ilias Apalodimas <ilias.apalodimas@linaro.org> | 2021-03-10 15:35:11 +0200 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2021-04-12 17:44:55 -0400 |
| commit | 92e84896112037833e429d629f87cedbeb255d5a (patch) | |
| tree | 5d36a324e3bbfcf05b094891995db2f5953c3a41 | |
| parent | ac3821430e0e2dc6ddb766950b0098c67d4969d3 (diff) | |
| download | u-boot-92e84896112037833e429d629f87cedbeb255d5a.zip u-boot-92e84896112037833e429d629f87cedbeb255d5a.tar.gz u-boot-92e84896112037833e429d629f87cedbeb255d5a.tar.bz2 | |
tee: optee: Change printing during optee_probe
Right now the error messages when optee has a version mismatch or shared
memory is not configured are done with a debug().
That's not very convenient since you have to enable debugging to figure
out what's going on, although this is an actual error.
So let's switch the debug() -> dev_err() and report those explicitly.
Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
| -rw-r--r-- | drivers/tee/optee/core.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/tee/optee/core.c b/drivers/tee/optee/core.c index b898c32..73dbb22 100644 --- a/drivers/tee/optee/core.c +++ b/drivers/tee/optee/core.c @@ -624,14 +624,14 @@ static int optee_probe(struct udevice *dev) u32 sec_caps; if (!is_optee_api(pdata->invoke_fn)) { - debug("%s: OP-TEE api uid mismatch\n", __func__); + dev_err(dev, "OP-TEE api uid mismatch\n"); return -ENOENT; } print_os_revision(dev, pdata->invoke_fn); if (!api_revision_is_compatible(pdata->invoke_fn)) { - debug("%s: OP-TEE api revision mismatch\n", __func__); + dev_err(dev, "OP-TEE api revision mismatch\n"); return -ENOENT; } @@ -642,7 +642,7 @@ static int optee_probe(struct udevice *dev) */ if (!exchange_capabilities(pdata->invoke_fn, &sec_caps) || !(sec_caps & OPTEE_SMC_SEC_CAP_DYNAMIC_SHM)) { - debug("%s: OP-TEE capabilities mismatch\n", __func__); + dev_err(dev, "OP-TEE capabilities mismatch\n"); return -ENOENT; } |