[Mesa-dev] [PATCH] gallium/radeon: return correct values for BE in r600_translate_colorswap

2016-02-24 Thread Oded Gabbay
Because I changed the swizzle check, I also need to adapt the return values for each check. It's basically almost the same as before, we just cross between STD and STD_REV, and cross between ALT and ALT_REV This fixes the rgba test in gl-1.0-readpixsanity (piglit) and also fixes tri-flat (mesa de

Re: [Mesa-dev] [PATCH] gallium/radeon: return correct values for BE in r600_translate_colorswap

2016-02-24 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Wed, Feb 24, 2016 at 1:46 PM, Oded Gabbay wrote: > Because I changed the swizzle check, I also need to adapt the return > values for each check. > > It's basically almost the same as before, we just cross between STD and > STD_REV, and cross between ALT and ALT

Re: [Mesa-dev] [PATCH] gallium/radeon: return correct values for BE in r600_translate_colorswap

2016-02-24 Thread Emil Velikov
Hi Oded, On 24 February 2016 at 12:46, Oded Gabbay wrote: > Because I changed the swizzle check, I also need to adapt the return > values for each check. > > It's basically almost the same as before, we just cross between STD and > STD_REV, and cross between ALT and ALT_REV > > This fixes the rgb

Re: [Mesa-dev] [PATCH] gallium/radeon: return correct values for BE in r600_translate_colorswap

2016-02-24 Thread Oded Gabbay
On Wed, Feb 24, 2016 at 5:56 PM, Emil Velikov wrote: > Hi Oded, > > On 24 February 2016 at 12:46, Oded Gabbay wrote: >> Because I changed the swizzle check, I also need to adapt the return >> values for each check. >> >> It's basically almost the same as before, we just cross between STD and >> S