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 <curroje...@riseup.net> Signed-off-by: Ben Widawsky <b...@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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev