[PATCH weston v5 00/11] drm: port the drm backend to the new init api
Bryce Harrington
bryce at osg.samsung.com
Wed Apr 13 10:25:04 UTC 2016
In following up on my earlier update of Giulio's drm backend config
patch, I've taken the liberty to try and also integrate a couple of
Benoit's other backend configuration patches into this patchset.
Giulio and Benoit took different approaches in their implementations.
I've attempted to reconcile them so that they are stylistically
consistent, along with incorporating pq's request to have the root
config structure incorporate versioning information. I went through all
the review comments against each of the backend config patches and
incorporated the suggestions people made. The resultant changes have
ended up a bit more extensive than I had expected, and I apologize for
any toe stepping I'm doing but my hope is to move the collective work
closer to being landable.
---
todo:
- Use backend-specific header #defines for struct_version values
v5:
- Add missing compositor-drm.h
- Integrate wayland, x11, and headless backend patches
- Implement struct versioning for wayland, x11, and headless backend
patches in the weston_backend_config structure, as requested by pq
- Drop bzero usage as suggested in review by Jan Engelhardt
- Don't change 'backend_init' entry point function name, as suggested
in review by Giulio
- Prefer use of load_backend_new() for actual module loading, as
suggested in Giulio's review of the x11 backend patch
- Refactor all backend initialization code paths and code style for
consistency.
- Squashed drm config struct versioning with drm patch; remainder moved
to a prerequisite patch.
v4:
- Update to current trunk
- Add missing param doc for mode in drm_output_choose_initial_mode
- Rebase to account for code changes by 91880f1e to make vt
switching configurable.
Benoit Gschwind (1):
x11: port the x11 backend to the new init api
Bryce Harrington (9):
Revert "main: Remove unused function load_backend_new()"
compositor: Version the backend configuration structures
drm: Fix gcc warning about missing braces.
compositor: Document refs for alternatives/assumptions for backend
configs
headless: port the headless backend to the new init api
drm: Code and comments reformatting for consistency with other backend
configs
drm: Don't hang onto the backend config object post-backend_init
Enforce destruction of all backend config objects after initialization
drm: Drop use of drm_config config wrapper
Giulio Camuffo (1):
drm: port the drm backend to the new init api
Makefile.am | 5 +
src/compositor-drm.c | 216 ++++++++++++++------------------
src/compositor-drm.h | 125 +++++++++++++++++++
src/compositor-headless.c | 69 +++++------
src/compositor-headless.h | 51 ++++++++
src/compositor-x11.c | 157 ++++++++----------------
src/compositor-x11.h | 60 +++++++++
src/compositor.h | 24 +++-
src/main.c | 304 +++++++++++++++++++++++++++++++++++++++++++++-
9 files changed, 734 insertions(+), 277 deletions(-)
create mode 100644 src/compositor-drm.h
create mode 100644 src/compositor-headless.h
create mode 100644 src/compositor-x11.h
--
1.9.1
More information about the wayland-devel
mailing list