On Thu, May 24, 2012 at 09:49:15PM +0300, Ville Syrj?l? wrote: > On Thu, May 24, 2012 at 11:31:32AM -0700, Jesse Barnes wrote: > > On Thu, 24 May 2012 21:08:58 +0300 > > ville.syrjala at linux.intel.com wrote: > > > > > From: Ville Syrj?l? <ville.syrjala at linux.intel.com> > > > > > > Take fb->offset[0] into account when calculating the linear and tile x/y > > > offsets. > > > > > > For non-tiled surfaces fb->offset[0] is simply added to the linear > > > byte offset. > > > > > > For tiled surfaces treat fb->offsets[0] as a byte offset into the > > > linearized view of the surface. So we end up converting fb->offsets[0] > > > into additional x and y offsets. > > > > Do you have code using a non-zero offsets[0]? At least for current > > code that would indicate some kind of problem... though hopefully we'll > > be adding planar support back again sometime soon. > > I did have some test app that used offsets[] at some point, but tbh I > didn't excercise these changes with it. > > I have a sort of semi working skeleton of a test app which I just modify > for various use cases as need arises. I really should try to clean it up > a bit and generalize it so that it wouldn't need constant code changes > to test different scenarios. Yeah, I want these little test apps as testcases in i-g-t. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48
