[PATCH xserver] xf86-video-modesetting: Lease planes as well if using atomic

Daniel Stone daniel at fooishbar.org
Wed Jun 27 09:41:54 UTC 2018


Hi,

On Wed, 27 Jun 2018 at 00:35, Keith Packard <keithp at keithp.com> wrote:
> @@ -3251,6 +3251,9 @@ drmmode_create_lease(RRLeasePtr lease, int *fd)
>
>      nobjects = ncrtc + noutput;
>
> +    if (ms->atomic_modeset)
> +        nobjects += ncrtc;

This seems like it definitely wants a comment as to why we add ncrtc
twice (for the primary plane). But, with that (and no need to pass it
through the list again):
Reviewed-by: Daniel Stone <daniels at collabora.com>

Cheers,
Daniel


More information about the xorg-devel mailing list