On Sun, Aug 08, 2010 at 02:24:11PM +0200, Daniel Vetter wrote: > In conclusion I think we need an if (IS_SNB(dev)) that sets dword 0, bit > 22 to 1 and ensures that dword 2, bit 0 is zero. For the rest of of the > IS_I965G branch we might as well write 0 instead of pitch and tiling_mode.
I've played around a bit and it looks like everyone is wrong (minus the current code;). Tested on my ilk (thinkpad t410) ymmv. Not setting pitch or tiling_mode is a bad idea and results in garbage on the screen. So hw does indeed read this values (despite what the docs seem to claim). But at least for my ilk here, published docs are correct: Or'ing it in with the pitch results in garbage, too. The original patch is therefore broken, 'cause it moves around the tiling_mode bit (with mentioning this in the changelog). I suspect something changed with sandybridge and whoever wrote the docs made a mess out of it. -Daniel -- Daniel Vetter Mail: [email protected] Mobile: +41 (0)79 365 57 48 _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
