On 09.12.2016 20:57, Daniel Stone wrote:
> Everyone else uses fb->fd rather than pulling the FD back out of GBM.
> Use that in the destroy callback too.
> 
> Signed-off-by: Daniel Stone <dani...@collabora.com>
> 

Makes sense.

Reviewed-by: Armin Krezović <krezovic.ar...@gmail.com>

> Differential Revision: https://phabricator.freedesktop.org/D1406
> ---
>  libweston/compositor-drm.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c
> index 3276ed0..a9bde0c 100644
> --- a/libweston/compositor-drm.c
> +++ b/libweston/compositor-drm.c
> @@ -243,10 +243,9 @@ static void
>  drm_fb_destroy_callback(struct gbm_bo *bo, void *data)
>  {
>       struct drm_fb *fb = data;
> -     struct gbm_device *gbm = gbm_bo_get_device(bo);
>  
>       if (fb->fb_id)
> -             drmModeRmFB(gbm_device_get_fd(gbm), fb->fb_id);
> +             drmModeRmFB(fb->fd, fb->fb_id);
>  
>       weston_buffer_reference(&fb->buffer_ref, NULL);
>  
> 


Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to