irqs_disable() can return void (#407)

Signed-off-by: John Levon <john.levon@nutanix.com>
Reviewed-by: Thanos Makatos <thanos.makatos@nutanix.com>

1 files changed, 3 insertions, 5 deletions

diff --git a/lib/irq.c b/lib/irq.c
index ecb6ce0..0531f59 100644
--- a/lib/irq.c
+++ b/lib/irq.c
@@ -92,7 +92,7 @@ handle_device_get_irq_info(vfu_ctx_t *vfu_ctx, uint32_t size,
     return dev_get_irqinfo(vfu_ctx, irq_info_in, irq_info_out);
 }
 
-static long
+static void
 irqs_disable(vfu_ctx_t *vfu_ctx, uint32_t index, uint32_t start, uint32_t count)
 {
     size_t i;
@@ -133,8 +133,6 @@ irqs_disable(vfu_ctx_t *vfu_ctx, uint32_t index, uint32_t start, uint32_t count)
             efds[i] = -1;
         }
     }
-
-    return 0;
 }
 
 static int
@@ -331,8 +329,8 @@ handle_device_set_irqs(vfu_ctx_t *vfu_ctx, uint32_t size,
 
     if ((data_type == VFIO_IRQ_SET_DATA_NONE && irq_set->count == 0) ||
         (data_type == VFIO_IRQ_SET_DATA_EVENTFD && nr_fds == 0)) {
-        return irqs_disable(vfu_ctx, irq_set->index,
-                            irq_set->start, irq_set->count);
+        irqs_disable(vfu_ctx, irq_set->index, irq_set->start, irq_set->count);
+        return 0;
     }
 
     vfu_log(vfu_ctx, LOG_DEBUG, "setting IRQ %s flags=%#x range [%u, %u)",