[Mesa-dev] [PATCH 08/11] i965: Use the Mesa surface formats for float RGB surfaces.

[Kenneth Graunke]

On 04/16/2013 05:21 PM, Eric Anholt wrote:
> ---
>   src/mesa/drivers/dri/i965/brw_wm_surface_state.c |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> index 94e712e..c1e3b14 100644
> --- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> +++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> @@ -562,10 +562,10 @@ brw_format_for_mesa_format(gl_format mesa_format)
>         [MESA_FORMAT_XRGB2101010_UNORM] = BRW_SURFACEFORMAT_B10G10R10X2_UNORM,
>         [MESA_FORMAT_XBGR16161616_UNORM] = BRW_SURFACEFORMAT_R16G16B16X16_UNORM,
>         [MESA_FORMAT_XBGR16161616_SNORM] = 0,
> -      [MESA_FORMAT_XBGR16161616_FLOAT] = 0,
> +      [MESA_FORMAT_XBGR16161616_FLOAT] = BRW_SURFACEFORMAT_R16G16B16X16_FLOAT,
>         [MESA_FORMAT_XBGR16161616_UINT] = 0,
>         [MESA_FORMAT_XBGR16161616_SINT] = 0,
> -      [MESA_FORMAT_XBGR32323232_FLOAT] = 0,
> +      [MESA_FORMAT_XBGR32323232_FLOAT] = BRW_SURFACEFORMAT_R32G32B32X32_FLOAT,
>         [MESA_FORMAT_XBGR32323232_UINT] = 0,
>         [MESA_FORMAT_XBGR32323232_SINT] = 0,
>      };

I thought these were backwards, but on a closer look, they appear 
correct to me.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>