diff options
author | J-Alves <joao.alves@arm.com> | 2021-06-03 10:32:45 +0100 |
---|---|---|
committer | J-Alves <joao.alves@arm.com> | 2021-06-03 10:33:42 +0100 |
commit | 69bd5cc30415b19f099f15c2b64ee4667846b3e1 (patch) | |
tree | f695cfe24a160926cd41ce89f9820fc64f14f786 /tftf/tests | |
parent | eee1452b9c8714b62ecf4a443858564811e07362 (diff) | |
download | tf-a-tests-69bd5cc30415b19f099f15c2b64ee4667846b3e1.tar.gz |
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
Diffstat (limited to 'tftf/tests')
-rw-r--r-- | tftf/tests/runtime_services/secure_service/spm_common.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tftf/tests/runtime_services/secure_service/spm_common.c b/tftf/tests/runtime_services/secure_service/spm_common.c index 12b70a9a2..179ef1cb9 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 @@ bool is_ffa_direct_response(smc_ret_values ret) 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 @@ bool is_expected_ffa_return(smc_ret_values ret, uint32_t func_id) 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; } |