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

Oded Gabbay oded.gabbay at gmail.com
Thu Feb 25 21:09:18 UTC 2016


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 <oded.gabbay at gmail.com>
Cc: "11.1 11.2" <mesa-stable at lists.freedesktop.org>
---
 src/gallium/drivers/r600/r600_state_common.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/gallium/drivers/r600/r600_state_common.c b/src/gallium/drivers/r600/r600_state_common.c
index c3346f2..614b0fb 100644
--- a/src/gallium/drivers/r600/r600_state_common.c
+++ b/src/gallium/drivers/r600/r600_state_common.c
@@ -2704,6 +2704,12 @@ uint32_t r600_translate_colorformat(enum chip_class chip, enum pipe_format forma
 
 uint32_t r600_colorformat_endian_swap(uint32_t colorformat)
 {
+	/*
+	 * No need to do endian swaps on colors, as mesa<-->pipe formats
+	 * conversion take into account the endian issue
+	 */
+	return ENDIAN_NONE;
+
 	if (R600_BIG_ENDIAN) {
 		switch(colorformat) {
 		/* 8-bit buffers. */
-- 
2.5.0



More information about the mesa-dev mailing list