diff options
author | Simon Ser <contact@emersion.fr> | 2021-04-06 17:44:31 +0200 |
---|---|---|
committer | Kenny Levinsen <kl@kl.wtf> | 2021-04-29 19:51:57 +0200 |
commit | 218955ce959e92a3c4be27340d8033c2b6710972 (patch) | |
tree | d2f45dbc9764e153c15ffb5e088fe5c162e47989 /backend/drm | |
parent | 7aba881c474edc5b03af013d81e55a18195296c4 (diff) |
backend/drm: remove mode arg from drm_connector_set_mode
All of the information is in wlr_output_state.
Diffstat (limited to 'backend/drm')
-rw-r--r-- | backend/drm/backend.c | 2 | ||||
-rw-r--r-- | backend/drm/drm.c | 46 |
2 files changed, 19 insertions, 29 deletions
diff --git a/backend/drm/backend.c b/backend/drm/backend.c index 5ec709a5..c494898e 100644 --- a/backend/drm/backend.c +++ b/backend/drm/backend.c @@ -128,7 +128,7 @@ static void handle_session_active(struct wl_listener *listener, void *data) { .mode_type = WLR_OUTPUT_STATE_MODE_FIXED, .mode = mode, }; - drm_connector_set_mode(conn, &state, mode); + drm_connector_set_mode(conn, &state); } } else { wlr_log(WLR_INFO, "DRM fd paused"); diff --git a/backend/drm/drm.c b/backend/drm/drm.c index 18e17c29..e71debd8 100644 --- a/backend/drm/drm.c +++ b/backend/drm/drm.c @@ -427,21 +427,6 @@ static bool drm_connector_test(struct wlr_output *output) { return true; } -static struct wlr_output_mode *drm_connector_get_pending_mode( - struct wlr_drm_connector *conn) { - struct wlr_output *output = &conn->output; - - switch (output->pending.mode_type) { - case WLR_OUTPUT_STATE_MODE_FIXED: - return output->pending.mode; - case WLR_OUTPUT_STATE_MODE_CUSTOM:; - drmModeModeInfo mode = {0}; - drm_connector_state_mode(conn, &output->pending, &mode); - return wlr_drm_connector_add_mode(output, &mode); - } - abort(); -} - static bool drm_connector_commit_buffer(struct wlr_output *output) { struct wlr_drm_connector *conn = get_drm_connector_from_output(output); struct wlr_drm_backend *drm = conn->backend; @@ -517,23 +502,21 @@ static bool drm_connector_commit(struct wlr_output *output) { if (output->pending.committed & (WLR_OUTPUT_STATE_MODE | WLR_OUTPUT_STATE_ENABLED)) { - struct wlr_output_mode *wlr_mode = output->current_mode; + struct wlr_output_state state = output->pending; - bool enable = (output->pending.committed & WLR_OUTPUT_STATE_ENABLED) ? - output->pending.enabled : output->enabled; - if (!enable) { - wlr_mode = NULL; - } + if ((state.committed & WLR_OUTPUT_STATE_MODE) && + state.mode_type == WLR_OUTPUT_STATE_MODE_CUSTOM) { + drmModeModeInfo mode = {0}; + drm_connector_state_mode(conn, &output->pending, &mode); - if (output->pending.committed & WLR_OUTPUT_STATE_MODE) { - assert(enable); - wlr_mode = drm_connector_get_pending_mode(conn); - if (wlr_mode == NULL) { + state.mode_type = WLR_OUTPUT_STATE_MODE_FIXED; + state.mode = wlr_drm_connector_add_mode(output, &mode); + if (state.mode == NULL) { return false; } } - if (!drm_connector_set_mode(conn, &output->pending, wlr_mode)) { + if (!drm_connector_set_mode(conn, &state)) { return false; } } else if (output->pending.committed & WLR_OUTPUT_STATE_BUFFER) { @@ -730,15 +713,22 @@ static void attempt_enable_needs_modeset(struct wlr_drm_backend *drm) { .mode_type = WLR_OUTPUT_STATE_MODE_FIXED, .mode = conn->desired_mode, }; - drm_connector_set_mode(conn, &state, conn->desired_mode); + drm_connector_set_mode(conn, &state); } } } bool drm_connector_set_mode(struct wlr_drm_connector *conn, - const struct wlr_output_state *state, struct wlr_output_mode *wlr_mode) { + const struct wlr_output_state *state) { struct wlr_drm_backend *drm = conn->backend; + struct wlr_output_mode *wlr_mode = NULL; + if (drm_connector_state_active(conn, state)) { + assert(state->committed & WLR_OUTPUT_STATE_MODE); + assert(state->mode_type == WLR_OUTPUT_STATE_MODE_FIXED); + wlr_mode = state->mode; + } + conn->desired_enabled = wlr_mode != NULL; conn->desired_mode = wlr_mode; |