Re: [PATCH v15 06/34] compositor-drm: Don't restore original CRTC mode

2018-02-06 Thread Pekka Paalanen
On Tue, 6 Feb 2018 13:53:21 + Daniel Stone wrote: > Hi Pekka, > > On 6 February 2018 at 13:30, Pekka Paalanen wrote: > > On Mon, 5 Feb 2018 18:44:15 + > > Daniel Stone wrote: > >> @@ -4296,7 +4290,8 @@ drm_output_destroy(struct weston_output *base) > >>*/ > >>

Re: [PATCH v15 06/34] compositor-drm: Don't restore original CRTC mode

2018-02-06 Thread Daniel Stone
Hi Pekka, On 6 February 2018 at 13:30, Pekka Paalanen wrote: > On Mon, 5 Feb 2018 18:44:15 + > Daniel Stone wrote: >> @@ -4296,7 +4290,8 @@ drm_output_destroy(struct weston_output *base) >>*/ >> if (output->cursor_plane) >> drm_plane_destr

Re: [PATCH v15 06/34] compositor-drm: Don't restore original CRTC mode

2018-02-06 Thread Pekka Paalanen
On Mon, 5 Feb 2018 18:44:15 + Daniel Stone 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

[PATCH v15 06/34] compositor-drm: Don't restore original CRTC mode

2018-02-05 Thread Daniel Stone
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 Reviewed-by: Pekka Paalanen --- libweston/compositor-drm.c