On Fri, 27 Jul 2012 18:12:26 +0800, Zhigang Gong <zhigang.gong at linux.intel.com> wrote: > On Fri, Jul 27, 2012 at 10:10:32AM +0100, Chris Wilson wrote: > > On Fri, 27 Jul 2012 17:06:21 +0800, zhigang.gong at linux.intel.com wrote: > > > - if (attachment == DRI2BufferFrontLeft) > > > + if (attachment == DRI2BufferFrontLeft && is_glamor_pixmap) > > is_glamor_pixmap is only true for FrontLeft, mind resending with this > > simplification? > > Sure, the simplified version is as below. Thanks. Thanks for the patch, pushed -Chris -- Chris Wilson, Intel Open Source Technology Centre
