On Fri, 7 Apr 2017 17:21:19 +0300 Pekka Paalanen <[email protected]> wrote:
> On Tue, 4 Apr 2017 17:54:34 +0100 > Daniel Stone <[email protected]> wrote: > > > We make the differentiation where planes are an abstract framebuffer > > with a position within a CRTC/output, and sprites are special cases of > > planes that are neither the primary (base/framebuffer) nor cursor plane. > > > > drm_sprite, OTOH, contains nothing that's actually specific to sprites, > > and we end up duplicating a lot of code to deal with them, especially > > when we come to use an entirely plane-based interface with atomic > > modesetting. > > > > Rename drm_sprite to drm_plane, to reflect that it's actually generic. > > > > No functional changes. > > > > Signed-off-by: Daniel Stone <[email protected]> > > Reviewed-by: Pekka Paalanen <[email protected]> > > Reviewed-by: Quentin Glidic <[email protected]> > > > > Differential Revision: https://phabricator.freedesktop.org/D1408 > > > > Signed-off-by: Daniel Stone <[email protected]> > > Tags. Tags! > > > --- > > libweston/compositor-drm.c | 203 > > +++++++++++++++++++++++---------------------- > > 1 file changed, 106 insertions(+), 97 deletions(-) > > > > diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c > > index 88975007..8a8dd1c2 100644 > > --- a/libweston/compositor-drm.c > > +++ b/libweston/compositor-drm.c > > > @@ -902,10 +913,8 @@ drm_output_repaint(struct weston_output *output_base, > > .request.sequence = 1, > > }; > > > > - /* XXX: Set output much earlier, so we don't attempt to place > > - * planes on entirely the wrong output. */ > > What a strange commit to remove this comment in. Hi, I dropped this hunk from this patch. Patches 13-16 pushed: 863e66b..08d4edf master -> master Thanks, pq
pgpWuvd1Sv0E9.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
