[RFC] compositor: remove OpenWFD backend

Kristian Høgsberg hoegsberg at gmail.com
Wed Aug 1 07:09:32 PDT 2012


On Wed, Aug 01, 2012 at 03:56:45PM +0200, Benjamin Franzke wrote:
> Hi Pekka,
> 
> I agree with the removal, if that helps to speed up development.
> I have not the time to fix it right now, and as far as i know no one
> else is intrested.
> Though i think this shouldn't mean, that no one may reintroduce it at
> some point.
> 
> Acked-by: Benjamin Franzke <benjaminfranzke at googlemail.com>

Thanks Benjamin.  If you or somebody else are willing to maintain it
again, we can certainly pull it out of git put it back in.

Kristian

> 2012/8/1 Pekka Paalanen <ppaalanen at gmail.com>:
> > This backend has not seen even build testing for months, presumably does
> > not even compile, and is starting to hinder development a little.
> >
> > Signed-off-by: Pekka Paalanen <ppaalanen at gmail.com>
> > Cc: Benjamin Franzke <benjaminfranzke at googlemail.com>
> > ---
> >
> > Hi,
> >
> > does anyone care enough about this backend to step up and make sure it
> > works, and fix what is needed?
> >
> > I am asking, because I am working on evdev input for the Android
> > backend, and need to separate the udev stuff away from evdev.c.
> > The current proposition is to move that code into compositor-drm.c,
> > and that would certainly break the openwfd backend.
> >
> > Thanks,
> > pq
> >
> >  configure.ac             |    9 -
> >  src/Makefile.am          |   11 +-
> >  src/compositor-openwfd.c |  687 ----------------------------------------------
> >  src/compositor.c         |    2 -
> >  4 files changed, 1 insertions(+), 708 deletions(-)
> >  delete mode 100644 src/compositor-openwfd.c
> >
> > diff --git a/configure.ac b/configure.ac
> > index 4e44eb6..e67ad50 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -97,15 +97,6 @@ if test x$enable_drm_compositor = xyes; then
> >  fi
> >
> >
> > -AC_ARG_ENABLE(openwfd-compositor, [  --enable-openwfd-compositor],,
> > -             enable_openwfd_compositor=no)
> > -AM_CONDITIONAL(ENABLE_OPENWFD_COMPOSITOR, test x$enable_openwfd_compositor = xyes)
> > -if test x$enable_openwfd_compositor = xyes; then
> > -  AC_DEFINE([BUILD_OPENWFD_COMPOSITOR], [1], [Build the OpenWF compositor])
> > -  PKG_CHECK_MODULES(OPENWFD_COMPOSITOR, [openwfd gbm])
> > -fi
> > -
> > -
> >  AC_ARG_ENABLE(wayland-compositor, [  --enable-wayland-compositor],,
> >               enable_wayland_compositor=yes)
> >  AM_CONDITIONAL(ENABLE_WAYLAND_COMPOSITOR,
> > diff --git a/src/Makefile.am b/src/Makefile.am
> > index f7cb90c..03f610e 100644
> > --- a/src/Makefile.am
> > +++ b/src/Makefile.am
> > @@ -70,8 +70,7 @@ module_LTLIBRARIES =                          \
> >         $(tablet_shell)                         \
> >         $(x11_backend)                          \
> >         $(drm_backend)                          \
> > -       $(wayland_backend)                      \
> > -       $(openwfd_backend)
> > +       $(wayland_backend)
> >
> >  # Do not install, since the binary produced via autotools is unusable.
> >  # The real backend is built by the Android build system.
> > @@ -123,14 +122,6 @@ wayland_backend_la_CFLAGS =                        \
> >  wayland_backend_la_SOURCES = compositor-wayland.c
> >  endif
> >
> > -if ENABLE_OPENWFD_COMPOSITOR
> > -openwfd_backend = openwfd-backend.la
> > -openwfd_backend_la_LDFLAGS = -module -avoid-version
> > -openwfd_backend_la_LIBADD = $(COMPOSITOR_LIBS) $(OPENWFD_COMPOSITOR_LIBS)
> > -openwfd_backend_la_CFLAGS = $(OPENWFD_COMPOSITOR_CFLAGS) $(GCC_CFLAGS)
> > -openwfd_backend_la_SOURCES = compositor-openwfd.c tty.c evdev.c evdev.h
> > -endif
> > -
> >  if ENABLE_ANDROID_COMPOSITOR
> >  android_backend = android-backend.la
> >  android_backend_la_LDFLAGS = -module -avoid-version
> > diff --git a/src/compositor-openwfd.c b/src/compositor-openwfd.c
> > deleted file mode 100644
> > index 38bdd35..0000000
> > --- a/src/compositor-openwfd.c
> > +++ /dev/null
> > @@ -1,687 +0,0 @@
> > -/*
> > - * Copyright © 2011 Benjamin Franzke
> > - * Copyright © 2011 Intel Corporation
> > - *
> > - * Permission to use, copy, modify, distribute, and sell this software and
> > - * its documentation for any purpose is hereby granted without fee, provided
> > - * that the above copyright notice appear in all copies and that both that
> > - * copyright notice and this permission notice appear in supporting
> > - * documentation, and that the name of the copyright holders not be used in
> > - * advertising or publicity pertaining to distribution of the software
> > - * without specific, written prior permission.  The copyright holders make
> > - * no representations about the suitability of this software for any
> > - * purpose.  It is provided "as is" without express or implied warranty.
> > - *
> > - * THE COPYRIGHT HOLDERS DISCLAIM ALL WARRANTIES WITH REGARD TO THIS
> > - * SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND
> > - * FITNESS, IN NO EVENT SHALL THE COPYRIGHT HOLDERS BE LIABLE FOR ANY
> > - * SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER
> > - * RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF
> > - * CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN
> > - * CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
> > - */
> > -
> > -#define _GNU_SOURCE
> > -
> > -#include <stdlib.h>
> > -#include <string.h>
> > -
> > -#include <sys/time.h>
> > -
> > -#include <WF/wfd.h>
> > -#include <WF/wfdext.h>
> > -
> > -#include <gbm.h>
> > -
> > -#include "compositor.h"
> > -#include "evdev.h"
> > -#include "log.h"
> > -
> > -struct wfd_compositor {
> > -       struct weston_compositor base;
> > -
> > -       struct udev *udev;
> > -       struct gbm_device *gbm;
> > -       WFDDevice dev;
> > -
> > -       WFDEvent event;
> > -       int wfd_fd;
> > -       struct wl_event_source *wfd_source;
> > -
> > -       struct tty *tty;
> > -
> > -       uint32_t start_time;
> > -       uint32_t used_pipelines;
> > -};
> > -
> > -struct wfd_mode {
> > -       struct weston_mode base;
> > -       WFDPortMode mode;
> > -};
> > -
> > -struct wfd_output {
> > -       struct weston_output   base;
> > -
> > -       WFDPort port;
> > -
> > -       WFDPipeline pipeline;
> > -       WFDint pipeline_id;
> > -
> > -       WFDPortMode mode;
> > -       WFDSource source[2];
> > -
> > -       struct gbm_bo *bo[2];
> > -       EGLImageKHR image[2];
> > -       GLuint rbo[2];
> > -       uint32_t current;
> > -};
> > -
> > -union wfd_geometry {
> > -       struct {
> > -               WFDint x, y;
> > -               WFDint width, height;
> > -       } g;
> > -
> > -       WFDint array[4];
> > -};
> > -
> > -static void
> > -wfd_output_repaint(struct weston_output *output_base)
> > -{
> > -       struct wfd_output *output = (struct wfd_output *) output_base;
> > -       struct weston_surface *surface;
> > -       struct wfd_compositor *compositor =
> > -               (struct wfd_compositor *) output->base.compositor;
> > -
> > -       glFramebufferRenderbuffer(GL_FRAMEBUFFER,
> > -                                 GL_COLOR_ATTACHMENT0,
> > -                                 GL_RENDERBUFFER,
> > -                                 output->rbo[output->current]);
> > -
> > -       if (glCheckFramebufferStatus(GL_FRAMEBUFFER) != GL_FRAMEBUFFER_COMPLETE)
> > -               return -1;
> > -
> > -       wl_list_for_each_reverse(surface, &compositor->surface_list, link)
> > -               weston_surface_draw(surface, output);
> > -
> > -       glFlush();
> > -
> > -       output->current ^= 1;
> > -
> > -       wfdBindSourceToPipeline(compositor->dev, output->pipeline,
> > -                               output->source[output->current ^ 1],
> > -                               WFD_TRANSITION_AT_VSYNC, NULL);
> > -
> > -       wfdDeviceCommit(compositor->dev,
> > -                       WFD_COMMIT_PIPELINE, output->pipeline);
> > -
> > -       return 0;
> > -}
> > -
> > -static int
> > -init_egl(struct wfd_compositor *ec)
> > -{
> > -       EGLint major, minor;
> > -       const char *extensions;
> > -       int fd;
> > -       static const EGLint context_attribs[] = {
> > -               EGL_CONTEXT_CLIENT_VERSION, 2,
> > -               EGL_NONE
> > -       };
> > -
> > -       fd = wfdGetDeviceAttribi(ec->dev, WFD_DEVICE_ID);
> > -       if (fd < 0)
> > -               return -1;
> > -
> > -       ec->wfd_fd = fd;
> > -       ec->gbm = gbm_create_device(ec->wfd_fd);
> > -       ec->base.egl_display = eglGetDisplay(ec->gbm);
> > -       if (ec->base.egl_display == NULL) {
> > -               weston_log("failed to create display\n");
> > -               return -1;
> > -       }
> > -
> > -       if (!eglInitialize(ec->base.egl_display, &major, &minor)) {
> > -               weston_log("failed to initialize display\n");
> > -               return -1;
> > -       }
> > -
> > -       extensions = eglQueryString(ec->base.egl_display, EGL_EXTENSIONS);
> > -       if (!strstr(extensions, "EGL_KHR_surfaceless_gles2")) {
> > -               weston_log("EGL_KHR_surfaceless_gles2 not available\n");
> > -               return -1;
> > -       }
> > -
> > -       if (!eglBindAPI(EGL_OPENGL_ES_API)) {
> > -               weston_log("failed to bind api EGL_OPENGL_ES_API\n");
> > -               return -1;
> > -       }
> > -
> > -       ec->base.egl_context =
> > -               eglCreateContext(ec->base.egl_display, NULL,
> > -                                EGL_NO_CONTEXT, context_attribs);
> > -       if (ec->base.egl_context == NULL) {
> > -               weston_log("failed to create context\n");
> > -               return -1;
> > -       }
> > -
> > -       if (!eglMakeCurrent(ec->base.egl_display, EGL_NO_SURFACE,
> > -                           EGL_NO_SURFACE, ec->base.egl_context)) {
> > -               weston_log("failed to make context current\n");
> > -               return -1;
> > -       }
> > -
> > -       return 0;
> > -}
> > -
> > -static int
> > -wfd_output_prepare_scanout_surface(struct weston_output *output_base,
> > -                                  struct weston_surface *es)
> > -{
> > -       return -1;
> > -}
> > -
> > -static int
> > -wfd_output_set_cursor(struct weston_output *output_base,
> > -                     struct weston_input_device *input)
> > -{
> > -       return -1;
> > -}
> > -
> > -static void
> > -wfd_output_destroy(struct weston_output *output_base)
> > -{
> > -       struct wfd_output *output = (struct wfd_output *) output_base;
> > -       struct wfd_compositor *ec =
> > -               (struct wfd_compositor *) output->base.compositor;
> > -       int i;
> > -
> > -       glFramebufferRenderbuffer(GL_FRAMEBUFFER,
> > -                                 GL_COLOR_ATTACHMENT0,
> > -                                 GL_RENDERBUFFER,
> > -                                 0);
> > -
> > -       glBindRenderbuffer(GL_RENDERBUFFER, 0);
> > -       glDeleteRenderbuffers(2, output->rbo);
> > -
> > -       for (i = 0; i < 2; i++) {
> > -               ec->base.destroy_image(ec->base.egl_display, output->image[i]);
> > -               gbm_bo_destroy(output->bo[i]);
> > -               wfdDestroySource(ec->dev, output->source[i]);
> > -       }
> > -
> > -       ec->used_pipelines &= ~(1 << output->pipeline_id);
> > -       wfdDestroyPipeline(ec->dev, output->pipeline);
> > -       wfdDestroyPort(ec->dev, output->port);
> > -
> > -       weston_output_destroy(&output->base);
> > -       wl_list_remove(&output->base.link);
> > -
> > -       free(output);
> > -}
> > -
> > -static int
> > -wfd_output_add_mode(struct wfd_output *output, WFDPortMode mode)
> > -{
> > -       struct wfd_compositor *ec =
> > -               (struct wfd_compositor *) output->base.compositor;
> > -       struct wfd_mode *wmode;
> > -
> > -       wmode = malloc(sizeof *wmode);
> > -       if (wmode == NULL)
> > -               return -1;
> > -
> > -       wmode->base.flags = 0;
> > -       wmode->base.width = wfdGetPortModeAttribi(ec->dev, output->port, mode,
> > -                                                 WFD_PORT_MODE_WIDTH);
> > -       wmode->base.height = wfdGetPortModeAttribi(ec->dev, output->port, mode,
> > -                                                  WFD_PORT_MODE_HEIGHT);
> > -       wmode->base.refresh = wfdGetPortModeAttribi(ec->dev, output->port, mode,
> > -                                                   WFD_PORT_MODE_REFRESH_RATE);
> > -       wmode->mode = mode;
> > -       wl_list_insert(output->base.mode_list.prev, &wmode->base.link);
> > -
> > -       return 0;
> > -}
> > -
> > -static int
> > -create_output_for_port(struct wfd_compositor *ec,
> > -                      WFDHandle port,
> > -                      int x, int y)
> > -{
> > -       struct wfd_output *output;
> > -       int i;
> > -       WFDint num_pipelines, *pipelines;
> > -       WFDint num_modes;
> > -       union wfd_geometry geometry;
> > -       struct wfd_mode *mode;
> > -       WFDPortMode *modes;
> > -       WFDfloat physical_size[2];
> > -
> > -       output = malloc(sizeof *output);
> > -       if (output == NULL)
> > -               return -1;
> > -
> > -       memset(output, 0, sizeof *output);
> > -       memset(&geometry, 0, sizeof geometry);
> > -
> > -       output->port = port;
> > -       wl_list_init(&output->base.mode_list);
> > -
> > -       wfdSetPortAttribi(ec->dev, output->port,
> > -                         WFD_PORT_POWER_MODE, WFD_POWER_MODE_ON);
> > -
> > -       num_modes = wfdGetPortModes(ec->dev, output->port, NULL, 0);
> > -       if (num_modes < 1) {
> > -               weston_log("failed to get port mode\n");
> > -               goto cleanup_port;
> > -       }
> > -
> > -       modes = malloc(sizeof(WFDPortMode) * num_modes);
> > -       if (modes == NULL)
> > -               goto cleanup_port;
> > -
> > -       output->base.compositor = &ec->base;
> > -       num_modes = wfdGetPortModes(ec->dev, output->port, modes, num_modes);
> > -       for (i = 0; i < num_modes; ++i)
> > -               wfd_output_add_mode(output, modes[i]);
> > -
> > -       free(modes);
> > -
> > -       wfdGetPortAttribiv(ec->dev, output->port,
> > -                          WFD_PORT_NATIVE_RESOLUTION,
> > -                          2, &geometry.array[2]);
> > -
> > -       output->base.current = NULL;
> > -       wl_list_for_each(mode, &output->base.mode_list, base.link) {
> > -               if (mode->base.width == geometry.g.width &&
> > -                   mode->base.height == geometry.g.height) {
> > -                       output->base.current = &mode->base;
> > -                       break;
> > -               }
> > -       }
> > -       if (output->base.current == NULL) {
> > -               weston_log("failed to find a native mode\n");
> > -               goto cleanup_port;
> > -       }
> > -
> > -       mode = (struct wfd_mode *) output->base.current;
> > -       mode->base.flags = WL_OUTPUT_MODE_CURRENT | WL_OUTPUT_MODE_PREFERRED;
> > -
> > -       wfdSetPortMode(ec->dev, output->port, mode->mode);
> > -
> > -       wfdEnumeratePipelines(ec->dev, NULL, 0, NULL);
> > -
> > -       num_pipelines = wfdGetPortAttribi(ec->dev, output->port,
> > -                                         WFD_PORT_PIPELINE_ID_COUNT);
> > -       if (num_pipelines < 1) {
> > -               weston_log("failed to get a bindable pipeline\n");
> > -               goto cleanup_port;
> > -       }
> > -       pipelines = calloc(num_pipelines, sizeof *pipelines);
> > -       if (pipelines == NULL)
> > -               goto cleanup_port;
> > -
> > -       wfdGetPortAttribiv(ec->dev, output->port,
> > -                          WFD_PORT_BINDABLE_PIPELINE_IDS,
> > -                          num_pipelines, pipelines);
> > -
> > -       output->pipeline_id = WFD_INVALID_PIPELINE_ID;
> > -       for (i = 0; i < num_pipelines; ++i) {
> > -               if (!(ec->used_pipelines & (1 << pipelines[i]))) {
> > -                       output->pipeline_id = pipelines[i];
> > -                       break;
> > -               }
> > -       }
> > -       if (output->pipeline_id == WFD_INVALID_PIPELINE_ID) {
> > -               weston_log("no pipeline found for port: %d\n", port);
> > -               goto cleanup_pipelines;
> > -       }
> > -
> > -       ec->used_pipelines |= (1 << output->pipeline_id);
> > -
> > -       wfdGetPortAttribfv(ec->dev, output->port,
> > -                          WFD_PORT_PHYSICAL_SIZE,
> > -                          2, physical_size);
> > -
> > -       weston_output_init(&output->base, &ec->base, x, y,
> > -                        physical_size[0], physical_size[1], 0);
> > -
> > -       output->pipeline = wfdCreatePipeline(ec->dev, output->pipeline_id, NULL);
> > -       if (output->pipeline == WFD_INVALID_HANDLE) {
> > -               weston_log("failed to create a pipeline\n");
> > -               goto cleanup_weston_output;
> > -       }
> > -
> > -       glGenRenderbuffers(2, output->rbo);
> > -       for (i = 0; i < 2; i++) {
> > -               glBindRenderbuffer(GL_RENDERBUFFER, output->rbo[i]);
> > -
> > -               output->bo[i] =
> > -                       gbm_bo_create(ec->gbm,
> > -                                     output->base.current->width,
> > -                                     output->base.current->height,
> > -                                     GBM_BO_FORMAT_XRGB8888,
> > -                                     GBM_BO_USE_SCANOUT | GBM_BO_USE_RENDERING);
> > -               output->image[i] = ec->base.create_image(ec->base.egl_display,
> > -                                                        NULL,
> > -                                                        EGL_NATIVE_PIXMAP_KHR,
> > -                                                        output->bo[i], NULL);
> > -
> > -               weston_log("output->image[i]: %p\n", output->image[i]);
> > -               ec->base.image_target_renderbuffer_storage(GL_RENDERBUFFER,
> > -                                                          output->image[i]);
> > -               output->source[i] =
> > -                       wfdCreateSourceFromImage(ec->dev, output->pipeline,
> > -                                                output->image[i], NULL);
> > -
> > -               if (output->source[i] == WFD_INVALID_HANDLE) {
> > -                       weston_log("failed to create source\n");
> > -                       goto cleanup_pipeline;
> > -               }
> > -       }
> > -
> > -       output->current = 0;
> > -       glFramebufferRenderbuffer(GL_FRAMEBUFFER,
> > -                                 GL_COLOR_ATTACHMENT0,
> > -                                 GL_RENDERBUFFER,
> > -                                 output->rbo[output->current]);
> > -
> > -       wfdSetPipelineAttribiv(ec->dev, output->pipeline,
> > -                              WFD_PIPELINE_SOURCE_RECTANGLE,
> > -                              4, geometry.array);
> > -       wfdSetPipelineAttribiv(ec->dev, output->pipeline,
> > -                              WFD_PIPELINE_DESTINATION_RECTANGLE,
> > -                              4, geometry.array);
> > -
> > -       wfdBindSourceToPipeline(ec->dev, output->pipeline,
> > -                               output->source[output->current ^ 1],
> > -                               WFD_TRANSITION_AT_VSYNC, NULL);
> > -
> > -       wfdBindPipelineToPort(ec->dev, output->port, output->pipeline);
> > -
> > -       wfdDeviceCommit(ec->dev, WFD_COMMIT_ENTIRE_DEVICE, WFD_INVALID_HANDLE);
> > -
> > -       output->base.origin = output->base.current;
> > -       output->base.repaint = wfd_output_repaint;
> > -       output->base.prepare_scanout_surface =
> > -               wfd_output_prepare_scanout_surface;
> > -       output->base.set_hardware_cursor = wfd_output_set_cursor;
> > -       output->base.destroy = wfd_output_destroy;
> > -       output->base.assign_planes = NULL;
> > -       output->base.set_backlight = NULL;
> > -       output->base.set_dpms = NULL;
> > -       output->base.switch_mode = NULL;
> > -
> > -       wl_list_insert(ec->base.output_list.prev, &output->base.link);
> > -
> > -       return 0;
> > -
> > -cleanup_pipeline:
> > -       wfdDestroyPipeline(ec->dev, output->pipeline);
> > -cleanup_weston_output:
> > -       weston_output_destroy(&output->base);
> > -cleanup_pipelines:
> > -       free(pipelines);
> > -cleanup_port:
> > -       wfdDestroyPort(ec->dev, output->port);
> > -       free(output);
> > -
> > -       return -1;
> > -}
> > -
> > -static int
> > -create_outputs(struct wfd_compositor *ec, int option_connector)
> > -{
> > -       int x = 0, y = 0;
> > -       WFDint i, num, *ports;
> > -       WFDPort port = WFD_INVALID_HANDLE;
> > -
> > -       num = wfdEnumeratePorts(ec->dev, NULL, 0, NULL);
> > -       ports = calloc(num, sizeof *ports);
> > -       if (ports == NULL)
> > -               return -1;
> > -
> > -       num = wfdEnumeratePorts(ec->dev, ports, num, NULL);
> > -       if (num < 1)
> > -               return -1;
> > -
> > -       for (i = 0; i < num; ++i) {
> > -               port = wfdCreatePort(ec->dev, ports[i], NULL);
> > -               if (port == WFD_INVALID_HANDLE)
> > -                       continue;
> > -
> > -               if (wfdGetPortAttribi(ec->dev, port, WFD_PORT_ATTACHED) &&
> > -                   (option_connector == 0 || ports[i] == option_connector)) {
> > -                       create_output_for_port(ec, port, x, y);
> > -
> > -                       x += container_of(ec->base.output_list.prev,
> > -                                         struct weston_output,
> > -                                         link)->current->width;
> > -               } else {
> > -                       wfdDestroyPort(ec->dev, port);
> > -               }
> > -       }
> > -
> > -       free(ports);
> > -
> > -       return 0;
> > -}
> > -
> > -static int
> > -handle_port_state_change(struct wfd_compositor *ec)
> > -{
> > -       struct wfd_output *output, *next;
> > -       WFDint output_port_id;
> > -       int x = 0, y = 0;
> > -       int x_offset = 0, y_offset = 0;
> > -       WFDPort port;
> > -       WFDint port_id;
> > -       WFDboolean state;
> > -
> > -       port_id = wfdGetEventAttribi(ec->dev, ec->event,
> > -                                    WFD_EVENT_PORT_ATTACH_PORT_ID);
> > -       state = wfdGetEventAttribi(ec->dev, ec->event,
> > -                                  WFD_EVENT_PORT_ATTACH_STATE);
> > -
> > -       if (state) {
> > -               struct weston_output *last_output =
> > -                       container_of(ec->base.output_list.prev,
> > -                                    struct weston_output, link);
> > -
> > -               /* XXX: not yet needed, we die with 0 outputs */
> > -               if (!wl_list_empty(&ec->base.output_list))
> > -                       x = last_output->x +
> > -                               last_output->current->width;
> > -               else
> > -                       x = 0;
> > -               y = 0;
> > -
> > -               port = wfdCreatePort(ec->dev, port_id, NULL);
> > -               if (port == WFD_INVALID_HANDLE)
> > -                       return -1;
> > -
> > -               create_output_for_port(ec, port, x, y);
> > -
> > -               return 0;
> > -       }
> > -
> > -       wl_list_for_each_safe(output, next, &ec->base.output_list, base.link) {
> > -               output_port_id =
> > -                       wfdGetPortAttribi(ec->dev, output->port, WFD_PORT_ID);
> > -
> > -               if (!state && output_port_id == port_id) {
> > -                       x_offset += output->base.current->width;
> > -                       wfd_output_destroy(&output->base);
> > -                       continue;
> > -               }
> > -
> > -               if (x_offset != 0 || y_offset != 0) {
> > -                       weston_output_move(&output->base,
> > -                                        output->base.x - x_offset,
> > -                                        output->base.y - y_offset);
> > -               }
> > -       }
> > -
> > -       if (ec->used_pipelines == 0)
> > -               wl_display_terminate(ec->base.wl_display);
> > -
> > -       return 0;
> > -}
> > -
> > -static int
> > -on_wfd_event(int fd, uint32_t mask, void *data)
> > -{
> > -       struct wfd_compositor *c = data;
> > -       struct wfd_output *output = NULL, *output_iter;
> > -       WFDEventType type;
> > -       const WFDtime timeout = 0;
> > -       WFDint pipeline_id;
> > -       WFDint bind_time;
> > -
> > -       type = wfdDeviceEventWait(c->dev, c->event, timeout);
> > -
> > -       switch (type) {
> > -       case WFD_EVENT_PIPELINE_BIND_SOURCE_COMPLETE:
> > -               pipeline_id =
> > -                       wfdGetEventAttribi(c->dev, c->event,
> > -                                          WFD_EVENT_PIPELINE_BIND_PIPELINE_ID);
> > -
> > -               bind_time =
> > -                       wfdGetEventAttribi(c->dev, c->event,
> > -                                          WFD_EVENT_PIPELINE_BIND_TIME_EXT);
> > -
> > -               wl_list_for_each(output_iter, &c->base.output_list, base.link) {
> > -                       if (output_iter->pipeline_id == pipeline_id)
> > -                               output = output_iter;
> > -               }
> > -
> > -               if (output == NULL)
> > -                       return 1;
> > -
> > -               weston_output_finish_frame(&output->base,
> > -                                        c->start_time + bind_time);
> > -               break;
> > -       case WFD_EVENT_PORT_ATTACH_DETACH:
> > -               handle_port_state_change(c);
> > -               break;
> > -       default:
> > -               return 1;
> > -       }
> > -
> > -       return 1;
> > -}
> > -
> > -static void
> > -wfd_destroy(struct weston_compositor *ec)
> > -{
> > -       struct wfd_compositor *d = (struct wfd_compositor *) ec;
> > -
> > -       weston_compositor_shutdown(ec);
> > -
> > -       udev_unref(d->udev);
> > -
> > -       wfdDestroyDevice(d->dev);
> > -
> > -       tty_destroy(d->tty);
> > -
> > -       free(d);
> > -}
> > -
> > -/* FIXME: Just add a stub here for now
> > - * handle drm{Set,Drop}Master in owfdrm somehow */
> > -static void
> > -vt_func(struct weston_compositor *compositor, int event)
> > -{
> > -       return;
> > -}
> > -
> > -static const char default_seat[] = "seat0";
> > -
> > -static struct weston_compositor *
> > -wfd_compositor_create(struct wl_display *display,
> > -                     int connector, const char *seat, int tty)
> > -{
> > -       struct wfd_compositor *ec;
> > -       struct wl_event_loop *loop;
> > -       struct timeval tv;
> > -
> > -       ec = malloc(sizeof *ec);
> > -       if (ec == NULL)
> > -               return NULL;
> > -
> > -       memset(ec, 0, sizeof *ec);
> > -
> > -       /* XXX: This is totally broken and doesn't even compile. */
> > -       if (weston_compositor_init(&ec->base, display) < 0)
> > -               return NULL;
> > -
> > -       gettimeofday(&tv, NULL);
> > -       ec->start_time = tv.tv_sec * 1000 + tv.tv_usec / 1000;
> > -
> > -       ec->udev = udev_new();
> > -       if (ec->udev == NULL) {
> > -               weston_log("failed to initialize udev context\n");
> > -               return NULL;
> > -       }
> > -
> > -       ec->dev = wfdCreateDevice(WFD_DEFAULT_DEVICE_ID, NULL);
> > -       if (ec->dev == WFD_INVALID_HANDLE) {
> > -               weston_log("failed to create wfd device\n");
> > -               return NULL;
> > -       }
> > -
> > -       ec->event = wfdCreateEvent(ec->dev, NULL);
> > -       if (ec->event == WFD_INVALID_HANDLE) {
> > -               weston_log("failed to create wfd event\n");
> > -               return NULL;
> > -       }
> > -
> > -       ec->base.wl_display = display;
> > -       if (init_egl(ec) < 0) {
> > -               weston_log("failed to initialize egl\n");
> > -               return NULL;
> > -       }
> > -
> > -       ec->base.destroy = wfd_destroy;
> > -       ec->base.focus = 1;
> > -
> > -       glGenFramebuffers(1, &ec->base.fbo);
> > -       glBindFramebuffer(GL_FRAMEBUFFER, ec->base.fbo);
> > -
> > -       if (weston_compositor_init_gl(&ec->base) < 0)
> > -               return NULL;
> > -
> > -       if (create_outputs(ec, connector) < 0) {
> > -               weston_log("failed to create outputs\n");
> > -               return NULL;
> > -       }
> > -
> > -       evdev_input_create(&ec->base, ec->udev, seat);
> > -
> > -       loop = wl_display_get_event_loop(ec->base.wl_display);
> > -       ec->wfd_source =
> > -               wl_event_loop_add_fd(loop,
> > -                                    wfdDeviceEventGetFD(ec->dev, ec->event),
> > -                                    WL_EVENT_READABLE, on_wfd_event, ec);
> > -       ec->tty = tty_create(&ec->base, vt_func, tty);
> > -
> > -       return &ec->base;
> > -}
> > -
> > -WL_EXPORT struct weston_compositor *
> > -backend_init(struct wl_display *display, int argc, char *argv[])
> > -{
> > -       int connector = 0, tty = 0;
> > -       const char *seat;
> > -
> > -       const struct weston_option wfd_options[] = {
> > -               { WESTON_OPTION_INTEGER, "connector", 0, &connector },
> > -               { WESTON_OPTION_STRING, "seat", 0, &seat },
> > -               { WESTON_OPTION_INTEGER, "tty", 0, &tty },
> > -       };
> > -
> > -       parse_options(&wfd_options, ARRAY_LENGTH(wfd_options), argc, argv);
> > -
> > -       return wfd_compositor_create(display, connector, seat, tty);
> > -}
> > diff --git a/src/compositor.c b/src/compositor.c
> > index 4f5388d..933019c 100644
> > --- a/src/compositor.c
> > +++ b/src/compositor.c
> > @@ -3501,8 +3501,6 @@ int main(int argc, char *argv[])
> >                         backend = "wayland-backend.so";
> >                 else if (getenv("DISPLAY"))
> >                         backend = "x11-backend.so";
> > -               else if (getenv("OPENWFD"))
> > -                       backend = "openwfd-backend.so";
> >                 else
> >                         backend = "drm-backend.so";
> >         }
> > --
> > 1.7.8.6
> >


More information about the wayland-devel mailing list