Re: [Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-27 Thread Jason Ekstrand
On Tue, Feb 27, 2018 at 11:55 AM, Rafael Antognolli < rafael.antogno...@intel.com> wrote: > On Tue, Feb 27, 2018 at 11:46:12AM -0800, Jason Ekstrand wrote: > > On Tue, Feb 27, 2018 at 9:35 AM, Rafael Antognolli < > rafael.antogno...@intel.com > > > wrote: > > > > On Mon, Feb 26, 2018 at 05:04:

Re: [Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-27 Thread Rafael Antognolli
On Tue, Feb 27, 2018 at 11:46:12AM -0800, Jason Ekstrand wrote: > On Tue, Feb 27, 2018 at 9:35 AM, Rafael Antognolli > > wrote: > > On Mon, Feb 26, 2018 at 05:04:37PM -0800, Jason Ekstrand wrote: > > On Wed, Feb 21, 2018 at 1:45 PM, Rafael Antognolli < > rafael.antogno...@intel.com >

Re: [Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-27 Thread Jason Ekstrand
On Tue, Feb 27, 2018 at 9:35 AM, Rafael Antognolli < rafael.antogno...@intel.com> wrote: > On Mon, Feb 26, 2018 at 05:04:37PM -0800, Jason Ekstrand wrote: > > On Wed, Feb 21, 2018 at 1:45 PM, Rafael Antognolli < > rafael.antogno...@intel.com > > > wrote: > > > > The size of the clear color str

Re: [Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-27 Thread Rafael Antognolli
On Mon, Feb 26, 2018 at 05:04:37PM -0800, Jason Ekstrand wrote: > On Wed, Feb 21, 2018 at 1:45 PM, Rafael Antognolli > > wrote: > > The size of the clear color struct (expected by the hardware) is 8 > dwords (isl_dev.ss.clear_value_state_size here). But we still need to > track the s

Re: [Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-26 Thread Jason Ekstrand
On Wed, Feb 21, 2018 at 1:45 PM, Rafael Antognolli < rafael.antogno...@intel.com> wrote: > The size of the clear color struct (expected by the hardware) is 8 > dwords (isl_dev.ss.clear_value_state_size here). But we still need to > track the size of the clear color, used when memcopying it to/from

Re: [Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-26 Thread Rafael Antognolli
On Mon, Feb 26, 2018 at 01:07:54PM -0800, Jordan Justen wrote: > On 2018-02-21 13:45:14, Rafael Antognolli wrote: > > The size of the clear color struct (expected by the hardware) is 8 > > dwords (isl_dev.ss.clear_value_state_size here). But we still need to > > track the size of the clear color, u

Re: [Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-26 Thread Jordan Justen
On 2018-02-21 13:45:14, Rafael Antognolli wrote: > The size of the clear color struct (expected by the hardware) is 8 > dwords (isl_dev.ss.clear_value_state_size here). But we still need to > track the size of the clear color, used when memcopying it to/from the > state buffer. For that we keep isl

[Mesa-dev] [PATCH v3 05/13] intel/genxml: Add Clear Color struct.

2018-02-21 Thread Rafael Antognolli
The size of the clear color struct (expected by the hardware) is 8 dwords (isl_dev.ss.clear_value_state_size here). But we still need to track the size of the clear color, used when memcopying it to/from the state buffer. For that we keep isl_dev.ss.clear_value_size. Signed-off-by: Rafael Antognol