[PATCH weston 4/5] gl-renderer: fix pixel format used in texture uploads when using R/RG textures

[Emil Velikov]

On 29 November 2017 at 14:25, Arnaud Vrac <rawoul at gmail.com> wrote:
> From: Arnaud Vrac <avrac at freebox.fr>
>
> In glTexImage2D / glTexSubImage2D calls, the only pixel formats allowed
> for the GL_R8 and GL_RG internal formats are respectively GL_RED and
s/GL_RG/GL_RG8/ ^^

> GL_RG [1].
>
> Make sure we match this requirement, as some drivers will fail with the
> current code.
>
AFAICT the patch is spot on. Trivial suggestion below:
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>


I'd suggest using something like the following. A reference to the
offending code is always nice, plus CC-ing the author gives you an
extra reviewer ;-)

Fixes: 00a03d2f724 ("gl-renderer: add support of WL_SHM_FORMAT_NV12")
Cc: Vincent Abriou <vincent.abriou at st.com>

HTH
Emil