Re: [PATCH 1/5] clocksource: sun5i: Switch to request_irq

[Maxime Ripard]

On Sun, Jan 18, 2015 at 07:40:56PM +0100, Daniel Lezcano wrote:
> On 01/11/2015 12:40 PM, Maxime Ripard wrote:
> >The current code uses setup_irq, while it could perfectly use the much simpler
> >request_irq. Switch to that.
> >
> >Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx>
> >---
> >  drivers/clocksource/timer-sun5i.c | 10 ++--------
> >  1 file changed, 2 insertions(+), 8 deletions(-)
> >
> >diff --git a/drivers/clocksource/timer-sun5i.c b/drivers/clocksource/timer-sun5i.c
> >index 02268448dc85..738853873a3b 100644
> >--- a/drivers/clocksource/timer-sun5i.c
> >+++ b/drivers/clocksource/timer-sun5i.c
> >@@ -130,13 +130,6 @@ static irqreturn_t sun5i_timer_interrupt(int irq, void *dev_id)
> >  	return IRQ_HANDLED;
> >  }
> >
> >-static struct irqaction sun5i_timer_irq = {
> >-	.name = "sun5i_timer0",
> >-	.flags = IRQF_TIMER | IRQF_IRQPOLL,
> >-	.handler = sun5i_timer_interrupt,
> >-	.dev_id = &sun5i_clockevent,
> >-};
> >-
> >  static u64 sun5i_timer_sched_read(void)
> >  {
> >  	return ~readl(timer_base + TIMER_CNTVAL_LO_REG(1));
> >@@ -178,7 +171,8 @@ static void __init sun5i_timer_init(struct device_node *node)
> >
> >  	ticks_per_jiffy = DIV_ROUND_UP(rate, HZ);
> >
> >-	ret = setup_irq(irq, &sun5i_timer_irq);
> >+	ret = request_irq(irq, sun5i_timer_interrupt, IRQF_TIMER | IRQF_IRQPOLL,
> >+			  "sun5i_timer", &sun5i_clockevent);
> 
> Is the timer name change done in purpose ?

Hmm, no, it's a mistake. I'll resend a v2.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Attachment: signature.asc
Description: Digital signature