Commit 1c56cf6b authored by Laurent Pinchart's avatar Laurent Pinchart
Browse files

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: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
parent bfa76bb1

drivers/clocksource/sh_tmu.c

+8 −14
Original line number Diff line number Diff line
@@ -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)) {

CRA Git | Maintained and supported by SUSTech CRA and CCSE