From 2486c7dd540d6f318265d8b49c0c6205cc9e9315 Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Wed, 10 May 2017 12:15:01 -0700 Subject: [PATCH] intel/isl: Refactor gen6_choose_image_alignment_el Reviewed-by: Topi Pohjolainen Reviewed-by: Chad Versace --- src/intel/isl/isl_gen6.c | 32 ++++++++++++++------------------ 1 file changed, 14 insertions(+), 18 deletions(-) diff --git a/src/intel/isl/isl_gen6.c b/src/intel/isl/isl_gen6.c index b746903..6da0be7 100644 --- a/src/intel/isl/isl_gen6.c +++ b/src/intel/isl/isl_gen6.c @@ -88,6 +88,8 @@ isl_gen6_choose_image_alignment_el(const struct isl_device *dev, * | format | halign | valign | * +------------------------+--------+--------+ * | YUV 4:2:2 formats | 4 | * | + * | BC1-5 | 4 | 4 | + * | FXT1 | 8 | 4 | * | uncompressed formats | 4 | * | * +------------------------+--------+--------+ * @@ -110,38 +112,32 @@ isl_gen6_choose_image_alignment_el(const struct isl_device *dev, */ if (isl_format_is_compressed(info->format)) { + /* Compressed formats have an alignment equal to their block size */ *image_align_el = isl_extent3d(1, 1, 1); return; } - if (isl_format_is_yuv(info->format)) { - *image_align_el = isl_extent3d(4, 2, 1); - return; - } - - if (info->samples > 1) { - *image_align_el = isl_extent3d(4, 4, 1); - return; - } - - if (isl_surf_usage_is_depth_or_stencil(info->usage) && - !ISL_DEV_USE_SEPARATE_STENCIL(dev)) { - /* interleaved depthstencil buffer */ - *image_align_el = isl_extent3d(4, 4, 1); - return; - } - if (isl_surf_usage_is_depth(info->usage)) { - /* separate depth buffer */ + /* depth buffer (possibly interleaved with stencil) */ *image_align_el = isl_extent3d(4, 4, 1); return; } if (isl_surf_usage_is_stencil(info->usage)) { /* separate stencil buffer */ + assert(!isl_surf_usage_is_depth(info->usage)); *image_align_el = isl_extent3d(4, 2, 1); return; } + if (info->samples > 1) { + *image_align_el = isl_extent3d(4, 4, 1); + return; + } + + /* For everything else, 4x2 is always a valid alignment. Since this is + * also the smallest alignment we can specify, we use 4x2 for everything + * else because it uses the least memory. + */ *image_align_el = isl_extent3d(4, 2, 1); } -- 2.7.4