On Thu, Apr 25, 2013 at 06:47:58PM +1000, Benjamin Herrenschmidt wrote: >On Thu, 2013-04-25 at 16:13 +0800, Gavin Shan wrote: >> It should be "chip_data" (not "irq_data"). Hopefully, you haven't >> get time to see the reply. Otherwise, it would a bit confused ;-) > >Doesn't ics-opal already use chip_data ? >
Yeah, Ben. that have been used now. So we can't use it for other purposes :-) >I was thinking just duplicating the irq_chip (including chip_data) so it >can be used by ics-opal just fine for all calls, just then overriding >the eoi callback and using container_of to get to the PHB. > >Any reason that wouldn't work ? > It should work and I had the code (with your idea implemented) and verified that on simulator. I'll send next version (together with the changes on f/w) for review after it works correctly on real hardware box. (I hope it can be done as early as possible to catch 3.10 merge window). Thanks, Gavin _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
