On Tue, Jun 2, 2015 at 2:05 AM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > On Wed, May 27, 2015 at 5:26 AM, Gregory Fong <gregory.0xf0@xxxxxxxxx> 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 devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html