aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOlivier Deprez <olivier.deprez@arm.com>2021-06-04 19:47:47 +0200
committerTrustedFirmware Code Review <review@review.trustedfirmware.org>2021-06-04 19:47:47 +0200
commit5de3d67e9fcc8b03efd28f6ae5fd4e0e7a345209 (patch)
treef695cfe24a160926cd41ce89f9820fc64f14f786
parentcfae10dfe9e4a2464b3d290b8f39be1cdeb7ac29 (diff)
parent69bd5cc30415b19f099f15c2b64ee4667846b3e1 (diff)
downloadtf-a-tests-5de3d67e9fcc8b03efd28f6ae5fd4e0e7a345209.tar.gz
Merge changes from topic "tests_tidy_log"
* changes: fix(spm): change error logs to verbose fix(cactus): remove misleading ERROR log
-rw-r--r--spm/cactus/cactus_tests/cactus_message_loop.c1
-rw-r--r--tftf/tests/runtime_services/secure_service/spm_common.c6
2 files changed, 3 insertions, 4 deletions
diff --git a/spm/cactus/cactus_tests/cactus_message_loop.c b/spm/cactus/cactus_tests/cactus_message_loop.c
index 11207dca..fde70744 100644
--- a/spm/cactus/cactus_tests/cactus_message_loop.c
+++ b/spm/cactus/cactus_tests/cactus_message_loop.c
@@ -49,7 +49,6 @@ bool cactus_handle_cmd(smc_ret_values *cmd_args, smc_ret_values *ret,
}
}
- ERROR("Unhandled test command!\n");
*ret = cactus_error_resp(ffa_dir_msg_dest(*cmd_args),
ffa_dir_msg_source(*cmd_args),
CACTUS_ERROR_UNHANDLED);
diff --git a/tftf/tests/runtime_services/secure_service/spm_common.c b/tftf/tests/runtime_services/secure_service/spm_common.c
index 12b70a9a..179ef1cb 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;
}