Linus Torvalds wrote: > Quite frankly, I'd actually prefer to just reinstate "dev" to the irq > registration instead. > > Why? Because that field is how we are able to track multiple interrupt > registrations that share an IRQ. Now, the "request_irq()" logic has a > special rule that actually tests that NULL is a valid cookie if the > IRQF_SHARED bit isn't set, but isn't it a nice thing to double-check > regardless? [...] > This, in contrast, *really* sucks as a cookie. In fact, it's useless. If > there are multiple tp3780i instances on the same irq, they will always > have the same cookie. AFAICS that will never happen for mwave hardware. Nonetheless, it's a good point that NULL fails for disambiguation, so I created the attached, which should do what you want. I'll push it after the compiler gives it a green light (unless it needs further revisions) (note, for mwave I couldn't use pSettings, since that might fail the ambiguity test) Jeff