| From 3ced2ffe9af02a62cf7f6c0482177de18fd6beed Mon Sep 17 00:00:00 2001 |
| From: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> |
| Date: Mon, 17 Feb 2014 11:27:49 +0100 |
| Subject: clocksource: sh_tmu: Use request_irq() instead of setup_irq() |
| |
| 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> |
| (cherry picked from commit 1c56cf6b048e1e1bbe08faf38b5592b373905ac5) |
| Signed-off-by: Simon Horman <horms+renesas@verge.net.au> |
| --- |
| drivers/clocksource/sh_tmu.c | 22 ++++++++-------------- |
| 1 file changed, 8 insertions(+), 14 deletions(-) |
| |
| diff --git a/drivers/clocksource/sh_tmu.c b/drivers/clocksource/sh_tmu.c |
| index ecd7b60bfdfa..8613cc90bb74 100644 |
| --- a/drivers/clocksource/sh_tmu.c |
| +++ b/drivers/clocksource/sh_tmu.c |
| @@ -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)) { |
| -- |
| 2.1.2 |
| |