fix(spm): change error logs to verbose
Helper functions to validate return of FF-A calls or responses
from other endpoints in 'spm_common', were printing ERROR also in cases
where this was actually expected.
Changing those ERROR messages to VERBOSE, given ERROR as referred was
misleading.
Signed-off-by: J-Alves <joao.alves@arm.com>
Change-Id: Id3feac197e8dbff78a616392b5ac7ca914c219df
diff --git a/tftf/tests/runtime_services/secure_service/spm_common.c b/tftf/tests/runtime_services/secure_service/spm_common.c
index 12b70a9..179ef1c 100644
--- a/tftf/tests/runtime_services/secure_service/spm_common.c
+++ b/tftf/tests/runtime_services/secure_service/spm_common.c
@@ -19,7 +19,7 @@
bool is_ffa_call_error(smc_ret_values ret)
{
if (ffa_func_id(ret) == FFA_ERROR) {
- ERROR("FF-A call returned error (%x): %d\n",
+ VERBOSE("FF-A call returned error (%x): %d\n",
ffa_func_id(ret), ffa_error_code(ret));
return true;
}
@@ -38,7 +38,7 @@
return true;
}
- ERROR("%x is not FF-A response.\n", ffa_func_id(ret));
+ VERBOSE("%x is not FF-A response.\n", ffa_func_id(ret));
/* To log error in case it is FFA_ERROR*/
is_ffa_call_error(ret);
@@ -54,7 +54,7 @@
return true;
}
- ERROR("Expecting %x, FF-A return was %x\n", func_id, ffa_func_id(ret));
+ VERBOSE("Expecting %x, FF-A return was %x\n", func_id, ffa_func_id(ret));
return false;
}