On 1 May 2016 at 13:43, Michel Dänzer <mic...@daenzer.net> wrote: > On 24.04.2016 20:27, Marek Olšák wrote: >> From: Marek Olšák <marek.ol...@amd.com> >> >> --- >> src/gallium/drivers/r300/r300_reg.h | 20 +++++++---------- >> src/gallium/drivers/r300/r300_texture.c | 38 >> ++++++++++++++++++++++++++++++--- >> 2 files changed, 43 insertions(+), 15 deletions(-) >> >> diff --git a/src/gallium/drivers/r300/r300_reg.h >> b/src/gallium/drivers/r300/r300_reg.h >> index 9c373c5..9c93b84 100644 >> --- a/src/gallium/drivers/r300/r300_reg.h >> +++ b/src/gallium/drivers/r300/r300_reg.h >> @@ -1700,10 +1700,7 @@ USE OR OTHER DEALINGS IN THE SOFTWARE. >> #define R300_TX_OFFSET_6 0x4558 >> #define R300_TX_OFFSET_7 0x455C >> >> -# define R300_TXO_ENDIAN_NO_SWAP (0 << 0) >> -# define R300_TXO_ENDIAN_BYTE_SWAP (1 << 0) >> -# define R300_TXO_ENDIAN_WORD_SWAP (2 << 0) >> -# define R300_TXO_ENDIAN_HALFDW_SWAP (3 << 0) >> +# define R300_TXO_ENDIAN(x) ((x) << 0) >> # define R300_TXO_MACRO_TILE_LINEAR (0 << 2) >> # define R300_TXO_MACRO_TILE_TILED (1 << 2) >> # define R300_TXO_MACRO_TILE(x) ((x) << 2) >> @@ -2418,10 +2415,7 @@ USE OR OTHER DEALINGS IN THE SOFTWARE. >> # define R300_COLOR_MICROTILE_ENABLE (1 << 17) >> # define R300_COLOR_MICROTILE_ENABLE_SQUARE (2 << 17) /* Only >> available in 16-bit */ >> # define R300_COLOR_MICROTILE(x) ((x) << 17) >> -# define R300_COLOR_ENDIAN_NO_SWAP (0 << 19) >> -# define R300_COLOR_ENDIAN_WORD_SWAP (1 << 19) >> -# define R300_COLOR_ENDIAN_DWORD_SWAP (2 << 19) >> -# define R300_COLOR_ENDIAN_HALF_DWORD_SWAP (3 << 19) >> +# define R300_COLOR_ENDIAN(x) ((x) << 19) >> # define R500_COLOR_FORMAT_ARGB10101010 (0 << 21) >> # define R500_COLOR_FORMAT_UV1010 (1 << 21) >> # define R500_COLOR_FORMAT_CI8 (2 << 21) /* 2D only */ > > Did you or anyone else test this patch? Back when I fixed the classic > r300 driver to work on big endian hosts, none of these bits seemed to > have any effect unfortunately. It's possible I did something wrong then > though. > Afaict an updated version got tested over at bugzilla https://bugs.freedesktop.org/show_bug.cgi?id=71789
Marek, if you can mention the corrected shift, bugreport and CC stable in the commit message I'll be very grateful. Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
