[PATCH] Fix inconsistent default eg_tile_split in evergreen_accel.c

Mario Kleiner mario.kleiner.de at gmail.com
Sat Jun 27 17:45:58 PDT 2015


eg_tile_split() maps the default: switch-case to tile split
flag 6, like for a tile split size of 4096 Bytes. All other
instances of tile split mappings or reverse mappings in both
the ddx and in Mesa's radeon gallium drivers assign the default:
case to tile split flag 4, consistent with a default of 1024
Bytes.

Make this one outlier consistent with other instances of
mappings in the code to avoid surprises.

Signed-off-by: Mario Kleiner <mario.kleiner.de at gmail.com>
---
 src/evergreen_accel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/evergreen_accel.c b/src/evergreen_accel.c
index 41ebc1a..43146bb 100644
--- a/src/evergreen_accel.c
+++ b/src/evergreen_accel.c
@@ -79,9 +79,9 @@ unsigned eg_tile_split(unsigned tile_split)
 	case 128:	tile_split = 1;	break;
 	case 256:	tile_split = 2;	break;
 	case 512:	tile_split = 3;	break;
+	default:
 	case 1024:	tile_split = 4;	break;
 	case 2048:	tile_split = 5;	break;
-	default:
 	case 4096:	tile_split = 6;	break;
 	}
 	return tile_split;
-- 
1.9.1



More information about the xorg-driver-ati mailing list