[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