Re: sja1000 interrupt problem

[Oliver Hartkopp <socketcan@hartkopp.net>]

On 13.11.2013 10:52, Wolfgang Grandegger wrote:

> 
> In Linux-CAN we have something similar:
> 
> http://lxr.linux.no/#linux/drivers/net/can/sja1000/ems_pcmcia.c#L90
> 

Indeed.

I think reworking the sja1000.c driver (as suggested by Kurt) won't make it.
It only touches the generic irq handling.

The peak_pci driver creates (depending on the number of channels) e.g.
two/four AFAICS pretty *independent* sja1000 netdevices.

Currently at the end of the generic sja1000 interrupt handling the according
irq bit in the PITA is cleared. This is not necessarily at the end of the
interrupt chain.

What we would need to set up a similar handling as we have in EMS PCMCIA or
the EMS PCI (referenced by Pavel) is a "group of sja1000 netdevices" which is
placed on a single PEAK PCI adapter.

Indeed there is already a chain of sja1000 netdevices:

http://lxr.linux.no/#linux+v3.12/drivers/net/can/sja1000/peak_pci.c#L645

BUT this is only used to clean up all channels when the PCI device is removed
or some errors occur at creation time.

IMO the existing chain of netdevices is not only needed for the device removal
but also for the interrupt handling.

When ever the interrupt for the PCI adapter occurs all channels have to be
handled (in a private peak_pci_interrupt() function) and finally the PITA has
to be cleared there too.

That change won't make use of the possibility to clear single IRQ bits in the
PITA anymore. And the PITA has to be checked first (e.g. to check if we have a
new interrupt somewhere later in the interrupt chain) to skip the irq handling
when it's obsolete.

Any thoughts?

Regards,
Oliver