<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Oct 24, 2018 at 12:34 PM Erik Faye-Lund <<a href="mailto:erik.faye-lund@collabora.com">erik.faye-lund@collabora.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, 2018-10-18 at 15:42 -0400, Marek Olšák wrote:<br>
> I think you need something like this:<br>
> <br>
<a href="https://cgit.freedesktop.org/~mareko/mesa/commit/?h=amd-extension-pack&id=ad774f9db1d735811a8d830ad90a2f8208aa0a7b" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~mareko/mesa/commit/?h=amd-extension-pack&id=ad774f9db1d735811a8d830ad90a2f8208aa0a7b</a><br>
> <br>
<br>
Thanks, this looks correct to me.<br>
<br>
I've ported some of the piglit s3tc tests to gles2, and they all seem<br>
to pass with your patch:<br>
<br>
<a href="https://patchwork.freedesktop.org/series/51467/" rel="noreferrer" target="_blank">https://patchwork.freedesktop.org/series/51467/</a><br>
<br>
Do you mind if I add a "Tested-by"-tag and submit a v2 with your patch<br>
instead?<br></blockquote><div><br></div><div>Feel free to do it.<br></div><div><br></div><div>Marek</div></div></div>