Reviewed-by: Marek Olšák <marek.ol...@amd.com> Marek
On Wed, Feb 24, 2016 at 1:46 PM, Oded Gabbay <oded.gab...@gmail.com> 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 rgba test in gl-1.0-readpixsanity (piglit) and also > fixes tri-flat (mesa demos). > > Signed-off-by: Oded Gabbay <oded.gab...@gmail.com> > --- > src/gallium/drivers/radeon/r600_texture.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/src/gallium/drivers/radeon/r600_texture.c > b/src/gallium/drivers/radeon/r600_texture.c > index 1df0c30..9a3ccb5 100644 > --- a/src/gallium/drivers/radeon/r600_texture.c > +++ b/src/gallium/drivers/radeon/r600_texture.c > @@ -1304,13 +1304,13 @@ unsigned r600_translate_colorswap(enum pipe_format > format) > return V_0280A0_SWAP_ALT_REV; /* WXYZ */ > #else > if (HAS_SWIZZLE(1,W) && HAS_SWIZZLE(2,X)) > - return V_0280A0_SWAP_STD; /* ZWXY */ > + return V_0280A0_SWAP_STD_REV; /* ZWXY */ > else if (HAS_SWIZZLE(1,X) && HAS_SWIZZLE(2,W)) > - return V_0280A0_SWAP_STD_REV; /* YXWZ */ > + return V_0280A0_SWAP_STD; /* YXWZ */ > else if (HAS_SWIZZLE(1,W) && HAS_SWIZZLE(2,Z)) > - return V_0280A0_SWAP_ALT; /* XWZY */ > + return V_0280A0_SWAP_ALT_REV; /* XWZY */ > else if (HAS_SWIZZLE(1,Z) && HAS_SWIZZLE(2,W)) > - return V_0280A0_SWAP_ALT_REV; /* YZWX */ > + return V_0280A0_SWAP_ALT; /* YZWX */ > #endif > break; > } > -- > 2.5.0 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev