[Intel-gfx] [PATCH 4/4 v2] drm/i915: Never read FORCEWAKE

Daniel Vetter daniel at ffwll.ch
Mon Sep 3 10:30:31 CEST 2012


On Sun, Sep 02, 2012 at 09:41:44AM +0100, Chris Wilson wrote:
> On Sat,  1 Sep 2012 22:59:50 -0700, Ben Widawsky <ben at bwidawsk.net> wrote:
> > The same designer from the previous patch has told us to never read
> > FORCEWAKE. We only do this for the POSTING_READ(), so simply change that
> > to something within the same cacheline (for no reason in particular
> > other than it sounds nice). In the _mt case we can leverage
> > the gtfifodbg check for the POSTING_READ.
> 
> The first two are Reviewed-by: Chris Wilson <chris at chris-wilson.oc.uk>
> The real meat of the change in the second two look good, so
> Acked-by: Chris Wilson <chris at chris-wilson.co.uk>
All slurped into dinq (with some frobbing due to the vlv patches), thanks.
-Daniel
-- 
Daniel Vetter
Mail: daniel at ffwll.ch
Mobile: +41 (0)79 365 57 48



More information about the Intel-gfx mailing list