<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">Il giorno ven 30 nov 2018, 12:26 Erik Faye-Lund <<a href="mailto:erik.faye-lund@collabora.com">erik.faye-lund@collabora.com</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 2018-11-23 at 11:53 +0100, Erik Faye-Lund wrote:<br>
> OK, so here's a v2 of this series. These are the changes since v1:<br>
> - Removed double-semicolons in patch #25<br>
> - Removed default-case and questionable comment in patch #25<br>
> - Dropped patch #30, as it would regress functionality on two drivers<br>
> <br>
> Please review :)<br>
<br>
Ping? Any takers?<br></blockquote></div></div><div dir="auto">I read all the patches and didn't spot anything wrong, but please don't consider this a proper review.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> Erik Faye-Lund (29):<br>
>   mesa/main: make _mesa_has_tessellation return bool<br>
>   mesa/main: rename format-check function<br>
>   mesa/main: clean up S3_s3tc check<br>
>   mesa/main: clean up OES_texture_float_linear check<br>
>   mesa/main: clean up ES2_compatibility check<br>
>   mesa/main: clean up integer texture check<br>
>   mesa/main: use _mesa_has_FOO_bar for compressed format checks<br>
>   mesa/main: do not allow s3tc enums on gles1<br>
>   mesa/main: do not allow etc2 enums on gles1<br>
>   mesa/main: do not allow astc enums on gles1<br>
>   mesa/main: do not allow depth-texture enums on gles1<br>
>   mesa/main: do not allow stencil-texture enums on gles1<br>
>   mesa/main: do not allow ARB_texture_rgb10_a2ui enums before gles3<br>
>   mesa/main: do not allow integer-texture enums before gles3<br>
>   mesa/main: do not allow ARB_depth_buffer_float enums before gles3<br>
>   mesa/main: do not allow EXT_packed_float enums before gles3<br>
>   mesa/main: do not allow rg-textures enums before gles3<br>
>   mesa/main: do not allow EXT_texture_shared_exponent enums before<br>
> gles3<br>
>   mesa/main: do not allow MESA_ycbcr_texture enums on gles<br>
>   mesa/main: do not allow type_2_10_10_10_REV enums before gles3<br>
>   mesa/main: do not allow floating-point texture enums on gles1<br>
>   mesa/main: do not allow snorm-texture enums before gles3<br>
>   mesa/main: do not allow sRGB texture enums before gles3<br>
>   mesa/main: do not allow EXT_texture_sRGB_R8 enums before gles3<br>
>   mesa/main: split float-texture support checking in two<br>
>   mesa/main: require EXT_texture_type_2_10_10_10_REV for gles3<br>
>   mesa/main: require EXT_texture_sRGB for gles3<br>
>   mesa/st: do not probe for the same texture-formats twice<br>
>   mesa/main: do not require float-texture filtering for es3<br>
> <br>
>  src/mesa/main/context.h                |  59 ++++++-<br>
>  src/mesa/main/glformats.c              | 221 +++++++++++++--------<br>
> ----<br>
>  src/mesa/main/glformats.h              |   6 +-<br>
>  src/mesa/main/teximage.c               |   4 +-<br>
>  src/mesa/main/version.c                |   8 +-<br>
>  src/mesa/state_tracker/st_extensions.c |   8 +-<br>
>  6 files changed, 184 insertions(+), 122 deletions(-)<br>
> <br>
<br>
</blockquote></div></div></div>