Yegor Yefremov wrote: > I'm trying to get sja1000_platfrom.c on a arm based device. The SJA1000 > shares interrupt line with some other peripherals, so I need add IRQF_SHARED > to the priv->irq_flags. If I define this in my platform definition file, it > will be overwritten by priv->irq_flags = res_irq->flags & IRQF_TRIGGER_MASK, > so I added IRQF_SHARED to the AND mask. What do you think about it? > > Regards, > Yeogr > > --------------------------------------------------------------------------------------------------- > SJA1000: allow shared interrupt definition > > Signed-off-by: Yegor Yefremov <[email protected]> > > diff --git a/drivers/net/can/sja1000/sja1000_platform.c > b/drivers/net/can/sja1000/sja1000_platform.c > index 628374c..bec0d3d 100644 > --- a/drivers/net/can/sja1000/sja1000_platform.c > +++ b/drivers/net/can/sja1000/sja1000_platform.c > @@ -90,7 +90,7 @@ static int sp_probe(struct platform_device *pdev) > priv = netdev_priv(dev); > > dev->irq = res_irq->start; > - priv->irq_flags = res_irq->flags & IRQF_TRIGGER_MASK; > + priv->irq_flags = res_irq->flags & (IRQF_TRIGGER_MASK | IRQF_SHARED); > priv->reg_base = addr; > priv->read_reg = sp_read_reg; > priv->write_reg = sp_write_reg
Fine for me. Wolfgang. _______________________________________________ Socketcan-core mailing list [email protected] https://lists.berlios.de/mailman/listinfo/socketcan-core
