refactor: don't pass sender/receiver ID
Don't pass `sender_vm_id` or `receiver_vm_id` to
`api_ffa_msg_send_direct_req` and `api_ffa_msg_send_direct_resp`; they
can be retrieved from `args` so it is redundant to pass them twice.
Change-Id: I9abcb4d034b0cff1a05dcd76d1fe04efdfc4cbe7
Signed-off-by: Karl Meakin <karl.meakin@arm.com>
diff --git a/src/api.c b/src/api.c
index 4728f31..1db5b30 100644
--- a/src/api.c
+++ b/src/api.c
@@ -2821,12 +2821,12 @@
* range of registers (x4-x17, instead of x4-x7) to be used as part of the
* message payload.
*/
-struct ffa_value api_ffa_msg_send_direct_req(ffa_id_t sender_vm_id,
- ffa_id_t receiver_vm_id,
- struct ffa_value args,
+struct ffa_value api_ffa_msg_send_direct_req(struct ffa_value args,
struct vcpu *current,
struct vcpu **next)
{
+ ffa_id_t sender_vm_id = ffa_sender(args);
+ ffa_id_t receiver_vm_id = ffa_receiver(args);
struct ffa_value ret;
struct vm *receiver_vm;
struct vm_locked receiver_locked;
@@ -3061,12 +3061,12 @@
* of registers (x4-x17, instead of x4-x7) to be used as part of the
* message payload.
*/
-struct ffa_value api_ffa_msg_send_direct_resp(ffa_id_t sender_vm_id,
- ffa_id_t receiver_vm_id,
- struct ffa_value args,
+struct ffa_value api_ffa_msg_send_direct_resp(struct ffa_value args,
struct vcpu *current,
struct vcpu **next)
{
+ ffa_id_t sender_vm_id = ffa_sender(args);
+ ffa_id_t receiver_vm_id = ffa_receiver(args);
struct vcpu_locked current_locked;
struct vcpu_locked next_locked = (struct vcpu_locked){
.vcpu = NULL,