Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-31 Thread Ilia Mirkin
On Thu, Mar 27, 2014 at 12:16 PM, Ilia Mirkin wrote: > On Thu, Mar 27, 2014 at 11:35 AM, Jose Fonseca wrote: >> >> >> - Original Message - >>> On Wed, Mar 26, 2014 at 4:30 AM, Roland Scheidegger >>> wrote: >>> > Am 26.03.2014 03:29, schrieb Marek Olšák: >>> >> My reasoning was that it wo

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-27 Thread Roland Scheidegger
Am 27.03.2014 03:04, schrieb Marek Olšák: > On Wed, Mar 26, 2014 at 4:30 AM, Roland Scheidegger > wrote: >> Am 26.03.2014 03:29, schrieb Marek Olšák: >>> My reasoning was that it would be better to specify a raw clear value >>> and clear value size for buffers, which are always untyped, and >>> p

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-27 Thread Ilia Mirkin
On Thu, Mar 27, 2014 at 11:35 AM, Jose Fonseca wrote: > > > - Original Message - >> On Wed, Mar 26, 2014 at 4:30 AM, Roland Scheidegger >> wrote: >> > Am 26.03.2014 03:29, schrieb Marek Olšák: >> >> My reasoning was that it would be better to specify a raw clear value >> >> and clear valu

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-27 Thread Jose Fonseca
- Original Message - > On Wed, Mar 26, 2014 at 4:30 AM, Roland Scheidegger > wrote: > > Am 26.03.2014 03:29, schrieb Marek Olšák: > >> My reasoning was that it would be better to specify a raw clear value > >> and clear value size for buffers, which are always untyped, and > >> pipe_colo

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-26 Thread Marek Olšák
On Wed, Mar 26, 2014 at 4:30 AM, Roland Scheidegger wrote: > Am 26.03.2014 03:29, schrieb Marek Olšák: >> My reasoning was that it would be better to specify a raw clear value >> and clear value size for buffers, which are always untyped, and >> pipe_color_union for textures, which are always type

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-25 Thread Roland Scheidegger
Am 26.03.2014 03:29, schrieb Marek Olšák: > My reasoning was that it would be better to specify a raw clear value > and clear value size for buffers, which are always untyped, and > pipe_color_union for textures, which are always typed, so that drivers > can easily implement the texture clearing on

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-25 Thread Marek Olšák
The first 2 patches are: Reviewed-by: Marek Olšák Marek On Wed, Mar 26, 2014 at 12:23 AM, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > --- > > v1 -> v2: add docs section to context.rst > > src/gallium/docs/source/context.rst | 5 + > src/gallium/include/pipe/p_context.h | 11 +

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-25 Thread Marek Olšák
My reasoning was that it would be better to specify a raw clear value and clear value size for buffers, which are always untyped, and pipe_color_union for textures, which are always typed, so that drivers can easily implement the texture clearing on top of pipe_context::clear. I also suggested tha

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-25 Thread Ilia Mirkin
On Tue, Mar 25, 2014 at 7:57 PM, Brian Paul wrote: > On 03/25/2014 05:23 PM, Ilia Mirkin wrote: >> >> Signed-off-by: Ilia Mirkin >> --- >> >> v1 -> v2: add docs section to context.rst >> >> src/gallium/docs/source/context.rst | 5 + >> src/gallium/include/pipe/p_context.h | 11 ++

Re: [Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-25 Thread Brian Paul
On 03/25/2014 05:23 PM, Ilia Mirkin wrote: Signed-off-by: Ilia Mirkin --- v1 -> v2: add docs section to context.rst src/gallium/docs/source/context.rst | 5 + src/gallium/include/pipe/p_context.h | 11 +++ 2 files changed, 16 insertions(+) diff --git a/src/gallium/docs/sourc

[Mesa-dev] [PATCHv2 1/3] gallium: add interface to clear buffers

2014-03-25 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- v1 -> v2: add docs section to context.rst src/gallium/docs/source/context.rst | 5 + src/gallium/include/pipe/p_context.h | 11 +++ 2 files changed, 16 insertions(+) diff --git a/src/gallium/docs/source/context.rst b/src/gallium/docs/source/context