In preparation for follow-on changes to support frame presentation
through EGLDevice+EGLOutput, this change includes the following:
  - Rename drm_backend::format   to gbm_format
  - Rename drm_output::format    to gbm_format
  - Rename drm_output::surface   to gbm_surface
  - Rename drm_output::cursor_bo to gbm_cursor_bo

Signed-off-by: Miguel A Vico Moya <[email protected]>
Reviewed-by: Andy Ritger <[email protected]>
---
 src/compositor-drm.c | 58 ++++++++++++++++++++++++++--------------------------
 1 file changed, 29 insertions(+), 29 deletions(-)

diff --git a/src/compositor-drm.c b/src/compositor-drm.c
index b3eca88..ecb340d 100644
--- a/src/compositor-drm.c
+++ b/src/compositor-drm.c
@@ -107,7 +107,7 @@ struct drm_backend {
        uint32_t crtc_allocator;
        uint32_t connector_allocator;
        struct wl_listener session_listener;
-       uint32_t format;
+       uint32_t gbm_format;
 
        /* we need these parameters in order to not fail drmModeAddFB2()
         * due to out of bounds dimensions, and then mistakenly set
@@ -170,7 +170,7 @@ struct drm_output {
        drmModeCrtcPtr original_crtc;
        struct drm_edid edid;
        drmModePropertyPtr dpms_prop;
-       uint32_t format;
+       uint32_t gbm_format;
 
        enum dpms_enum dpms;
 
@@ -178,8 +178,8 @@ struct drm_output {
        int page_flip_pending;
        int destroy_pending;
 
-       struct gbm_surface *surface;
-       struct gbm_bo *cursor_bo[2];
+       struct gbm_surface *gbm_surface;
+       struct gbm_bo *gbm_cursor_bo[2];
        struct weston_plane cursor_plane;
        struct weston_plane fb_plane;
        struct weston_view *cursor_view;
@@ -439,7 +439,7 @@ drm_output_release_fb(struct drm_output *output, struct 
drm_fb *fb)
                if (fb->is_client_buffer)
                        gbm_bo_destroy(fb->bo);
                else
-                       gbm_surface_release_buffer(output->surface,
+                       gbm_surface_release_buffer(output->gbm_surface,
                                                   fb->bo);
        }
 }
@@ -468,7 +468,7 @@ drm_output_check_scanout_format(struct drm_output *output,
                pixman_region32_fini(&r);
        }
 
-       if (output->format == format)
+       if (output->gbm_format == format)
                return format;
 
        return 0;
@@ -531,16 +531,16 @@ drm_output_render_gl(struct drm_output *output, 
pixman_region32_t *damage)
        output->base.compositor->renderer->repaint_output(&output->base,
                                                          damage);
 
-       bo = gbm_surface_lock_front_buffer(output->surface);
+       bo = gbm_surface_lock_front_buffer(output->gbm_surface);
        if (!bo) {
                weston_log("failed to lock front buffer: %m\n");
                return;
        }
 
-       output->next = drm_fb_get_from_bo(bo, b, output->format);
+       output->next = drm_fb_get_from_bo(bo, b, output->gbm_format);
        if (!output->next) {
                weston_log("failed to get drm_fb for bo\n");
-               gbm_surface_release_buffer(output->surface, bo);
+               gbm_surface_release_buffer(output->gbm_surface, bo);
                return;
        }
 }
@@ -1210,7 +1210,7 @@ drm_output_set_cursor(struct drm_output *output)
                pixman_region32_fini(&output->cursor_plane.damage);
                pixman_region32_init(&output->cursor_plane.damage);
                output->current_cursor ^= 1;
-               bo = output->cursor_bo[output->current_cursor];
+               bo = output->gbm_cursor_bo[output->current_cursor];
 
                cursor_bo_update(b, bo, ev);
                handle = gbm_bo_get_handle(bo).s32;
@@ -1362,7 +1362,7 @@ drm_output_destroy(struct weston_output *output_base)
                drm_output_fini_pixman(output);
        } else {
                gl_renderer->output_destroy(output_base);
-               gbm_surface_destroy(output->surface);
+               gbm_surface_destroy(output->gbm_surface);
        }
 
        weston_plane_release(&output->fb_plane);
@@ -1463,7 +1463,7 @@ drm_output_switch_mode(struct weston_output *output_base, 
struct weston_mode *mo
                }
        } else {
                gl_renderer->output_destroy(&output->base);
-               gbm_surface_destroy(output->surface);
+               gbm_surface_destroy(output->gbm_surface);
 
                if (drm_output_init_egl(output, b) < 0) {
                        weston_log("failed to init output egl state with "
@@ -1595,8 +1595,8 @@ static int
 drm_backend_create_gl_renderer(struct drm_backend *b)
 {
        EGLint format[3] = {
-               b->format,
-               fallback_format_for(b->format),
+               b->gbm_format,
+               fallback_format_for(b->gbm_format),
                0,
        };
        int n_formats = 2;
@@ -1853,18 +1853,18 @@ static int
 drm_output_init_egl(struct drm_output *output, struct drm_backend *b)
 {
        EGLint format[2] = {
-               output->format,
-               fallback_format_for(output->format),
+               output->gbm_format,
+               fallback_format_for(output->gbm_format),
        };
        int i, flags, n_formats = 1;
 
-       output->surface = gbm_surface_create(b->gbm,
+       output->gbm_surface = gbm_surface_create(b->gbm,
                                             output->base.current_mode->width,
                                             output->base.current_mode->height,
                                             format[0],
                                             GBM_BO_USE_SCANOUT |
                                             GBM_BO_USE_RENDERING);
-       if (!output->surface) {
+       if (!output->gbm_surface) {
                weston_log("failed to create gbm surface\n");
                return -1;
        }
@@ -1872,28 +1872,28 @@ drm_output_init_egl(struct drm_output *output, struct 
drm_backend *b)
        if (format[1])
                n_formats = 2;
        if (gl_renderer->output_window_create(&output->base,
-                                      (EGLNativeWindowType)output->surface,
-                                      output->surface,
+                                      (EGLNativeWindowType)output->gbm_surface,
+                                      output->gbm_surface,
                                       gl_renderer->opaque_attribs,
                                       format,
                                       n_formats) < 0) {
                weston_log("failed to create gl renderer output state\n");
-               gbm_surface_destroy(output->surface);
+               gbm_surface_destroy(output->gbm_surface);
                return -1;
        }
 
        flags = GBM_BO_USE_CURSOR | GBM_BO_USE_WRITE;
 
        for (i = 0; i < 2; i++) {
-               if (output->cursor_bo[i])
+               if (output->gbm_cursor_bo[i])
                        continue;
 
-               output->cursor_bo[i] =
+               output->gbm_cursor_bo[i] =
                        gbm_bo_create(b->gbm, b->cursor_width, b->cursor_height,
                                GBM_FORMAT_ARGB8888, flags);
        }
 
-       if (output->cursor_bo[0] == NULL || output->cursor_bo[1] == NULL) {
+       if (output->gbm_cursor_bo[0] == NULL || output->gbm_cursor_bo[1] == 
NULL) {
                weston_log("cursor buffers unavailable, using gl cursors\n");
                b->cursors_are_broken = 1;
        }
@@ -2375,9 +2375,9 @@ create_output_for_connector(struct drm_backend *b,
        free(s);
 
        if (get_gbm_format_from_section(section,
-                                       b->format,
-                                       &output->format) == -1)
-               output->format = b->format;
+                                       b->gbm_format,
+                                       &output->gbm_format) == -1)
+               output->gbm_format = b->gbm_format;
 
        weston_config_section_get_string(section, "seat", &s, "");
        setup_output_seat_constraint(b, &output->base, s);
@@ -2992,7 +2992,7 @@ recorder_binding(struct weston_keyboard *keyboard, 
uint32_t time, uint32_t key,
                              struct drm_output, base.link);
 
        if (!output->recorder) {
-               if (output->format != GBM_FORMAT_XRGB8888) {
+               if (output->gbm_format != GBM_FORMAT_XRGB8888) {
                        weston_log("failed to start vaapi recorder: "
                                   "output format not supported\n");
                        return;
@@ -3118,7 +3118,7 @@ drm_backend_create(struct weston_compositor *compositor,
        section = weston_config_get_section(config, "core", NULL, NULL);
        if (get_gbm_format_from_section(section,
                                        GBM_FORMAT_XRGB8888,
-                                       &b->format) == -1)
+                                       &b->gbm_format) == -1)
                goto err_base;
 
        b->use_pixman = param->use_pixman;
-- 
2.7.1

_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to