On Wed, 5 Sep 2012 23:36:46 +0200, Daniel Vetter <[email protected]> wrote: > On Wed, Sep 5, 2012 at 11:29 PM, Chris Wilson <[email protected]> > wrote: > > On Wed, 5 Sep 2012 21:24:40 +0200, Daniel Vetter <[email protected]> > > wrote: > >> Only enables the interrupt and puts a irq handler into place, doesn't > >> do anything yet. > >> > >> Unfortunately there's no gmbus interrupt support for gen2/3 (safe for > >> pnv, but there the irq is marked as "Test mode"). > > > > The basics look good, but the paranoia says I'd like for the interrupt > > to only be enabled as required. > > I do that ;-) The real interrupt generation is also controlled by > GMBUS4 - as long as that's 0, no interrupt shows up anywhere. And > frobbing GMBUS4 is much easier than adding a bunch of of spinlocks > around frobbing SDE_IIR ...
Indeed, killing it at source is preferrable all round. Are you sure GMBUS4 is always initialised to zero? So this patch looks sound, Reviewed-by: Chris Wilson <[email protected]> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
