<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 14, 2016 at 2:08 PM, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thursday, April 14, 2016 11:37:57 AM PDT Jason Ekstrand wrote:<br>
> The surface format table hasn't entirely been kept up-to-date.  This commit<br>
> marks a couple more compressed formats as sampleable on gen8+ and adds the<br>
> A4B4G4R4 format as renderable on gen9.<br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_surface_formats.c | 24 +++++++++++<br>
+------------<br>
>  1 file changed, 12 insertions(+), 12 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/brw_surface_formats.c b/src/mesa/<br>
drivers/dri/i965/brw_surface_formats.c<br>
> index 118b18d..61e1408 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_surface_formats.c<br>
> +++ b/src/mesa/drivers/dri/i965/brw_surface_formats.c<br>
> @@ -204,7 +204,7 @@ const struct brw_surface_format_info surface_formats[] =<br>
{<br>
>     SF(50, 50,  x,  x,  x,  x,  x,  x,  x,    x,   P8A8_UNORM_PALETTE0)<br>
>     SF(50, 50,  x,  x,  x,  x,  x,  x,  x,    x,   P8A8_UNORM_PALETTE1)<br>
>     SF( x,  x,  x,  x,  x,  x,  x,  x,  x,    x,   A1B5G5R5_UNORM)<br>
> -   SF( x,  x,  x,  x,  x,  x,  x,  x,  x,    x,   A4B4G4R4_UNORM)<br>
> +   SF( Y,  Y,  x,  Y, 90,  x,  x,  x,  x,    x,   A4B4G4R4_UNORM)<br>
<br>
</span>According to Shared Functions > 3D Sampler > State > SURFACE_STATE, this<br>
is only filterable on CHV/SKL+.  It is not supported for Chroma Key<br>
anywhere.  Please make this:<br></blockquote><div><br></div><div>empirical evidence suggests that at least sampling works fine on BDW.  I'll have to do a little research for gen7.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
   SF(90, 90,  x,  x, 90,  x,  x,  x,  x,    x,   A4B4G4R4_UNORM)<br>
<br>
With that fixed, these two patches are:<br>
Reviewed-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
</blockquote></div><br></div></div>