On Wed, 2018-06-27 at 10:41 +0100, Daniel Stone wrote: > Hi, > > On Wed, 27 Jun 2018 at 00:35, Keith Packard <kei...@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 <dani...@collabora.com>
Added such comment, and merged the series: remote: I: patch #232471 updated using rev 4a11f66e4690f9e94dc61cb264f5ef78dbdb255a. remote: E: failed to find patch for rev d83efc47b7a524b4f8d4a993c27a3e402a98fa7c. remote: I: 1 patch(es) updated to state Accepted. To ssh://git.freedesktop.org/git/xorg/xserver 2faf4cef8b..d83efc47b7 master -> master - ajax _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
