* [PATCH -next] irqchip: renesas-h8s: Replace ctrl_outw/ctrl_inw with writew/readw
@ 2015-12-16 4:30 Guenter Roeck
2015-12-17 16:31 ` Yoshinori Sato
0 siblings, 1 reply; 2+ messages in thread
From: Guenter Roeck @ 2015-12-16 4:30 UTC (permalink / raw)
To: Yoshinori Sato
Cc: Thomas Gleixner, Jason Cooper, Marc Zyngier, uclinux-h8-devel,
linux-kernel, Guenter Roeck, Daniel Lezcano
Commit 13ae42a3b1c1 ("h8300: Rename ctlr_out/in[bwl] to
raw_read/write[bwl]") changed the function names, but not all callers,
resulting in
drivers/irqchip/irq-renesas-h8s.c: In function ‘h8s_disable_irq’:
drivers/irqchip/irq-renesas-h8s.c:43:9: error:
implicit declaration of function ‘ctrl_inw’
drivers/irqchip/irq-renesas-h8s.c:44:2: error:
implicit declaration of function ‘ctrl_outw’
Fixes: 13ae42a3b1c1 ("h8300: Rename ctlr_out/in[bwl] to raw_read/write[bwl]")
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
drivers/irqchip/irq-renesas-h8s.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/irqchip/irq-renesas-h8s.c b/drivers/irqchip/irq-renesas-h8s.c
index 8098ead1eb22..af8c6c61c824 100644
--- a/drivers/irqchip/irq-renesas-h8s.c
+++ b/drivers/irqchip/irq-renesas-h8s.c
@@ -40,8 +40,8 @@ static void h8s_disable_irq(struct irq_data *data)
addr = IPRA + ((ipr_table[irq - 16] & 0xf0) >> 3);
pos = (ipr_table[irq - 16] & 0x0f) * 4;
pri = ~(0x000f << pos);
- pri &= ctrl_inw(addr);
- ctrl_outw(pri, addr);
+ pri &= readw(addr);
+ writew(pri, addr);
}
static void h8s_enable_irq(struct irq_data *data)
@@ -54,9 +54,9 @@ static void h8s_enable_irq(struct irq_data *data)
addr = IPRA + ((ipr_table[irq - 16] & 0xf0) >> 3);
pos = (ipr_table[irq - 16] & 0x0f) * 4;
pri = ~(0x000f << pos);
- pri &= ctrl_inw(addr);
+ pri &= readw(addr);
pri |= 1 << pos;
- ctrl_outw(pri, addr);
+ writew(pri, addr);
}
struct irq_chip h8s_irq_chip = {
@@ -90,7 +90,7 @@ static int __init h8s_intc_of_init(struct device_node *intc,
/* All interrupt priority is 0 (disable) */
/* IPRA to IPRK */
for (n = 0; n <= 'k' - 'a'; n++)
- ctrl_outw(0x0000, IPRA + (n * 2));
+ writew(0x0000, IPRA + (n * 2));
domain = irq_domain_add_linear(intc, NR_IRQS, &irq_ops, NULL);
BUG_ON(!domain);
--
2.1.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH -next] irqchip: renesas-h8s: Replace ctrl_outw/ctrl_inw with writew/readw
2015-12-16 4:30 [PATCH -next] irqchip: renesas-h8s: Replace ctrl_outw/ctrl_inw with writew/readw Guenter Roeck
@ 2015-12-17 16:31 ` Yoshinori Sato
0 siblings, 0 replies; 2+ messages in thread
From: Yoshinori Sato @ 2015-12-17 16:31 UTC (permalink / raw)
To: Guenter Roeck
Cc: Thomas Gleixner, Jason Cooper, Marc Zyngier, uclinux-h8-devel,
linux-kernel, Daniel Lezcano
On Wed, 16 Dec 2015 13:30:37 +0900,
Guenter Roeck wrote:
>
> Commit 13ae42a3b1c1 ("h8300: Rename ctlr_out/in[bwl] to
> raw_read/write[bwl]") changed the function names, but not all callers,
> resulting in
>
> drivers/irqchip/irq-renesas-h8s.c: In function ‘h8s_disable_irq’:
> drivers/irqchip/irq-renesas-h8s.c:43:9: error:
> implicit declaration of function ‘ctrl_inw’
> drivers/irqchip/irq-renesas-h8s.c:44:2: error:
> implicit declaration of function ‘ctrl_outw’
>
> Fixes: 13ae42a3b1c1 ("h8300: Rename ctlr_out/in[bwl] to raw_read/write[bwl]")
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Applied.
Thanks.
> ---
> drivers/irqchip/irq-renesas-h8s.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/irqchip/irq-renesas-h8s.c b/drivers/irqchip/irq-renesas-h8s.c
> index 8098ead1eb22..af8c6c61c824 100644
> --- a/drivers/irqchip/irq-renesas-h8s.c
> +++ b/drivers/irqchip/irq-renesas-h8s.c
> @@ -40,8 +40,8 @@ static void h8s_disable_irq(struct irq_data *data)
> addr = IPRA + ((ipr_table[irq - 16] & 0xf0) >> 3);
> pos = (ipr_table[irq - 16] & 0x0f) * 4;
> pri = ~(0x000f << pos);
> - pri &= ctrl_inw(addr);
> - ctrl_outw(pri, addr);
> + pri &= readw(addr);
> + writew(pri, addr);
> }
>
> static void h8s_enable_irq(struct irq_data *data)
> @@ -54,9 +54,9 @@ static void h8s_enable_irq(struct irq_data *data)
> addr = IPRA + ((ipr_table[irq - 16] & 0xf0) >> 3);
> pos = (ipr_table[irq - 16] & 0x0f) * 4;
> pri = ~(0x000f << pos);
> - pri &= ctrl_inw(addr);
> + pri &= readw(addr);
> pri |= 1 << pos;
> - ctrl_outw(pri, addr);
> + writew(pri, addr);
> }
>
> struct irq_chip h8s_irq_chip = {
> @@ -90,7 +90,7 @@ static int __init h8s_intc_of_init(struct device_node *intc,
> /* All interrupt priority is 0 (disable) */
> /* IPRA to IPRK */
> for (n = 0; n <= 'k' - 'a'; n++)
> - ctrl_outw(0x0000, IPRA + (n * 2));
> + writew(0x0000, IPRA + (n * 2));
>
> domain = irq_domain_add_linear(intc, NR_IRQS, &irq_ops, NULL);
> BUG_ON(!domain);
> --
> 2.1.4
>
--
Yoshinori Sato
<ysato@users.sourceforge.jp>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-12-17 16:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-16 4:30 [PATCH -next] irqchip: renesas-h8s: Replace ctrl_outw/ctrl_inw with writew/readw Guenter Roeck
2015-12-17 16:31 ` Yoshinori Sato
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).