commit d0123afc1f89665837c8cd20445caeab9503343f
author Karl Meakin <karl.meakin@arm.com> Mon Mar 17 16:46:38 2025 +0000
committer J-Alves <joao.alves@arm.com> Fri Jun 13 14:27:39 2025 +0100
tree 75fe0e31110f4d5a6933684c50bbcca9e91fbdd2
parent b9afc8975ddbd98bdcd3507199a0521bd6f94843

chore: fix clang-tidy warnings

Change-Id: I2cd96000b7216ac88cb7a7ca586c1168e0229b1d
Signed-off-by: Karl Meakin <karl.meakin@arm.com>

src/api.c

diff --git a/src/api.c b/src/api.c
index 0d9bd6a..5fb4daf 100644
--- a/src/api.c
+++ b/src/api.c
@@ -655,12 +655,8 @@
 			 * NOT_SUPPORTED, that is not an error. If there are no
 			 * secure partitions the SPMC returns NOT_SUPPORTED.
 			 */
-			if ((ffa_func_id(ret) == FFA_ERROR_32) &&
-			    (ffa_error_code(ret) == FFA_NOT_SUPPORTED)) {
-				return true;
-			}
-
-			return false;
+			return (ffa_func_id(ret) == FFA_ERROR_32) &&
+			       (ffa_error_code(ret) == FFA_NOT_SUPPORTED);
 		}
 
 		if (!api_ffa_fill_partition_info_from_regs(
@@ -1114,14 +1110,16 @@
  */
 bool api_extended_args_are_zero(struct ffa_value *args)
 {
-	if (args->extended_val.arg8 != 0U || args->extended_val.arg9 != 0U ||
-	    args->extended_val.arg10 != 0U || args->extended_val.arg11 != 0U ||
-	    args->extended_val.arg12 != 0U || args->extended_val.arg13 != 0U ||
-	    args->extended_val.arg14 != 0U || args->extended_val.arg15 != 0U ||
-	    args->extended_val.arg16 != 0U || args->extended_val.arg17 != 0U) {
-		return false;
-	}
-	return true;
+	return (args->extended_val.arg8 == 0U &&
+		args->extended_val.arg9 == 0U &&
+		args->extended_val.arg10 == 0U &&
+		args->extended_val.arg11 == 0U &&
+		args->extended_val.arg12 == 0U &&
+		args->extended_val.arg13 == 0U &&
+		args->extended_val.arg14 == 0U &&
+		args->extended_val.arg15 == 0U &&
+		args->extended_val.arg16 == 0U &&
+		args->extended_val.arg17 == 0U);
 }
 
 static void api_ffa_msg_wait_rx_release(struct vcpu *current)