Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-26 Thread Emil Velikov
On 26 February 2016 at 07:35, Oded Gabbay wrote: > On Fri, Feb 26, 2016 at 9:32 AM, Michel Dänzer wrote: >> On 26.02.2016 16:14, Oded Gabbay wrote: >>> On Fri, Feb 26, 2016 at 5:01 AM, Michel Dänzer wrote: [ Dropping

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-26 Thread Oded Gabbay
On Fri, Feb 26, 2016 at 9:35 AM, Oded Gabbay wrote: > On Fri, Feb 26, 2016 at 9:32 AM, Michel Dänzer wrote: >> On 26.02.2016 16:14, Oded Gabbay wrote: >>> On Fri, Feb 26, 2016 at 5:01 AM, Michel Dänzer wrote: [ Dropping

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Oded Gabbay
On Fri, Feb 26, 2016 at 9:32 AM, Michel Dänzer wrote: > On 26.02.2016 16:14, Oded Gabbay wrote: >> On Fri, Feb 26, 2016 at 5:01 AM, Michel Dänzer wrote: >>> >>> [ Dropping mesa-stable list from Cc, since sending patches there by >>> e-mail before they've

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Michel Dänzer
On 26.02.2016 16:14, Oded Gabbay wrote: > On Fri, Feb 26, 2016 at 5:01 AM, Michel Dänzer wrote: >> >> [ Dropping mesa-stable list from Cc, since sending patches there by >> e-mail before they've landed on master is basically noise ] > > Problem is that I sometimes later

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Oded Gabbay
On Fri, Feb 26, 2016 at 5:01 AM, Michel Dänzer wrote: > > [ Dropping mesa-stable list from Cc, since sending patches there by > e-mail before they've landed on master is basically noise ] Problem is that I sometimes later forget to add stable :) > > On 26.02.2016 06:09, Oded

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Michel Dänzer
[ Dropping mesa-stable list from Cc, since sending patches there by e-mail before they've landed on master is basically noise ] On 26.02.2016 06:09, Oded Gabbay wrote: > Since the rework on gallium pipe formats, there is no more need to do > endian swap of the colorformat in the h/w, because the

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Oded Gabbay
On Thu, Feb 25, 2016 at 11:23 PM, Matt Turner wrote: > On Thu, Feb 25, 2016 at 1:19 PM, Oded Gabbay wrote: >> On Thu, Feb 25, 2016 at 11:16 PM, Matt Turner wrote: >>> On Thu, Feb 25, 2016 at 1:09 PM, Oded Gabbay

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Matt Turner
On Thu, Feb 25, 2016 at 1:19 PM, Oded Gabbay wrote: > On Thu, Feb 25, 2016 at 11:16 PM, Matt Turner wrote: >> On Thu, Feb 25, 2016 at 1:09 PM, Oded Gabbay wrote: >>> Since the rework on gallium pipe formats, there is no more need

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Oded Gabbay
On Thu, Feb 25, 2016 at 11:16 PM, Matt Turner wrote: > On Thu, Feb 25, 2016 at 1:09 PM, Oded Gabbay wrote: >> Since the rework on gallium pipe formats, there is no more need to do >> endian swap of the colorformat in the h/w, because the conversion

Re: [Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Matt Turner
On Thu, Feb 25, 2016 at 1:09 PM, Oded Gabbay wrote: > Since the rework on gallium pipe formats, there is no more need to do > endian swap of the colorformat in the h/w, because the conversion between > mesa format and gallium (pipe) format takes endianess into account (see

[Mesa-dev] [PATCH 2/3] gallium/r600: Don't let h/w do endian swap for colorformat

2016-02-25 Thread Oded Gabbay
Since the rework on gallium pipe formats, there is no more need to do endian swap of the colorformat in the h/w, because the conversion between mesa format and gallium (pipe) format takes endianess into account (see the big #if in p_format.h). Signed-off-by: Oded Gabbay