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

Daniel Stone daniel at fooishbar.org
Tue Feb 6 13:53:21 UTC 2018


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?

Cheers,
Daniel


More information about the wayland-devel mailing list