On 07/24/2018 10:31 PM, Eric Anholt wrote:
Tapani Pälli <[email protected]> writes:

These bits were missing, found when extending the Piglit test.

Fixes: 7f467d4f73 "mesa: GL_EXT_texture_norm16 extension plumbing"
Signed-off-by: Tapani Pälli <[email protected]>

Aren't you missing the RGB16 and R/RG/RGB/RGBA16_SNORM cases here?  They
aren't required for rendering, but if the implementation supports them
we should answer correctly.


If I've understood correctly, SNORM ones are supported only with EXT_render_snorm so those ones I was planning to add separately. For RGB16 I guess we would need to query the driver. Not sure what would be the best way to achieve this, it seems that at least i965 relies on _mesa_query_internal_format_default which just says to support anything (is broken).

Thanks;

// Tapani
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to