On Thu, Mar 02, 2017 at 11:03:44AM -0800, Jason Ekstrand wrote: > On Mar 2, 2017 10:57 AM, "Nanley Chery" <nanleych...@gmail.com> wrote: > > On Wed, Mar 01, 2017 at 04:46:42PM -0800, Jason Ekstrand wrote: > > I didn't re-read every patch, just the ones I made comments on. Assuming > > only the commented patches made real changes, the series is > > > > Reviewed-by: Jason Ekstrand <jason@jlekstrand> > > > > I really like the direction things are going. > > > > Thanks! > > I had to update one patch due to rebasing conflicts that occurred today. > Please see: > [PATCH v3] anv/descriptor_set: Store aux usage of sampled image descriptors > > > I might have split that into two patches with the refactoring separate from > the aux usage change bit they're fine squashed together. >
That's a good point. I will try to keep that in mind next time. > Also, updating the patch, "anv/image: Add anv_layout_to_aux_usage()", > caused the patches that modify transition_depth_buffer() to change. The > > > Yup. I saw that. Looks fine to me. > > end result looks about the same to me. Does your Rb still apply? > > > Yup. Land it. > With the fix discussed on IRC, this series has been pushed. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
