clocksource: sh_tmu: Use request_irq() instead of setup_irq()
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Mon, 17 Feb 2014 10:27:49 +0000 (11:27 +0100)
committerLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Wed, 16 Apr 2014 10:03:17 +0000 (12:03 +0200)
The driver claims it needs to register an interrupt handler too early
for request_irq(). This might have been true in the past, but the only
meaningful difference between request_irq() and setup_irq() today is an
additional kzalloc() call in request_irq(). As the driver calls
kmalloc() itself we know that the slab allocator is available, we can
thus switch to request_irq().

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
drivers/clocksource/sh_tmu.c

index ecd7b60bfdfa9d2323053f325b8f9d4447988c9d..8613cc90bb74260d4c48d1fbb319536f901216b5 100644 (file)
@@ -38,7 +38,7 @@
 struct sh_tmu_priv {
        void __iomem *mapbase;
        struct clk *clk;
-       struct irqaction irqaction;
+       int irq;
        struct platform_device *pdev;
        unsigned long rate;
        unsigned long periodic;
@@ -401,10 +401,11 @@ static void sh_tmu_register_clockevent(struct sh_tmu_priv *p,
 
        clockevents_config_and_register(ced, 1, 0x300, 0xffffffff);
 
-       ret = setup_irq(p->irqaction.irq, &p->irqaction);
+       ret = request_irq(p->irq, sh_tmu_interrupt,
+                         IRQF_TIMER | IRQF_IRQPOLL | IRQF_NOBALANCING,
+                         dev_name(&p->pdev->dev), p);
        if (ret) {
-               dev_err(&p->pdev->dev, "failed to request irq %d\n",
-                       p->irqaction.irq);
+               dev_err(&p->pdev->dev, "failed to request irq %d\n", p->irq);
                return;
        }
 }
@@ -425,7 +426,7 @@ static int sh_tmu_setup(struct sh_tmu_priv *p, struct platform_device *pdev)
 {
        struct sh_timer_config *cfg = pdev->dev.platform_data;
        struct resource *res;
-       int irq, ret;
+       int ret;
        ret = -ENXIO;
 
        memset(p, 0, sizeof(*p));
@@ -444,8 +445,8 @@ static int sh_tmu_setup(struct sh_tmu_priv *p, struct platform_device *pdev)
                goto err0;
        }
 
-       irq = platform_get_irq(p->pdev, 0);
-       if (irq < 0) {
+       p->irq = platform_get_irq(p->pdev, 0);
+       if (p->irq < 0) {
                dev_err(&p->pdev->dev, "failed to get irq\n");
                goto err0;
        }
@@ -457,13 +458,6 @@ static int sh_tmu_setup(struct sh_tmu_priv *p, struct platform_device *pdev)
                goto err0;
        }
 
-       /* setup data for setup_irq() (too early for request_irq()) */
-       p->irqaction.name = dev_name(&p->pdev->dev);
-       p->irqaction.handler = sh_tmu_interrupt;
-       p->irqaction.dev_id = p;
-       p->irqaction.irq = irq;
-       p->irqaction.flags = IRQF_TIMER | IRQF_IRQPOLL | IRQF_NOBALANCING;
-
        /* get hold of clock */
        p->clk = clk_get(&p->pdev->dev, "tmu_fck");
        if (IS_ERR(p->clk)) {