Re: [Mesa-dev] [PATCH 13/30] i965: Combine render target resolve code

2017-06-06 Thread Jason Ekstrand
On Tue, Jun 6, 2017 at 12:54 PM, Chad Versace wrote: > There's a patch on your branch I didn't see on mesa-dev. >Subject: i965: Be a bit more conservative about certain resolves > It has my r-b. > > I have comments on this patch... > > On Fri 26 May 2017, Jason Ekstrand wrote: > > We have two

Re: [Mesa-dev] [PATCH 13/30] i965: Combine render target resolve code

2017-06-06 Thread Chad Versace
There's a patch on your branch I didn't see on mesa-dev. Subject: i965: Be a bit more conservative about certain resolves It has my r-b. I have comments on this patch... On Fri 26 May 2017, Jason Ekstrand wrote: > We have two different bits of resolve code for render targets: one in > brw_dra

[Mesa-dev] [PATCH 13/30] i965: Combine render target resolve code

2017-05-26 Thread Jason Ekstrand
We have two different bits of resolve code for render targets: one in brw_draw where it's always been and one in brw_context to deal with sRGB on gen9. Let's pull them together. --- src/mesa/drivers/dri/i965/brw_context.c | 47 - src/mesa/drivers/dri/i965/brw_draw.