irqchip/armada-370-xp: Use unsigned int type for virqs
authorMarek Behún <kabel@kernel.org>
Thu, 11 Jul 2024 11:57:40 +0000 (13:57 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 30 Jul 2024 11:35:46 +0000 (13:35 +0200)
The return type of irq_find_mapping() and irq_linear_revmap() is
unsigned int. Use the unsigned int type for the variables storing the
return value.

Signed-off-by: Marek Behún <kabel@kernel.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/all/20240711115748.30268-3-kabel@kernel.org
drivers/irqchip/irq-armada-370-xp.c

index 7016b206bdddc354dc00beabc8321d9020779d21..b29f3bbfb1c31bd85cbcc0ff6edea2ffb81d27be 100644 (file)
@@ -462,10 +462,10 @@ static const struct irq_domain_ops ipi_domain_ops = {
 static void ipi_resume(void)
 {
        for (int i = 0; i < IPI_DOORBELL_END; i++) {
-               int virq;
+               unsigned int virq;
 
                virq = irq_find_mapping(ipi_domain, i);
-               if (virq <= 0)
+               if (!virq)
                        continue;
                if (irq_percpu_is_enabled(virq)) {
                        struct irq_data *d;
@@ -539,7 +539,7 @@ static void armada_xp_mpic_reenable_percpu(void)
        /* Re-enable per-CPU interrupts that were enabled before suspend */
        for (unsigned int irq = 0; irq < MPIC_MAX_PER_CPU_IRQS; irq++) {
                struct irq_data *data;
-               int virq;
+               unsigned int virq;
 
                virq = irq_linear_revmap(armada_370_xp_mpic_domain, irq);
                if (virq == 0)
@@ -734,7 +734,7 @@ static void armada_370_xp_mpic_resume(void)
        nirqs = (readl(main_int_base + MPIC_INT_CONTROL) >> 2) & 0x3ff;
        for (irq_hw_number_t irq = 0; irq < nirqs; irq++) {
                struct irq_data *data;
-               int virq;
+               unsigned int virq;
 
                virq = irq_linear_revmap(armada_370_xp_mpic_domain, irq);
                if (virq == 0)