On Mon,  5 Feb 2018 18:44:15 +0000
Daniel Stone <dani...@collabora.com> wrote:

> When leaving Weston, don't attempt to restore the previous CRTC
> settings. The framebuffer may well have disappeared, and in every
> likelihood, whoever gets the KMS device afterwards will be repainting
> anyway.
> 
> Signed-off-by: Daniel Stone <dani...@collabora.com>
> Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk>
> ---
>  libweston/compositor-drm.c | 31 ++++++++++++-------------------
>  1 file changed, 12 insertions(+), 19 deletions(-)
> 
> diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c
> index 5c3abee39..d57967972 100644
> --- a/libweston/compositor-drm.c
> +++ b/libweston/compositor-drm.c

> @@ -4296,7 +4290,8 @@ drm_output_destroy(struct weston_output *base)
>                */
>               if (output->cursor_plane)
>                       drm_plane_destroy(output->cursor_plane);
> -             drm_plane_destroy(output->scanout_plane);
> +             if (output->scanout_plane)
> +                     drm_plane_destroy(output->scanout_plane);

This hunk belong in the previous patch to make it correct.

>       }
>  
>       wl_list_for_each_safe(drm_mode, next, &output->base.mode_list,

Otherwise R-b stands.


Thanks,
pq

Attachment: pgpq5t3R4yMJl.pgp
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