On Tue, Jun 2, 2015 at 2:05 AM, Linus Walleij <[email protected]> wrote: > On Wed, May 27, 2015 at 5:26 AM, Gregory Fong <[email protected]> wrote: >> I've now actually attempted to use the gpiolib irqchip code. This >> driver can't directly use gpiochip_irqchip_add() because of the >> multiple gpiochip : one irqchip map. At first, I thought it might be >> possible to simply add a new argument (or break things into a helper >> function) to allow setting the associated IRQ domain, but then I can't >> use the generic map and unmap functions which expect the irq_domain >> host_data member to be struct gpiochip *, which makes no sense in this >> case. That puts me right back to implementing a special version of >> the map and unmap function. > > I see. >
I think I was wrong, it isn't actually necessary to have a single irq_chip as long as all of the IRQs use the same handler. I have a preliminary implementation doing that instead that seems to work fine. Once I've had a chance to verify that it works in all cases, will post that as v3. Cheers, Gregory -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
