Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Jason Ekstrand
On Tue, Jun 6, 2017 at 10:11 PM, Chad Versace wrote: > On Tue 06 Jun 2017, Jason Ekstrand wrote: > > On Tue, Jun 6, 2017 at 6:00 PM, Chad Versace wrote: > > > > > On Tue 06 Jun 2017, Jason Ekstrand wrote: > > > > On Tue, Jun 6, 2017 at 1:32 PM, Jason

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Chad Versace
On Tue 06 Jun 2017, Jason Ekstrand wrote: > On Tue, Jun 6, 2017 at 6:00 PM, Chad Versace wrote: > > > On Tue 06 Jun 2017, Jason Ekstrand wrote: > > > On Tue, Jun 6, 2017 at 1:32 PM, Jason Ekstrand > > wrote: > > > > > > > On Tue, Jun 6, 2017 at 1:22 PM,

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Jason Ekstrand
On Tue, Jun 6, 2017 at 6:00 PM, Chad Versace wrote: > On Tue 06 Jun 2017, Jason Ekstrand wrote: > > On Tue, Jun 6, 2017 at 1:32 PM, Jason Ekstrand > wrote: > > > > > On Tue, Jun 6, 2017 at 1:22 PM, Chad Versace > > > > wrote:

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Jason Ekstrand
On Tue, Jun 6, 2017 at 5:41 PM, Chad Versace wrote: > On Tue 06 Jun 2017, Jason Ekstrand wrote: > > On Tue, Jun 6, 2017 at 1:22 PM, Chad Versace > > wrote: > > > > > On Fri 26 May 2017, Jason Ekstrand wrote: > > > > This enum describes all of the

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Chad Versace
On Tue 06 Jun 2017, Jason Ekstrand wrote: > On Tue, Jun 6, 2017 at 1:32 PM, Jason Ekstrand wrote: > > > On Tue, Jun 6, 2017 at 1:22 PM, Chad Versace > > wrote: > > > >> On Fri 26 May 2017, Jason Ekstrand wrote: > > How about a section after the

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Chad Versace
On Tue 06 Jun 2017, Jason Ekstrand wrote: > On Tue, Jun 6, 2017 at 1:22 PM, Chad Versace > wrote: > > > On Fri 26 May 2017, Jason Ekstrand wrote: > > > This enum describes all of the states that a auxiliary compressed > > > surface can have. All of the states as well

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Jason Ekstrand
On Tue, Jun 6, 2017 at 1:32 PM, Jason Ekstrand wrote: > On Tue, Jun 6, 2017 at 1:22 PM, Chad Versace > wrote: > >> On Fri 26 May 2017, Jason Ekstrand wrote: >> > This enum describes all of the states that a auxiliary compressed >> > surface can

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Jason Ekstrand
On Tue, Jun 6, 2017 at 1:22 PM, Chad Versace wrote: > On Fri 26 May 2017, Jason Ekstrand wrote: > > This enum describes all of the states that a auxiliary compressed > > surface can have. All of the states as well as normative language for > > referring to each of the

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-06 Thread Chad Versace
On Fri 26 May 2017, Jason Ekstrand wrote: > This enum describes all of the states that a auxiliary compressed > surface can have. All of the states as well as normative language for > referring to each of the compression operations is provided in the > truly colossal comment for the new

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-02 Thread Nanley Chery
On Fri, Jun 02, 2017 at 01:05:47PM -0700, Jason Ekstrand wrote: > On Fri, Jun 2, 2017 at 12:57 PM, Nanley Chery wrote: > > > On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand wrote: > > > This enum describes all of the states that a auxiliary compressed > >

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-02 Thread Jason Ekstrand
On Fri, Jun 2, 2017 at 12:57 PM, Nanley Chery wrote: > On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand wrote: > > This enum describes all of the states that a auxiliary compressed > ^ >

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-06-02 Thread Nanley Chery
On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand wrote: > This enum describes all of the states that a auxiliary compressed ^ an > surface can have. All of the states as well as normative language

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-05-31 Thread Pohjolainen, Topi
On Wed, May 31, 2017 at 08:35:22AM -0700, Jason Ekstrand wrote: > On Wed, May 31, 2017 at 5:40 AM, Pohjolainen, Topi < > topi.pohjolai...@gmail.com> wrote: > > > On Tue, May 30, 2017 at 08:29:55PM +0300, Pohjolainen, Topi wrote: > > > On Tue, May 30, 2017 at 07:32:24AM -0700, Jason Ekstrand

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-05-31 Thread Jason Ekstrand
On Wed, May 31, 2017 at 5:40 AM, Pohjolainen, Topi < topi.pohjolai...@gmail.com> wrote: > On Tue, May 30, 2017 at 08:29:55PM +0300, Pohjolainen, Topi wrote: > > On Tue, May 30, 2017 at 07:32:24AM -0700, Jason Ekstrand wrote: > > > On Tue, May 30, 2017 at 1:43 AM, Pohjolainen, Topi < > > >

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-05-31 Thread Pohjolainen, Topi
On Tue, May 30, 2017 at 08:29:55PM +0300, Pohjolainen, Topi wrote: > On Tue, May 30, 2017 at 07:32:24AM -0700, Jason Ekstrand wrote: > > On Tue, May 30, 2017 at 1:43 AM, Pohjolainen, Topi < > > topi.pohjolai...@gmail.com> wrote: > > > > > On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-05-30 Thread Pohjolainen, Topi
On Tue, May 30, 2017 at 07:32:24AM -0700, Jason Ekstrand wrote: > On Tue, May 30, 2017 at 1:43 AM, Pohjolainen, Topi < > topi.pohjolai...@gmail.com> wrote: > > > On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand wrote: > > > This enum describes all of the states that a auxiliary compressed

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-05-30 Thread Jason Ekstrand
On Tue, May 30, 2017 at 1:43 AM, Pohjolainen, Topi < topi.pohjolai...@gmail.com> wrote: > On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand wrote: > > This enum describes all of the states that a auxiliary compressed > > surface can have. All of the states as well as normative language

Re: [Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-05-30 Thread Pohjolainen, Topi
On Fri, May 26, 2017 at 04:30:18PM -0700, Jason Ekstrand wrote: > This enum describes all of the states that a auxiliary compressed > surface can have. All of the states as well as normative language for > referring to each of the compression operations is provided in the > truly colossal comment

[Mesa-dev] [PATCH 14/30] intel/isl: Add an enum for describing auxiliary compression state

2017-05-26 Thread Jason Ekstrand
This enum describes all of the states that a auxiliary compressed surface can have. All of the states as well as normative language for referring to each of the compression operations is provided in the truly colossal comment for the new isl_aux_state enum. There is also a diagram showing how