refactor(interrupts): remove api_interrupt_inject
api_interrupt_inject and vcpu_interrupt_inject were performing the
same function, as the next argument was never being used. So just
use vcpu_interrupt_inject.
Also remove unnecessary if statement from vcpu_interrupt_inject.
Signed-off-by: Daniel Boulby <daniel.boulby@arm.com>
Change-Id: I36c5157b7961e5a85b3e485f0a309b5dddf7c21a
diff --git a/src/ffa/spmc/interrupts.c b/src/ffa/spmc/interrupts.c
index b69a47a..8f0c9ac 100644
--- a/src/ffa/spmc/interrupts.c
+++ b/src/ffa/spmc/interrupts.c
@@ -584,8 +584,7 @@
}
bool ffa_interrupts_inject_notification_pending_interrupt(
- struct vcpu_locked target_locked, struct vcpu_locked current_locked,
- struct vm_locked receiver_locked)
+ struct vcpu_locked target_locked, struct vm_locked receiver_locked)
{
struct vm *next_vm = target_locked.vcpu->vm;
bool ret = false;
@@ -602,9 +601,8 @@
receiver_locked, vcpu_index(target_locked.vcpu)) ||
(vm_are_global_notifications_pending(receiver_locked) &&
!vm_notifications_is_npi_injected(receiver_locked)))) {
- api_interrupt_inject_locked(target_locked,
- HF_NOTIFICATION_PENDING_INTID,
- current_locked, NULL);
+ vcpu_interrupt_inject(target_locked,
+ HF_NOTIFICATION_PENDING_INTID);
vm_notifications_set_npi_injected(receiver_locked, true);
ret = true;
}