Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1449094 > unrolled thread
| Started by | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| First post | 2016-07-24 10:00 +0200 |
| Last post | 2016-07-25 06:20 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
linux-next: build failure after merge of the sh tree Stephen Rothwell <sfr@canb.auug.org.au> - 2016-07-24 10:00 +0200
Re: linux-next: build failure after merge of the sh tree Rich Felker <dalias@libc.org> - 2016-07-25 06:00 +0200
Re: linux-next: build failure after merge of the sh tree Stephen Rothwell <sfr@canb.auug.org.au> - 2016-07-25 06:20 +0200
| From | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| Date | 2016-07-24 10:00 +0200 |
| Subject | linux-next: build failure after merge of the sh tree |
| Message-ID | <rYmum-5dy-5@gated-at.bofh.it> |
Hi Rich,
After merging the sh tree, today's linux-next build (powerpc allyesconfig)
failed like this:
drivers/clocksource/jcore-pit.c:145:20: error: conflicting types for 'timer_interrupt'
static irqreturn_t timer_interrupt(int irq, void *dev_id)
^
In file included from arch/powerpc/include/asm/irqflags.h:11:0,
from include/linux/irqflags.h:15,
from include/linux/spinlock.h:53,
from include/linux/mmzone.h:7,
from include/linux/gfp.h:5,
from include/linux/slab.h:14,
from drivers/clocksource/jcore-pit.c:12:
arch/powerpc/include/asm/hw_irq.h:36:13: note: previous declaration of 'timer_interrupt' was here
extern void timer_interrupt(struct pt_regs *);
^
Caused by commit
e0aa0655c60b ("clocksource: add J-Core timer/clocksource driver")
I added this fix patch:
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Sun, 24 Jul 2016 17:42:19 +1000
Subject: [PATCH] clocksource: make J-Core timer_interrupt more unique
Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
drivers/clocksource/jcore-pit.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/clocksource/jcore-pit.c b/drivers/clocksource/jcore-pit.c
index b9d693bcb41e..373b9f954a5c 100644
--- a/drivers/clocksource/jcore-pit.c
+++ b/drivers/clocksource/jcore-pit.c
@@ -142,7 +142,7 @@ static int jcore_pit_cpu_notify(struct notifier_block *self,
return NOTIFY_OK;
}
-static irqreturn_t timer_interrupt(int irq, void *dev_id)
+static irqreturn_t jcore_timer_interrupt(int irq, void *dev_id)
{
struct jcore_pit *pit = this_cpu_ptr(dev_id);
@@ -224,7 +224,7 @@ static void __init jcore_pit_init(struct device_node *node)
goto out;
}
- err = request_irq(pit_irq, timer_interrupt,
+ err = request_irq(pit_irq, jcore_timer_interrupt,
IRQF_TIMER | IRQF_PERCPU, "jcore_pit", pit_percpu);
if (err) {
pr_err("pit irq request failed: %d\n", err);
--
2.8.1
--
Cheers,
Stephen Rothwell
[toc] | [next] | [standalone]
| From | Rich Felker <dalias@libc.org> |
|---|---|
| Date | 2016-07-25 06:00 +0200 |
| Message-ID | <rYFdD-80t-1@gated-at.bofh.it> |
| In reply to | #1449094 |
On Sun, Jul 24, 2016 at 05:55:01PM +1000, Stephen Rothwell wrote:
> Hi Rich,
>
> After merging the sh tree, today's linux-next build (powerpc allyesconfig)
> failed like this:
>
> drivers/clocksource/jcore-pit.c:145:20: error: conflicting types for 'timer_interrupt'
> static irqreturn_t timer_interrupt(int irq, void *dev_id)
> ^
> In file included from arch/powerpc/include/asm/irqflags.h:11:0,
> from include/linux/irqflags.h:15,
> from include/linux/spinlock.h:53,
> from include/linux/mmzone.h:7,
> from include/linux/gfp.h:5,
> from include/linux/slab.h:14,
> from drivers/clocksource/jcore-pit.c:12:
> arch/powerpc/include/asm/hw_irq.h:36:13: note: previous declaration of 'timer_interrupt' was here
> extern void timer_interrupt(struct pt_regs *);
> ^
>
> Caused by commit
>
> e0aa0655c60b ("clocksource: add J-Core timer/clocksource driver")
>
> I added this fix patch:
>
> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Date: Sun, 24 Jul 2016 17:42:19 +1000
> Subject: [PATCH] clocksource: make J-Core timer_interrupt more unique
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
> drivers/clocksource/jcore-pit.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/clocksource/jcore-pit.c b/drivers/clocksource/jcore-pit.c
> index b9d693bcb41e..373b9f954a5c 100644
> --- a/drivers/clocksource/jcore-pit.c
> +++ b/drivers/clocksource/jcore-pit.c
> @@ -142,7 +142,7 @@ static int jcore_pit_cpu_notify(struct notifier_block *self,
> return NOTIFY_OK;
> }
>
> -static irqreturn_t timer_interrupt(int irq, void *dev_id)
> +static irqreturn_t jcore_timer_interrupt(int irq, void *dev_id)
> {
> struct jcore_pit *pit = this_cpu_ptr(dev_id);
>
> @@ -224,7 +224,7 @@ static void __init jcore_pit_init(struct device_node *node)
> goto out;
> }
>
> - err = request_irq(pit_irq, timer_interrupt,
> + err = request_irq(pit_irq, jcore_timer_interrupt,
> IRQF_TIMER | IRQF_PERCPU, "jcore_pit", pit_percpu);
> if (err) {
> pr_err("pit irq request failed: %d\n", err);
> --
If it's acceptable I'll rebase this change into sh for-next and the
patch I send to the subsystem maintainer.
Rich
[toc] | [prev] | [next] | [standalone]
| From | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| Date | 2016-07-25 06:20 +0200 |
| Message-ID | <rYFwZ-8mq-5@gated-at.bofh.it> |
| In reply to | #1449269 |
Hi Rich, On Sun, 24 Jul 2016 23:57:30 -0400 Rich Felker <dalias@libc.org> wrote: > > If it's acceptable I'll rebase this change into sh for-next and the > patch I send to the subsystem maintainer. Sure. -- Cheers, Stephen Rothwell
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web