diff options
| author | Daniel Stone <daniels@collabora.com> | 2016-10-24 13:26:42 +0100 |
|---|---|---|
| committer | Daniel Stone <daniels@collabora.com> | 2017-08-28 20:07:13 +0100 |
| commit | 4202e50fc840101f606587e94182d84014b1fbc3 (patch) | |
| tree | efcd67068673c8ca77db9327bfeb01ff3aa0930a /libweston | |
| parent | ec27271492cc3ad86db31436f5aa38032f99a16f (diff) | |
compositor-drm: Rename fb_plane to scanout_plane
All planes being displayed have a framebuffer. What makes 'fb_plane'
special is that it's being displayed as the primary plane by KMS.
Previous patchsets renamed this to 'primary_plane' to match the KMS
terminology, namely the CRTC's base plane, which is controlled by
drmModeSetCrtc in the legacy API, and identified by PLANE_TYPE ==
"Primary" in the universal-plane API.
However, Weston uses 'primary_plane' internally to refer to the case
where client content is _not_ directly displayed on a plane, but
composited via the renderer, with the result of the compositing then
shown.
Rename to 'scanout_plane' as our least-ambiguous name, and document it a
bit.
Signed-off-by: Daniel Stone <daniels@collabora.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Emre Ucan <eucan@de.adit-jv.com>
Diffstat (limited to 'libweston')
| -rw-r--r-- | libweston/compositor-drm.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c index decd586f..b707a01f 100644 --- a/libweston/compositor-drm.c +++ b/libweston/compositor-drm.c @@ -235,7 +235,8 @@ struct drm_output { struct gbm_surface *gbm_surface; uint32_t gbm_format; - struct weston_plane fb_plane; + /* Plane for a fullscreen direct scanout view */ + struct weston_plane scanout_plane; /* The last framebuffer submitted to the kernel for this CRTC. */ struct drm_fb *fb_current; @@ -721,7 +722,7 @@ drm_output_prepare_scanout_view(struct drm_output *output, drm_fb_set_buffer(output->fb_pending, buffer); - return &output->fb_plane; + return &output->scanout_plane; } static struct drm_fb * @@ -2784,10 +2785,10 @@ drm_output_enable(struct weston_output *base) weston_plane_init(&output->cursor_plane, b->compositor, INT32_MIN, INT32_MIN); - weston_plane_init(&output->fb_plane, b->compositor, 0, 0); + weston_plane_init(&output->scanout_plane, b->compositor, 0, 0); weston_compositor_stack_plane(b->compositor, &output->cursor_plane, NULL); - weston_compositor_stack_plane(b->compositor, &output->fb_plane, + weston_compositor_stack_plane(b->compositor, &output->scanout_plane, &b->compositor->primary_plane); weston_log("Output %s, (connector %d, crtc %d)\n", @@ -2830,7 +2831,7 @@ drm_output_deinit(struct weston_output *base) else drm_output_fini_egl(output); - weston_plane_release(&output->fb_plane); + weston_plane_release(&output->scanout_plane); weston_plane_release(&output->cursor_plane); drmModeFreeProperty(output->dpms_prop); |