fix: few typos and verbosity
Change-Id: I604e6b74216f8fb4d1b679c0dec777fa8f15b185
Signed-off-by: Olivier Deprez <olivier.deprez@arm.com>
diff --git a/src/arch/aarch64/plat/ffa/spmc.c b/src/arch/aarch64/plat/ffa/spmc.c
index 19ddb28..82428de 100644
--- a/src/arch/aarch64/plat/ffa/spmc.c
+++ b/src/arch/aarch64/plat/ffa/spmc.c
@@ -493,7 +493,6 @@
{
struct ffa_value ret = (struct ffa_value){.func = FFA_SUCCESS_32};
struct vm_locked vm_locked;
- const char *error_string = "Notification bitmap already created.";
struct nwd_vms_locked nwd_vms_locked = nwd_vms_lock();
if (vm_id == HF_OTHER_WORLD_ID) {
@@ -509,7 +508,7 @@
/* Call has been used for the other world vm already */
if (vm_locked.vm->notifications.enabled != false) {
- dlog_error("%s\n", error_string);
+ dlog_verbose("Notification bitmap already created.");
ret = ffa_error(FFA_DENIED);
goto out;
}
@@ -523,7 +522,7 @@
/* If vm already exists bitmap has been created as well. */
vm_locked = plat_ffa_nwd_vm_find_locked(nwd_vms_locked, vm_id);
if (vm_locked.vm != NULL) {
- dlog_error("%s\n", error_string);
+ dlog_verbose("Notification bitmap already created.");
ret = ffa_error(FFA_DENIED);
goto out;
}
diff --git a/src/ffa_memory.c b/src/ffa_memory.c
index ab47929..377f6b3 100644
--- a/src/ffa_memory.c
+++ b/src/ffa_memory.c
@@ -536,7 +536,7 @@
fragment_constituent_counts,
fragment_count);
if (ret.func != FFA_SUCCESS_32) {
- dlog_verbose("Inconsistent modes.\n", fragment_count);
+ dlog_verbose("Inconsistent modes.\n");
return ret;
}
diff --git a/src/vcpu.c b/src/vcpu.c
index aba6449..87486f2 100644
--- a/src/vcpu.c
+++ b/src/vcpu.c
@@ -89,8 +89,8 @@
* Check whether the given vcpu_state is an off state, for the purpose of
* turning vCPUs on and off. Note that Aborted still counts as ON for the
* purposes of PSCI, because according to the PSCI specification (section
- * 5.7.1) a core is only considered to be off if it has been turned off
- * with a CPU_OFF call or hasn't yet been turned on with a CPU_ON call.
+ * 5.7.1) a core is only considered to be off if it has been turned off
+ * with a CPU_OFF call or hasn't yet been turned on with a CPU_ON call.
*/
bool vcpu_is_off(struct vcpu_locked vcpu)
{