On Wed, Nov 4, 2015 at 5:10 PM, Ilia Mirkin <imir...@alum.mit.edu> wrote:
> On Wed, Nov 4, 2015 at 8:03 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote:
>> ---
>> src/mesa/drivers/dri/i965/brw_image_load_store.c | 49
>> ++++++++++++++++++++++++
>> src/mesa/drivers/dri/i965/brw_image_load_store.h | 2 +
>> 2 files changed, 51 insertions(+)
>>
>> diff --git a/src/mesa/drivers/dri/i965/brw_image_load_store.c
>> b/src/mesa/drivers/dri/i965/brw_image_load_store.c
>> index b78d6b2..f2e2e36 100644
>> --- a/src/mesa/drivers/dri/i965/brw_image_load_store.c
>> +++ b/src/mesa/drivers/dri/i965/brw_image_load_store.c
>> @@ -72,6 +72,55 @@ const struct brw_image_format_info
>> brw_image_format_info[] = {
>> #undef IF
>>
>> uint32_t
>> +brw_image_format_for_gl_format(uint32_t gl_format)
>> +{
>> + switch (gl_format) {
>> + case GL_R8: return BRW_SURFACEFORMAT_R8_UNORM;
>> + case GL_R8_SNORM: return BRW_SURFACEFORMAT_R8_SNORM;
>> + case GL_R8UI: return BRW_SURFACEFORMAT_R8_UINT;
>> + case GL_R8I: return BRW_SURFACEFORMAT_R8_SINT;
>> + case GL_RG8: return BRW_SURFACEFORMAT_R8G8_UNORM;
>> + case GL_RG8_SNORM: return BRW_SURFACEFORMAT_R8G8_SNORM;
>> + case GL_RG8UI: return BRW_SURFACEFORMAT_R8G8_UINT;
>> + case GL_RG8I: return BRW_SURFACEFORMAT_R8G8_SINT;
>> + case GL_RGBA8: return BRW_SURFACEFORMAT_R8G8B8A8_UNORM;
>> + case GL_RGBA8_SNORM: return BRW_SURFACEFORMAT_R8G8B8A8_SNORM;
>> + case GL_RGBA8UI: return BRW_SURFACEFORMAT_R8G8B8A8_UINT;
>> + case GL_RGBA8I: return BRW_SURFACEFORMAT_R8G8B8A8_SINT;
>> + case GL_R11F_G11F_B10F: return BRW_SURFACEFORMAT_R11G11B10_FLOAT;
>> + case GL_RGB10_A2: return BRW_SURFACEFORMAT_R10G10B10A2_UNORM;
>> + case GL_RGB10_A2UI: return BRW_SURFACEFORMAT_R10G10B10A2_UINT;
>> + case GL_R16: return BRW_SURFACEFORMAT_R16_UNORM;
>> + case GL_R16_SNORM: return BRW_SURFACEFORMAT_R16_SNORM;
>> + case GL_R16F: return BRW_SURFACEFORMAT_R16_FLOAT;
>> + case GL_R16UI: return BRW_SURFACEFORMAT_R16_UINT;
>> + case GL_R16I: return BRW_SURFACEFORMAT_R16_SINT;
>> + case GL_RG16: return BRW_SURFACEFORMAT_R16G16_UNORM;
>> + case GL_RG16_SNORM: return BRW_SURFACEFORMAT_R16G16_SNORM;
>> + case GL_RG16F: return BRW_SURFACEFORMAT_R16G16_FLOAT;
>> + case GL_RG16UI: return BRW_SURFACEFORMAT_R16G16_UINT;
>> + case GL_RG16I: return BRW_SURFACEFORMAT_R16G16_SINT;
>> + case GL_RGBA16: return BRW_SURFACEFORMAT_R16G16B16A16_UNORM;
>> + case GL_RGBA16_SNORM: return BRW_SURFACEFORMAT_R16G16B16A16_SNORM;
>> + case GL_RGBA16F: return BRW_SURFACEFORMAT_R16G16B16A16_FLOAT;
>> + case GL_RGBA16UI: return BRW_SURFACEFORMAT_R16G16B16A16_UINT;
>> + case GL_RGBA16I: return BRW_SURFACEFORMAT_R16G16B16A16_SINT;
>> + case GL_R32F: return BRW_SURFACEFORMAT_R32_FLOAT;
>> + case GL_R32UI: return BRW_SURFACEFORMAT_R32_UINT;
>> + case GL_R32I: return BRW_SURFACEFORMAT_R32_SINT;
>> + case GL_RG32F: return BRW_SURFACEFORMAT_R32G32_FLOAT;
>> + case GL_RG32UI: return BRW_SURFACEFORMAT_R32G32_UINT;
>> + case GL_RG32I: return BRW_SURFACEFORMAT_R32G32_SINT;
>> + case GL_RGBA32F: return BRW_SURFACEFORMAT_R32G32B32A32_FLOAT;
>> + case GL_RGBA32UI: return BRW_SURFACEFORMAT_R32G32B32A32_UINT;
>> + case GL_RGBA32I: return BRW_SURFACEFORMAT_R32G32B32A32_SINT;
>> + default:
>> + assert(!"Invalid image format");
>> + return GL_NONE;
>
> Seems odd to be returning GL_NONE here where all the other returns are
> BRW_SURFACEFORMAT_*.
Oops. Yeah, that should be BRW_SURFACEFORMAT_RAW. I'll get it fixed
locally. However, if anyone ever hits that, they'll hit the assert so
it doesn't matter too much.
>> + }
>> +}
>> +
>> +uint32_t
>> brw_lower_image_format(const struct brw_device_info *devinfo,
>> uint32_t format)
>> {
>> diff --git a/src/mesa/drivers/dri/i965/brw_image_load_store.h
>> b/src/mesa/drivers/dri/i965/brw_image_load_store.h
>> index 2ab85cc..c18f701 100644
>> --- a/src/mesa/drivers/dri/i965/brw_image_load_store.h
>> +++ b/src/mesa/drivers/dri/i965/brw_image_load_store.h
>> @@ -51,6 +51,8 @@ struct brw_image_format_info {
>>
>> extern const struct brw_image_format_info brw_image_format_info[];
>>
>> +uint32_t brw_image_format_for_gl_format(uint32_t gl_format);
>> +
>> uint32_t brw_lower_image_format(const struct brw_device_info *devinfo,
>> uint32_t format);
>> #ifdef __cplusplus
>> --
>> 2.5.0.400.gff86faf
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
