[PATCH] compositor-drm: fix EGL format type

Kristian Høgsberg hoegsberg at gmail.com
Fri Oct 11 13:40:18 PDT 2013


On Wed, Oct 02, 2013 at 05:51:05PM +0100, Alex DAMIAN wrote:
> From: Alexandru DAMIAN <alexandru.damian at intel.com>
> 
> EGLInt is not always uint32_t so we need
> to make sure we use the right int size for the format.
> 
> Signed-off-by: Alexandru DAMIAN <alexandru.damian at intel.com>
> ---
>  src/compositor-drm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I changed this a bit to just use a temp EGLint in init_egl().  I
prefer that we store the GBM format code in it's native type.

thanks,
Kristian

> diff --git a/src/compositor-drm.c b/src/compositor-drm.c
> index 2770c85..9228f85 100644
> --- a/src/compositor-drm.c
> +++ b/src/compositor-drm.c
> @@ -85,7 +85,7 @@ struct drm_compositor {
>  	uint32_t crtc_allocator;
>  	uint32_t connector_allocator;
>  	struct wl_listener session_listener;
> -	uint32_t format;
> +	EGLint format;
>  
>  	/* we need these parameters in order to not fail drmModeAddFB2()
>  	 * due to out of bounds dimensions, and then mistakenly set
> -- 
> 1.8.1.2
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list