Re: [Mesa-dev] [PATCH v2 0/6] Flag new aux state on aux state updates

2017-09-18 Thread Iago Toral
Hi Jason, thanks for the quick review! I agree with all your points and will update the series accordingly before pushing. Iago On Fri, 2017-09-15 at 17:11 -0700, Jason Ekstrand wrote: > I don't think we need patches 2 or 6 and the commit message from 2 > should probably go on 5.  I left a comment

Re: [Mesa-dev] [PATCH v2 0/6] Flag new aux state on aux state updates

2017-09-15 Thread Jason Ekstrand
I don't think we need patches 2 or 6 and the commit message from 2 should probably go on 5. I left a comment on patch 4. Other than that, 1, 3, 4, and 5 are Reviewed-by: Jason Ekstrand On Fri, Sep 15, 2017 at 3:02 AM, Iago Toral Quiroga wrote: >

[Mesa-dev] [PATCH v2 0/6] Flag new aux state on aux state updates

2017-09-15 Thread Iago Toral Quiroga
Jason, Ken: I think this series addresses all your feedback, let me know if you think I missed anything. Maybe you also prefer to squash some of the patches, let me know if that is the case. Iago Toral Quiroga (6): i965: rename BRW_NEW_FAST_CLEAR_COLOR to BRW_NEW_AUX_STATE i965: emit