* [PATCH] gpio: pl061: drop references to "virtual" IRQ
@ 2013-10-11 17:42 Linus Walleij
2013-10-12 18:59 ` Baruch Siach
0 siblings, 1 reply; 2+ messages in thread
From: Linus Walleij @ 2013-10-11 17:42 UTC (permalink / raw)
To: linux-gpio
Cc: Alexandre Courbot, Linus Walleij, Haojian Zhuang, Rob Herring,
Baruch Siach
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.
Cc: Haojian Zhuang <haojian.zhuang@linaro.org>
Cc: Rob Herring <rob.herring@calxeda.com>
Cc: Baruch Siach <baruch@tkos.co.il>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
drivers/gpio/gpio-pl061.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpio/gpio-pl061.c b/drivers/gpio/gpio-pl061.c
index 4274e2e..f22f7f3 100644
--- a/drivers/gpio/gpio-pl061.c
+++ b/drivers/gpio/gpio-pl061.c
@@ -238,15 +238,15 @@ static struct irq_chip pl061_irqchip = {
.irq_set_type = pl061_irq_type,
};
-static int pl061_irq_map(struct irq_domain *d, unsigned int virq,
- irq_hw_number_t hw)
+static int pl061_irq_map(struct irq_domain *d, unsigned int irq,
+ irq_hw_number_t hwirq)
{
struct pl061_gpio *chip = d->host_data;
- irq_set_chip_and_handler_name(virq, &pl061_irqchip, handle_simple_irq,
+ irq_set_chip_and_handler_name(irq, &pl061_irqchip, handle_simple_irq,
"pl061");
- irq_set_chip_data(virq, chip);
- irq_set_irq_type(virq, IRQ_TYPE_NONE);
+ irq_set_chip_data(irq, chip);
+ irq_set_irq_type(irq, IRQ_TYPE_NONE);
return 0;
}
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] gpio: pl061: drop references to "virtual" IRQ
2013-10-11 17:42 [PATCH] gpio: pl061: drop references to "virtual" IRQ Linus Walleij
@ 2013-10-12 18:59 ` Baruch Siach
0 siblings, 0 replies; 2+ messages in thread
From: Baruch Siach @ 2013-10-12 18:59 UTC (permalink / raw)
To: Linus Walleij; +Cc: linux-gpio, Alexandre Courbot, Haojian Zhuang, Rob Herring
Hi Linus,
On Fri, Oct 11, 2013 at 07:42:07PM +0200, Linus Walleij wrote:
> 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.
>
> Cc: Haojian Zhuang <haojian.zhuang@linaro.org>
> Cc: Rob Herring <rob.herring@calxeda.com>
> Cc: Baruch Siach <baruch@tkos.co.il>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Acked-by: Baruch Siach <baruch@tkos.co.il>
baruch
> ---
> drivers/gpio/gpio-pl061.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpio/gpio-pl061.c b/drivers/gpio/gpio-pl061.c
> index 4274e2e..f22f7f3 100644
> --- a/drivers/gpio/gpio-pl061.c
> +++ b/drivers/gpio/gpio-pl061.c
> @@ -238,15 +238,15 @@ static struct irq_chip pl061_irqchip = {
> .irq_set_type = pl061_irq_type,
> };
>
> -static int pl061_irq_map(struct irq_domain *d, unsigned int virq,
> - irq_hw_number_t hw)
> +static int pl061_irq_map(struct irq_domain *d, unsigned int irq,
> + irq_hw_number_t hwirq)
> {
> struct pl061_gpio *chip = d->host_data;
>
> - irq_set_chip_and_handler_name(virq, &pl061_irqchip, handle_simple_irq,
> + irq_set_chip_and_handler_name(irq, &pl061_irqchip, handle_simple_irq,
> "pl061");
> - irq_set_chip_data(virq, chip);
> - irq_set_irq_type(virq, IRQ_TYPE_NONE);
> + irq_set_chip_data(irq, chip);
> + irq_set_irq_type(irq, IRQ_TYPE_NONE);
>
> return 0;
> }
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-10-12 19:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-11 17:42 [PATCH] gpio: pl061: drop references to "virtual" IRQ Linus Walleij
2013-10-12 18:59 ` Baruch Siach
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).