On Wed, Oct 10, 2012 at 10:42:52PM +0100, Lespiau, Damien wrote: > On Wed, Oct 10, 2012 at 10:09 PM, Paulo Zanoni <przanoni at gmail.com> wrote: > > From: Paulo Zanoni <paulo.r.zanoni at intel.com> > > > > Introduced in commit 87f8020ec9e3069597746040a4e8655189bc0c1a: > > drm/i915: implement WaDisableEarlyCull for VLV and IVB > > > > Notice that the original patch sent to the mailing list did not > > include the Haswell chunk, it was added later. > > > > The bit set by the commit does not exist on Haswell machines (at least > > that's what the documentation says). Also, the commit gives me a GPU > > hang every time we're loading the driver. So let's revert the Haswell > > chunk, making the patch do only what its title actually says. > > > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com> > > Indeed the bit definition has disappeared in the documentation, but > the wa is listed in the wa database. A hang is very handy to decide > who is right :) > > Reviewed-by: Damien Lespiau <damien.lespiau at intel.com> Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch