commit 3f5d627bae1b791cc056ac2f87820a37547feec4
author AlexeiFedorov <Alexei.Fedorov@arm.com> Mon Oct 23 16:27:37 2023 +0100
committer Soby Mathew <soby.mathew@arm.com> Tue Oct 31 10:28:33 2023 +0000
tree 79854fabca38387a84129790b20db3041f2360a9
parent 4201310827ae8491f7e416c5dbc1e2f7b6e72d09

fix(rmm-eac5): fix MISRA C defects

Fixes MISRA C 2012 Rule 2.3, 12.1 and 18.4 violations.

Change-Id: I7b96776da05c636fde89d915b0aae5a7134baab4
Signed-off-by: AlexeiFedorov <Alexei.Fedorov@arm.com>

runtime/rsi/host_call.c

diff --git a/runtime/rsi/host_call.c b/runtime/rsi/host_call.c
index a35eac1..8ebf8d9 100644
--- a/runtime/rsi/host_call.c
+++ b/runtime/rsi/host_call.c
@@ -35,7 +35,7 @@
 	unsigned long ipa = rec->regs[1];
 	unsigned long page_ipa;
 	struct granule *gr;
-	unsigned char *data;
+	uintptr_t data;
 	struct rsi_host_call *host_call;
 	unsigned int i;
 
@@ -59,16 +59,17 @@
 		}
 		return;
 	case WALK_INVALID_PARAMS:
+	default:
 		assert(false);
 		break;
 	}
 
 	/* Map Realm data granule to RMM address space */
 	gr = find_granule(walk_result.pa);
-	data = (unsigned char *)granule_map(gr, SLOT_RSI_CALL);
-	assert(data != NULL);
+	data = (uintptr_t)granule_map(gr, SLOT_RSI_CALL);
+	assert(data != 0UL);
 
-	host_call = (struct rsi_host_call *)(data + (ipa - page_ipa));
+	host_call = (struct rsi_host_call *)(data + ipa - page_ipa);
 
 	if (rec_exit != NULL) {
 		/* Copy host call arguments to REC exit data structure */
@@ -97,7 +98,7 @@
 	}
 
 	/* Unmap Realm data granule */
-	buffer_unmap(data);
+	buffer_unmap((void *)data);
 
 	/* Unlock last level RTT */
 	granule_unlock(walk_result.llt);