gpio: tc3589x: drop references to "virtual" IRQ
authorLinus Walleij <linus.walleij@linaro.org>
Fri, 11 Oct 2013 17:06:12 +0000 (19:06 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Fri, 11 Oct 2013 17:06:12 +0000 (19:06 +0200)
Rename the argument "virq" to just "irq", this IRQ isn't any
more "virtual" than any other Linux IRQ number, we use "hwirq"
for the actual hw-numbers, "virq" is just bogus.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-tc3589x.c

index 4a5de273c230d159883c4430f573416bcf0d4ea4..ddb5fefaa715096823fad4858a7d6bb2c5f7bf5d 100644 (file)
@@ -96,27 +96,27 @@ static int tc3589x_gpio_direction_input(struct gpio_chip *chip,
 }
 
 /**
- * tc3589x_gpio_irq_get_virq(): Map an interrupt on a chip to a virtual IRQ
+ * tc3589x_gpio_irq_get_irq(): Map a hardware IRQ on a chip to a Linux IRQ
  *
  * @tc3589x_gpio: tc3589x_gpio_irq controller to operate on.
- * @irq: index of the interrupt requested in the chip IRQs
+ * @irq: index of the hardware interrupt requested in the chip IRQs
  *
  * Useful for drivers to request their own IRQs.
  */
-static int tc3589x_gpio_irq_get_virq(struct tc3589x_gpio *tc3589x_gpio,
-                                    int irq)
+static int tc3589x_gpio_irq_get_irq(struct tc3589x_gpio *tc3589x_gpio,
+                                    int hwirq)
 {
        if (!tc3589x_gpio)
                return -EINVAL;
 
-       return irq_create_mapping(tc3589x_gpio->domain, irq);
+       return irq_create_mapping(tc3589x_gpio->domain, hwirq);
 }
 
 static int tc3589x_gpio_to_irq(struct gpio_chip *chip, unsigned offset)
 {
        struct tc3589x_gpio *tc3589x_gpio = to_tc3589x_gpio(chip);
 
-       return tc3589x_gpio_irq_get_virq(tc3589x_gpio, offset);
+       return tc3589x_gpio_irq_get_irq(tc3589x_gpio, offset);
 }
 
 static struct gpio_chip template_chip = {
@@ -242,9 +242,9 @@ static irqreturn_t tc3589x_gpio_irq(int irq, void *dev)
                while (stat) {
                        int bit = __ffs(stat);
                        int line = i * 8 + bit;
-                       int virq = tc3589x_gpio_irq_get_virq(tc3589x_gpio, line);
+                       int irq = tc3589x_gpio_irq_get_irq(tc3589x_gpio, line);
 
-                       handle_nested_irq(virq);
+                       handle_nested_irq(irq);
                        stat &= ~(1 << bit);
                }
 
@@ -254,31 +254,31 @@ static irqreturn_t tc3589x_gpio_irq(int irq, void *dev)
        return IRQ_HANDLED;
 }
 
-static int tc3589x_gpio_irq_map(struct irq_domain *d, unsigned int virq,
+static int tc3589x_gpio_irq_map(struct irq_domain *d, unsigned int irq,
                                irq_hw_number_t hwirq)
 {
        struct tc3589x *tc3589x_gpio = d->host_data;
 
-       irq_set_chip_data(virq, tc3589x_gpio);
-       irq_set_chip_and_handler(virq, &tc3589x_gpio_irq_chip,
+       irq_set_chip_data(irq, tc3589x_gpio);
+       irq_set_chip_and_handler(irq, &tc3589x_gpio_irq_chip,
                                handle_simple_irq);
-       irq_set_nested_thread(virq, 1);
+       irq_set_nested_thread(irq, 1);
 #ifdef CONFIG_ARM
-       set_irq_flags(virq, IRQF_VALID);
+       set_irq_flags(irq, IRQF_VALID);
 #else
-       irq_set_noprobe(virq);
+       irq_set_noprobe(irq);
 #endif
 
        return 0;
 }
 
-static void tc3589x_gpio_irq_unmap(struct irq_domain *d, unsigned int virq)
+static void tc3589x_gpio_irq_unmap(struct irq_domain *d, unsigned int irq)
 {
 #ifdef CONFIG_ARM
-       set_irq_flags(virq, 0);
+       set_irq_flags(irq, 0);
 #endif
-       irq_set_chip_and_handler(virq, NULL, NULL);
-       irq_set_chip_data(virq, NULL);
+       irq_set_chip_and_handler(irq, NULL, NULL);
+       irq_set_chip_data(irq, NULL);
 }
 
 static struct irq_domain_ops tc3589x_irq_ops = {