<div dir="ltr"><div>Reviewed-by: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>></div><div><br></div><div>Marek<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 18, 2018 at 7:07 PM, Bas Nieuwenhuizen <span dir="ltr"><<a href="mailto:bas@basnieuwenhuizen.nl" target="_blank">bas@basnieuwenhuizen.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We're not sharing 32_32_32 formats between different GPUs, so we<br>
do not have to align for vega on pre-vega cards.<br>
<br>
Fixes: e361970ed73 "radv: Add support for IMG_DATA_FORMAT_32_32_32."<br>
---<br>
src/amd/common/ac_surface.c | 4 ++--<br>
1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c<br>
index 9e742dc8a45..d7da9950256 100644<br>
--- a/src/amd/common/ac_surface.c<br>
+++ b/src/amd/common/ac_surface.c<br>
@@ -301,10 +301,10 @@ static int gfx6_compute_level(ADDR_HANDLE addrlib,<br>
*/<br>
if (config->info.levels == 1 &&<br>
AddrSurfInfoIn->tileMode == ADDR_TM_LINEAR_ALIGNED &&<br>
- AddrSurfInfoIn->bpp) {<br>
+ AddrSurfInfoIn->bpp &&<br>
+ util_is_power_of_two_or_zero(<wbr>AddrSurfInfoIn->bpp)) {<br>
unsigned alignment = 256 / (AddrSurfInfoIn->bpp / 8);<br>
<br>
- assert(util_is_power_of_two_<wbr>or_zero(AddrSurfInfoIn->bpp));<br>
AddrSurfInfoIn->width = align(AddrSurfInfoIn->width, alignment);<br>
}<br>
<span class="HOEnZb"><font color="#888888"> <br>
-- <br>
2.17.0<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div>