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

Pekka Paalanen ppaalanen at gmail.com
Tue Feb 6 14:01:43 UTC 2018


On Tue, 6 Feb 2018 13:53:21 +0000
Daniel Stone <daniel at fooishbar.org> wrote:

> Hi Pekka,
> 
> On 6 February 2018 at 13:30, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> > On Mon,  5 Feb 2018 18:44:15 +0000
> > Daniel Stone <daniels at collabora.com> wrote:  
> >> @@ -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.  
> 
> Yes, fixup landed in the wrong patch. Thanks for catching that. Is
> 'otherwise looks ok' on the previous patch R-b with this hunk moved
> back?

Yes, it is!


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20180206/6de61f99/attachment.sig>


More information about the wayland-devel mailing list