[Mesa-dev] [PATCH] egl/dri: fix error value with unknown drm format

[Tapani Pälli]

matches the spec

Reviewed-by: Tapani Pälli <tapani.palli at intel.com>

On 11/13/18 4:10 PM, Lionel Landwerlin wrote:
> According to the EGL_EXT_image_dma_buf_import spec, creating an EGL
> image with a DRM format not supported should yield the BAD_MATCH
> error :
> 
> "
>         * If <target> is EGL_LINUX_DMA_BUF_EXT, and the EGL_LINUX_DRM_FOURCC_EXT
>           attribute is set to a format not supported by the EGL, EGL_BAD_MATCH
>           is generated.
> "
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> Fixes: 20de7f9f226401 ("egl/dri2: support for creating images out of dma buffers")
> ---
>   src/egl/drivers/dri2/egl_dri2.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
> index 3b63aebbf9a..198ba73247f 100644
> --- a/src/egl/drivers/dri2/egl_dri2.c
> +++ b/src/egl/drivers/dri2/egl_dri2.c
> @@ -2310,7 +2310,7 @@ dri2_check_dma_buf_format(const _EGLImageAttribs *attrs)
>   {
>      unsigned plane_n = dri2_num_fourcc_format_planes(attrs->DMABufFourCC.Value);
>      if (plane_n == 0) {
> -      _eglError(EGL_BAD_ATTRIBUTE, "invalid format");
> +      _eglError(EGL_BAD_MATCH, "unknown drm fourcc format");
>         return 0;
>      }
>   
>