Re: [PATCH] parport: Use the PCI IRQ if offered

[Alan Cox]

On Tue, 31 Mar 2009 15:31:56 -0400
Gabor Kovacs <kgabor@xxxxxxxxxxxxxx> wrote:

> Hi Alan,
> 
> just to confirm, IRQ detection works fine on my Lava parallel card. 
> 
> The instrument driver module that registers an irq handler via 
> parport_register_device gets false events when parport irq is shared e.g. with 
> an ethernet controller. Should irq sharing awareness go to our handler 
> function or parport_irq_handler/parport_generic_irq still needs some 
> additional patch? (Or have I just missed this second patch?)

I suspect it may have to go in the handler functions to avoid the risk of
losing any state reading the registers to see an IRQ is pending. The
ideal case would be to keep it in the common code path however.

I've not looked at that so far as a lot of hardware is just fine without
the checks. I guess try it and see with your actual hardware
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/