On Monday, 2018-03-19 17:45:19 +0100, Guido Günther wrote:
> Use stride instead of width for buffer calculation.
> 
> Signed-off-by: Guido Günther <[email protected]>
> ---
>  clients/simple-dmabuf-drm.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/clients/simple-dmabuf-drm.c b/clients/simple-dmabuf-drm.c
> index efe3b7f5..a184d7e7 100644
> --- a/clients/simple-dmabuf-drm.c
> +++ b/clients/simple-dmabuf-drm.c
> @@ -222,13 +222,15 @@ fd_alloc_bo(struct buffer *buf)
>  {
>       int flags = DRM_FREEDRENO_GEM_CACHE_WCOMBINE;
>       int size = buf->width * buf->height * buf->bpp / 8;

You forgot to remove the initialisation here ^

> -     buf->fd_dev = fd_device_new(buf->drm_fd);
>  
> +     buf->fd_dev = fd_device_new(buf->drm_fd);
> +     buf->stride = ALIGN(buf->width, 32) * buf->bpp / 8;
> +     size = buf->stride * buf->height;
> +     buf->fd_dev = fd_device_new(buf->drm_fd);
>       buf->fd_bo = fd_bo_new(buf->fd_dev, size, flags);
>  
>       if (!buf->fd_bo)
>               return 0;
> -     buf->stride = ALIGN(buf->width, 32) * buf->bpp / 8;
>       return 1;
>  }
>  
> -- 
> 2.16.1
> 
> _______________________________________________
> wayland-devel mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to