On Wed, Oct 22, 2014 at 05:29:24PM -0700, Matt Roper wrote: > On Tue, Oct 07, 2014 at 08:43:46AM +0000, Jindal, Sonika wrote: > > Hi, > > > > Did anybody get a chance to look at this patch? > > > > Thanks, > > Sonika > > Looks like we waited a bit too long and the codebase has evolved, so I > needed to make some tweaks to your patches to get them to apply cleanly > on the latest di-nightly. However the overall changes here look good to > me, and we don't seem to be missing any details from the bspec, so > > Reviewed-by: Matt Roper <matthew.d.roper@xxxxxxxxx> > > After tweaking your patches to apply against the latest di-nightly, your > i-g-t test runs properly for me on IVB, as well as another simple test I > wrote myself, so you can also put > > Tested-by: Matt Roper <matthew.d.roper@xxxxxxxxx> > > Let me know if you want the rebased copies of the patches I used for > testing. Lazy maintainer here. That would be awesome ;-) Since you've frobbed the patches a bit pls also put your sob onto them. Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx