egl/gbm: Fix config validation
authorRobin Murphy <robin.murphy@arm.com>
Sat, 21 Sep 2019 17:07:28 +0000 (18:07 +0100)
committerEric Engestrom <eric.engestrom@intel.com>
Tue, 1 Oct 2019 13:45:15 +0000 (14:45 +0100)
commit563f8974d81c3716538036781df3ecbf7e7686c0
treef945ce890275feb8a26ec14ee8fe3ad35285b85d
parent4943c89d6d3f522b3100be9187ff045f84f032b1
egl/gbm: Fix config validation

In converting to shift/size-based validation, we lost a condition from
the ARGB/XRGB equivalence check, which left it working one way round
but not the other, and broke applications like glmark2-es2-drm on some
platforms. Restore the equivalent check that *both* configs actually
have an alpha channel before considering a mismatch.

Fixes: 7b4ed2b513ef ("egl: Convert configs to use shifts and sizes instead of masks")
Signed-off-by: Robin Murphy <robin.murphy@arm.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
src/egl/drivers/dri2/platform_drm.c