LGTM :) Reviewed-by: Plamena Manolova <[email protected]>
On Thu, Feb 2, 2017 at 12:27 PM, Tapani Pälli <[email protected]> wrote: > Earlier changes introduced is_ycrcb flag which checks the component > order of u and v components. Condition for setting the flag was > incorrect, with ycrcb we are supposed to have cr before cb. > > This patch (together with a fix in our gralloc) fixes corrupted > rendering from 'test-opengl-gl2_yuvtex' native test and corrupted > gallery thumbnail in application switcher on Android-IA. > > Fixes: 51727b1cf57e8c4630767eb9ead207b102ffa489 > Signed-off-by: Tapani Pälli <[email protected]> > --- > src/egl/drivers/dri2/platform_android.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/egl/drivers/dri2/platform_android.c > b/src/egl/drivers/dri2/platform_android.c > index 36bd119..511b696 100644 > --- a/src/egl/drivers/dri2/platform_android.c > +++ b/src/egl/drivers/dri2/platform_android.c > @@ -629,7 +629,7 @@ droid_create_image_from_prime_fd_yuv(_EGLDisplay > *disp, _EGLContext *ctx, > * so they can be interpreted as offsets. */ > offsets[0] = (size_t)ycbcr.y; > /* We assume here that all the planes are located in one DMA-buf. */ > - is_ycrcb = (size_t)ycbcr.cb < (size_t)ycbcr.cr; > + is_ycrcb = (size_t)ycbcr.cr < (size_t)ycbcr.cb; > if (is_ycrcb) { > offsets[1] = (size_t)ycbcr.cr; > offsets[2] = (size_t)ycbcr.cb; > -- > 2.9.3 > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev >
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
