[Mesa-dev] [PATCH] i965/l3: Remove redundant is_cherryview check
Ben Widawsky
ben at bwidawsk.net
Tue Oct 4 20:30:39 UTC 2016
All mobile parts (so far) are GT1. The check added extra confusion
because it appeared Broxton was missing when it wasn't. Replace it with
a comment.
Alternatively, I'd be willing to add an is_broxton check.
Cc: Francisco Jerez <currojerez at riseup.net>
Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
---
src/intel/common/gen_l3_config.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/intel/common/gen_l3_config.c b/src/intel/common/gen_l3_config.c
index b172ef6..eb4e8ae 100644
--- a/src/intel/common/gen_l3_config.c
+++ b/src/intel/common/gen_l3_config.c
@@ -258,7 +258,8 @@ get_l3_way_size(const struct gen_device_info *devinfo)
if (devinfo->is_baytrail)
return 2;
- else if (devinfo->is_cherryview || devinfo->gt == 1)
+ /* XXX: Cherryview and Broxton are always gt1 */
+ else if (devinfo->gt == 1)
return 4;
else
--
2.10.0
More information about the mesa-dev
mailing list