Re: [Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-24 Thread Brian Paul
Will do. BTW, I had to add #include "main/framebuffer.h" in st_atom_rasterizer.c -Brian On 03/24/2016 05:29 AM, Bas Nieuwenhuizen wrote: Can someone push this patch for me? Thanks, Bas Nieuwenhuizen On Wed, Mar 23, 2016 at 4:21 PM, Brian Paul wrote: No regressions

Re: [Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-24 Thread Bas Nieuwenhuizen
Can someone push this patch for me? Thanks, Bas Nieuwenhuizen On Wed, Mar 23, 2016 at 4:21 PM, Brian Paul wrote: > No regressions here, Bas. > > -Brian > > > On 03/22/2016 01:27 PM, Brian Paul wrote: >> >> If you can wait until tomorrow, Bas, I'll do an overight piglit run to

Re: [Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-23 Thread Brian Paul
No regressions here, Bas. -Brian On 03/22/2016 01:27 PM, Brian Paul wrote: If you can wait until tomorrow, Bas, I'll do an overight piglit run to check for regressions. -Brian On 03/22/2016 12:31 PM, Marek Olšák wrote: Reviewed-by: Marek Olšák Somebody from Intel or

Re: [Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-22 Thread Bas Nieuwenhuizen
Sure, no problem. - Bas On Tue, Mar 22, 2016 at 8:27 PM, Brian Paul wrote: > If you can wait until tomorrow, Bas, I'll do an overight piglit run to check > for regressions. > > -Brian > > > On 03/22/2016 12:31 PM, Marek Olšák wrote: >> >> Reviewed-by: Marek Olšák

Re: [Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-22 Thread Brian Paul
If you can wait until tomorrow, Bas, I'll do an overight piglit run to check for regressions. -Brian On 03/22/2016 12:31 PM, Marek Olšák wrote: Reviewed-by: Marek Olšák Somebody from Intel or VMWare might want to take a look too. Marek On Tue, Mar 22, 2016 at 2:58 AM,

Re: [Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-22 Thread Marek Olšák
Reviewed-by: Marek Olšák Somebody from Intel or VMWare might want to take a look too. Marek On Tue, Mar 22, 2016 at 2:58 AM, Bas Nieuwenhuizen wrote: > This removes any dependency on driver validation of the number of > framebuffer samples. > >

Re: [Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-21 Thread eocallaghan
Too quick, very nice cleanup, thanks. Reviewed-by: Edward O'Callaghan On 2016-03-22 12:58, Bas Nieuwenhuizen wrote: This removes any dependency on driver validation of the number of framebuffer samples. Signed-off-by: Bas Nieuwenhuizen

[Mesa-dev] [PATCH] mesa: replace gl_context->Multisample._Enabled with _mesa_is_multisample_enabled.

2016-03-21 Thread Bas Nieuwenhuizen
This removes any dependency on driver validation of the number of framebuffer samples. Signed-off-by: Bas Nieuwenhuizen --- src/mesa/drivers/dri/i965/brw_util.h | 5 +++-- src/mesa/drivers/dri/i965/gen6_cc.c| 6 +++---