On 08/09/2012 02:56 PM, Yegor Yefremov wrote: > Am 09.08.2012 14:46, schrieb Schmitt, Sven (EVM/8): >> Second try: >> >> Signed-off-by: Sven Schmitt >> >> --- linux-3.5/drivers/net/can/sja1000/sja1000_platform.c.orig >> +++ linux-3.5/drivers/net/can/sja1000/sja1000_platform.c >> @@ -109,7 +109,9 @@ static int sp_probe(struct platform_devi >> priv = netdev_priv(dev); >> >> dev->irq = res_irq->start; >> - priv->irq_flags = res_irq->flags & (IRQF_TRIGGER_MASK | >> IRQF_SHARED); >> + priv->irq_flags &= IRQF_TRIGGER_MASK; >> + if (res_irq->flags & IORESOURCE_IRQ_SHAREABLE) >> + priv->irq_flags |= IRQF_SHARED; >> priv->reg_base = addr; >> /* The CAN clock frequency is half the oscillator clock >> frequency */ >> priv->can.clock.freq = pdata->osc_freq / 2; >> >> >> note: patch is compile tested only. >> >> Sven > > O.K. Nice catch. I just picked the wrong header file > include/linux/interrupt.h, it should be include/linux/ioport.h. Now > I'll have to change my board file as well. A quick serach showed no mainline board using shared irq, have I missed one? > You can add my > > Signed-off-by: Yegor Yefremov You mean Acked-by? Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |