Revert "Add gst-recorder implementation"

This reverts commit 562c0c1bb2ef74ccbfda1bae4f84a61828119674.

Conflicts:
	recipes-graphics/wayland/weston_%.bbappend

Change-Id: I7b6e5a3969fc89c8279d47aaaf4f68a7544a181b
Signed-off-by: Ronan Le Martret <ronan.lemartret@iot.bzh>

5 files changed, 0 insertions, 4908 deletions

diff --git a/recipes-graphics/wayland/weston/0001-Add-virtual-output-support.patch b/recipes-graphics/wayland/weston/0001-Add-virtual-output-support.patch
deleted file mode 100644
index 6373f94e3..000000000
--- a/recipes-graphics/wayland/weston/0001-Add-virtual-output-support.patch
+++ /dev/null
@@ -1,375 +0,0 @@
-From c0bb07ba816524d69de22c22fcb7f7b9b95fbb11 Mon Sep 17 00:00:00 2001
-From: Damian Hobson-Garcia <dhobsong@igel.co.jp>
-Date: Thu, 27 Apr 2017 16:47:00 +0900
-Subject: [PATCH 1/3] Add virtual output support
-
-Following patch ported to Weston 1.11 with minor updates
-----------
-Author: Grigory Kletsko <grigory.kletsko@cogentembedded.com>
-Date:   Wed Nov 2 17:14:43 2016 +0300
-
-To enable virtual output set "virtual" property in core section
-to desirable number of virtual outputs. Then add settings to
-each virtual output in output sections. Name of the outputs
-will be virtual1, virtual2... etc.
-------------
----
- src/compositor-drm.c | 310 +++++++++++++++++++++++++++++++++++++++++++++++++++
- 1 file changed, 310 insertions(+)
-
-diff --git a/src/compositor-drm.c b/src/compositor-drm.c
-index abc9408..fc5a2ff 100644
---- a/src/compositor-drm.c
-+++ b/src/compositor-drm.c
-@@ -199,6 +199,11 @@ struct drm_output {
- 
- 	struct vaapi_recorder *recorder;
- 	struct wl_listener recorder_frame_listener;
-+
-+	/* not real output device */
-+	int virtual;
-+	/* Timer for updating frame */
-+	struct wl_event_source *virtual_finish_frame_timer;
- };
- 
- /*
-@@ -1474,6 +1479,33 @@ drm_output_destroy(struct weston_output *output_base)
- 	free(output);
- }
- 
-+static void
-+virtual_output_destroy(struct weston_output *output_base)
-+{
-+	struct drm_output *output = (struct drm_output *) output_base;
-+	struct drm_backend *c =
-+		(struct drm_backend *) output->base.compositor;
-+
-+	c->crtc_allocator &= ~(1 << output->crtc_id);
-+	c->connector_allocator &= ~(1 << output->connector_id);
-+
-+	if (c->use_pixman) {
-+		drm_output_fini_pixman(output);
-+	} else {
-+		gl_renderer->output_destroy(output_base);
-+		gbm_surface_destroy(output->gbm_surface);
-+	}
-+
-+	weston_plane_release(&output->fb_plane);
-+	weston_plane_release(&output->cursor_plane);
-+
-+	weston_output_destroy(&output->base);
-+
-+
-+	wl_event_source_remove(output->virtual_finish_frame_timer);
-+	free(output);
-+}
-+
- /**
-  * Find the closest-matching mode for a given target
-  *
-@@ -2649,6 +2681,270 @@ err_free:
- }
- 
- static void
-+virtual_output_start_repaint_loop(struct weston_output *output)
-+{
-+	struct timespec now;
-+
-+	weston_compositor_read_presentation_clock(output->compositor, &now);
-+	weston_output_finish_frame(output, &now, WP_PRESENTATION_FEEDBACK_INVALID);
-+}
-+
-+
-+static int
-+virtual_output_repaint(struct weston_output *output_base,
-+		   pixman_region32_t *damage)
-+{
-+	struct drm_output *output = (struct drm_output *) output_base;
-+	struct timespec ts;
-+	uint32_t msec_next;
-+	uint32_t msec_current;
-+
-+	msec_next = (output->base.frame_time + 1000000UL / output->base.current_mode->refresh) ;
-+
-+	if (output->destroy_pending)
-+		return -1;
-+
-+	if (!output->next)
-+		drm_output_render(output, damage);
-+	if (!output->next)
-+		return -1;
-+
-+	drm_output_set_cursor(output);
-+
-+	output->page_flip_pending = 1;
-+
-+	weston_compositor_read_presentation_clock(output_base->compositor, &ts);
-+
-+	msec_current = ts.tv_sec * 1000 + ts.tv_nsec / 1000000;
-+
-+	/*
-+	 * If we somehow late with updating frame, then fireup timer immediately (1 msec)
-+	 */
-+	wl_event_source_timer_update(output->virtual_finish_frame_timer, (msec_next > msec_current)  ?
-+				     msec_next - msec_current : 1);
-+
-+	return 0;
-+}
-+
-+static int
-+virtual_finish_frame_handler(void *data)
-+{
-+	struct drm_output *output = (struct drm_output *) data;
-+	struct timespec ts;
-+
-+	/* We don't set page_flip_pending on start_repaint_loop, in that case
-+	 * we just want to page flip to the current buffer to get an accurate
-+	 * timestamp */
-+	if (output->page_flip_pending) {
-+		drm_output_release_fb(output, output->current);
-+		output->current = output->next;
-+		output->next = NULL;
-+	}
-+
-+	output->page_flip_pending = 0;
-+
-+	if (output->destroy_pending)
-+		drm_output_destroy(&output->base);
-+	else if (!output->vblank_pending) {
-+		weston_compositor_read_presentation_clock(output->base.compositor, &ts);
-+
-+		weston_output_finish_frame(&output->base, &ts,
-+			WP_PRESENTATION_FEEDBACK_INVALID);
-+
-+		/* We can't call this from frame_notify, because the output's
-+		 * repaint needed flag is cleared just after that */
-+		if (output->recorder)
-+			weston_output_schedule_repaint(&output->base);
-+	}
-+
-+	return 1;
-+}
-+
-+/*
-+ * Virtual output connector that could be used for simulating output
-+ * device for clients and/or streaming of video
-+ */
-+static int
-+create_output_for_virtual_connector(struct drm_backend *b,
-+				    int x, int y, struct udev_device *drm_device)
-+{
-+	struct wl_event_loop *loop;
-+	static int virtual_id = 1; /* as other outputs numbered */
-+	struct drm_output *output;
-+	struct drm_mode *drm_mode, *next, *current;
-+	struct weston_mode *m;
-+	struct weston_config_section *section;
-+	int width, height, scale, fps;
-+	int recorded_output;
-+	char name[32], *s;
-+	enum weston_drm_backend_output_mode mode;
-+	struct weston_drm_backend_output_config config = {{ 0 }};
-+	uint32_t transform;
-+	int valid_mode;
-+	drmModeModeInfo crtc_mode;
-+
-+	output = zalloc(sizeof *output);
-+	if (output == NULL)
-+		return -1;
-+
-+	output->base.subpixel = WL_OUTPUT_SUBPIXEL_NONE; //drm_subpixel_to_wayland(connector->subpixel);
-+	output->base.make = "CogentEmbedded,Inc";
-+	output->base.serial_number = "";
-+	wl_list_init(&output->base.mode_list);
-+
-+	snprintf(name, 32, "virtual%d", virtual_id++);
-+	output->base.name = strdup(name);
-+
-+	section = weston_config_get_section(b->compositor->config, "output", "name",
-+					    output->base.name);
-+
-+	weston_config_section_get_bool(section, "recorder", &recorded_output, 0);
-+	if (recorded_output) {
-+		char model[64];
-+		char *ip;
-+		int port;
-+
-+		weston_config_section_get_string(section, "ip", &ip, "<nil>");
-+		weston_config_section_get_int(section, "port", &port, -1);
-+		snprintf(model, 64, "Virtual RTP %s:%d", ip, port);
-+		output->base.model = strdup(model);
-+	} else {
-+		output->base.model = "Virtual Display";
-+	}
-+
-+	mode = b->configure_output(b->compositor, b->use_current_mode,
-+				   output->base.name, &config);
-+
-+	if (mode == WESTON_DRM_BACKEND_OUTPUT_PREFERRED) {
-+		if (config.modeline && sscanf(config.modeline, "%dx%d@%d", &width, &height, &fps) >= 3)
-+			valid_mode = 1;
-+	}
-+
-+	weston_config_section_get_int(section, "scale", &scale, 1);
-+	weston_config_section_get_string(section, "transform", &s, "normal");
-+	if (weston_parse_transform(s, &transform) < 0)
-+		weston_log("Invalid transform \"%s\" for output %s\n",
-+			   s, output->base.name);
-+	free(s);
-+
-+	if (parse_gbm_format(config.gbm_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);
-+	free(s);
-+
-+	output->pipe = 0;
-+	b->crtc_allocator |= (1 << output->crtc_id);
-+	output->connector_id = 0;
-+	b->connector_allocator |= (1 << output->connector_id);
-+
-+	/* this is virtual output */
-+	output->virtual = 1;
-+
-+
-+	output->original_crtc = NULL;
-+	output->dpms_prop = NULL;
-+
-+	/* set static mode */
-+	if (valid_mode) {
-+		/* TODO: calculate proper mode settings to get desirable framerate */
-+		drmModeModeInfo static_drm_mode = {
-+			width * height * fps,
-+			width, 0, 0, width, width,
-+			height, 0, 0, height, height,
-+			fps * 1000,
-+			0, //flags
-+			0, //type
-+			"virtual"
-+		};
-+
-+		drm_mode = drm_output_add_mode(output, &static_drm_mode);
-+		if (!drm_mode)
-+			goto err_free;
-+
-+		drm_mode->base.refresh = fps * 1000;
-+	}
-+
-+	if (mode == WESTON_DRM_BACKEND_OUTPUT_OFF) {
-+		weston_log("Disabling output %s\n", output->base.name);
-+		drmModeSetCrtc(b->drm.fd, output->crtc_id,
-+			       0, 0, 0, 0, 0, NULL);
-+		goto err_free;
-+	}
-+
-+	current = drm_output_choose_initial_mode(b, output, mode, &config,
-+						 &crtc_mode);
-+	if (!current)
-+		goto err_free;
-+	output->base.current_mode = &current->base;
-+	output->base.current_mode->flags |= WL_OUTPUT_MODE_CURRENT;
-+
-+	weston_output_init(&output->base, b->compositor, x, y,
-+			   100, 100 * height / width, /* FIXME: calculate proper mm_width and mm_height */
-+			   config.base.transform, config.base.scale);
-+	if (b->use_pixman) {
-+		if (drm_output_init_pixman(output, b) < 0) {
-+			weston_log("Failed to init output pixman state\n");
-+			goto err_output;
-+		}
-+	} else if (drm_output_init_egl(output, b) < 0) {
-+		weston_log("Failed to init output gl state\n");
-+		goto err_output;
-+	}
-+
-+	output->backlight = NULL;
-+
-+	weston_compositor_add_output(b->compositor, &output->base);
-+
-+	output->base.connection_internal = 1;
-+
-+	loop = wl_display_get_event_loop(b->compositor->wl_display);
-+	output->virtual_finish_frame_timer = wl_event_loop_add_timer(loop, virtual_finish_frame_handler, output);
-+
-+	output->base.start_repaint_loop = virtual_output_start_repaint_loop;
-+	output->base.repaint = virtual_output_repaint;
-+	output->base.destroy = virtual_output_destroy;
-+	output->base.assign_planes = NULL;
-+	output->base.set_backlight = NULL;
-+	output->base.set_dpms = NULL;
-+	output->base.switch_mode = drm_output_switch_mode;
-+
-+	output->base.gamma_size = 0;
-+	output->base.set_gamma = drm_output_set_gamma;
-+
-+	weston_plane_init(&output->cursor_plane, b->compositor, 0, 0);
-+	weston_plane_init(&output->fb_plane, b->compositor, 0, 0);
-+
-+	weston_compositor_stack_plane(b->compositor, &output->cursor_plane, NULL);
-+	weston_compositor_stack_plane(b->compositor, &output->fb_plane,
-+				      &b->compositor->primary_plane);
-+
-+	weston_log("Output %s, ()\n",
-+		   output->base.name);
-+	wl_list_for_each(m, &output->base.mode_list, link)
-+		weston_log_continue(STAMP_SPACE "mode %dx%d@%.1f\n",
-+				    m->width, m->height, m->refresh / 1000.0);
-+
-+	return 0;
-+
-+err_output:
-+	weston_output_destroy(&output->base);
-+err_free:
-+	wl_list_for_each_safe(drm_mode, next, &output->base.mode_list,
-+							base.link) {
-+		wl_list_remove(&drm_mode->base.link);
-+		free(drm_mode);
-+	}
-+
-+	b->crtc_allocator &= ~(1 << output->crtc_id);
-+	b->connector_allocator &= ~(1 << output->connector_id);
-+	free(output);
-+
-+	return -1;
-+}
-+
-+static void
- create_sprites(struct drm_backend *b)
- {
- 	struct drm_sprite *sprite;
-@@ -2721,10 +3017,12 @@ static int
- create_outputs(struct drm_backend *b, uint32_t option_connector,
- 	       struct udev_device *drm_device)
- {
-+	struct weston_config_section *section;
- 	drmModeConnector *connector;
- 	drmModeRes *resources;
- 	int i;
- 	int x = 0, y = 0;
-+	int virtual;
- 
- 	resources = drmModeGetResources(b->drm.fd);
- 	if (!resources) {
-@@ -2770,6 +3068,18 @@ create_outputs(struct drm_backend *b, uint32_t option_connector,
- 		drmModeFreeConnector(connector);
- 	}
- 
-+	section = weston_config_get_section(b->compositor->config, "core", NULL, NULL);
-+	weston_config_section_get_int(section, "virtual", &virtual, 0);
-+
-+	for (i = 0; i < virtual; i++) {
-+		if (create_output_for_virtual_connector(b, x, y,
-+							drm_device) < 0)
-+			continue;
-+		x += container_of(b->compositor->output_list.prev,
-+				  struct weston_output,
-+				  link)->width;
-+	}
-+
- 	if (wl_list_empty(&b->compositor->output_list)) {
- 		weston_log("No currently active connector found.\n");
- 		drmModeFreeResources(resources);
--- 
-1.9.1
-
diff --git a/recipes-graphics/wayland/weston/0002-Add-gst-recorder-for-h264-output-streaming.patch b/recipes-graphics/wayland/weston/0002-Add-gst-recorder-for-h264-output-streaming.patch
deleted file mode 100644
index 36c4d93e5..000000000
--- a/recipes-graphics/wayland/weston/0002-Add-gst-recorder-for-h264-output-streaming.patch
+++ /dev/null
@@ -1,4201 +0,0 @@
-From 3ff09fecbe55d2f84f9a83f8965dffb913c00db7 Mon Sep 17 00:00:00 2001
-From: Damian Hobson-Garcia <dhobsong@igel.co.jp>
-Date: Thu, 27 Apr 2017 16:47:00 +0900
-Subject: [PATCH 2/3] Add gst-recorder for h264 output streaming
-
-Following patch ported to Weston 1.11 with minor updates
---------
-To use gst-recorder run weston with arg --gst-record. Add
-recorder=true property to output section of one of the outputs
-(real or virtual). Use properties ip, port to set host of RTP
-stream. Use property bitrate to set desirable maximum h264 bitrate.
---------
----
- Makefile.am                   |   17 +
- configure.ac                  |   12 +
- src/compositor-drm.c          |  199 ++++++-
- src/compositor-drm.h          |    3 +
- src/gst-recorder.c            | 1213 +++++++++++++++++++++++++++++++++++++++++
- src/gst-recorder.h            |   58 ++
- src/main.c                    |    4 +-
- src/media-ctl/libmediactl.c   |  955 ++++++++++++++++++++++++++++++++
- src/media-ctl/libv4l2subdev.c |  759 ++++++++++++++++++++++++++
- src/media-ctl/mediactl-priv.h |   64 +++
- src/media-ctl/mediactl.h      |  423 ++++++++++++++
- src/media-ctl/tools.h         |   32 ++
- src/media-ctl/v4l2subdev.h    |  258 +++++++++
- 13 files changed, 3995 insertions(+), 2 deletions(-)
- create mode 100644 src/gst-recorder.c
- create mode 100644 src/gst-recorder.h
- create mode 100644 src/media-ctl/libmediactl.c
- create mode 100644 src/media-ctl/libv4l2subdev.c
- create mode 100644 src/media-ctl/mediactl-priv.h
- create mode 100644 src/media-ctl/mediactl.h
- create mode 100644 src/media-ctl/tools.h
- create mode 100644 src/media-ctl/v4l2subdev.h
-
-diff --git a/Makefile.am b/Makefile.am
-index 98cd683..301c444 100644
---- a/Makefile.am
-+++ b/Makefile.am
-@@ -328,6 +328,23 @@ drm_backend_la_SOURCES += src/vaapi-recorder.c src/vaapi-recorder.h
- drm_backend_la_LIBADD += $(LIBVA_LIBS)
- drm_backend_la_CFLAGS += $(LIBVA_CFLAGS)
- endif
-+
-+if ENABLE_GST_RECORDER
-+drm_backend_la_SOURCES +=			\
-+	src/gst-recorder.c 			\
-+	src/gst-recorder.h			\
-+	src/v4l2-device.h			\
-+	src/media-ctl/libmediactl.c		\
-+	src/media-ctl/libv4l2subdev.c		\
-+	src/media-ctl/mediactl-priv.h		\
-+	src/media-ctl/mediactl.h		\
-+	src/media-ctl/tools.h			\
-+	src/media-ctl/v4l2subdev.h
-+drm_backend_la_LIBADD += $(LIBVA_LIBS) 		\
-+	-lgstallocators-1.0			\
-+	-lgstvideo-1.0
-+drm_backend_la_CFLAGS += $(LIBVA_CFLAGS)
-+endif
- endif
- 
- if ENABLE_WAYLAND_COMPOSITOR
-diff --git a/configure.ac b/configure.ac
-index 1d11864..c0717b3 100644
---- a/configure.ac
-+++ b/configure.ac
-@@ -249,6 +249,17 @@ if test "x$enable_rpi_compositor" = "xyes"; then
- fi
- AM_CONDITIONAL(INSTALL_RPI_COMPOSITOR, test "x$have_bcm_host" = "xyes")
- 
-+AC_ARG_ENABLE(gst-recorder, [  --enable-gst-recorder],,
-+	      enable_gst_recorder=yes)
-+if test x$enable_gst_recorder != xno; then
-+  AC_DEFINE([BUILD_GST_RECORDER], [1], [Build the gst recorder])
-+  PKG_CHECK_MODULES(GSTREAMER, [gstreamer-1.0 >= 1.0])
-+
-+  CPPFLAGS="$CPPFLAGS $GSTREAMER_CFLAGS"
-+  LIBS="$LIBS $GSTREAMER_LIBS -lgstapp-1.0"
-+fi
-+AM_CONDITIONAL(ENABLE_GST_RECORDER, test "x$enable_gst_recorder" != xno)
-+
- 
- AC_ARG_ENABLE([fbdev-compositor], [  --enable-fbdev-compositor],,
-               enable_fbdev_compositor=yes)
-@@ -734,4 +745,5 @@ AC_MSG_RESULT([
- 	libwebp Support			${have_webp}
- 	libunwind Support		${have_libunwind}
- 	VA H.264 encoding Support	${have_libva}
-+	GStreamer H.264 enc. Support	${enable_gst_recorder}
- ])
-diff --git a/src/compositor-drm.c b/src/compositor-drm.c
-index fc5a2ff..8b65637 100644
---- a/src/compositor-drm.c
-+++ b/src/compositor-drm.c
-@@ -57,6 +57,7 @@
- #include "v4l2-renderer.h"
- #include "launcher-util.h"
- #include "vaapi-recorder.h"
-+#include "gst-recorder.h"
- #include "presentation-time-server-protocol.h"
- #include "linux-dmabuf.h"
- 
-@@ -117,6 +118,8 @@ struct drm_backend {
- 
- 	int use_v4l2;
- 
-+	int enable_recorder;
-+
- 	uint32_t prev_state;
- 
- 	struct udev_input input;
-@@ -197,7 +200,12 @@ struct drm_output {
- 	int current_image;
- 	pixman_region32_t previous_damage;
- 
-+#ifdef BUILD_VAAPI_RECORDER
- 	struct vaapi_recorder *recorder;
-+#endif
-+#ifdef BUILD_GST_RECORDER
-+	struct gst_recorder *recorder;
-+#endif
- 	struct wl_listener recorder_frame_listener;
- 
- 	/* not real output device */
-@@ -2350,6 +2358,23 @@ parse_modeline(const char *s, drmModeModeInfo *mode)
- 	return 0;
- }
- 
-+static int parse_crop_rect(const char *s, struct v4l2_rect* crop)
-+{
-+	crop->left = 0;
-+	crop->top = 0;
-+	crop->width = 0;
-+	crop->height = 0;
-+
-+	if (sscanf(s, "%dx%d@%dx%d",
-+		   &crop->width,
-+		   &crop->height,
-+		   &crop->top,
-+		   &crop->left) != 4)
-+		return -1;
-+
-+	return 0;
-+}
-+
- static void
- setup_output_seat_constraint(struct drm_backend *b,
- 			     struct weston_output *output,
-@@ -3478,7 +3503,171 @@ recorder_binding(struct weston_keyboard *keyboard, uint32_t time, uint32_t key,
- 		recorder_destroy(output);
- 	}
- }
--#else
-+#endif
-+
-+#ifdef BUILD_GST_RECORDER
-+static void
-+recorder_destroy(struct drm_backend *c, struct drm_output *output)
-+{
-+	wl_list_remove(&output->recorder_frame_listener.link);
-+
-+	gst_recorder_destroy(output->recorder);
-+	output->recorder = NULL;
-+
-+	output->base.disable_planes--;
-+
-+	weston_log("[gst recorder] done\n");
-+}
-+
-+static void
-+recorder_frame_notify(struct wl_listener *listener, void *data)
-+{
-+	int ret = 0;
-+	struct drm_output *output;
-+	struct drm_backend *c;
-+	int fd;
-+
-+	output = container_of(listener, struct drm_output,
-+			      recorder_frame_listener);
-+	c = (struct drm_backend *) output->base.compositor->backend;
-+
-+	if (!output->recorder) {
-+		weston_log("%s: output have no recorder enabled\n",
-+			output->base.name);
-+		return;
-+	}
-+
-+	if (!output->current) {
-+		weston_log("%s: frame notify while current frame == NULL\n",
-+			   output->base.name);
-+		return;
-+	}
-+
-+	ret = drmPrimeHandleToFD(c->drm.fd, output->current->handle, DRM_CLOEXEC, &fd);
-+	if (!ret) {
-+		ret = gst_recorder_frame_dmafd(output->recorder, fd,
-+					       output->current->stride);
-+	}
-+
-+	if (ret < 0) {
-+		weston_log("[gst recorder] aborted: %m\n");
-+		recorder_destroy(c, output);
-+	}
-+}
-+
-+static int
-+recorder_enable(struct drm_backend *c, struct drm_output *output)
-+{
-+	int enable_recorder = 0;
-+	struct gst_recorder_settings *settings;
-+	struct weston_config_section *section;
-+	char* s;
-+	struct v4l2_rect crop = { .width = output->base.current_mode->width,
-+				  .height = output->base.current_mode->height,
-+				  .top = 0,
-+				  .left = 0 };
-+
-+	if (output->recorder)
-+		return -1;
-+
-+	section = weston_config_get_section(c->compositor->config, "output", "name",
-+					    output->base.name);
-+
-+	weston_config_section_get_bool(section, "recorder", &enable_recorder, 0);
-+
-+	if (!enable_recorder)
-+		return 0;
-+
-+	/* TODO: add support for NV16 or NV12 */
-+	if (output->gbm_format != GBM_FORMAT_XRGB8888) {
-+		weston_log("[gst recorder] %s: "
-+			   "output format not supported\n", output->base.name);
-+		return -1;
-+	}
-+
-+	settings = malloc(sizeof(* settings));
-+	weston_config_section_get_string(section, "ip", &settings->ip, NULL);
-+	if (!settings->ip)
-+		goto err;
-+	weston_config_section_get_int(section, "port", &settings->port, -1);
-+	/* default gives about 16 Mbit/s at 1280x720@60FPS */
-+	weston_config_section_get_int(section, "bitrate", &settings->bitrate, 300000);
-+
-+	settings->width = output->base.current_mode->width;
-+	settings->height = output->base.current_mode->height;
-+
-+	settings->crop = crop;
-+
-+	weston_config_section_get_string(section, "crop", &s, NULL);
-+	if (s) {
-+		if (parse_crop_rect(s, &settings->crop)) {
-+			weston_log("[gst recorder] %s:"
-+				   " failed to parse crop parameter\n",
-+				   output->base.name);
-+			goto err;
-+		}
-+	}
-+
-+
-+	output->recorder =
-+		gst_recorder_create(settings);
-+	if (!output->recorder) {
-+		weston_log("[gst recorder] %s:"
-+			" failed to create gst recorder\n",
-+			output->base.name);
-+		goto err;
-+	}
-+
-+	output->base.disable_planes++;
-+
-+	output->recorder_frame_listener.notify = recorder_frame_notify;
-+	wl_signal_add(&output->base.frame_signal,
-+		      &output->recorder_frame_listener);
-+
-+	weston_output_schedule_repaint(&output->base);
-+
-+	weston_log("[gst recorder] %s:"
-+		" recorder initialized\n",
-+		output->base.name);
-+
-+	return 0;
-+err:
-+	weston_log("[gst recorder] %s:"
-+		" invalid settings\n",
-+		output->base.name);
-+	free(settings);
-+	return -1;
-+}
-+
-+static void
-+recorders_enable(struct drm_backend *c)
-+{
-+	struct drm_output *output;
-+
-+	wl_list_for_each(output, &c->compositor->output_list, base.link) {
-+		recorder_enable(c, output);
-+	}
-+}
-+
-+static void
-+recorder_binding(struct weston_keyboard *keyboard, uint32_t time, uint32_t key,
-+		 void *data)
-+{
-+	struct drm_backend *c = data;
-+	struct drm_output *output;
-+
-+	/* fix this */
-+	wl_list_for_each(output, &c->compositor->output_list, base.link) {
-+		if (!output->recorder)
-+			recorder_enable(c, output);
-+		else
-+			recorder_destroy(c, output);
-+	}
-+}
-+#endif
-+
-+#if !defined(BUILD_VAAPI_RECORDER) && !defined(BUILD_GST_RECORDER)
-+
- static void
- recorder_binding(struct weston_keyboard *keyboard, uint32_t time, uint32_t key,
- 		 void *data)
-@@ -3571,6 +3760,7 @@ drm_backend_create(struct weston_compositor *compositor,
- 	b->compositor = compositor;
- 	b->use_pixman = config->use_pixman;
- 	b->use_v4l2 = config->use_v4l2;
-+	b->enable_recorder = config->enable_recorder;
- 	b->configure_output = config->configure_output;
- 	b->use_current_mode = config->use_current_mode;
- 
-@@ -3627,6 +3817,10 @@ drm_backend_create(struct weston_compositor *compositor,
- 		}
- 	}
- 
-+#ifdef BUILD_GST_RECORDER
-+	gst_recorder_init();
-+#endif
-+
- 	b->base.destroy = drm_destroy;
- 	b->base.restore = drm_restore;
- 
-@@ -3688,6 +3882,9 @@ drm_backend_create(struct weston_compositor *compositor,
- 					    recorder_binding, b);
- 	weston_compositor_add_debug_binding(compositor, KEY_W,
- 					    renderer_switch_binding, b);
-+	/* enable GST recording-streaming */
-+	if (b->enable_recorder)
-+		recorders_enable(b);
- 
- 	if (compositor->renderer->import_dmabuf) {
- 		if (linux_dmabuf_setup(compositor) < 0)
-diff --git a/src/compositor-drm.h b/src/compositor-drm.h
-index d3d15af..65305d9 100644
---- a/src/compositor-drm.h
-+++ b/src/compositor-drm.h
-@@ -90,6 +90,9 @@ struct weston_drm_backend_config {
- 	/** Whether to use the v4l2 renderer insted of the OpenGL ES renderer. */
- 	bool use_v4l2;
- 
-+	/** Whether to use the v4l2 renderer insted of the OpenGL ES renderer. */
-+	bool enable_recorder;
-+
- 	/** The seat to be used for input and output.
- 	 *
- 	 * If NULL the default "seat0" will be used.  The backend will
-diff --git a/src/gst-recorder.c b/src/gst-recorder.c
-new file mode 100644
-index 0000000..2e3b359
---- /dev/null
-+++ b/src/gst-recorder.c
-@@ -0,0 +1,1213 @@
-+/*
-+ * Copyright © 2016 Cogent Embedded Inc
-+ *
-+ * 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.
-+ */
-+
-+/*
-+ * TODO:
-+ * 1) Add format parameter to virtual display to render in another format
-+ * with v4l2-renderer
-+ * 2) Add capability to use already NV12 rendered frame
-+ */
-+#include "config.h"
-+
-+#include <stdlib.h>
-+#include <stdint.h>
-+#include <string.h>
-+#include <unistd.h>
-+#include <assert.h>
-+#include <errno.h>
-+
-+#include <sys/types.h>
-+#include <sys/stat.h>
-+#include <sys/mman.h>
-+#include <sys/ioctl.h>
-+#include <fcntl.h>
-+
-+#include <pthread.h>
-+
-+#include "compositor.h"
-+#include "gst-recorder.h"
-+
-+/* VSP includes */
-+#include <media-ctl/mediactl.h>
-+#include <media-ctl/v4l2subdev.h>
-+
-+/* Gstreamer includes */
-+#include <gst/gst.h>
-+#include <gst/app/gstappsrc.h>
-+#include <gst/rtp/gstrtpbuffer.h>
-+
-+#include <gst/video/video.h>
-+#include <gst/video/gstvideometa.h>
-+#include <gst/video/gstvideopool.h>
-+#include <gst/allocators/gstdmabuf.h>
-+
-+struct vsp_data;
-+
-+#define DEFAULT_FPS		60
-+
-+typedef enum _vsp_port_n {
-+	VSP_PORT_INPUT	= 0,
-+	VSP_PORT_INPUT0 = VSP_PORT_INPUT,
-+	VSP_PORT_INPUT1,
-+	VSP_PORT_INPUT2,
-+	VSP_PORT_INPUT3,
-+	VSP_PORT_OUTPUT
-+} vsp_port_n;
-+
-+struct gst_recorder {
-+	struct gst_recorder_settings *set;
-+	int frame_count;
-+	int input_count;
-+
-+	int error;
-+	int destroying;
-+	pthread_t worker_thread;
-+	pthread_mutex_t mutex;
-+	pthread_cond_t input_cond;
-+
-+	struct {
-+		int valid;
-+		int prime_fd, stride;
-+	} input;
-+
-+	/* GLib */
-+	GMainContext *gcontext;
-+	/* Gstreamer stuff */
-+	GstElement *pipeline;
-+	/* AppSrc */
-+	GstAppSrc *appsrc;
-+	/* ...and source pad */
-+	GstPad *appsrc_pad;
-+	/* OMX encoder buffer pool */
-+	GstBufferPool *omx_pool;
-+	/* bus */
-+	GstBus *bus;
-+	/* timestamp */
-+	GstClockTime timestamp;
-+	uint32_t ts_last_frame;
-+	/* to be removed */
-+	guint callback_tag;
-+
-+	struct vsp_data *vsp;
-+};
-+
-+/*******************************************************************************
-+ * VSP related code
-+ ******************************************************************************/
-+
-+#define VSP_OUTPUT_BUFFERS_PLANE	2
-+
-+/* #define VSP_OUTPUT_NV16			1 */
-+
-+/* ...number of input/output pads (WPF1-3 are not implemented) */
-+#define VSP_PADS_NUM                    1
-+
-+/*******************************************************************************
-+ * Local types definition
-+ ******************************************************************************/
-+
-+struct vsp_media_pad
-+{
-+	struct media_pad	   *infmt_pad;
-+	struct media_pad	   *outfmt_pad;
-+	struct media_entity	   *entity;
-+	int                     fd;
-+};
-+
-+typedef struct vsp_media_pad vsp_media_pad_t;
-+
-+struct vsp_data
-+{
-+	/* ...media device */
-+	struct media_device *media;
-+
-+	/* ...VSP input/output pads */
-+	vsp_media_pad_t input, output;
-+
-+	/* mutex */
-+	pthread_mutex_t mutex;
-+
-+	/* user count */
-+	int users;
-+};
-+
-+struct vsp_data *vsp_g = NULL;
-+
-+/* ...type declarations */
-+typedef struct vsp_data vsp_data_t;
-+
-+static int
-+gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride);
-+
-+/* ...module initialization (a bit of salami) */
-+static vsp_data_t *
-+vsp_init(const char *devname)
-+{
-+	vsp_data_t                     *vsp;
-+	struct media_device            *media;
-+	const struct media_device_info *info;
-+	const char                     *dev;
-+	char                            buf[256];
-+	char                           *endp, *p;
-+	struct media_entity            *entity;
-+
-+	/* ...create data structure */
-+	if ((vsp = malloc(sizeof(*vsp))) == NULL)
-+	{
-+		weston_log("failed to allocate memory\n");
-+		errno = ENOMEM;
-+		return NULL;
-+	}
-+	memset(vsp, 0, sizeof(*vsp));
-+
-+	pthread_mutex_init(&vsp->mutex, NULL);
-+
-+	/* ...create media device */
-+	if ((vsp->media = media = media_device_new(devname)) == NULL)
-+	{
-+		weston_log("failed to open device '%s'\n", devname);
-+		goto error;
-+	}
-+	else if ((errno = -media_device_enumerate(media)) != 0)
-+	{
-+		weston_log("failed to enumerate device '%s'\n", devname);
-+		goto error_media;
-+	}
-+	else if ((info = media_get_info(media)) == NULL)
-+	{
-+		weston_log("failed to get media info data\n");
-+		goto error_media;
-+	}
-+	else
-+	{
-+		dev = ((p = strchr(info->bus_info, ':')) ? p + 1 : info->bus_info);
-+		weston_log("open media device: %s (%s)\n", info->bus_info, dev);
-+	}
-+
-+	/* ...reset links */
-+	if (media_reset_links(media) != 0)
-+	{
-+		weston_log("failed to reset media device\n");
-+		goto error_media;
-+	}
-+
-+	/* ...setup RPF.0:1 -> WPF.0:0 link */
-+	snprintf(buf, sizeof(buf), "'%s rpf.0':1 -> '%s wpf.0':0 [1]", dev, dev);
-+	if (media_parse_setup_link(media, buf, &endp) != 0)
-+	{
-+		weston_log("failed to setup link '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	/* ...setup WPF.0:1 -> WPF.0 output link */
-+	snprintf(buf, sizeof(buf), "'%s wpf.0':1 -> '%s wpf.0 output':0 [1]", dev, dev);
-+	if (media_parse_setup_link(media, buf, &endp) != 0)
-+	{
-+		weston_log("failed to setup link '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	/* ...specify input/output-format of RPF pad */
-+	snprintf(buf, sizeof(buf), "'%s rpf.0':0", dev);
-+	if ((vsp->input.infmt_pad = media_parse_pad(media, buf, NULL)) == NULL)
-+	{
-+		weston_log("failed to parse pad '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	snprintf(buf, sizeof(buf), "'%s rpf.0':1", dev);
-+	if ((vsp->input.outfmt_pad = media_parse_pad(media, buf, NULL)) == NULL)
-+	{
-+		weston_log("failed to parse pad '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	snprintf(buf, sizeof(buf), "%s rpf.0", dev);
-+	if ((vsp->input.entity = media_get_entity_by_name(media, buf, strlen(buf))) == NULL)
-+	{
-+		weston_log("failed to parse entity '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	/* ...get input file-descriptor */
-+	snprintf(buf, sizeof(buf), "%s rpf.0 input", dev);
-+	if ((entity = media_get_entity_by_name(media, buf, strlen(buf))) == NULL)
-+	{
-+		weston_log("entity '%s' not found\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+	else if (v4l2_subdev_open(entity) != 0)
-+	{
-+		weston_log("failed to open subdev '%s'\n", buf);
-+		goto error_media;
-+	}
-+	else if ((vsp->input.fd = open(media_entity_get_devname(entity), O_RDWR/* | O_NONBLOCK*/)) < 0)
-+	{
-+		weston_log("failed to open device '%s'\n", media_entity_get_devname(entity));
-+		goto error_media;
-+	}
-+	else
-+	{
-+		weston_log("input pad setup ('%s':'%s')\n", buf, media_entity_get_devname(entity));
-+	}
-+
-+	/* ...specify input/output formats of WPF pad */
-+	snprintf(buf, sizeof(buf), "'%s wpf.0':0", dev);
-+	if ((vsp->output.infmt_pad = media_parse_pad(media, buf, NULL)) == NULL)
-+	{
-+		weston_log("failed to parse pad '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	snprintf(buf, sizeof(buf), "'%s wpf.0':1", dev);
-+	if ((vsp->output.outfmt_pad = media_parse_pad(media, buf, NULL)) == NULL)
-+	{
-+		weston_log("failed to parse pad '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	snprintf(buf, sizeof(buf), "%s wpf.0", dev);
-+	if ((vsp->output.entity = media_get_entity_by_name(media, buf, strlen(buf))) == NULL)
-+	{
-+		weston_log("failed to parse entity '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+
-+	/* ...get a file descriptor for the output */
-+	snprintf(buf, sizeof(buf), "%s wpf.0 output", dev);
-+	if ((entity = media_get_entity_by_name(media, buf, strlen(buf))) == NULL)
-+	{
-+		weston_log("failed to get entity '%s'\n", buf);
-+		errno = EINVAL;
-+		goto error_media;
-+	}
-+	else if (v4l2_subdev_open(entity) != 0)
-+	{
-+		weston_log("failed to open subdev '%s'\n", buf);
-+		goto error_media;
-+	}
-+	else if ((vsp->output.fd = open(media_entity_get_devname(entity), O_RDWR | O_NONBLOCK)) < 0)
-+	{
-+		weston_log("failed to open device '%s'\n", media_entity_get_devname(entity));
-+		goto error_media;
-+	}
-+	else
-+	{
-+		weston_log("output pad setup (%s:%s)\n", buf, media_entity_get_devname(entity));
-+	}
-+
-+	weston_log("vsp-device '%s' created\n", devname);
-+
-+	return vsp;
-+
-+error_media:
-+	/* ...destroy media device and all associated structures */
-+	media_device_unref(vsp->media);
-+
-+error:
-+	/* ...destroy data structure */
-+	free(vsp);
-+	return NULL;
-+}
-+
-+static void
-+vsp_deinit(vsp_data_t *vsp)
-+{
-+	/* ...destroy media device and all associated structures */
-+	media_device_unref(vsp->media);
-+
-+	/* ...destroy data structure */
-+	free(vsp);
-+}
-+
-+/* ...set V4L2 device format */
-+static int
-+vsp_set_format(int fd, struct v4l2_format *fmt)
-+{
-+	/* ...set format */
-+	if (ioctl(fd, VIDIOC_S_FMT, fmt) < 0) {
-+		weston_log("format set (fd=%d) failed: %d\n",
-+			fd, errno);
-+	}
-+
-+	return 0;
-+}
-+
-+/* ...start streaming on specific V4L2 device */
-+static int
-+vsp_streaming_enable(vsp_data_t *vsp, vsp_port_n port, int enable)
-+{
-+	vsp_media_pad_t *pad = (port == VSP_PORT_INPUT) ? &vsp->input : &vsp->output;
-+	int fd = pad->fd;
-+	int type = (port == VSP_PORT_INPUT) ?
-+		V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE : V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
-+
-+	return ioctl(fd, (enable ? VIDIOC_STREAMON : VIDIOC_STREAMOFF), &type);
-+}
-+
-+/* ...prepare VSP filter for operation */
-+static int
-+vsp_set_formats(vsp_data_t *vsp, int width, int height, struct v4l2_rect* crop)
-+{
-+	vsp_media_pad_t *input = &vsp->input, *output = &vsp->output;
-+	struct v4l2_mbus_framefmt mfmt = { .width = width, .height = height };
-+	struct v4l2_format format;
-+
-+	/* ...configure RPF input pads; specify pixel format and size (one of YUV variants) */
-+	mfmt.width = width;
-+	mfmt.height = height;
-+	mfmt.code = V4L2_MBUS_FMT_ARGB8888_1X32;
-+	if (v4l2_subdev_set_format(input->infmt_pad->entity,
-+		&mfmt, input->infmt_pad->index, V4L2_SUBDEV_FORMAT_ACTIVE) < 0) {
-+		weston_log("VSP: input pad in format set failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	 /* set a crop paramters */
-+	if (v4l2_subdev_set_selection(input->infmt_pad->entity, crop, input->infmt_pad->index,
-+				      V4L2_SEL_TGT_CROP, V4L2_SUBDEV_FORMAT_ACTIVE)) {
-+		weston_log("set crop parameter failed: %dx%d@(%d,%d).\n",
-+			   crop->width, crop->height, crop->left, crop->top);
-+		return -1;
-+	}
-+
-+	/* ...output is NV12 or NV16 or I420*/
-+	mfmt.width = crop->width;
-+	mfmt.height = crop->height;
-+	mfmt.code = V4L2_MBUS_FMT_AYUV8_1X32;
-+	if (v4l2_subdev_set_format(input->outfmt_pad->entity,
-+		&mfmt, input->outfmt_pad->index, V4L2_SUBDEV_FORMAT_ACTIVE) < 0) {
-+		weston_log("VSP: input pad out format set failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	/* ...specify input format */
-+	memset(&format, 0, sizeof(format));
-+	format.type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE;
-+	format.fmt.pix_mp.width = /* crop-> */width;
-+	format.fmt.pix_mp.height = /* crop-> */height;
-+	format.fmt.pix_mp.pixelformat = V4L2_PIX_FMT_ABGR32;
-+	format.fmt.pix_mp.num_planes = 1;
-+	/* ...set input port format */
-+	if (vsp_set_format(input->fd, &format) < 0) {
-+		weston_log("VSP: input set format failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	/* ...both input and output are ARGB8888 always (now effective) */
-+	if (v4l2_subdev_set_format(output->infmt_pad->entity,
-+		&mfmt, output->infmt_pad->index, V4L2_SUBDEV_FORMAT_ACTIVE) < 0) {
-+		weston_log("VSP: output pad in format set failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	/* ...specify cropping area, probably? - tbd */
-+	if (v4l2_subdev_set_format(output->outfmt_pad->entity,
-+		&mfmt, output->outfmt_pad->index, V4L2_SUBDEV_FORMAT_ACTIVE) < 0) {
-+		weston_log("VSP: output pad in format set failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	/* ...setup output pads */
-+	memset(&format, 0, sizeof(format));
-+	format.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
-+	format.fmt.pix_mp.width = crop->width;
-+	format.fmt.pix_mp.height = crop->height;
-+#ifdef VSP_OUTPUT_NV16
-+	format.fmt.pix_mp.pixelformat = V4L2_PIX_FMT_NV16M;
-+#else
-+	format.fmt.pix_mp.pixelformat = V4L2_PIX_FMT_NV12M;
-+#endif
-+	format.fmt.pix_mp.num_planes = VSP_OUTPUT_BUFFERS_PLANE;
-+	/* ...set output buffer format */
-+	if (vsp_set_format(output->fd, &format) < 0) {
-+		weston_log("VSP: output set format failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	return 0;
-+}
-+
-+static int
-+vsp_request_buffers(vsp_data_t *vsp, vsp_port_n port, unsigned int num)
-+{
-+	vsp_media_pad_t *pad = (port == VSP_PORT_INPUT) ? &vsp->input : &vsp->output;
-+	struct v4l2_requestbuffers reqbuf;
-+	int fd = pad->fd;
-+
-+	/* ...input buffers are DMA-fd, output buffers allocated by kernel */
-+	memset(&reqbuf, 0, sizeof(reqbuf));
-+	reqbuf.type = (port == VSP_PORT_INPUT) ?
-+		V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE : V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
-+	reqbuf.memory = V4L2_MEMORY_DMABUF;
-+	reqbuf.count = num;
-+	if (ioctl(fd, VIDIOC_REQBUFS, &reqbuf) < 0) {
-+		weston_log("VSP: %s REQBUFS failed: %d\n",
-+			(port == VSP_PORT_INPUT) ? "input" : "output", errno);
-+		return -1;
-+	}
-+
-+	if (reqbuf.count != num) {
-+		weston_log("VSP: %s failed to request %d (!= %d) bufs\n",
-+			(port == VSP_PORT_INPUT) ? "input" : "output", num, reqbuf.count);
-+		return -1;
-+	}
-+	return 0;
-+}
-+
-+
-+/* ...enqueue dmafd buffer */
-+static int
-+vsp_input_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd)
-+{
-+	vsp_media_pad_t    *pad = &vsp->input;
-+	struct v4l2_buffer  buf;
-+	struct v4l2_plane   planes[1];
-+
-+	/* ...set buffer parameters */
-+	memset(&buf, 0, sizeof(buf));
-+	memset(planes, 0, sizeof(planes));
-+	buf.type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE;
-+	buf.memory = V4L2_MEMORY_DMABUF;
-+	buf.index = i;
-+	buf.m.planes = planes;
-+	buf.length = 1;
-+	buf.m.planes[0].m.fd = dmafd;
-+
-+	/* ...submit buffer */
-+	if (ioctl(pad->fd, VIDIOC_QBUF, &buf) < 0) {
-+		weston_log("VSP: input dmafd (%d) buffer (%i) queue failed: %d\n",
-+			dmafd, i, errno);
-+		return -1;
-+	}
-+
-+	return 0;
-+}
-+
-+/* ...dequeue dmafd buffer */
-+static int
-+vsp_input_buffer_dequeue_dmafd(vsp_data_t *vsp)
-+{
-+	vsp_media_pad_t    *pad = &vsp->input;
-+	struct v4l2_buffer  buf;
-+	struct v4l2_plane   planes[1];
-+
-+	/* ...set buffer parameters */
-+	memset(&buf, 0, sizeof(buf));
-+	memset(planes, 0, sizeof(planes));
-+	buf.type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE;
-+	buf.memory = V4L2_MEMORY_DMABUF;
-+	buf.m.planes = planes;
-+	buf.length = 1;
-+
-+	if (ioctl(pad->fd, VIDIOC_DQBUF, &buf) < 0) {
-+		weston_log("VSP: input dmafd buffer de-queue failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	/* ...return buffer index */
-+	return buf.index;
-+}
-+
-+/* ...enqueue output buffer */
-+static int
-+vsp_output_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd[])
-+{
-+	vsp_media_pad_t    *pad = &vsp->output;
-+	struct v4l2_plane   planes[2];
-+	struct v4l2_buffer  buf;
-+
-+	/* ...set buffer parameters (single-plane ARGB always) */
-+	memset(&buf, 0, sizeof(buf));
-+	memset(planes, 0, sizeof(planes));
-+	buf.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
-+	buf.memory = V4L2_MEMORY_DMABUF;
-+	buf.index = i;
-+	buf.m.planes = planes;
-+	buf.length = VSP_OUTPUT_BUFFERS_PLANE;
-+	buf.m.planes[0].m.fd = dmafd[0];
-+	buf.m.planes[1].m.fd = dmafd[1];
-+
-+	/* ...submit buffer */
-+	if (ioctl(pad->fd, VIDIOC_QBUF, &buf) < 0) {
-+		weston_log("VSP: output dmafd queue failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	return 0;
-+}
-+
-+/* ...dequeue output buffer */
-+static int
-+vsp_output_buffer_dequeue_dmafd(vsp_data_t *vsp)
-+{
-+	vsp_media_pad_t    *pad = &vsp->output;
-+	struct v4l2_buffer  buf;
-+	struct v4l2_plane   planes[2];
-+
-+	/* ...set buffer parameters */
-+	memset(&buf, 0, sizeof(buf));
-+	memset(planes, 0, sizeof(planes));
-+	buf.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
-+	buf.memory = V4L2_MEMORY_DMABUF;
-+	buf.m.planes = planes;
-+	buf.length = VSP_OUTPUT_BUFFERS_PLANE;
-+
-+	if (ioctl(pad->fd, VIDIOC_DQBUF, &buf) < 0) {
-+		weston_log("VSP: output dmafd de-queue failed: %d\n", errno);
-+		return -1;
-+	}
-+
-+	/* ...return dequeue buffer index */
-+	return buf.index;
-+}
-+
-+/* ...get capturing interface file descriptor */
-+static int
-+vsp_capture_fd(vsp_data_t *vsp)
-+{
-+	return vsp->output.fd;
-+}
-+
-+/*******************************************************************************
-+ * Gstreamer stuff
-+ ******************************************************************************/
-+
-+static void
-+print_one_tag (const GstTagList * list, const gchar * tag, gpointer user_data)
-+{
-+	int i, num;
-+
-+	num = gst_tag_list_get_tag_size (list, tag);
-+	for (i = 0; i < num; ++i) {
-+		const GValue *val;
-+
-+		/* Note: when looking for specific tags, use the gst_tag_list_get_xyz() API,
-+		 * we only use the GValue approach here because it is more generic */
-+		val = gst_tag_list_get_value_index (list, tag, i);
-+		if (G_VALUE_HOLDS_STRING (val)) {
-+			weston_log("\t%20s : %s\n", tag, g_value_get_string (val));
-+		} else if (G_VALUE_HOLDS_UINT (val)) {
-+			weston_log("\t%20s : %u\n", tag, g_value_get_uint (val));
-+		} else if (G_VALUE_HOLDS_DOUBLE (val)) {
-+			weston_log("\t%20s : %g\n", tag, g_value_get_double (val));
-+		} else if (G_VALUE_HOLDS_BOOLEAN (val)) {
-+			weston_log("\t%20s : %s\n", tag,
-+					(g_value_get_boolean (val)) ? "true" : "false");
-+		} else if (GST_VALUE_HOLDS_BUFFER (val)) {
-+			GstBuffer *buf = gst_value_get_buffer (val);
-+			guint buffer_size = gst_buffer_get_size (buf);
-+
-+			weston_log("\t%20s : buffer of size %u\n", tag, buffer_size);
-+		} else if (GST_VALUE_HOLDS_DATE_TIME (val)) {
-+			GstDateTime *dt = g_value_get_boxed (val);
-+			gchar *dt_str = gst_date_time_to_iso8601_string (dt);
-+
-+			weston_log("\t%20s : %s\n", tag, dt_str);
-+			g_free (dt_str);
-+		} else {
-+			weston_log("\t%20s : tag of type '%s'\n", tag, G_VALUE_TYPE_NAME (val));
-+		}
-+	}
-+}
-+
-+static gboolean
-+gst_bus_callback(GstBus *bus, GstMessage *message, gpointer user_data)
-+{
-+	GTimeVal time;
-+	struct gst_recorder *r = user_data;
-+
-+	if (!r->pipeline) {
-+		weston_log("gst_pipeline: unexpected gst bus callback event, while pipeline==null\n");
-+		return TRUE;
-+	}
-+
-+	g_get_current_time(&time);
-+
-+	switch (GST_MESSAGE_TYPE(message))
-+	{
-+		case GST_MESSAGE_QOS:
-+			{
-+				GstFormat format;
-+				guint64 processed;
-+				guint64 dropped;
-+
-+				gst_message_parse_qos_stats (message, &format, &processed, &dropped);
-+				weston_log("gst_pipeline: qos from: %s processed %lld, dropped %lld\n",
-+					GST_OBJECT_NAME (message->src),
-+					processed, dropped);
-+			}
-+		break;
-+		case GST_MESSAGE_STREAM_STATUS:
-+			{
-+				const GValue *val;
-+
-+				val = gst_message_get_stream_status_object (message);
-+				weston_log("gst_pipeline: stream status type %s, value %p\n",
-+					G_VALUE_TYPE_NAME(val),
-+					g_value_get_object(val));
-+			}
-+		break;
-+		case GST_MESSAGE_TAG:
-+			{
-+				GstTagList *tags = NULL;
-+
-+				gst_message_parse_tag (message, &tags);
-+				weston_log("gst_pipeline: tag from element %s:\n",
-+					GST_OBJECT_NAME (message->src));
-+				gst_tag_list_foreach (tags, print_one_tag, NULL);
-+				weston_log("\n");
-+			}
-+		break;
-+		case GST_MESSAGE_STATE_CHANGED:
-+			{
-+				GstState oldstate, newstate;
-+
-+				gst_message_parse_state_changed(message, &oldstate, &newstate, NULL);
-+				weston_log("gst_pipeline: element %s changed state from %s to %s.\n",
-+					GST_OBJECT_NAME (message->src),
-+					gst_element_state_get_name (oldstate),
-+					gst_element_state_get_name (newstate));
-+
-+				/* if gstreamer become ready */
-+				if ((GST_MESSAGE_SRC(message) == GST_OBJECT(r->appsrc)) &&
-+					(newstate == GST_STATE_PAUSED)) {
-+					weston_log("gst_pipeline: pipeline ready\n");
-+				}
-+			}
-+		break;
-+		case GST_MESSAGE_ERROR:
-+			{
-+				GError *err;
-+				gchar *debug_info;
-+				gst_message_parse_error(message, &err, &debug_info);
-+				weston_log("gst_pipeline: error received from element %s: %s\n",
-+					GST_OBJECT_NAME(message->src), err->message);
-+				weston_log("gst_pipeline: debugging information: %s\n",
-+					debug_info ? debug_info : "none");
-+				g_clear_error (&err);
-+				g_free (debug_info);
-+			}
-+		break;
-+		case GST_MESSAGE_WARNING:
-+			{
-+				GError *err;
-+				gchar *debug_info;
-+				gst_message_parse_warning(message, &err, &debug_info);
-+				weston_log("gst_pipeline: warning received from element %s: %s\n",
-+					GST_OBJECT_NAME(message->src), err->message);
-+				weston_log("gst_pipeline: debugging information: %s\n",
-+					debug_info ? debug_info : "none");
-+				g_clear_error (&err);
-+				g_free (debug_info);
-+			}
-+		break;
-+		default:
-+			weston_log("gst_pipeline: %s from %s\n",
-+				GST_MESSAGE_TYPE_NAME(message), GST_OBJECT_NAME (message->src));
-+		break;
-+	}
-+	return TRUE;
-+}
-+
-+static void *
-+worker_thread_function(void *data)
-+{
-+	GstMessage *msg;
-+	struct gst_recorder *r = data;
-+
-+	pthread_mutex_lock(&r->mutex);
-+
-+	while (!r->destroying) {
-+		if (!r->input.valid)
-+			pthread_cond_wait(&r->input_cond, &r->mutex);
-+
-+		/* If the thread is awaken by destroy_worker_thread(),
-+		 * there might not be valid input */
-+		if (!r->input.valid)
-+			continue;
-+
-+		/* TODO: move it to separate thread? */
-+		g_main_context_iteration(r->gcontext, FALSE);
-+
-+		do {
-+			msg = gst_bus_pop_filtered(r->bus,
-+						   GST_MESSAGE_ANY);
-+			if (msg) {
-+				gst_bus_callback(r->bus, msg, r);
-+			}
-+		} while (msg);
-+
-+		/* check input */
-+		gst_recorder_process_dmafd(r, r->input.prime_fd, r->input.stride);
-+
-+		r->input.valid = 0;
-+	}
-+
-+	pthread_mutex_unlock(&r->mutex);
-+
-+	return NULL;
-+}
-+
-+static int
-+setup_worker_thread(struct gst_recorder *r)
-+{
-+	r->gcontext = g_main_context_new();
-+	pthread_mutex_init(&r->mutex, NULL);
-+	pthread_cond_init(&r->input_cond, NULL);
-+	pthread_create(&r->worker_thread, NULL, worker_thread_function, r);
-+
-+	return 1;
-+}
-+
-+static void
-+destroy_worker_thread(struct gst_recorder *r)
-+{
-+	/* Make sure the worker thread finishes */
-+	r->destroying = 1;
-+
-+	pthread_cond_signal(&r->input_cond);
-+
-+	pthread_join(r->worker_thread, NULL);
-+
-+	pthread_mutex_destroy(&r->mutex);
-+	pthread_cond_destroy(&r->input_cond);
-+}
-+
-+void weston_debug_function(GstDebugCategory* category, GstDebugLevel level,
-+                          const gchar* file, const char* function,
-+                          gint line, GObject* object, GstDebugMessage* message,
-+                          gpointer data)
-+{
-+	weston_log("[GST]:%s %s:%d %s\n", file, function, line, gst_debug_message_get(message));
-+}
-+
-+void
-+gst_recorder_init(void)
-+{
-+	gst_init(NULL, 0);
-+
-+	/* VSP init */
-+	vsp_g = vsp_init("/dev/media0");
-+	if (!vsp_g)
-+		weston_log("[gst recorder] VSP init failed\n");
-+}
-+
-+static int
-+gst_recorder_find_omx_pool(struct gst_recorder *r)
-+{
-+	int ret = 0;
-+	GstCaps *caps;
-+	GstQuery *query;
-+	GstBufferPool *pool;
-+	GstStructure *config;
-+	guint size, min, max;
-+
-+	caps = gst_caps_new_simple (	"video/x-raw",
-+#ifdef VSP_OUTPUT_NV16
-+					"format", G_TYPE_STRING, "NV16",
-+#else
-+					"format", G_TYPE_STRING, "NV12",
-+#endif
-+					"width", G_TYPE_INT, r->set->crop.width,
-+					"height", G_TYPE_INT, r->set->crop.height,
-+					"framerate", GST_TYPE_FRACTION, 0, DEFAULT_FPS,
-+					NULL);
-+
-+	/* find a pool for the negotiated caps now */
-+	query = gst_query_new_allocation (caps, TRUE);
-+
-+	if (!gst_pad_peer_query (r->appsrc_pad, query)) {
-+		/* query failed, not a problem, we use the query defaults */
-+		weston_log("allocation query failed\n");
-+		ret = -1;
-+		goto err;
-+	}
-+
-+	weston_log("goot %d pools\n", gst_query_get_n_allocation_pools (query));
-+	if (gst_query_get_n_allocation_pools (query) > 0) {
-+		/* we got configuration from our peer, parse them */
-+		gst_query_parse_nth_allocation_pool (query, 0, &pool, &size, &min, &max);
-+		weston_log(" pool settings size %d, min %d, max %d\n", size, min, max);
-+	} else {
-+		weston_log("no pool queried\n");
-+		ret = -1;
-+		goto err;
-+	}
-+
-+	config = gst_buffer_pool_get_config (pool);
-+	gst_buffer_pool_config_add_option (config, GST_BUFFER_POOL_OPTION_VIDEO_META);
-+	gst_buffer_pool_config_set_params (config, caps, size, min, max);
-+	gst_buffer_pool_set_config (pool, config);
-+
-+	/* and activate */
-+	gst_buffer_pool_set_active (pool, TRUE);
-+
-+	r->omx_pool = pool;
-+
-+err:
-+	gst_query_unref (query);
-+	return ret;
-+}
-+
-+static int
-+gst_recorder_omx_buffer_acquire(struct gst_recorder *r, GstBuffer **ret_buf, int fd[])
-+{
-+	unsigned int i;
-+	GstFlowReturn ret;
-+	GstBuffer *buf;
-+	guint n_mem;
-+	GstMemory *mem;
-+
-+	ret = gst_buffer_pool_acquire_buffer(r->omx_pool, &buf, NULL);
-+	if (ret != GST_FLOW_OK) {
-+		weston_log("OMX buffer acquire failed\n");
-+		return -1;
-+	}
-+
-+	n_mem = gst_buffer_n_memory(buf);
-+	if (n_mem < 1) {
-+		weston_log("Buffer with no mem!\n");
-+		goto err_release;
-+	}
-+
-+	for (i = 0; i < n_mem; i++) {
-+		mem = gst_buffer_peek_memory (buf, i);
-+		if (!gst_is_dmabuf_memory (mem)) {
-+			weston_log("Mem not dmabuf\n");
-+			goto err_release;
-+		}
-+		fd[i] = gst_dmabuf_memory_get_fd (mem);
-+	}
-+
-+	*ret_buf = buf;
-+
-+	return 0;
-+err_release:
-+	gst_buffer_pool_release_buffer(r->omx_pool, buf);
-+	return -1;
-+}
-+
-+static int
-+gst_recorder_omx_buffer_release(struct gst_recorder *r, GstBuffer *buf)
-+{
-+	gst_buffer_pool_release_buffer(r->omx_pool, buf);
-+
-+	return 0;
-+}
-+
-+struct gst_recorder *
-+gst_recorder_create(struct gst_recorder_settings *settings)
-+{
-+	struct gst_recorder *r;
-+	char gst_pipe[1024];
-+	char *ptr = gst_pipe;
-+	GError *perror = NULL;
-+
-+	weston_log("gst_recorder_create (%dx%d) crop %dx%d at %d,%d\n",
-+		   settings->width, settings->height, settings->crop.width,
-+		   settings->crop.height, settings->crop.top, settings->crop.left);
-+
-+	if (!vsp_g) {
-+		weston_log("gst_recorder_create: no VSP\n");
-+		return NULL;
-+	}
-+
-+	r = calloc(1, sizeof *r);
-+	if (!r)
-+		return NULL;
-+	memset(r, 0, sizeof *r);
-+
-+	r->set = settings;
-+	r->timestamp = 0;
-+
-+	r->vsp = vsp_g;
-+	vsp_g->users++;
-+
-+	/* GST init */
-+	/* source (GST_FORMAT_BYTES) */
-+	ptr += sprintf(ptr,
-+		"appsrc name=src ! ");
-+
-+	/* omx */
-+	ptr += sprintf(ptr,
-+		"omxh264enc target-bitrate=%d control-rate=2 name=my_encoder ! "
-+		"video/x-h264,width=%d,height=%d ! ",
-+		r->set->bitrate, r->set->crop.width, r->set->crop.height);
-+
-+	/* rtp payloader */
-+	ptr += sprintf(ptr,
-+		"rtph264pay config-interval=1 name=my_h264pay ! queue  ! ");
-+
-+	/* usp sink */
-+	ptr += sprintf(ptr,
-+		"udpsink host=%s ", r->set->ip);
-+	if (r->set->port > 0)
-+		ptr += sprintf(ptr,
-+			" port=%d name=my_udpsink", r->set->port);
-+
-+	weston_log("gst_pipeline: starting: %s\n", gst_pipe);
-+
-+	/* launch */
-+	r->pipeline = gst_parse_launch (gst_pipe, &perror);
-+	if (!r->pipeline) {
-+		weston_log("gst_pipeline: can not start pipeline: %s\n", perror->message);
-+		goto err_gst;
-+	}
-+
-+	/* get appsrc */
-+	r->appsrc = gst_bin_get_by_name(GST_BIN (r->pipeline), "src");
-+	if (!r->appsrc) {
-+		weston_log("gst_pipeline: can not get appsrc\n");
-+		goto err_gst;
-+	}
-+
-+	/* get bus */
-+	r->bus = gst_pipeline_get_bus (GST_PIPELINE(r->pipeline));
-+	if (!r->bus) {
-+		weston_log("gst_pipeline: can not get bus\n");
-+		goto err_gst;
-+	}
-+
-+	setup_worker_thread(r);
-+
-+	/* setup caps */
-+	g_object_set(G_OBJECT(r->appsrc), "caps",
-+		gst_caps_new_simple (	"video/x-raw",
-+#ifdef VSP_OUTPUT_NV16
-+					"format", G_TYPE_STRING, "NV16",
-+#else
-+					"format", G_TYPE_STRING, "NV12",
-+#endif
-+					"width", G_TYPE_INT, r->set->crop.width,
-+					"height", G_TYPE_INT, r->set->crop.height,
-+					"framerate", GST_TYPE_FRACTION, 0, DEFAULT_FPS,
-+					NULL), NULL);
-+
-+	r->appsrc_pad = gst_element_get_static_pad(GST_ELEMENT_CAST(r->appsrc), "src");
-+	if (!r->appsrc_pad)
-+		weston_log("Failed to get src0 pad of appsrc\n");
-+
-+	/* set playing */
-+	if (gst_element_set_state (r->pipeline, GST_STATE_PLAYING) == GST_STATE_CHANGE_FAILURE) {
-+		weston_log("gst_pipeline: can not change state to PLAYING\n");
-+		goto err_gst;
-+	}
-+
-+	if (gst_recorder_find_omx_pool(r) != 0) {
-+		weston_log("failed to find OMX buffer pool\n");
-+		goto err_gst_stop;
-+	}
-+
-+	weston_log("gst_recorder_create done\n");
-+
-+	return r;
-+
-+err_gst_stop:
-+	gst_element_set_state (r->pipeline, GST_STATE_NULL);
-+	destroy_worker_thread(r);
-+err_gst:
-+	free(r->pipeline);
-+	free(r);
-+
-+	return NULL;
-+}
-+
-+void
-+gst_recorder_destroy(struct gst_recorder *r)
-+{
-+	r->vsp->users--;
-+
-+	if (r->pipeline) {
-+		gst_element_set_state (r->pipeline, GST_STATE_NULL);
-+		gst_object_unref(r->omx_pool);
-+
-+		destroy_worker_thread(r);
-+
-+		gst_object_unref(GST_OBJECT(r->bus));
-+		gst_object_unref (r->pipeline);
-+		r->pipeline = NULL;
-+	}
-+	free(r);
-+}
-+
-+static int
-+gst_recorder_set_timestamp(struct gst_recorder *r, GstBuffer *buffer)
-+{
-+	uint32_t cur_time = weston_compositor_get_time();
-+
-+	if (r->timestamp == 0) {
-+		/* first frame assume around DEFAULT_FPS FPS */
-+		GST_BUFFER_DURATION(buffer) = gst_util_uint64_scale_int(1, GST_SECOND, DEFAULT_FPS);
-+	} else {
-+		uint32_t delta = cur_time - r->ts_last_frame;
-+		/* delta in mS */
-+		GST_BUFFER_DURATION(buffer) = gst_util_uint64_scale_int(delta, GST_SECOND, 1000);
-+	}
-+
-+	r->timestamp += GST_BUFFER_DURATION(buffer);
-+	GST_BUFFER_PTS(buffer) = r->timestamp;
-+	GST_BUFFER_DTS(buffer) = r->timestamp;
-+
-+	r->ts_last_frame = cur_time;
-+
-+	return 0;
-+}
-+
-+
-+static int
-+gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
-+{
-+	int ret;
-+	GstBuffer *buf;
-+	int omx_fd[2];
-+
-+        /* get GST buffer */
-+	if (gst_recorder_omx_buffer_acquire(r, &buf, omx_fd) < 0) {
-+		weston_log("VSP: can not acquire GST buffer, dropping frame\n");
-+		return 0;
-+	}
-+
-+	pthread_mutex_lock(&r->vsp->mutex);
-+	/* setup vsp */
-+	if (vsp_set_formats(r->vsp, r->set->width, r->set->height, &r->set->crop) < 0) {
-+		weston_log("VSP: format set failed\n");
-+		goto err;
-+	}
-+
-+	/* input */
-+	if (vsp_request_buffers(r->vsp, VSP_PORT_INPUT, 1) < 0) {
-+		weston_log("VSP: input buffer allocation failed\n");
-+		goto err_vsp;
-+	}
-+
-+	/* output */
-+	if (vsp_request_buffers(r->vsp, VSP_PORT_OUTPUT, 1) < 0) {
-+		weston_log("VSP: output buffer allocation failed\n");
-+		goto err_vsp;
-+	}
-+
-+	/* queue output biffer */
-+	if (vsp_output_buffer_queue_dmafd(r->vsp, 0, omx_fd) < 0) {
-+		weston_log("can not queue OMX buffer %d to VSP\n", 0);
-+		gst_recorder_omx_buffer_release(r, buf);
-+		goto err_vsp;
-+	}
-+
-+	/* queue input vsp buffer */
-+	if (vsp_input_buffer_queue_dmafd(r->vsp, 0, fd) < 0) {
-+		weston_log("VSP: failed to queue input buffer\n");
-+		goto err_vsp;
-+	}
-+
-+	/* start input */
-+	if (vsp_streaming_enable(r->vsp, VSP_PORT_INPUT, 1) < 0) {
-+		weston_log("VSP: failed to start input\n");
-+		goto err_vsp;
-+	}
-+
-+	/* start output */
-+	if (vsp_streaming_enable(r->vsp, VSP_PORT_OUTPUT, 1) < 0) {
-+		weston_log("VSP: failed to start output\n");
-+		goto err_vsp;
-+	}
-+
-+	/* dequeue input (do we need this?) */
-+	if (vsp_input_buffer_dequeue_dmafd(r->vsp) < 0) {
-+		weston_log("VSP: failed to dequeue input buffer\n");
-+		/* don't care */
-+	}
-+
-+	/* dequeue output */
-+	if (vsp_output_buffer_dequeue_dmafd(r->vsp) < 0) {
-+		weston_log("VSP: failed to dequeu output buffer\n");
-+		gst_recorder_omx_buffer_release(r, buf);
-+		/* fall through */
-+	} else {
-+		/* set timestamp */
-+		gst_recorder_set_timestamp(r, buf);
-+
-+		ret = gst_app_src_push_buffer(r->appsrc, buf);
-+		r->frame_count++;
-+
-+		if (ret != GST_FLOW_OK) {
-+			/* some error, stop sending data */
-+			weston_log("gst_pipeline: some error %d\n", ret);
-+		}
-+
-+	}
-+	/* stop input */
-+	vsp_streaming_enable(r->vsp, VSP_PORT_INPUT, 0);
-+	/* stop output */
-+	vsp_streaming_enable(r->vsp, VSP_PORT_OUTPUT, 0);
-+
-+	/* deinit */
-+	vsp_request_buffers(r->vsp, VSP_PORT_INPUT, 0);
-+	vsp_request_buffers(r->vsp, VSP_PORT_OUTPUT, 0);
-+
-+	pthread_mutex_unlock(&r->vsp->mutex);
-+	return 0;
-+
-+err_vsp:
-+	/* drop gst buffer */
-+	/* finish vsp here */
-+err:
-+	pthread_mutex_unlock(&r->vsp->mutex);
-+	return -1;
-+}
-+
-+int
-+gst_recorder_frame_dmafd(struct gst_recorder *r, int fd, int stride)
-+{
-+	int ret = 0;
-+	
-+	pthread_mutex_lock(&r->mutex);
-+
-+	if (r->error) {
-+		errno = r->error;
-+		ret = -1;
-+		goto unlock;
-+	}
-+		
-+	/* The mutex is never released while encoding, so this point should
-+	 * never be reached if input.valid is true. */
-+	assert(!r->input.valid);
-+
-+	r->input.prime_fd = fd;
-+	r->input.stride = stride;
-+	r->input.valid = 1;
-+	pthread_cond_signal(&r->input_cond);
-+
-+unlock:
-+	pthread_mutex_unlock(&r->mutex);
-+
-+	return 0;
-+}
-diff --git a/src/gst-recorder.h b/src/gst-recorder.h
-new file mode 100644
-index 0000000..e1c53ff
---- /dev/null
-+++ b/src/gst-recorder.h
-@@ -0,0 +1,58 @@
-+/*
-+ * Copyright © 2016 Cogent Embedded Inc
-+ *
-+ * 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.
-+ */
-+
-+#ifndef _GST_RECORDER_H_
-+#define _GST_RECORDER_H_
-+
-+/* VSP includes */
-+#include <media-ctl/mediactl.h>
-+#include <media-ctl/v4l2subdev.h>
-+
-+struct gst_recorder;
-+
-+struct gst_recorder_settings {
-+	int width;
-+	int height;
-+	int bitrate;
-+	char *ip;
-+	int port;
-+	int latency_test;
-+	int refresh_ratio;
-+
-+	/* Cropping */
-+	struct v4l2_rect crop;
-+};
-+
-+void
-+gst_recorder_init(void);
-+struct gst_recorder *
-+gst_recorder_create(struct gst_recorder_settings *settings);
-+void
-+gst_recorder_destroy(struct gst_recorder *r);
-+int
-+gst_recorder_frame(struct gst_recorder *r, int fd, int stride);
-+int
-+gst_recorder_frame_mmap(struct gst_recorder *r, void *data, int stride);
-+int
-+gst_recorder_frame_dmafd(struct gst_recorder *r, int fd, int stride);
-+
-+#endif /* _GST_RECORDER_H_ */
-diff --git a/src/main.c b/src/main.c
-index c2168eb..e287a88 100644
---- a/src/main.c
-+++ b/src/main.c
-@@ -278,7 +278,8 @@ usage(int error_code)
- 		"  --tty=TTY\t\tThe tty to use\n"
- 		"  --use-pixman\t\tUse the pixman (CPU) renderer\n"
- 		"  --use-v4l2\t\tUse the v4l2 renderer\n"
--		"  --current-mode\tPrefer current KMS mode over EDID preferred mode\n\n");
-+		"  --current-mode\tPrefer current KMS mode over EDID preferred mode\n"
-+		"  --gst-record\t\tEnable GStreamer recording\n\n");
- #endif
- 
- #if defined(BUILD_FBDEV_COMPOSITOR)
-@@ -752,6 +753,7 @@ load_drm_backend(struct weston_compositor *c, const char *backend,
- 		{ WESTON_OPTION_BOOLEAN, "current-mode", 0, &config.use_current_mode },
- 		{ WESTON_OPTION_BOOLEAN, "use-pixman", 0, &config.use_pixman },
- 		{ WESTON_OPTION_BOOLEAN, "use-v4l2", 0, &config.use_v4l2 },
-+		{ WESTON_OPTION_BOOLEAN, "gst-record", 0, &config.enable_recorder },
- 	};
- 
- 	parse_options(options, ARRAY_LENGTH(options), argc, argv);
-diff --git a/src/media-ctl/libmediactl.c b/src/media-ctl/libmediactl.c
-new file mode 100644
-index 0000000..f15b1a3
---- /dev/null
-+++ b/src/media-ctl/libmediactl.c
-@@ -0,0 +1,955 @@
-+/*
-+ * Media controller interface library
-+ *
-+ * Copyright (C) 2010-2014 Ideas on board SPRL
-+ *
-+ * Contact: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-+ *
-+ * This program is free software; you can redistribute it and/or modify
-+ * it under the terms of the GNU Lesser General Public License as published
-+ * by the Free Software Foundation; either version 2.1 of the License, or
-+ * (at your option) any later version.
-+ *
-+ * This program is distributed in the hope that it will be useful,
-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-+ * GNU Lesser General Public License for more details.
-+ *
-+ * You should have received a copy of the GNU Lesser General Public License
-+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
-+ */
-+
-+#include "config.h"
-+
-+#include <sys/ioctl.h>
-+#include <sys/stat.h>
-+#include <sys/types.h>
-+
-+#include <ctype.h>
-+#include <errno.h>
-+#include <fcntl.h>
-+#include <stdbool.h>
-+#include <stdio.h>
-+#include <stdlib.h>
-+#include <string.h>
-+#include <unistd.h>
-+
-+#include <linux/media.h>
-+#include <linux/videodev2.h>
-+
-+#include "mediactl.h"
-+#include "mediactl-priv.h"
-+#include "tools.h"
-+
-+/* -----------------------------------------------------------------------------
-+ * Graph access
-+ */
-+
-+struct media_pad *media_entity_remote_source(struct media_pad *pad)
-+{
-+	unsigned int i;
-+
-+	if (!(pad->flags & MEDIA_PAD_FL_SINK))
-+		return NULL;
-+
-+	for (i = 0; i < pad->entity->num_links; ++i) {
-+		struct media_link *link = &pad->entity->links[i];
-+
-+		if (!(link->flags & MEDIA_LNK_FL_ENABLED))
-+			continue;
-+
-+		if (link->sink == pad)
-+			return link->source;
-+	}
-+
-+	return NULL;
-+}
-+
-+struct media_entity *media_get_entity_by_name(struct media_device *media,
-+					      const char *name, size_t length)
-+{
-+	unsigned int i;
-+
-+	/* A match is impossible if the entity name is longer than the maximum
-+	 * size we can get from the kernel.
-+	 */
-+	if (length >= FIELD_SIZEOF(struct media_entity_desc, name))
-+		return NULL;
-+
-+	for (i = 0; i < media->entities_count; ++i) {
-+		struct media_entity *entity = &media->entities[i];
-+
-+		if (strncmp(entity->info.name, name, length) == 0 &&
-+		    entity->info.name[length] == '\0')
-+			return entity;
-+	}
-+
-+	return NULL;
-+}
-+
-+struct media_entity *media_get_entity_by_id(struct media_device *media,
-+					    __u32 id)
-+{
-+	bool next = id & MEDIA_ENT_ID_FLAG_NEXT;
-+	unsigned int i;
-+
-+	id &= ~MEDIA_ENT_ID_FLAG_NEXT;
-+
-+	for (i = 0; i < media->entities_count; ++i) {
-+		struct media_entity *entity = &media->entities[i];
-+
-+		if ((entity->info.id == id && !next) ||
-+		    (entity->info.id > id && next))
-+			return entity;
-+	}
-+
-+	return NULL;
-+}
-+
-+unsigned int media_get_entities_count(struct media_device *media)
-+{
-+	return media->entities_count;
-+}
-+
-+struct media_entity *media_get_entity(struct media_device *media, unsigned int index)
-+{
-+	if (index >= media->entities_count)
-+		return NULL;
-+
-+	return &media->entities[index];
-+}
-+
-+const struct media_pad *media_entity_get_pad(struct media_entity *entity, unsigned int index)
-+{
-+	if (index >= entity->info.pads)
-+		return NULL;
-+
-+	return &entity->pads[index];
-+}
-+
-+unsigned int media_entity_get_links_count(struct media_entity *entity)
-+{
-+	return entity->num_links;
-+}
-+
-+const struct media_link *media_entity_get_link(struct media_entity *entity, unsigned int index)
-+{
-+	if (index >= entity->num_links)
-+		return NULL;
-+
-+	return &entity->links[index];
-+}
-+
-+const char *media_entity_get_devname(struct media_entity *entity)
-+{
-+	return entity->devname[0] ? entity->devname : NULL;
-+}
-+
-+struct media_entity *media_get_default_entity(struct media_device *media,
-+					      unsigned int type)
-+{
-+	switch (type) {
-+	case MEDIA_ENT_T_DEVNODE_V4L:
-+		return media->def.v4l;
-+	case MEDIA_ENT_T_DEVNODE_FB:
-+		return media->def.fb;
-+	case MEDIA_ENT_T_DEVNODE_ALSA:
-+		return media->def.alsa;
-+	case MEDIA_ENT_T_DEVNODE_DVB:
-+		return media->def.dvb;
-+	}
-+
-+	return NULL;
-+}
-+
-+const struct media_device_info *media_get_info(struct media_device *media)
-+{
-+	return &media->info;
-+}
-+
-+const char *media_get_devnode(struct media_device *media)
-+{
-+	return media->devnode;
-+}
-+
-+const struct media_entity_desc *media_entity_get_info(struct media_entity *entity)
-+{
-+	return &entity->info;
-+}
-+
-+/* -----------------------------------------------------------------------------
-+ * Open/close
-+ */
-+
-+static int media_device_open(struct media_device *media)
-+{
-+	int ret;
-+
-+	if (media->fd != -1)
-+		return 0;
-+
-+	media_dbg(media, "Opening media device %s\n", media->devnode);
-+
-+	media->fd = open(media->devnode, O_RDWR);
-+	if (media->fd < 0) {
-+		ret = -errno;
-+		media_dbg(media, "%s: Can't open media device %s\n",
-+			  __func__, media->devnode);
-+		return ret;
-+	}
-+
-+	return 0;
-+}
-+
-+static void media_device_close(struct media_device *media)
-+{
-+	if (media->fd != -1) {
-+		close(media->fd);
-+		media->fd = -1;
-+	}
-+}
-+
-+/* -----------------------------------------------------------------------------
-+ * Link setup
-+ */
-+
-+int media_setup_link(struct media_device *media,
-+		     struct media_pad *source,
-+		     struct media_pad *sink,
-+		     __u32 flags)
-+{
-+	struct media_link *link;
-+	struct media_link_desc ulink;
-+	unsigned int i;
-+	int ret;
-+
-+	ret = media_device_open(media);
-+	if (ret < 0)
-+		goto done;
-+
-+	for (i = 0; i < source->entity->num_links; i++) {
-+		link = &source->entity->links[i];
-+
-+		if (link->source->entity == source->entity &&
-+		    link->source->index == source->index &&
-+		    link->sink->entity == sink->entity &&
-+		    link->sink->index == sink->index)
-+			break;
-+	}
-+
-+	if (i == source->entity->num_links) {
-+		media_dbg(media, "%s: Link not found\n", __func__);
-+		ret = -ENOENT;
-+		goto done;
-+	}
-+
-+	/* source pad */
-+	ulink.source.entity = source->entity->info.id;
-+	ulink.source.index = source->index;
-+	ulink.source.flags = MEDIA_PAD_FL_SOURCE;
-+
-+	/* sink pad */
-+	ulink.sink.entity = sink->entity->info.id;
-+	ulink.sink.index = sink->index;
-+	ulink.sink.flags = MEDIA_PAD_FL_SINK;
-+
-+	ulink.flags = flags | (link->flags & MEDIA_LNK_FL_IMMUTABLE);
-+
-+	ret = ioctl(media->fd, MEDIA_IOC_SETUP_LINK, &ulink);
-+	if (ret == -1) {
-+		ret = -errno;
-+		media_dbg(media, "%s: Unable to setup link (%s)\n",
-+			  __func__, strerror(errno));
-+		goto done;
-+	}
-+
-+	link->flags = ulink.flags;
-+	link->twin->flags = ulink.flags;
-+
-+	ret = 0;
-+
-+done:
-+	return ret;
-+}
-+
-+int media_reset_links(struct media_device *media)
-+{
-+	unsigned int i, j;
-+	int ret;
-+
-+	for (i = 0; i < media->entities_count; ++i) {
-+		struct media_entity *entity = &media->entities[i];
-+
-+		for (j = 0; j < entity->num_links; j++) {
-+			struct media_link *link = &entity->links[j];
-+
-+			if (link->flags & MEDIA_LNK_FL_IMMUTABLE ||
-+			    link->source->entity != entity)
-+				continue;
-+
-+			ret = media_setup_link(media, link->source, link->sink,
-+					       link->flags & ~MEDIA_LNK_FL_ENABLED);
-+			if (ret < 0)
-+				return ret;
-+		}
-+	}
-+
-+	return 0;
-+}
-+
-+/* -----------------------------------------------------------------------------
-+ * Entities, pads and links enumeration
-+ */
-+
-+static struct media_link *media_entity_add_link(struct media_entity *entity)
-+{
-+	if (entity->num_links >= entity->max_links) {
-+		struct media_link *links = entity->links;
-+		unsigned int max_links = entity->max_links * 2;
-+		unsigned int i;
-+
-+		links = realloc(links, max_links * sizeof *links);
-+		if (links == NULL)
-+			return NULL;
-+
-+		for (i = 0; i < entity->num_links; ++i)
-+			links[i].twin->twin = &links[i];
-+
-+		entity->max_links = max_links;
-+		entity->links = links;
-+	}
-+
-+	return &entity->links[entity->num_links++];
-+}
-+
-+static int media_enum_links(struct media_device *media)
-+{
-+	__u32 id;
-+	int ret = 0;
-+
-+	for (id = 1; id <= media->entities_count; id++) {
-+		struct media_entity *entity = &media->entities[id - 1];
-+		struct media_links_enum links;
-+		unsigned int i;
-+
-+		links.entity = entity->info.id;
-+		links.pads = calloc(entity->info.pads, sizeof(struct media_pad_desc));
-+		links.links = calloc(entity->info.links, sizeof(struct media_link_desc));
-+
-+		if (ioctl(media->fd, MEDIA_IOC_ENUM_LINKS, &links) < 0) {
-+			ret = -errno;
-+			media_dbg(media,
-+				  "%s: Unable to enumerate pads and links (%s).\n",
-+				  __func__, strerror(errno));
-+			free(links.pads);
-+			free(links.links);
-+			return ret;
-+		}
-+
-+		for (i = 0; i < entity->info.pads; ++i) {
-+			entity->pads[i].entity = entity;
-+			entity->pads[i].index = links.pads[i].index;
-+			entity->pads[i].flags = links.pads[i].flags;
-+		}
-+
-+		for (i = 0; i < entity->info.links; ++i) {
-+			struct media_link_desc *link = &links.links[i];
-+			struct media_link *fwdlink;
-+			struct media_link *backlink;
-+			struct media_entity *source;
-+			struct media_entity *sink;
-+
-+			source = media_get_entity_by_id(media, link->source.entity);
-+			sink = media_get_entity_by_id(media, link->sink.entity);
-+
-+			if (source == NULL || sink == NULL) {
-+				media_dbg(media,
-+					  "WARNING entity %u link %u from %u/%u to %u/%u is invalid!\n",
-+					  id, i, link->source.entity,
-+					  link->source.index,
-+					  link->sink.entity,
-+					  link->sink.index);
-+				ret = -EINVAL;
-+			} else {
-+				fwdlink = media_entity_add_link(source);
-+				fwdlink->source = &source->pads[link->source.index];
-+				fwdlink->sink = &sink->pads[link->sink.index];
-+				fwdlink->flags = link->flags;
-+
-+				backlink = media_entity_add_link(sink);
-+				backlink->source = &source->pads[link->source.index];
-+				backlink->sink = &sink->pads[link->sink.index];
-+				backlink->flags = link->flags;
-+
-+				fwdlink->twin = backlink;
-+				backlink->twin = fwdlink;
-+			}
-+		}
-+
-+		free(links.pads);
-+		free(links.links);
-+	}
-+
-+	return ret;
-+}
-+
-+#ifdef HAVE_LIBUDEV
-+
-+#include <libudev.h>
-+
-+static inline int media_udev_open(struct udev **udev)
-+{
-+	*udev = udev_new();
-+	if (*udev == NULL)
-+		return -ENOMEM;
-+	return 0;
-+}
-+
-+static inline void media_udev_close(struct udev *udev)
-+{
-+	if (udev != NULL)
-+		udev_unref(udev);
-+}
-+
-+static int media_get_devname_udev(struct udev *udev,
-+		struct media_entity *entity)
-+{
-+	struct udev_device *device;
-+	dev_t devnum;
-+	const char *p;
-+	int ret = -ENODEV;
-+
-+	if (udev == NULL)
-+		return -EINVAL;
-+
-+	devnum = makedev(entity->info.v4l.major, entity->info.v4l.minor);
-+	media_dbg(entity->media, "looking up device: %u:%u\n",
-+		  major(devnum), minor(devnum));
-+	device = udev_device_new_from_devnum(udev, 'c', devnum);
-+	if (device) {
-+		p = udev_device_get_devnode(device);
-+		if (p) {
-+			strncpy(entity->devname, p, sizeof(entity->devname));
-+			entity->devname[sizeof(entity->devname) - 1] = '\0';
-+		}
-+		ret = 0;
-+	}
-+
-+	udev_device_unref(device);
-+
-+	return ret;
-+}
-+
-+#else	/* HAVE_LIBUDEV */
-+
-+struct udev;
-+
-+static inline int media_udev_open(struct udev **udev) { return 0; }
-+
-+static inline void media_udev_close(struct udev *udev) { }
-+
-+static inline int media_get_devname_udev(struct udev *udev,
-+		struct media_entity *entity)
-+{
-+	return -ENOTSUP;
-+}
-+
-+#endif	/* HAVE_LIBUDEV */
-+
-+static int media_get_devname_sysfs(struct media_entity *entity)
-+{
-+	struct stat devstat;
-+	char devname[32];
-+	char sysname[32];
-+	char target[1024];
-+	char *p;
-+	int ret;
-+
-+	sprintf(sysname, "/sys/dev/char/%u:%u", entity->info.v4l.major,
-+		entity->info.v4l.minor);
-+	ret = readlink(sysname, target, sizeof(target) - 1);
-+	if (ret < 0)
-+		return -errno;
-+
-+	target[ret] = '\0';
-+	p = strrchr(target, '/');
-+	if (p == NULL)
-+		return -EINVAL;
-+
-+	sprintf(devname, "/dev/%s", p + 1);
-+	ret = stat(devname, &devstat);
-+	if (ret < 0)
-+		return -errno;
-+
-+	/* Sanity check: udev might have reordered the device nodes.
-+	 * Make sure the major/minor match. We should really use
-+	 * libudev.
-+	 */
-+	if (major(devstat.st_rdev) == entity->info.v4l.major &&
-+	    minor(devstat.st_rdev) == entity->info.v4l.minor)
-+		strcpy(entity->devname, devname);
-+
-+	return 0;
-+}
-+
-+static int media_enum_entities(struct media_device *media)
-+{
-+	struct media_entity *entity;
-+	struct udev *udev;
-+	unsigned int size;
-+	__u32 id;
-+	int ret;
-+
-+	ret = media_udev_open(&udev);
-+	if (ret < 0)
-+		media_dbg(media, "Can't get udev context\n");
-+
-+	for (id = 0, ret = 0; ; id = entity->info.id) {
-+		size = (media->entities_count + 1) * sizeof(*media->entities);
-+		media->entities = realloc(media->entities, size);
-+
-+		entity = &media->entities[media->entities_count];
-+		memset(entity, 0, sizeof(*entity));
-+		entity->fd = -1;
-+		entity->info.id = id | MEDIA_ENT_ID_FLAG_NEXT;
-+		entity->media = media;
-+
-+		ret = ioctl(media->fd, MEDIA_IOC_ENUM_ENTITIES, &entity->info);
-+		if (ret < 0) {
-+			ret = errno != EINVAL ? -errno : 0;
-+			break;
-+		}
-+
-+		/* Number of links (for outbound links) plus number of pads (for
-+		 * inbound links) is a good safe initial estimate of the total
-+		 * number of links.
-+		 */
-+		entity->max_links = entity->info.pads + entity->info.links;
-+
-+		entity->pads = malloc(entity->info.pads * sizeof(*entity->pads));
-+		entity->links = malloc(entity->max_links * sizeof(*entity->links));
-+		if (entity->pads == NULL || entity->links == NULL) {
-+			ret = -ENOMEM;
-+			break;
-+		}
-+
-+		media->entities_count++;
-+
-+		if (entity->info.flags & MEDIA_ENT_FL_DEFAULT) {
-+			switch (entity->info.type) {
-+			case MEDIA_ENT_T_DEVNODE_V4L:
-+				media->def.v4l = entity;
-+				break;
-+			case MEDIA_ENT_T_DEVNODE_FB:
-+				media->def.fb = entity;
-+				break;
-+			case MEDIA_ENT_T_DEVNODE_ALSA:
-+				media->def.alsa = entity;
-+				break;
-+			case MEDIA_ENT_T_DEVNODE_DVB:
-+				media->def.dvb = entity;
-+				break;
-+			}
-+		}
-+
-+		/* Find the corresponding device name. */
-+		if (media_entity_type(entity) != MEDIA_ENT_T_DEVNODE &&
-+		    media_entity_type(entity) != MEDIA_ENT_T_V4L2_SUBDEV)
-+			continue;
-+
-+		/* Try to get the device name via udev */
-+		if (!media_get_devname_udev(udev, entity))
-+			continue;
-+
-+		/* Fall back to get the device name via sysfs */
-+		media_get_devname_sysfs(entity);
-+	}
-+
-+	media_udev_close(udev);
-+	return ret;
-+}
-+
-+int media_device_enumerate(struct media_device *media)
-+{
-+	int ret;
-+
-+	if (media->entities)
-+		return 0;
-+
-+	ret = media_device_open(media);
-+	if (ret < 0)
-+		return ret;
-+
-+	ret = ioctl(media->fd, MEDIA_IOC_DEVICE_INFO, &media->info);
-+	if (ret < 0) {
-+		ret = -errno;
-+		media_dbg(media, "%s: Unable to retrieve media device "
-+			  "information for device %s (%s)\n", __func__,
-+			  media->devnode, strerror(errno));
-+		goto done;
-+	}
-+
-+	media_dbg(media, "Enumerating entities\n");
-+
-+	ret = media_enum_entities(media);
-+	if (ret < 0) {
-+		media_dbg(media,
-+			  "%s: Unable to enumerate entities for device %s (%s)\n",
-+			  __func__, media->devnode, strerror(-ret));
-+		goto done;
-+	}
-+
-+	media_dbg(media, "Found %u entities\n", media->entities_count);
-+	media_dbg(media, "Enumerating pads and links\n");
-+
-+	ret = media_enum_links(media);
-+	if (ret < 0) {
-+		media_dbg(media,
-+			  "%s: Unable to enumerate pads and linksfor device %s\n",
-+			  __func__, media->devnode);
-+		goto done;
-+	}
-+
-+	ret = 0;
-+
-+done:
-+	return ret;
-+}
-+
-+/* -----------------------------------------------------------------------------
-+ * Create/destroy
-+ */
-+
-+static void media_debug_default(void *ptr, ...)
-+{
-+}
-+
-+void media_debug_set_handler(struct media_device *media,
-+			     void (*debug_handler)(void *, ...),
-+			     void *debug_priv)
-+{
-+	if (debug_handler) {
-+		media->debug_handler = debug_handler;
-+		media->debug_priv = debug_priv;
-+	} else {
-+		media->debug_handler = media_debug_default;
-+		media->debug_priv = NULL;
-+	}
-+}
-+
-+static struct media_device *__media_device_new(void)
-+{
-+	struct media_device *media;
-+
-+	media = calloc(1, sizeof(*media));
-+	if (media == NULL)
-+		return NULL;
-+
-+	media->fd = -1;
-+	media->refcount = 1;
-+
-+	media_debug_set_handler(media, NULL, NULL);
-+
-+	return media;
-+}
-+
-+struct media_device *media_device_new(const char *devnode)
-+{
-+	struct media_device *media;
-+
-+	media = __media_device_new();
-+	if (media == NULL)
-+		return NULL;
-+
-+	media->devnode = strdup(devnode);
-+	if (media->devnode == NULL) {
-+		media_device_unref(media);
-+		return NULL;
-+	}
-+
-+	return media;
-+}
-+
-+struct media_device *media_device_new_emulated(struct media_device_info *info)
-+{
-+	struct media_device *media;
-+
-+	media = __media_device_new();
-+	if (media == NULL)
-+		return NULL;
-+
-+	media->info = *info;
-+
-+	return media;
-+}
-+
-+struct media_device *media_device_ref(struct media_device *media)
-+{
-+	media->refcount++;
-+	return media;
-+}
-+
-+void media_device_unref(struct media_device *media)
-+{
-+	unsigned int i;
-+
-+	media->refcount--;
-+	if (media->refcount > 0)
-+		return;
-+
-+	for (i = 0; i < media->entities_count; ++i) {
-+		struct media_entity *entity = &media->entities[i];
-+
-+		free(entity->pads);
-+		free(entity->links);
-+		if (entity->fd != -1)
-+			close(entity->fd);
-+	}
-+
-+	free(media->entities);
-+	free(media->devnode);
-+	free(media);
-+}
-+
-+int media_device_add_entity(struct media_device *media,
-+			    const struct media_entity_desc *desc,
-+			    const char *devnode)
-+{
-+	struct media_entity **defent = NULL;
-+	struct media_entity *entity;
-+	unsigned int size;
-+
-+	size = (media->entities_count + 1) * sizeof(*media->entities);
-+	entity = realloc(media->entities, size);
-+	if (entity == NULL)
-+		return -ENOMEM;
-+
-+	media->entities = entity;
-+	media->entities_count++;
-+
-+	entity = &media->entities[media->entities_count - 1];
-+	memset(entity, 0, sizeof *entity);
-+
-+	entity->fd = -1;
-+	entity->media = media;
-+	strncpy(entity->devname, devnode, sizeof entity->devname);
-+	entity->devname[sizeof entity->devname - 1] = '\0';
-+
-+	entity->info.id = 0;
-+	entity->info.type = desc->type;
-+	entity->info.flags = 0;
-+	memcpy(entity->info.name, desc->name, sizeof entity->info.name);
-+
-+	switch (entity->info.type) {
-+	case MEDIA_ENT_T_DEVNODE_V4L:
-+		defent = &media->def.v4l;
-+		entity->info.v4l = desc->v4l;
-+		break;
-+	case MEDIA_ENT_T_DEVNODE_FB:
-+		defent = &media->def.fb;
-+		entity->info.fb = desc->fb;
-+		break;
-+	case MEDIA_ENT_T_DEVNODE_ALSA:
-+		defent = &media->def.alsa;
-+		entity->info.alsa = desc->alsa;
-+		break;
-+	case MEDIA_ENT_T_DEVNODE_DVB:
-+		defent = &media->def.dvb;
-+		entity->info.dvb = desc->dvb;
-+		break;
-+	}
-+
-+	if (desc->flags & MEDIA_ENT_FL_DEFAULT) {
-+		entity->info.flags |= MEDIA_ENT_FL_DEFAULT;
-+		if (defent)
-+			*defent = entity;
-+	}
-+
-+	return 0;
-+}
-+
-+struct media_pad *media_parse_pad(struct media_device *media,
-+				  const char *p, char **endp)
-+{
-+	unsigned int entity_id, pad;
-+	struct media_entity *entity;
-+	char *end;
-+
-+	/* endp can be NULL. To avoid spreading NULL checks across the function,
-+	 * set endp to &end in that case.
-+	 */
-+	if (endp == NULL)
-+		endp = &end;
-+
-+	for (; isspace(*p); ++p);
-+
-+	if (*p == '"' || *p == '\'') {
-+		for (end = (char *)p + 1; *end && *end != '"' && *end != '\''; ++end);
-+		if (*end != '"' && *end != '\'') {
-+			media_dbg(media, "missing matching '\"'\n");
-+			*endp = end;
-+			return NULL;
-+		}
-+
-+		entity = media_get_entity_by_name(media, p + 1, end - p - 1);
-+		if (entity == NULL) {
-+			media_dbg(media, "no such entity \"%.*s\"\n", end - p - 1, p + 1);
-+			*endp = (char *)p + 1;
-+			return NULL;
-+		}
-+
-+		++end;
-+	} else {
-+		entity_id = strtoul(p, &end, 10);
-+		entity = media_get_entity_by_id(media, entity_id);
-+		if (entity == NULL) {
-+			media_dbg(media, "no such entity %d\n", entity_id);
-+			*endp = (char *)p;
-+			return NULL;
-+		}
-+	}
-+	for (; isspace(*end); ++end);
-+
-+	if (*end != ':') {
-+		media_dbg(media, "Expected ':'\n", *end);
-+		*endp = end;
-+		return NULL;
-+	}
-+
-+	for (p = end + 1; isspace(*p); ++p);
-+
-+	pad = strtoul(p, &end, 10);
-+
-+	if (pad >= entity->info.pads) {
-+		media_dbg(media, "No pad '%d' on entity \"%s\". Maximum pad number is %d\n",
-+				pad, entity->info.name, entity->info.pads - 1);
-+		*endp = (char *)p;
-+		return NULL;
-+	}
-+
-+	for (p = end; isspace(*p); ++p);
-+	*endp = (char *)p;
-+
-+	return &entity->pads[pad];
-+}
-+
-+struct media_link *media_parse_link(struct media_device *media,
-+				    const char *p, char **endp)
-+{
-+	struct media_link *link;
-+	struct media_pad *source;
-+	struct media_pad *sink;
-+	unsigned int i;
-+	char *end;
-+
-+	source = media_parse_pad(media, p, &end);
-+	if (source == NULL) {
-+		*endp = end;
-+		return NULL;
-+	}
-+
-+	if (end[0] != '-' || end[1] != '>') {
-+		*endp = end;
-+		media_dbg(media, "Expected '->'\n");
-+		return NULL;
-+	}
-+
-+	p = end + 2;
-+
-+	sink = media_parse_pad(media, p, &end);
-+	if (sink == NULL) {
-+		*endp = end;
-+		return NULL;
-+	}
-+
-+	*endp = end;
-+
-+	for (i = 0; i < source->entity->num_links; i++) {
-+		link = &source->entity->links[i];
-+
-+		if (link->source == source && link->sink == sink)
-+			return link;
-+	}
-+
-+	media_dbg(media, "No link between \"%s\":%d and \"%s\":%d\n",
-+			source->entity->info.name, source->index,
-+			sink->entity->info.name, sink->index);
-+	return NULL;
-+}
-+
-+int media_parse_setup_link(struct media_device *media,
-+			   const char *p, char **endp)
-+{
-+	struct media_link *link;
-+	__u32 flags;
-+	char *end;
-+
-+	link = media_parse_link(media, p, &end);
-+	if (link == NULL) {
-+		media_dbg(media,
-+			  "%s: Unable to parse link\n", __func__);
-+		*endp = end;
-+		return -EINVAL;
-+	}
-+
-+	p = end;
-+	if (*p++ != '[') {
-+		media_dbg(media, "Unable to parse link flags: expected '['.\n");
-+		*endp = (char *)p - 1;
-+		return -EINVAL;
-+	}
-+
-+	flags = strtoul(p, &end, 10);
-+	for (p = end; isspace(*p); p++);
-+	if (*p++ != ']') {
-+		media_dbg(media, "Unable to parse link flags: expected ']'.\n");
-+		*endp = (char *)p - 1;
-+		return -EINVAL;
-+	}
-+
-+	for (; isspace(*p); p++);
-+	*endp = (char *)p;
-+
-+	media_dbg(media,
-+		  "Setting up link %u:%u -> %u:%u [%u]\n",
-+		  link->source->entity->info.id, link->source->index,
-+		  link->sink->entity->info.id, link->sink->index,
-+		  flags);
-+
-+	return media_setup_link(media, link->source, link->sink, flags);
-+}
-+
-+void media_print_streampos(struct media_device *media, const char *p,
-+			   const char *end)
-+{
-+	int pos;
-+
-+	pos = end - p + 1;
-+
-+	if (pos < 0)
-+		pos = 0;
-+	if (pos > strlen(p))
-+		pos = strlen(p);
-+
-+	media_dbg(media, "\n");
-+	media_dbg(media, " %s\n", p);
-+	media_dbg(media, " %*s\n", pos, "^");
-+}
-+
-+int media_parse_setup_links(struct media_device *media, const char *p)
-+{
-+	char *end;
-+	int ret;
-+
-+	do {
-+		ret = media_parse_setup_link(media, p, &end);
-+		if (ret < 0) {
-+			media_print_streampos(media, p, end);
-+			return ret;
-+		}
-+
-+		p = end + 1;
-+	} while (*end == ',');
-+
-+	return *end ? -EINVAL : 0;
-+}
-diff --git a/src/media-ctl/libv4l2subdev.c b/src/media-ctl/libv4l2subdev.c
-new file mode 100644
-index 0000000..4ede4fa
---- /dev/null
-+++ b/src/media-ctl/libv4l2subdev.c
-@@ -0,0 +1,759 @@
-+/*
-+ * V4L2 subdev interface library
-+ *
-+ * Copyright (C) 2010-2014 Ideas on board SPRL
-+ *
-+ * Contact: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-+ *
-+ * This program is free software; you can redistribute it and/or modify
-+ * it under the terms of the GNU Lesser General Public License as published
-+ * by the Free Software Foundation; either version 2.1 of the License, or
-+ * (at your option) any later version.
-+ *
-+ * This program is distributed in the hope that it will be useful,
-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-+ * GNU Lesser General Public License for more details.
-+ *
-+ * You should have received a copy of the GNU Lesser General Public License
-+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
-+ */
-+
-+#include <sys/ioctl.h>
-+#include <sys/stat.h>
-+#include <sys/types.h>
-+
-+#include <ctype.h>
-+#include <errno.h>
-+#include <fcntl.h>
-+#include <stdbool.h>
-+#include <stdio.h>
-+#include <stdlib.h>
-+#include <string.h>
-+#include <unistd.h>
-+
-+#include <linux/v4l2-subdev.h>
-+
-+#include "mediactl.h"
-+#include "mediactl-priv.h"
-+#include "tools.h"
-+#include "v4l2subdev.h"
-+
-+int v4l2_subdev_open(struct media_entity *entity)
-+{
-+	if (entity->fd != -1)
-+		return 0;
-+
-+	entity->fd = open(entity->devname, O_RDWR);
-+	if (entity->fd == -1) {
-+		int ret = -errno;
-+		media_dbg(entity->media,
-+			  "%s: Failed to open subdev device node %s\n", __func__,
-+			  entity->devname);
-+		return ret;
-+	}
-+
-+	return 0;
-+}
-+
-+void v4l2_subdev_close(struct media_entity *entity)
-+{
-+	close(entity->fd);
-+	entity->fd = -1;
-+}
-+
-+int v4l2_subdev_get_format(struct media_entity *entity,
-+	struct v4l2_mbus_framefmt *format, unsigned int pad,
-+	enum v4l2_subdev_format_whence which)
-+{
-+	struct v4l2_subdev_format fmt;
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(&fmt, 0, sizeof(fmt));
-+	fmt.pad = pad;
-+	fmt.which = which;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_FMT, &fmt);
-+	if (ret < 0)
-+		return -errno;
-+
-+	*format = fmt.format;
-+	return 0;
-+}
-+
-+int v4l2_subdev_set_format(struct media_entity *entity,
-+	struct v4l2_mbus_framefmt *format, unsigned int pad,
-+	enum v4l2_subdev_format_whence which)
-+{
-+	struct v4l2_subdev_format fmt;
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(&fmt, 0, sizeof(fmt));
-+	fmt.pad = pad;
-+	fmt.which = which;
-+	fmt.format = *format;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_FMT, &fmt);
-+	if (ret < 0)
-+		return -errno;
-+
-+	*format = fmt.format;
-+	return 0;
-+}
-+
-+int v4l2_subdev_get_selection(struct media_entity *entity,
-+	struct v4l2_rect *rect, unsigned int pad, unsigned int target,
-+	enum v4l2_subdev_format_whence which)
-+{
-+	union {
-+		struct v4l2_subdev_selection sel;
-+		struct v4l2_subdev_crop crop;
-+	} u;
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(&u.sel, 0, sizeof(u.sel));
-+	u.sel.pad = pad;
-+	u.sel.target = target;
-+	u.sel.which = which;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_SELECTION, &u.sel);
-+	if (ret >= 0) {
-+		*rect = u.sel.r;
-+		return 0;
-+	}
-+	if (errno != ENOTTY || target != V4L2_SEL_TGT_CROP)
-+		return -errno;
-+
-+	memset(&u.crop, 0, sizeof(u.crop));
-+	u.crop.pad = pad;
-+	u.crop.which = which;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_CROP, &u.crop);
-+	if (ret < 0)
-+		return -errno;
-+
-+	*rect = u.crop.rect;
-+	return 0;
-+}
-+
-+int v4l2_subdev_set_selection(struct media_entity *entity,
-+	struct v4l2_rect *rect, unsigned int pad, unsigned int target,
-+	enum v4l2_subdev_format_whence which)
-+{
-+	union {
-+		struct v4l2_subdev_selection sel;
-+		struct v4l2_subdev_crop crop;
-+	} u;
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(&u.sel, 0, sizeof(u.sel));
-+	u.sel.pad = pad;
-+	u.sel.target = target;
-+	u.sel.which = which;
-+	u.sel.r = *rect;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_SELECTION, &u.sel);
-+	if (ret >= 0) {
-+		*rect = u.sel.r;
-+		return 0;
-+	}
-+	if (errno != ENOTTY || target != V4L2_SEL_TGT_CROP)
-+		return -errno;
-+
-+	memset(&u.crop, 0, sizeof(u.crop));
-+	u.crop.pad = pad;
-+	u.crop.which = which;
-+	u.crop.rect = *rect;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_CROP, &u.crop);
-+	if (ret < 0)
-+		return -errno;
-+
-+	*rect = u.crop.rect;
-+	return 0;
-+}
-+
-+#if 0
-+int v4l2_subdev_get_dv_timings_caps(struct media_entity *entity,
-+	struct v4l2_dv_timings_cap *caps)
-+{
-+	unsigned int pad = caps->pad;
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(caps, 0, sizeof(*caps));
-+	caps->pad = pad;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_DV_TIMINGS_CAP, caps);
-+	if (ret < 0)
-+		return -errno;
-+
-+	return 0;
-+}
-+
-+int v4l2_subdev_query_dv_timings(struct media_entity *entity,
-+	struct v4l2_dv_timings *timings)
-+{
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(timings, 0, sizeof(*timings));
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_QUERY_DV_TIMINGS, timings);
-+	if (ret < 0)
-+		return -errno;
-+
-+	return 0;
-+}
-+
-+int v4l2_subdev_get_dv_timings(struct media_entity *entity,
-+	struct v4l2_dv_timings *timings)
-+{
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(timings, 0, sizeof(*timings));
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_DV_TIMINGS, timings);
-+	if (ret < 0)
-+		return -errno;
-+
-+	return 0;
-+}
-+
-+int v4l2_subdev_set_dv_timings(struct media_entity *entity,
-+	struct v4l2_dv_timings *timings)
-+{
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_DV_TIMINGS, timings);
-+	if (ret < 0)
-+		return -errno;
-+
-+	return 0;
-+}
-+#endif
-+
-+int v4l2_subdev_get_frame_interval(struct media_entity *entity,
-+				   struct v4l2_fract *interval)
-+{
-+	struct v4l2_subdev_frame_interval ival;
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(&ival, 0, sizeof(ival));
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_G_FRAME_INTERVAL, &ival);
-+	if (ret < 0)
-+		return -errno;
-+
-+	*interval = ival.interval;
-+	return 0;
-+}
-+
-+int v4l2_subdev_set_frame_interval(struct media_entity *entity,
-+				   struct v4l2_fract *interval)
-+{
-+	struct v4l2_subdev_frame_interval ival;
-+	int ret;
-+
-+	ret = v4l2_subdev_open(entity);
-+	if (ret < 0)
-+		return ret;
-+
-+	memset(&ival, 0, sizeof(ival));
-+	ival.interval = *interval;
-+
-+	ret = ioctl(entity->fd, VIDIOC_SUBDEV_S_FRAME_INTERVAL, &ival);
-+	if (ret < 0)
-+		return -errno;
-+
-+	*interval = ival.interval;
-+	return 0;
-+}
-+
-+static int v4l2_subdev_parse_format(struct media_device *media,
-+				    struct v4l2_mbus_framefmt *format,
-+				    const char *p, char **endp)
-+{
-+	enum v4l2_mbus_pixelcode code;
-+	unsigned int width, height;
-+	char *end;
-+
-+	/*
-+	 * Compatibility with the old syntax: consider space as valid
-+	 * separator between the media bus pixel code and the size.
-+	 */
-+	for (; isspace(*p); ++p);
-+	for (end = (char *)p;
-+	     *end != '/' && *end != ' ' && *end != '\0'; ++end);
-+
-+	code = v4l2_subdev_string_to_pixelcode(p, end - p);
-+	if (code == (enum v4l2_mbus_pixelcode)-1) {
-+		media_dbg(media, "Invalid pixel code '%.*s'\n", end - p, p);
-+		return -EINVAL;
-+	}
-+
-+	p = end + 1;
-+	width = strtoul(p, &end, 10);
-+	if (*end != 'x') {
-+		media_dbg(media, "Expected 'x'\n");
-+		return -EINVAL;
-+	}
-+
-+	p = end + 1;
-+	height = strtoul(p, &end, 10);
-+	*endp = end;
-+
-+	memset(format, 0, sizeof(*format));
-+	format->width = width;
-+	format->height = height;
-+	format->code = code;
-+
-+	return 0;
-+}
-+
-+static int v4l2_subdev_parse_rectangle(struct media_device *media,
-+				       struct v4l2_rect *r, const char *p,
-+				       char **endp)
-+{
-+	char *end;
-+
-+	if (*p++ != '(') {
-+		media_dbg(media, "Expected '('\n");
-+		*endp = (char *)p - 1;
-+		return -EINVAL;
-+	}
-+
-+	r->left = strtoul(p, &end, 10);
-+	if (*end != ',') {
-+		media_dbg(media, "Expected ','\n");
-+		*endp = end;
-+		return -EINVAL;
-+	}
-+
-+	p = end + 1;
-+	r->top = strtoul(p, &end, 10);
-+	if (*end++ != ')') {
-+		media_dbg(media, "Expected ')'\n");
-+		*endp = end - 1;
-+		return -EINVAL;
-+	}
-+	if (*end != '/') {
-+		media_dbg(media, "Expected '/'\n");
-+		*endp = end;
-+		return -EINVAL;
-+	}
-+
-+	p = end + 1;
-+	r->width = strtoul(p, &end, 10);
-+	if (*end != 'x') {
-+		media_dbg(media, "Expected 'x'\n");
-+		*endp = end;
-+		return -EINVAL;
-+	}
-+
-+	p = end + 1;
-+	r->height = strtoul(p, &end, 10);
-+	*endp = end;
-+
-+	return 0;
-+}
-+
-+static int v4l2_subdev_parse_frame_interval(struct media_device *media,
-+					    struct v4l2_fract *interval,
-+					    const char *p, char **endp)
-+{
-+	char *end;
-+
-+	for (; isspace(*p); ++p);
-+
-+	interval->numerator = strtoul(p, &end, 10);
-+
-+	for (p = end; isspace(*p); ++p);
-+	if (*p++ != '/') {
-+		media_dbg(media, "Expected '/'\n");
-+		*endp = (char *)p - 1;
-+		return -EINVAL;
-+	}
-+
-+	for (; isspace(*p); ++p);
-+	interval->denominator = strtoul(p, &end, 10);
-+
-+	*endp = end;
-+	return 0;
-+}
-+
-+/*
-+ * The debate over whether this function should be named icanhasstr() instead
-+ * has been strong and heated. If you feel like this would be an important
-+ * change, patches are welcome (or not).
-+ */
-+static bool strhazit(const char *str, const char **p)
-+{
-+	int len = strlen(str);
-+
-+	if (strncmp(str, *p, len))
-+		return false;
-+
-+	for (*p += len; isspace(**p); ++*p);
-+	return true;
-+}
-+
-+static struct media_pad *v4l2_subdev_parse_pad_format(
-+	struct media_device *media, struct v4l2_mbus_framefmt *format,
-+	struct v4l2_rect *crop, struct v4l2_rect *compose,
-+	struct v4l2_fract *interval, const char *p, char **endp)
-+{
-+	struct media_pad *pad;
-+	bool first;
-+	char *end;
-+	int ret;
-+
-+	for (; isspace(*p); ++p);
-+
-+	pad = media_parse_pad(media, p, &end);
-+	if (pad == NULL) {
-+		*endp = end;
-+		return NULL;
-+	}
-+
-+	for (p = end; isspace(*p); ++p);
-+	if (*p++ != '[') {
-+		media_dbg(media, "Expected '['\n");
-+		*endp = (char *)p - 1;
-+		return NULL;
-+	}
-+
-+	for (first = true; ; first = false) {
-+		for (; isspace(*p); p++);
-+
-+		/*
-+		 * Backward compatibility: if the first property starts with an
-+		 * uppercase later, process it as a format description.
-+		 */
-+		if (strhazit("fmt:", &p) || (first && isupper(*p))) {
-+			ret = v4l2_subdev_parse_format(media, format, p, &end);
-+			if (ret < 0) {
-+				*endp = end;
-+				return NULL;
-+			}
-+
-+			p = end;
-+			continue;
-+		}
-+
-+		/*
-+		 * Backward compatibility: crop rectangles can be specified
-+		 * implicitly without the 'crop:' property name.
-+		 */
-+		if (strhazit("crop:", &p) || *p == '(') {
-+			ret = v4l2_subdev_parse_rectangle(media, crop, p, &end);
-+			if (ret < 0) {
-+				*endp = end;
-+				return NULL;
-+			}
-+
-+			p = end;
-+			continue;
-+		}
-+
-+		if (strhazit("compose:", &p)) {
-+			ret = v4l2_subdev_parse_rectangle(media, compose, p, &end);
-+			if (ret < 0) {
-+				*endp = end;
-+				return NULL;
-+			}
-+
-+			for (p = end; isspace(*p); p++);
-+			continue;
-+		}
-+
-+		if (*p == '@') {
-+			ret = v4l2_subdev_parse_frame_interval(media, interval, ++p, &end);
-+			if (ret < 0) {
-+				*endp = end;
-+				return NULL;
-+			}
-+
-+			p = end;
-+			continue;
-+		}
-+
-+		break;
-+	}
-+
-+	if (*p != ']') {
-+		media_dbg(media, "Expected ']'\n");
-+		*endp = (char *)p;
-+		return NULL;
-+	}
-+
-+	*endp = (char *)p + 1;
-+	return pad;
-+}
-+
-+static int set_format(struct media_pad *pad,
-+		      struct v4l2_mbus_framefmt *format)
-+{
-+	int ret;
-+
-+	if (format->width == 0 || format->height == 0)
-+		return 0;
-+
-+	media_dbg(pad->entity->media,
-+		  "Setting up format %s %ux%u on pad %s/%u\n",
-+		  v4l2_subdev_pixelcode_to_string(format->code),
-+		  format->width, format->height,
-+		  pad->entity->info.name, pad->index);
-+
-+	ret = v4l2_subdev_set_format(pad->entity, format, pad->index,
-+				     V4L2_SUBDEV_FORMAT_ACTIVE);
-+	if (ret < 0) {
-+		media_dbg(pad->entity->media,
-+			  "Unable to set format: %s (%d)\n",
-+			  strerror(-ret), ret);
-+		return ret;
-+	}
-+
-+	media_dbg(pad->entity->media,
-+		  "Format set: %s %ux%u\n",
-+		  v4l2_subdev_pixelcode_to_string(format->code),
-+		  format->width, format->height);
-+
-+	return 0;
-+}
-+
-+static int set_selection(struct media_pad *pad, unsigned int target,
-+			 struct v4l2_rect *rect)
-+{
-+	int ret;
-+
-+	if (rect->left == -1 || rect->top == -1)
-+		return 0;
-+
-+	media_dbg(pad->entity->media,
-+		  "Setting up selection target %u rectangle (%u,%u)/%ux%u on pad %s/%u\n",
-+		  target, rect->left, rect->top, rect->width, rect->height,
-+		  pad->entity->info.name, pad->index);
-+
-+	ret = v4l2_subdev_set_selection(pad->entity, rect, pad->index,
-+					target, V4L2_SUBDEV_FORMAT_ACTIVE);
-+	if (ret < 0) {
-+		media_dbg(pad->entity->media,
-+			  "Unable to set selection rectangle: %s (%d)\n",
-+			  strerror(-ret), ret);
-+		return ret;
-+	}
-+
-+	media_dbg(pad->entity->media,
-+		  "Selection rectangle set: (%u,%u)/%ux%u\n",
-+		  rect->left, rect->top, rect->width, rect->height);
-+
-+	return 0;
-+}
-+
-+static int set_frame_interval(struct media_entity *entity,
-+			      struct v4l2_fract *interval)
-+{
-+	int ret;
-+
-+	if (interval->numerator == 0)
-+		return 0;
-+
-+	media_dbg(entity->media,
-+		  "Setting up frame interval %u/%u on entity %s\n",
-+		  interval->numerator, interval->denominator,
-+		  entity->info.name);
-+
-+	ret = v4l2_subdev_set_frame_interval(entity, interval);
-+	if (ret < 0) {
-+		media_dbg(entity->media,
-+			  "Unable to set frame interval: %s (%d)",
-+			  strerror(-ret), ret);
-+		return ret;
-+	}
-+
-+	media_dbg(entity->media, "Frame interval set: %u/%u\n",
-+		  interval->numerator, interval->denominator);
-+
-+	return 0;
-+}
-+
-+
-+static int v4l2_subdev_parse_setup_format(struct media_device *media,
-+					  const char *p, char **endp)
-+{
-+	struct v4l2_mbus_framefmt format = { 0, 0, 0 };
-+	struct media_pad *pad;
-+	struct v4l2_rect crop = { -1, -1, -1, -1 };
-+	struct v4l2_rect compose = crop;
-+	struct v4l2_fract interval = { 0, 0 };
-+	unsigned int i;
-+	char *end;
-+	int ret;
-+
-+	pad = v4l2_subdev_parse_pad_format(media, &format, &crop, &compose,
-+					   &interval, p, &end);
-+	if (pad == NULL) {
-+		media_print_streampos(media, p, end);
-+		media_dbg(media, "Unable to parse format\n");
-+		return -EINVAL;
-+	}
-+
-+	if (pad->flags & MEDIA_PAD_FL_SINK) {
-+		ret = set_format(pad, &format);
-+		if (ret < 0)
-+			return ret;
-+	}
-+
-+	ret = set_selection(pad, V4L2_SEL_TGT_CROP, &crop);
-+	if (ret < 0)
-+		return ret;
-+
-+	ret = set_selection(pad, V4L2_SEL_TGT_COMPOSE, &compose);
-+	if (ret < 0)
-+		return ret;
-+
-+	if (pad->flags & MEDIA_PAD_FL_SOURCE) {
-+		ret = set_format(pad, &format);
-+		if (ret < 0)
-+			return ret;
-+	}
-+
-+	ret = set_frame_interval(pad->entity, &interval);
-+	if (ret < 0)
-+		return ret;
-+
-+
-+	/* If the pad is an output pad, automatically set the same format on
-+	 * the remote subdev input pads, if any.
-+	 */
-+	if (pad->flags & MEDIA_PAD_FL_SOURCE) {
-+		for (i = 0; i < pad->entity->num_links; ++i) {
-+			struct media_link *link = &pad->entity->links[i];
-+			struct v4l2_mbus_framefmt remote_format;
-+
-+			if (!(link->flags & MEDIA_LNK_FL_ENABLED))
-+				continue;
-+
-+			if (link->source == pad &&
-+			    link->sink->entity->info.type == MEDIA_ENT_T_V4L2_SUBDEV) {
-+				remote_format = format;
-+				set_format(link->sink, &remote_format);
-+			}
-+		}
-+	}
-+
-+	*endp = end;
-+	return 0;
-+}
-+
-+int v4l2_subdev_parse_setup_formats(struct media_device *media, const char *p)
-+{
-+	char *end;
-+	int ret;
-+
-+	do {
-+		ret = v4l2_subdev_parse_setup_format(media, p, &end);
-+		if (ret < 0)
-+			return ret;
-+
-+		p = end + 1;
-+	} while (*end == ',');
-+
-+	return *end ? -EINVAL : 0;
-+}
-+
-+static struct {
-+	const char *name;
-+	enum v4l2_mbus_pixelcode code;
-+} mbus_formats[] = {
-+	{ "Y8", V4L2_MBUS_FMT_Y8_1X8},
-+	{ "Y10", V4L2_MBUS_FMT_Y10_1X10 },
-+	{ "Y12", V4L2_MBUS_FMT_Y12_1X12 },
-+	{ "YUYV", V4L2_MBUS_FMT_YUYV8_1X16 },
-+	{ "YUYV1_5X8", V4L2_MBUS_FMT_YUYV8_1_5X8 },
-+	{ "YUYV2X8", V4L2_MBUS_FMT_YUYV8_2X8 },
-+	{ "UYVY", V4L2_MBUS_FMT_UYVY8_1X16 },
-+	{ "UYVY1_5X8", V4L2_MBUS_FMT_UYVY8_1_5X8 },
-+	{ "UYVY2X8", V4L2_MBUS_FMT_UYVY8_2X8 },
-+	{ "SBGGR8", V4L2_MBUS_FMT_SBGGR8_1X8 },
-+	{ "SGBRG8", V4L2_MBUS_FMT_SGBRG8_1X8 },
-+	{ "SGRBG8", V4L2_MBUS_FMT_SGRBG8_1X8 },
-+	{ "SRGGB8", V4L2_MBUS_FMT_SRGGB8_1X8 },
-+	{ "SBGGR10", V4L2_MBUS_FMT_SBGGR10_1X10 },
-+	{ "SGBRG10", V4L2_MBUS_FMT_SGBRG10_1X10 },
-+	{ "SGRBG10", V4L2_MBUS_FMT_SGRBG10_1X10 },
-+	{ "SRGGB10", V4L2_MBUS_FMT_SRGGB10_1X10 },
-+	{ "SBGGR10_DPCM8", V4L2_MBUS_FMT_SBGGR10_DPCM8_1X8 },
-+	{ "SGBRG10_DPCM8", V4L2_MBUS_FMT_SGBRG10_DPCM8_1X8 },
-+	{ "SGRBG10_DPCM8", V4L2_MBUS_FMT_SGRBG10_DPCM8_1X8 },
-+	{ "SRGGB10_DPCM8", V4L2_MBUS_FMT_SRGGB10_DPCM8_1X8 },
-+	{ "SBGGR12", V4L2_MBUS_FMT_SBGGR12_1X12 },
-+	{ "SGBRG12", V4L2_MBUS_FMT_SGBRG12_1X12 },
-+	{ "SGRBG12", V4L2_MBUS_FMT_SGRBG12_1X12 },
-+	{ "SRGGB12", V4L2_MBUS_FMT_SRGGB12_1X12 },
-+	{ "AYUV32", V4L2_MBUS_FMT_AYUV8_1X32 },
-+	{ "ARGB32", V4L2_MBUS_FMT_ARGB8888_1X32 },
-+};
-+
-+const char *v4l2_subdev_pixelcode_to_string(enum v4l2_mbus_pixelcode code)
-+{
-+	unsigned int i;
-+
-+	for (i = 0; i < ARRAY_SIZE(mbus_formats); ++i) {
-+		if (mbus_formats[i].code == code)
-+			return mbus_formats[i].name;
-+	}
-+
-+	return "unknown";
-+}
-+
-+enum v4l2_mbus_pixelcode v4l2_subdev_string_to_pixelcode(const char *string,
-+							 unsigned int length)
-+{
-+	unsigned int i;
-+
-+	for (i = 0; i < ARRAY_SIZE(mbus_formats); ++i) {
-+		if (strncmp(mbus_formats[i].name, string, length) == 0)
-+			break;
-+	}
-+
-+	if (i == ARRAY_SIZE(mbus_formats))
-+		return (enum v4l2_mbus_pixelcode)-1;
-+
-+	return mbus_formats[i].code;
-+}
-diff --git a/src/media-ctl/mediactl-priv.h b/src/media-ctl/mediactl-priv.h
-new file mode 100644
-index 0000000..a0d3a55
---- /dev/null
-+++ b/src/media-ctl/mediactl-priv.h
-@@ -0,0 +1,64 @@
-+/*
-+ * Media controller interface library
-+ *
-+ * Copyright (C) 2010-2014 Ideas on board SPRL
-+ *
-+ * Contact: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-+ *
-+ * This program is free software; you can redistribute it and/or modify
-+ * it under the terms of the GNU Lesser General Public License as published
-+ * by the Free Software Foundation; either version 2.1 of the License, or
-+ * (at your option) any later version.
-+ *
-+ * This program is distributed in the hope that it will be useful,
-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-+ * GNU Lesser General Public License for more details.
-+ *
-+ * You should have received a copy of the GNU Lesser General Public License
-+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
-+ */
-+
-+#ifndef __MEDIA_PRIV_H__
-+#define __MEDIA_PRIV_H__
-+
-+#include <linux/media.h>
-+
-+#include "mediactl.h"
-+
-+struct media_entity {
-+	struct media_device *media;
-+	struct media_entity_desc info;
-+	struct media_pad *pads;
-+	struct media_link *links;
-+	unsigned int max_links;
-+	unsigned int num_links;
-+
-+	char devname[32];
-+	int fd;
-+};
-+
-+struct media_device {
-+	int fd;
-+	int refcount;
-+	char *devnode;
-+
-+	struct media_device_info info;
-+	struct media_entity *entities;
-+	unsigned int entities_count;
-+
-+	void (*debug_handler)(void *, ...);
-+	void *debug_priv;
-+
-+	struct {
-+		struct media_entity *v4l;
-+		struct media_entity *fb;
-+		struct media_entity *alsa;
-+		struct media_entity *dvb;
-+	} def;
-+};
-+
-+#define media_dbg(media, ...) \
-+	(media)->debug_handler((media)->debug_priv, __VA_ARGS__)
-+
-+#endif /* __MEDIA_PRIV_H__ */
-diff --git a/src/media-ctl/mediactl.h b/src/media-ctl/mediactl.h
-new file mode 100644
-index 0000000..77ac182
---- /dev/null
-+++ b/src/media-ctl/mediactl.h
-@@ -0,0 +1,423 @@
-+/*
-+ * Media controller interface library
-+ *
-+ * Copyright (C) 2010-2014 Ideas on board SPRL
-+ *
-+ * Contact: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-+ *
-+ * This program is free software; you can redistribute it and/or modify
-+ * it under the terms of the GNU Lesser General Public License as published
-+ * by the Free Software Foundation; either version 2.1 of the License, or
-+ * (at your option) any later version.
-+ *
-+ * This program is distributed in the hope that it will be useful,
-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-+ * GNU Lesser General Public License for more details.
-+ *
-+ * You should have received a copy of the GNU Lesser General Public License
-+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
-+ */
-+
-+#ifndef __MEDIA_H__
-+#define __MEDIA_H__
-+
-+#include <linux/media.h>
-+
-+struct media_link {
-+	struct media_pad *source;
-+	struct media_pad *sink;
-+	struct media_link *twin;
-+	__u32 flags;
-+	__u32 padding[3];
-+};
-+
-+struct media_pad {
-+	struct media_entity *entity;
-+	__u32 index;
-+	__u32 flags;
-+	__u32 padding[3];
-+};
-+
-+struct media_device;
-+struct media_entity;
-+
-+/**
-+ * @brief Create a new media device.
-+ * @param devnode - device node path.
-+ *
-+ * Create a media device instance for the given device node and return it. The
-+ * device node is not accessed by this function, device node access errors will
-+ * not be caught and reported here. The media device needs to be enumerated
-+ * before it can be accessed, see media_device_enumerate().
-+ *
-+ * Media devices are reference-counted, see media_device_ref() and
-+ * media_device_unref() for more information.
-+ *
-+ * @return A pointer to the new media device or NULL if memory cannot be
-+ * allocated.
-+ */
-+struct media_device *media_device_new(const char *devnode);
-+
-+/**
-+ * @brief Create a new emulated media device.
-+ * @param info - device information.
-+ *
-+ * Emulated media devices are userspace-only objects not backed by a kernel
-+ * media device. They are created for ALSA and V4L2 devices that are not
-+ * associated with a media controller device.
-+ *
-+ * Only device query functions are available for media devices. Enumerating or
-+ * setting up links is invalid.
-+ *
-+ * @return A pointer to the new media device or NULL if memory cannot be
-+ * allocated.
-+ */
-+struct media_device *media_device_new_emulated(struct media_device_info *info);
-+
-+/**
-+ * @brief Take a reference to the device.
-+ * @param media - device instance.
-+ *
-+ * Media devices are reference-counted. Taking a reference to a device prevents
-+ * it from being freed until all references are released. The reference count is
-+ * initialized to 1 when the device is created.
-+ *
-+ * @return A pointer to @a media.
-+ */
-+struct media_device *media_device_ref(struct media_device *media);
-+
-+/**
-+ * @brief Release a reference to the device.
-+ * @param media - device instance.
-+ *
-+ * Release a reference to the media device. When the reference count reaches 0
-+ * this function frees the device.
-+ */
-+void media_device_unref(struct media_device *media);
-+
-+/**
-+ * @brief Add an entity to an existing media device
-+ * @param media - device instance.
-+ * @param desc - description of the entity to be added
-+ * @param devnode - device node corresponding to the entity
-+ *
-+ * Entities are usually created and added to media devices automatically when
-+ * the media device is enumerated through the media controller API. However,
-+ * when an emulated media device (thus not backed with a kernel-side media
-+ * controller device) is created, entities need to be manually added.
-+ *
-+ * Entities can also be manually added to a successfully enumerated media device
-+ * to group several functions provided by separate kernel devices. The most
-+ * common use case is to group the audio and video functions of a USB webcam in
-+ * a single media device. Those functions are exposed through separate USB
-+ * interfaces and handled through unrelated kernel drivers, they must thus be
-+ * manually added to the same media device.
-+ *
-+ * This function adds a new entity to the given media device and initializes it
-+ * from the given entity description and device node name. Only the following
-+ * fields of the description are copied over to the new entity:
-+ *
-+ * - type
-+ * - flags (MEDIA_ENT_FL_DEFAULT only)
-+ * - name
-+ * - v4l, fb, alsa or dvb (depending on the device type)
-+ *
-+ * All other fields of the newly created entity id are initialized to 0,
-+ * including the entity ID.
-+ *
-+ * @return Zero on success or -ENOMEM if memory cannot be allocated.
-+ */
-+int media_device_add_entity(struct media_device *media,
-+			    const struct media_entity_desc *desc,
-+			    const char *devnode);
-+
-+/**
-+ * @brief Set a handler for debug messages.
-+ * @param media - device instance.
-+ * @param debug_handler - debug message handler
-+ * @param debug_priv - first argument to debug message handler
-+ *
-+ * Set a handler for debug messages that will be called whenever
-+ * debugging information is to be printed. The handler expects an
-+ * fprintf-like function.
-+ */
-+void media_debug_set_handler(
-+	struct media_device *media, void (*debug_handler)(void *, ...),
-+	void *debug_priv);
-+
-+/**
-+ * @brief Enumerate the device topology
-+ * @param media - device instance.
-+ *
-+ * Enumerate the media device entities, pads and links. Calling this function is
-+ * mandatory before accessing the media device contents.
-+ *
-+ * @return Zero on success or a negative error code on failure.
-+ */
-+int media_device_enumerate(struct media_device *media);
-+
-+/**
-+ * @brief Locate the pad at the other end of a link.
-+ * @param pad - sink pad at one end of the link.
-+ *
-+ * Locate the source pad connected to @a pad through an enabled link. As only one
-+ * link connected to a sink pad can be enabled at a time, the connected source
-+ * pad is guaranteed to be unique.
-+ *
-+ * @return A pointer to the connected source pad, or NULL if all links connected
-+ * to @a pad are disabled. Return NULL also if @a pad is not a sink pad.
-+ */
-+struct media_pad *media_entity_remote_source(struct media_pad *pad);
-+
-+/**
-+ * @brief Get information about a media entity
-+ * @param entity - media entity.
-+ *
-+ * The information structure is owned by the media entity object and will be
-+ * freed when the object is destroyed.
-+ *
-+ * @return A pointer to the media entity information
-+ */
-+const struct media_entity_desc *media_entity_get_info(struct media_entity *entity);
-+
-+/**
-+ * @brief Get an entity pad
-+ * @param entity - media entity.
-+ * @param index - pad index.
-+ *
-+ * This function returns a pointer to the pad object identified by its index
-+ * for the given entity. If the pad index is out of bounds it will return NULL.
-+ *
-+ * @return A pointer to the pad
-+ */
-+const struct media_pad *media_entity_get_pad(struct media_entity *entity,
-+					     unsigned int index);
-+
-+/**
-+ * @brief Get the number of links
-+ * @param entity - media entity.
-+ *
-+ * This function returns the total number of links that originate from or arrive
-+ * at the the media entity.
-+ *
-+ * @return The number of links for the entity
-+ */
-+unsigned int media_entity_get_links_count(struct media_entity *entity);
-+
-+/**
-+ * @brief Get an entity link
-+ * @param entity - media entity.
-+ * @param index - link index.
-+ *
-+ * This function returns a pointer to the link object identified by its index
-+ * for the given entity. If the link index is out of bounds it will return NULL.
-+ *
-+ * @return A pointer to the link
-+ */
-+const struct media_link *media_entity_get_link(struct media_entity *entity,
-+					       unsigned int index);
-+
-+/**
-+ * @brief Get the device node name for an entity
-+ * @param entity - media entity.
-+ *
-+ * This function returns the full path and name to the device node corresponding
-+ * to the given entity.
-+ *
-+ * @return A pointer to the device node name or NULL if the entity has no
-+ * associated device node
-+ */
-+const char *media_entity_get_devname(struct media_entity *entity);
-+
-+/**
-+ * @brief Get the type of an entity.
-+ * @param entity - the entity.
-+ *
-+ * @return The type of @a entity.
-+ */
-+static inline unsigned int media_entity_type(struct media_entity *entity)
-+{
-+	return media_entity_get_info(entity)->type & MEDIA_ENT_TYPE_MASK;
-+}
-+
-+/**
-+ * @brief Find an entity by its name.
-+ * @param media - media device.
-+ * @param name - entity name.
-+ * @param length - size of @a name.
-+ *
-+ * Search for an entity with a name equal to @a name.
-+ *
-+ * @return A pointer to the entity if found, or NULL otherwise.
-+ */
-+struct media_entity *media_get_entity_by_name(struct media_device *media,
-+	const char *name, size_t length);
-+
-+/**
-+ * @brief Find an entity by its ID.
-+ * @param media - media device.
-+ * @param id - entity ID.
-+ *
-+ * This function searches for an entity based on its ID using an exact match or
-+ * next ID method based on the given @a id. If @a id is ORed with
-+ * MEDIA_ENT_ID_FLAG_NEXT, the function will return the entity with the smallest
-+ * ID larger than @a id. Otherwise it will return the entity with an ID equal to
-+ * @a id.
-+ *
-+ * @return A pointer to the entity if found, or NULL otherwise.
-+ */
-+struct media_entity *media_get_entity_by_id(struct media_device *media,
-+	__u32 id);
-+
-+/**
-+ * @brief Get the number of entities
-+ * @param media - media device.
-+ *
-+ * This function returns the total number of entities in the media device. If
-+ * entities haven't been enumerated yet it will return 0.
-+ *
-+ * @return The number of entities in the media device
-+ */
-+unsigned int media_get_entities_count(struct media_device *media);
-+
-+/**
-+ * @brief Get the entities
-+ * @param media - media device.
-+ *
-+ * This function returns a pointer to the array of entities for the media
-+ * device. If entities haven't been enumerated yet it will return NULL.
-+ *
-+ * The array of entities is owned by the media device object and will be freed
-+ * when the media object is destroyed.
-+ *
-+ * @return A pointer to an array of entities
-+ */
-+struct media_entity *media_get_entity(struct media_device *media, unsigned int index);
-+
-+/**
-+ * @brief Get the default entity for a given type
-+ * @param media - media device.
-+ * @param type - entity type.
-+ *
-+ * This function returns the default entity of the requested type. @a type must
-+ * be one of
-+ *
-+ *	MEDIA_ENT_T_DEVNODE_V4L
-+ *	MEDIA_ENT_T_DEVNODE_FB
-+ *	MEDIA_ENT_T_DEVNODE_ALSA
-+ *	MEDIA_ENT_T_DEVNODE_DVB
-+ *
-+ * @return A pointer to the default entity for the type if it exists, or NULL
-+ * otherwise.
-+ */
-+struct media_entity *media_get_default_entity(struct media_device *media,
-+					      unsigned int type);
-+
-+/**
-+ * @brief Get the media device information
-+ * @param media - media device.
-+ *
-+ * The information structure is owned by the media device object and will be freed
-+ * when the media object is destroyed.
-+ *
-+ * @return A pointer to the media device information
-+ */
-+const struct media_device_info *media_get_info(struct media_device *media);
-+
-+/**
-+ * @brief Get the media device node name
-+ * @param media - media device.
-+ *
-+ * The device node name string is owned by the media device object and will be
-+ * freed when the media object is destroyed.
-+ *
-+ * @return A pointer to the media device node name
-+ */
-+const char *media_get_devnode(struct media_device *media);
-+
-+/**
-+ * @brief Configure a link.
-+ * @param media - media device.
-+ * @param source - source pad at the link origin.
-+ * @param sink - sink pad at the link target.
-+ * @param flags - configuration flags.
-+ *
-+ * Locate the link between @a source and @a sink, and configure it by applying
-+ * the new @a flags.
-+ *
-+ * Only the MEDIA_LINK_FLAG_ENABLED flag is writable.
-+ *
-+ * @return 0 on success, -1 on failure:
-+ *	   -ENOENT: link not found
-+ *	   - other error codes returned by MEDIA_IOC_SETUP_LINK
-+ */
-+int media_setup_link(struct media_device *media,
-+	struct media_pad *source, struct media_pad *sink,
-+	__u32 flags);
-+
-+/**
-+ * @brief Reset all links to the disabled state.
-+ * @param media - media device.
-+ *
-+ * Disable all links in the media device. This function is usually used after
-+ * opening a media device to reset all links to a known state.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int media_reset_links(struct media_device *media);
-+
-+/**
-+ * @brief Parse string to a pad on the media device.
-+ * @param media - media device.
-+ * @param p - input string
-+ * @param endp - pointer to string where parsing ended
-+ *
-+ * Parse NULL terminated string describing a pad and return its struct
-+ * media_pad instance.
-+ *
-+ * @return Pointer to struct media_pad on success, NULL on failure.
-+ */
-+struct media_pad *media_parse_pad(struct media_device *media,
-+				  const char *p, char **endp);
-+
-+/**
-+ * @brief Parse string to a link on the media device.
-+ * @param media - media device.
-+ * @param p - input string
-+ * @param endp - pointer to p where parsing ended
-+ *
-+ * Parse NULL terminated string p describing a link and return its struct
-+ * media_link instance.
-+ *
-+ * @return Pointer to struct media_link on success, NULL on failure.
-+ */
-+struct media_link *media_parse_link(struct media_device *media,
-+				    const char *p, char **endp);
-+
-+/**
-+ * @brief Parse string to a link on the media device and set it up.
-+ * @param media - media device.
-+ * @param p - input string
-+ *
-+ * Parse NULL terminated string p describing a link and its configuration
-+ * and configure the link.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int media_parse_setup_link(struct media_device *media,
-+			   const char *p, char **endp);
-+
-+/**
-+ * @brief Parse string to link(s) on the media device and set it up.
-+ * @param media - media device.
-+ * @param p - input string
-+ *
-+ * Parse NULL terminated string p describing link(s) separated by
-+ * commas (,) and configure the link(s).
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int media_parse_setup_links(struct media_device *media, const char *p);
-+
-+#endif
-diff --git a/src/media-ctl/tools.h b/src/media-ctl/tools.h
-new file mode 100644
-index 0000000..815534c
---- /dev/null
-+++ b/src/media-ctl/tools.h
-@@ -0,0 +1,32 @@
-+/*
-+ * Media controller test application
-+ *
-+ * Copyright (C) 2010-2014 Ideas on board SPRL
-+ *
-+ * Contact: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-+ *
-+ * This program is free software; you can redistribute it and/or modify
-+ * it under the terms of the GNU Lesser General Public License as published
-+ * by the Free Software Foundation; either version 2.1 of the License, or
-+ * (at your option) any later version.
-+ *
-+ * This program is distributed in the hope that it will be useful,
-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-+ * GNU Lesser General Public License for more details.
-+ *
-+ * You should have received a copy of the GNU Lesser General Public License
-+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
-+ */
-+
-+#ifndef __TOOLS_H__
-+#define __TOOLS_H__
-+
-+#define ARRAY_SIZE(array)	(sizeof(array) / sizeof((array)[0]))
-+#define FIELD_SIZEOF(t, f)	(sizeof(((t*)0)->f))
-+
-+void media_print_streampos(struct media_device *media, const char *p,
-+			   const char *end);
-+
-+#endif /* __TOOLS_H__ */
-+
-diff --git a/src/media-ctl/v4l2subdev.h b/src/media-ctl/v4l2subdev.h
-new file mode 100644
-index 0000000..1cb53ff
---- /dev/null
-+++ b/src/media-ctl/v4l2subdev.h
-@@ -0,0 +1,258 @@
-+/*
-+ * V4L2 subdev interface library
-+ *
-+ * Copyright (C) 2010-2014 Ideas on board SPRL
-+ *
-+ * Contact: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-+ *
-+ * This program is free software; you can redistribute it and/or modify
-+ * it under the terms of the GNU Lesser General Public License as published
-+ * by the Free Software Foundation; either version 2.1 of the License, or
-+ * (at your option) any later version.
-+ *
-+ * This program is distributed in the hope that it will be useful,
-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-+ * GNU Lesser General Public License for more details.
-+ *
-+ * You should have received a copy of the GNU Lesser General Public License
-+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
-+ */
-+
-+#ifndef __SUBDEV_H__
-+#define __SUBDEV_H__
-+
-+#include <linux/v4l2-subdev.h>
-+
-+struct media_entity;
-+
-+/**
-+ * @brief Open a sub-device.
-+ * @param entity - sub-device media entity.
-+ *
-+ * Open the V4L2 subdev device node associated with @a entity. The file
-+ * descriptor is stored in the media_entity structure.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_open(struct media_entity *entity);
-+
-+/**
-+ * @brief Close a sub-device.
-+ * @param entity - sub-device media entity.
-+ *
-+ * Close the V4L2 subdev device node associated with the @a entity and opened by
-+ * a previous call to v4l2_subdev_open() (either explicit or implicit).
-+ */
-+void v4l2_subdev_close(struct media_entity *entity);
-+
-+/**
-+ * @brief Retrieve the format on a pad.
-+ * @param entity - subdev-device media entity.
-+ * @param format - format to be filled.
-+ * @param pad - pad number.
-+ * @param which - identifier of the format to get.
-+ *
-+ * Retrieve the current format on the @a entity @a pad and store it in the
-+ * @a format structure.
-+ *
-+ * @a which is set to V4L2_SUBDEV_FORMAT_TRY to retrieve the try format stored
-+ * in the file handle, of V4L2_SUBDEV_FORMAT_ACTIVE to retrieve the current
-+ * active format.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_get_format(struct media_entity *entity,
-+	struct v4l2_mbus_framefmt *format, unsigned int pad,
-+	enum v4l2_subdev_format_whence which);
-+
-+/**
-+ * @brief Set the format on a pad.
-+ * @param entity - subdev-device media entity.
-+ * @param format - format.
-+ * @param pad - pad number.
-+ * @param which - identifier of the format to set.
-+ *
-+ * Set the format on the @a entity @a pad to @a format. The driver is allowed to
-+ * modify the requested format, in which case @a format is updated with the
-+ * modifications.
-+ *
-+ * @a which is set to V4L2_SUBDEV_FORMAT_TRY to set the try format stored in the
-+ * file handle, of V4L2_SUBDEV_FORMAT_ACTIVE to configure the device with an
-+ * active format.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_set_format(struct media_entity *entity,
-+	struct v4l2_mbus_framefmt *format, unsigned int pad,
-+	enum v4l2_subdev_format_whence which);
-+
-+/**
-+ * @brief Retrieve a selection rectangle on a pad.
-+ * @param entity - subdev-device media entity.
-+ * @param r - rectangle to be filled.
-+ * @param pad - pad number.
-+ * @param target - selection target
-+ * @param which - identifier of the format to get.
-+ *
-+ * Retrieve the @a target selection rectangle on the @a entity @a pad
-+ * and store it in the @a rect structure.
-+ *
-+ * @a which is set to V4L2_SUBDEV_FORMAT_TRY to retrieve the try
-+ * selection rectangle stored in the file handle, or
-+ * V4L2_SUBDEV_FORMAT_ACTIVE to retrieve the current active selection
-+ * rectangle.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_get_selection(struct media_entity *entity,
-+	struct v4l2_rect *rect, unsigned int pad, unsigned int target,
-+	enum v4l2_subdev_format_whence which);
-+
-+/**
-+ * @brief Set a selection rectangle on a pad.
-+ * @param entity - subdev-device media entity.
-+ * @param rect - crop rectangle.
-+ * @param pad - pad number.
-+ * @param target - selection target
-+ * @param which - identifier of the format to set.
-+ *
-+ * Set the @a target selection rectangle on the @a entity @a pad to @a
-+ * rect. The driver is allowed to modify the requested rectangle, in
-+ * which case @a rect is updated with the modifications.
-+ *
-+ * @a which is set to V4L2_SUBDEV_FORMAT_TRY to set the try crop rectangle
-+ * stored in the file handle, of V4L2_SUBDEV_FORMAT_ACTIVE to configure the
-+ * device with an active crop rectangle.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_set_selection(struct media_entity *entity,
-+	struct v4l2_rect *rect, unsigned int pad, unsigned int target,
-+	enum v4l2_subdev_format_whence which);
-+
-+/**
-+ * @brief Query the digital video capabilities of a pad.
-+ * @param entity - subdev-device media entity.
-+ * @param cap - capabilities to be filled.
-+ *
-+ * Retrieve the digital video capabilities of the @a entity pad specified by
-+ * @a cap.pad and store it in the @a cap structure.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_get_dv_timings_caps(struct media_entity *entity,
-+	struct v4l2_dv_timings_cap *caps);
-+
-+/**
-+ * @brief Query the digital video timings of a sub-device
-+ * @param entity - subdev-device media entity.
-+ * @param timings timings to be filled.
-+ *
-+ * Retrieve the detected digital video timings for the currently selected input
-+ * of @a entity and store them in the @a timings structure.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_query_dv_timings(struct media_entity *entity,
-+	struct v4l2_dv_timings *timings);
-+
-+/**
-+ * @brief Get the current digital video timings of a sub-device
-+ * @param entity - subdev-device media entity.
-+ * @param timings timings to be filled.
-+ *
-+ * Retrieve the current digital video timings for the currently selected input
-+ * of @a entity and store them in the @a timings structure.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_get_dv_timings(struct media_entity *entity,
-+	struct v4l2_dv_timings *timings);
-+
-+/**
-+ * @brief Set the digital video timings of a sub-device
-+ * @param entity - subdev-device media entity.
-+ * @param timings timings to be set.
-+ *
-+ * Set the digital video timings of @a entity to @a timings. The driver is
-+ * allowed to modify the requested format, in which case @a timings is updated
-+ * with the modifications.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_set_dv_timings(struct media_entity *entity,
-+	struct v4l2_dv_timings *timings);
-+
-+/**
-+ * @brief Retrieve the frame interval on a sub-device.
-+ * @param entity - subdev-device media entity.
-+ * @param interval - frame interval to be filled.
-+ *
-+ * Retrieve the current frame interval on subdev @a entity and store it in the
-+ * @a interval structure.
-+ *
-+ * Frame interval retrieving is usually supported only on devices at the
-+ * beginning of video pipelines, such as sensors.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+
-+int v4l2_subdev_get_frame_interval(struct media_entity *entity,
-+	struct v4l2_fract *interval);
-+
-+/**
-+ * @brief Set the frame interval on a sub-device.
-+ * @param entity - subdev-device media entity.
-+ * @param interval - frame interval.
-+ *
-+ * Set the frame interval on subdev @a entity to @a interval. The driver is
-+ * allowed to modify the requested frame interval, in which case @a interval is
-+ * updated with the modifications.
-+ *
-+ * Frame interval setting is usually supported only on devices at the beginning
-+ * of video pipelines, such as sensors.
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_set_frame_interval(struct media_entity *entity,
-+	struct v4l2_fract *interval);
-+
-+/**
-+ * @brief Parse a string and apply format, crop and frame interval settings.
-+ * @param media - media device.
-+ * @param p - input string
-+ * @param endp - pointer to string p where parsing ended (return)
-+ *
-+ * Parse string @a p and apply format, crop and frame interval settings to a
-+ * subdev pad specified in @a p. @a endp will be written a pointer where
-+ * parsing of @a p ended.
-+ *
-+ * Format strings are separeted by commas (,).
-+ *
-+ * @return 0 on success, or a negative error code on failure.
-+ */
-+int v4l2_subdev_parse_setup_formats(struct media_device *media, const char *p);
-+
-+/**
-+ * @brief Convert media bus pixel code to string.
-+ * @param code - input string
-+ *
-+ * Convert media bus pixel code @a code to a human-readable string.
-+ *
-+ * @return A pointer to a string on success, NULL on failure.
-+ */
-+const char *v4l2_subdev_pixelcode_to_string(enum v4l2_mbus_pixelcode code);
-+
-+/**
-+ * @brief Parse string to media bus pixel code.
-+ * @param string - input string
-+ * @param lenght - length of the string
-+ *
-+ * Parse human readable string @a string to an media bus pixel code.
-+ *
-+ * @return media bus pixelcode on success, -1 on failure.
-+ */
-+enum v4l2_mbus_pixelcode v4l2_subdev_string_to_pixelcode(const char *string,
-+							 unsigned int length);
-+#endif
--- 
-1.9.1
-
diff --git a/recipes-graphics/wayland/weston/0003-gst-recorder-Use-USERPTR-instead-of-DMABUF-for-VSP-o.patch b/recipes-graphics/wayland/weston/0003-gst-recorder-Use-USERPTR-instead-of-DMABUF-for-VSP-o.patch
deleted file mode 100644
index 095507c18..000000000
--- a/recipes-graphics/wayland/weston/0003-gst-recorder-Use-USERPTR-instead-of-DMABUF-for-VSP-o.patch
+++ /dev/null
@@ -1,261 +0,0 @@
-From ab3cf8044667bff4a9e2e5108c55dd198fa353be Mon Sep 17 00:00:00 2001
-From: Damian Hobson-Garcia <dhobsong@igel.co.jp>
-Date: Thu, 11 May 2017 12:05:56 +0900
-Subject: [PATCH 3/4] gst-recorder: Use USERPTR instead of DMABUF for VSP
- output
-
-The RCar-Gen3 encoder requires buffers to be allocated and managed
-externally when using dmabuf buffers.  Since different sized buffers
-are required for each output, the VSP cannot allocate these buffers
-externally.  The encoder provides its own buffers in USERPTR mode, so
-switch to that.
----
- src/gst-recorder.c | 100 +++++++++++++++++++++++++++++++++--------------------
- 1 file changed, 63 insertions(+), 37 deletions(-)
-
-diff --git a/src/gst-recorder.c b/src/gst-recorder.c
-index 2e3b359..271fb69 100644
---- a/src/gst-recorder.c
-+++ b/src/gst-recorder.c
-@@ -314,7 +314,7 @@ vsp_init(const char *devname)
- 		weston_log("failed to open subdev '%s'\n", buf);
- 		goto error_media;
- 	}
--	else if ((vsp->output.fd = open(media_entity_get_devname(entity), O_RDWR | O_NONBLOCK)) < 0)
-+	else if ((vsp->output.fd = open(media_entity_get_devname(entity), O_RDWR )) < 0)
- 	{
- 		weston_log("failed to open device '%s'\n", media_entity_get_devname(entity));
- 		goto error_media;
-@@ -467,7 +467,8 @@ vsp_request_buffers(vsp_data_t *vsp, vsp_port_n port, unsigned int num)
- 	memset(&reqbuf, 0, sizeof(reqbuf));
- 	reqbuf.type = (port == VSP_PORT_INPUT) ?
- 		V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE : V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
--	reqbuf.memory = V4L2_MEMORY_DMABUF;
-+	reqbuf.memory = (port == VSP_PORT_INPUT) ?
-+		V4L2_MEMORY_DMABUF : V4L2_MEMORY_USERPTR;
- 	reqbuf.count = num;
- 	if (ioctl(fd, VIDIOC_REQBUFS, &reqbuf) < 0) {
- 		weston_log("VSP: %s REQBUFS failed: %d\n",
-@@ -539,7 +540,8 @@ vsp_input_buffer_dequeue_dmafd(vsp_data_t *vsp)
- 
- /* ...enqueue output buffer */
- static int
--vsp_output_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd[])
-+vsp_output_buffer_queue_userptr(vsp_data_t *vsp, int i, void * omx_mem,
-+		int y_plane_size, int c_plane_size)
- {
- 	vsp_media_pad_t    *pad = &vsp->output;
- 	struct v4l2_plane   planes[2];
-@@ -549,16 +551,23 @@ vsp_output_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd[])
- 	memset(&buf, 0, sizeof(buf));
- 	memset(planes, 0, sizeof(planes));
- 	buf.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
--	buf.memory = V4L2_MEMORY_DMABUF;
-+	buf.memory = V4L2_MEMORY_USERPTR;
- 	buf.index = i;
- 	buf.m.planes = planes;
- 	buf.length = VSP_OUTPUT_BUFFERS_PLANE;
--	buf.m.planes[0].m.fd = dmafd[0];
--	buf.m.planes[1].m.fd = dmafd[1];
-+
-+	buf.m.planes[0].m.userptr = (unsigned long) omx_mem;
-+	buf.m.planes[1].m.userptr = (unsigned long) omx_mem + y_plane_size;
-+
-+	buf.m.planes[0].bytesused = y_plane_size;
-+	buf.m.planes[0].length = y_plane_size;
-+
-+	buf.m.planes[1].bytesused = c_plane_size;
-+	buf.m.planes[1].length = c_plane_size;
- 
- 	/* ...submit buffer */
- 	if (ioctl(pad->fd, VIDIOC_QBUF, &buf) < 0) {
--		weston_log("VSP: output dmafd queue failed: %d\n", errno);
-+		weston_log("VSP: output buffer queue failed: %d\n", errno);
- 		return -1;
- 	}
- 
-@@ -567,7 +576,7 @@ vsp_output_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd[])
- 
- /* ...dequeue output buffer */
- static int
--vsp_output_buffer_dequeue_dmafd(vsp_data_t *vsp)
-+vsp_output_buffer_dequeue_userptr(vsp_data_t *vsp)
- {
- 	vsp_media_pad_t    *pad = &vsp->output;
- 	struct v4l2_buffer  buf;
-@@ -577,12 +586,12 @@ vsp_output_buffer_dequeue_dmafd(vsp_data_t *vsp)
- 	memset(&buf, 0, sizeof(buf));
- 	memset(planes, 0, sizeof(planes));
- 	buf.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
--	buf.memory = V4L2_MEMORY_DMABUF;
-+	buf.memory = V4L2_MEMORY_USERPTR;
- 	buf.m.planes = planes;
- 	buf.length = VSP_OUTPUT_BUFFERS_PLANE;
- 
- 	if (ioctl(pad->fd, VIDIOC_DQBUF, &buf) < 0) {
--		weston_log("VSP: output dmafd de-queue failed: %d\n", errno);
-+		weston_log("VSP: output buffer de-queue failed: %d\n", errno);
- 		return -1;
- 	}
- 
-@@ -879,33 +888,25 @@ err:
- }
- 
- static int
--gst_recorder_omx_buffer_acquire(struct gst_recorder *r, GstBuffer **ret_buf, int fd[])
-+gst_recorder_omx_buffer_acquire(struct gst_recorder *r, GstBuffer **ret_buf, GstMapInfo *info)
- {
--	unsigned int i;
- 	GstFlowReturn ret;
- 	GstBuffer *buf;
--	guint n_mem;
--	GstMemory *mem;
- 
- 	ret = gst_buffer_pool_acquire_buffer(r->omx_pool, &buf, NULL);
- 	if (ret != GST_FLOW_OK) {
--		weston_log("OMX buffer acquire failed\n");
--		return -1;
-+		weston_log("GStreamer buffer acquire failed\n");
-+		goto err_release;
- 	}
- 
--	n_mem = gst_buffer_n_memory(buf);
--	if (n_mem < 1) {
--		weston_log("Buffer with no mem!\n");
-+	if (!gst_buffer_is_writable(buf)) {
-+		weston_log("GStreamer buffer not writable\n");
- 		goto err_release;
- 	}
- 
--	for (i = 0; i < n_mem; i++) {
--		mem = gst_buffer_peek_memory (buf, i);
--		if (!gst_is_dmabuf_memory (mem)) {
--			weston_log("Mem not dmabuf\n");
--			goto err_release;
--		}
--		fd[i] = gst_dmabuf_memory_get_fd (mem);
-+	if (!gst_buffer_map(buf, info, GST_MAP_WRITE)) {
-+		weston_log("Cannot map GStreamer buffer\n");
-+		goto err_release;
- 	}
- 
- 	*ret_buf = buf;
-@@ -959,7 +960,7 @@ gst_recorder_create(struct gst_recorder_settings *settings)
- 
- 	/* omx */
- 	ptr += sprintf(ptr,
--		"omxh264enc target-bitrate=%d control-rate=2 name=my_encoder ! "
-+		"omxh264enc target-bitrate=%d control-rate=2 no-copy=true name=my_encoder ! "
- 		"video/x-h264,width=%d,height=%d ! ",
- 		r->set->bitrate, r->set->crop.width, r->set->crop.height);
- 
-@@ -1012,6 +1013,12 @@ gst_recorder_create(struct gst_recorder_settings *settings)
- 					"framerate", GST_TYPE_FRACTION, 0, DEFAULT_FPS,
- 					NULL), NULL);
- 
-+	g_object_set(G_OBJECT(r->appsrc),
-+			"stream-type", 0,
-+			"format", GST_FORMAT_TIME,
-+			"is-live", TRUE,
-+			NULL);
-+
- 	r->appsrc_pad = gst_element_get_static_pad(GST_ELEMENT_CAST(r->appsrc), "src");
- 	if (!r->appsrc_pad)
- 		weston_log("Failed to get src0 pad of appsrc\n");
-@@ -1088,14 +1095,23 @@ gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
- {
- 	int ret;
- 	GstBuffer *buf;
--	int omx_fd[2];
-+	GstMapInfo info;
-+	int ysize;
-+	int csize;
- 
-         /* get GST buffer */
--	if (gst_recorder_omx_buffer_acquire(r, &buf, omx_fd) < 0) {
-+	if (gst_recorder_omx_buffer_acquire(r, &buf, &info) < 0) {
- 		weston_log("VSP: can not acquire GST buffer, dropping frame\n");
- 		return 0;
- 	}
- 
-+	ysize = r->set->crop.width * r->set->crop.height;
-+#ifdef VSP_OUTPUT_NV16
-+	csize = ysize;
-+#else
-+	csize = ysize / 2;
-+#endif
-+
- 	pthread_mutex_lock(&r->vsp->mutex);
- 	/* setup vsp */
- 	if (vsp_set_formats(r->vsp, r->set->width, r->set->height, &r->set->crop) < 0) {
-@@ -1116,7 +1132,7 @@ gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
- 	}
- 
- 	/* queue output biffer */
--	if (vsp_output_buffer_queue_dmafd(r->vsp, 0, omx_fd) < 0) {
-+	if (vsp_output_buffer_queue_userptr(r->vsp, 0, info.data, ysize, csize) < 0) {
- 		weston_log("can not queue OMX buffer %d to VSP\n", 0);
- 		gst_recorder_omx_buffer_release(r, buf);
- 		goto err_vsp;
-@@ -1147,11 +1163,15 @@ gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
- 	}
- 
- 	/* dequeue output */
--	if (vsp_output_buffer_dequeue_dmafd(r->vsp) < 0) {
--		weston_log("VSP: failed to dequeu output buffer\n");
-+	if (vsp_output_buffer_dequeue_userptr(r->vsp) < 0) {
-+		weston_log("VSP: failed to dequeue output buffer\n");
-+		gst_buffer_unmap(buf, &info);
- 		gst_recorder_omx_buffer_release(r, buf);
--		/* fall through */
- 	} else {
-+
-+		gst_buffer_unmap(buf, &info);
-+		gst_buffer_set_size(buf, ysize + csize);
-+
- 		/* set timestamp */
- 		gst_recorder_set_timestamp(r, buf);
- 
-@@ -1174,6 +1194,7 @@ gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
- 	vsp_request_buffers(r->vsp, VSP_PORT_OUTPUT, 0);
- 
- 	pthread_mutex_unlock(&r->vsp->mutex);
-+	close(fd);
- 	return 0;
- 
- err_vsp:
-@@ -1181,6 +1202,7 @@ err_vsp:
- 	/* finish vsp here */
- err:
- 	pthread_mutex_unlock(&r->vsp->mutex);
-+	close(fd);
- 	return -1;
- }
- 
-@@ -1197,9 +1219,13 @@ gst_recorder_frame_dmafd(struct gst_recorder *r, int fd, int stride)
- 		goto unlock;
- 	}
- 		
--	/* The mutex is never released while encoding, so this point should
--	 * never be reached if input.valid is true. */
--	assert(!r->input.valid);
-+	/* It is possible that the frame callback can be called mutiple
-+	 * times before the worker thread wakes up.  In this case
-+	 * drop all buf the first frame */
-+	if(r->input.valid) {
-+		close(fd);
-+		goto unlock;
-+	}
- 
- 	r->input.prime_fd = fd;
- 	r->input.stride = stride;
-@@ -1209,5 +1235,5 @@ gst_recorder_frame_dmafd(struct gst_recorder *r, int fd, int stride)
- unlock:
- 	pthread_mutex_unlock(&r->mutex);
- 
--	return 0;
-+	return ret;
- }
--- 
-1.9.1
-
diff --git a/recipes-graphics/wayland/weston/0004-gst-record-Specify-bytesused-and-length-of-VSP-input.patch b/recipes-graphics/wayland/weston/0004-gst-record-Specify-bytesused-and-length-of-VSP-input.patch
deleted file mode 100644
index 925d3a43e..000000000
--- a/recipes-graphics/wayland/weston/0004-gst-record-Specify-bytesused-and-length-of-VSP-input.patch
+++ /dev/null
@@ -1,61 +0,0 @@
-From 28a72656562930d27ef8117a489f8299537bdfb8 Mon Sep 17 00:00:00 2001
-From: Damian Hobson-Garcia <dhobsong@igel.co.jp>
-Date: Tue, 9 May 2017 18:07:52 +0900
-Subject: [PATCH] gst-record: Specify bytesused and length of VSP input buffer
-
-The bytesused=0 setting has been deprecated in newer kernels.
-
----
- src/gst-recorder.c | 8 ++++++--
- 1 file changed, 6 insertions(+), 2 deletions(-)
-
-diff --git a/src/gst-recorder.c b/src/gst-recorder.c
-index f6d12f8..de74250 100644
---- a/src/gst-recorder.c
-+++ b/src/gst-recorder.c
-@@ -487,7 +487,7 @@ vsp_request_buffers(vsp_data_t *vsp, vsp_port_n port, unsigned int num)
- 
- /* ...enqueue dmafd buffer */
- static int
--vsp_input_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd)
-+vsp_input_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd, int bytesused)
- {
- 	vsp_media_pad_t    *pad = &vsp->input;
- 	struct v4l2_buffer  buf;
-@@ -502,6 +502,8 @@ vsp_input_buffer_queue_dmafd(vsp_data_t *vsp, int i, int dmafd)
- 	buf.m.planes = planes;
- 	buf.length = 1;
- 	buf.m.planes[0].m.fd = dmafd;
-+	buf.m.planes[0].bytesused = bytesused;
-+	buf.m.planes[0].length = bytesused;
- 
- 	/* ...submit buffer */
- 	if (ioctl(pad->fd, VIDIOC_QBUF, &buf) < 0) {
-@@ -1098,6 +1100,7 @@ gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
- 	GstMapInfo info;
- 	int ysize;
- 	int csize;
-+	int rgbsize;
- 
-         /* get GST buffer */
- 	if (gst_recorder_omx_buffer_acquire(r, &buf, &info) < 0) {
-@@ -1111,6 +1114,7 @@ gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
- #else
- 	csize = ysize / 2;
- #endif
-+	rgbsize = stride * r->set->height;
- 
- 	pthread_mutex_lock(&r->vsp->mutex);
- 	/* setup vsp */
-@@ -1139,7 +1143,7 @@ gst_recorder_process_dmafd(struct gst_recorder *r, int fd, int stride)
- 	}
- 
- 	/* queue input vsp buffer */
--	if (vsp_input_buffer_queue_dmafd(r->vsp, 0, fd) < 0) {
-+	if (vsp_input_buffer_queue_dmafd(r->vsp, 0, fd, rgbsize) < 0) {
- 		weston_log("VSP: failed to queue input buffer\n");
- 		goto err_vsp;
- 	}
--- 
-1.9.1
-
diff --git a/recipes-graphics/wayland/weston_%.bbappend b/recipes-graphics/wayland/weston_%.bbappend
deleted file mode 100644
index b5de59212..000000000
--- a/recipes-graphics/wayland/weston_%.bbappend
+++ /dev/null
@@ -1,10 +0,0 @@
-FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
-
-SRC_URI_append_m3ulcb = " \
-    file://0001-Add-virtual-output-support.patch \
-    file://0002-Add-gst-recorder-for-h264-output-streaming.patch \
-    file://0003-gst-recorder-Use-USERPTR-instead-of-DMABUF-for-VSP-o.patch \
-    file://0004-gst-record-Specify-bytesused-and-length-of-VSP-input.patch \
-"
-
-DEPENDS_append_m3ulcb = " gstreamer1.0-plugins-base"