dvdec: Properly refuse incoming stream without framerate
[platform/upstream/gst-plugins-good.git] / ext / dv / gstdvdec.c
index e732702..ea2ea41 100644 (file)
@@ -1,5 +1,6 @@
 /* GStreamer
  * Copyright (C) <1999> Erik Walthinsen <omega@cse.ogi.edu>
+ *               <2005> Wim Taymans <wim@fluendo.com>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  *
  * You should have received a copy of the GNU Library General Public
  * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
+ * Free Software Foundation, Inc., 51 Franklin St, Fifth Floor,
+ * Boston, MA 02110-1301, USA.
+ */
+
+/**
+ * SECTION:element-dvdec
+ *
+ * dvdec decodes DV video into raw video. The element expects a full DV frame
+ * as input, which is 120000 bytes for NTSC and 144000 for PAL video.
+ *
+ * This element can perform simple frame dropping with the #GstDVDec:drop-factor
+ * property. Setting this property to a value N > 1 will only decode every 
+ * Nth frame.
+ *
+ * <refsect2>
+ * <title>Example launch line</title>
+ * |[
+ * gst-launch-1.0 filesrc location=test.dv ! dvdemux name=demux ! dvdec ! xvimagesink
+ * ]| This pipeline decodes and renders the raw DV stream to a videosink.
+ * </refsect2>
+ *
+ * Last reviewed on 2006-02-28 (0.10.3)
  */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
 #include <string.h>
+#include <math.h>
+#include <gst/video/video.h>
+#include <gst/video/gstvideometa.h>
+#include <gst/video/gstvideopool.h>
 
-/* First, include the header file for the plugin, to bring in the
- * object definition and other useful things.
- */
 #include "gstdvdec.h"
 
+/* sizes of one input buffer */
 #define NTSC_HEIGHT 480
 #define NTSC_BUFFER 120000
-#define NTSC_FRAMERATE 29.997
+#define NTSC_FRAMERATE_NUMERATOR 30000
+#define NTSC_FRAMERATE_DENOMINATOR 1001
 
 #define PAL_HEIGHT 576
 #define PAL_BUFFER 144000
-#define PAL_FRAMERATE 25.0
+#define PAL_FRAMERATE_NUMERATOR 25
+#define PAL_FRAMERATE_DENOMINATOR 1
 
-/* The ElementDetails structure gives a human-readable description
- * of the plugin, as well as author and version data.
- */
-static GstElementDetails dvdec_details = GST_ELEMENT_DETAILS (
-  "DV (smpte314) decoder plugin",
-  "Codec/Decoder/Video",
-  "Uses libdv to decode DV video (libdv.sourceforge.net)",
-  "Erik Walthinsen <omega@cse.ogi.edu>\n"
-  "Wim Taymans <wim.taymans@tvd.be>"
-);
-
-
-/* These are the signals that this element can fire.  They are zero-
- * based because the numbers themselves are private to the object.
- * LAST_SIGNAL is used for initialization of the signal array.
- */
-enum {
-  /* FILL ME */
-  LAST_SIGNAL
+#define PAL_NORMAL_PAR_X        59
+#define PAL_NORMAL_PAR_Y        54
+#define PAL_WIDE_PAR_X          118
+#define PAL_WIDE_PAR_Y          81
+
+#define NTSC_NORMAL_PAR_X       10
+#define NTSC_NORMAL_PAR_Y       11
+#define NTSC_WIDE_PAR_X         40
+#define NTSC_WIDE_PAR_Y         33
+
+#define DV_DEFAULT_QUALITY DV_QUALITY_BEST
+#define DV_DEFAULT_DECODE_NTH 1
+
+GST_DEBUG_CATEGORY_STATIC (dvdec_debug);
+#define GST_CAT_DEFAULT dvdec_debug
+
+enum
+{
+  PROP_0,
+  PROP_CLAMP_LUMA,
+  PROP_CLAMP_CHROMA,
+  PROP_QUALITY,
+  PROP_DECODE_NTH
 };
 
-/* Arguments are identified the same way, but cannot be zero, so you
- * must leave the ARG_0 entry in as a placeholder.
- */
-enum {
-  ARG_0,
-  ARG_CLAMP_LUMA,
-  ARG_CLAMP_CHROMA,
-  ARG_QUALITY,
-  /* FILL ME */
+const gint qualities[] = {
+  DV_QUALITY_DC,
+  DV_QUALITY_AC_1,
+  DV_QUALITY_AC_2,
+  DV_QUALITY_DC | DV_QUALITY_COLOR,
+  DV_QUALITY_AC_1 | DV_QUALITY_COLOR,
+  DV_QUALITY_AC_2 | DV_QUALITY_COLOR
 };
 
-/* The PadFactory structures describe what pads the element has or
- * can have.  They can be quite complex, but for this dvdec plugin
- * they are rather simple.
- */
-static GstStaticPadTemplate sink_temp =
-GST_STATIC_PAD_TEMPLATE 
-(
-  "sink" , 
-  GST_PAD_SINK,
-  GST_PAD_ALWAYS,
-  GST_STATIC_CAPS (
-    "video/x-dv, systemstream = (boolean) true"
-  )
-);
-
-static GstStaticPadTemplate video_src_temp =
-GST_STATIC_PAD_TEMPLATE
-(
-  "video",
-  GST_PAD_SRC,
-  GST_PAD_ALWAYS,
-  GST_STATIC_CAPS (
-    "video/x-raw-yuv, "
-    "format = (fourcc) YUY2, " 
-    "width = (int) 720, "
-    "height = (int) { " 
-      G_STRINGIFY(NTSC_HEIGHT) ", "
-      G_STRINGIFY(PAL_HEIGHT)
-    " }, "
-    "framerate = (double) { "
-      G_STRINGIFY(PAL_FRAMERATE) ", "
-      G_STRINGIFY(NTSC_FRAMERATE) 
-    " }; "
-
-    "video/x-raw-rgb, "
-    "bpp = (int) 32, "
-    "depth = (int) 32, "
-    "endianness = (int) " G_STRINGIFY(G_BIG_ENDIAN) ", "
-    "red_mask =   (int) 0x000000ff, "
-    "green_mask = (int) 0x0000ff00, "
-    "blue_mask =  (int) 0x00ff0000, "
-    "width = (int) 720, "
-    "height = (int) { "
-       G_STRINGIFY(NTSC_HEIGHT) ", "
-       G_STRINGIFY(PAL_HEIGHT)
-    " }, "
-    "framerate = (double) { "
-        G_STRINGIFY(PAL_FRAMERATE) ", "
-       G_STRINGIFY(NTSC_FRAMERATE) 
-    " }; "
-
-    "video/x-raw-rgb, "
-    "bpp = (int) 24, "
-    "depth = (int) 24, "
-    "endianness = (int) " G_STRINGIFY(G_BIG_ENDIAN) ", "
-    "red_mask =   (int) 0x000000ff, "
-    "green_mask = (int) 0x0000ff00, "
-    "blue_mask =  (int) 0x00ff0000, "
-    "width = (int) 720, "
-    "height = (int) { "
-       G_STRINGIFY(NTSC_HEIGHT) ", "
-       G_STRINGIFY(PAL_HEIGHT)
-    " }, "
-    "framerate = (double) { "
-        G_STRINGIFY(PAL_FRAMERATE) ", "
-        G_STRINGIFY(NTSC_FRAMERATE) 
-    " }"
-  )
-);
-
-static GstStaticPadTemplate audio_src_temp =
-GST_STATIC_PAD_TEMPLATE
-(
-  "audio",
-  GST_PAD_SRC,
-  GST_PAD_ALWAYS,
-  GST_STATIC_CAPS (
-    "audio/x-raw-int, "
-    "depth = (int) 16, "
-    "width = (int) 16, "
-    "signed = (boolean) TRUE, "
-    "channels = (int) 2, "
-    "endianness = (int) " G_STRINGIFY(G_LITTLE_ENDIAN) ", "
-    "rate = (int) [ 4000, 48000 ]"
-  )
-);
+static GstStaticPadTemplate sink_temp = GST_STATIC_PAD_TEMPLATE ("sink",
+    GST_PAD_SINK,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS ("video/x-dv, systemstream = (boolean) false")
+    );
+
+static GstStaticPadTemplate src_temp = GST_STATIC_PAD_TEMPLATE ("src",
+    GST_PAD_SRC,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS ("video/x-raw, "
+        "format = (string) { YUY2, BGRx, RGB }, "
+        "framerate = (fraction) [ 1/1, 60/1 ], "
+        "width = (int) 720, " "height = (int) { 576, 480 }")
+    );
 
 #define GST_TYPE_DVDEC_QUALITY (gst_dvdec_quality_get_type())
-GType
+static GType
 gst_dvdec_quality_get_type (void)
 {
   static GType qtype = 0;
+
   if (qtype == 0) {
-    static const GFlagsValue values[] = {
-      { DV_QUALITY_COLOR, "DV_QUALITY_COLOR", "Color or monochrome decoding" },
-      { DV_QUALITY_AC_1,  "DV_QUALITY_AC_1",  "AC 1 something" },
-      { DV_QUALITY_AC_2,  "DV_QUALITY_AC_2",  "AC 2 something" },
-      { 0, NULL, NULL }
+    static const GEnumValue values[] = {
+      {0, "Monochrome, DC (Fastest)", "fastest"},
+      {1, "Monochrome, first AC coefficient", "monochrome-ac"},
+      {2, "Monochrome, highest quality", "monochrome-best"},
+      {3, "Colour, DC, fastest", "colour-fastest"},
+      {4, "Colour, using only the first AC coefficient", "colour-ac"},
+      {5, "Highest quality colour decoding", "best"},
+      {0, NULL, NULL},
     };
-    qtype = g_flags_register_static ("GstDVDecQualityFlags", values);
+
+    qtype = g_enum_register_static ("GstDVDecQualityEnum", values);
   }
   return qtype;
 }
 
-/* A number of functon prototypes are given so we can refer to them later. */
-static void            gst_dvdec_base_init             (gpointer g_class);
-static void            gst_dvdec_class_init            (GstDVDecClass *klass);
-static void            gst_dvdec_init                  (GstDVDec *dvdec);
-
-static const GstQueryType* 
-                       gst_dvdec_get_src_query_types   (GstPad *pad);
-static gboolean        gst_dvdec_src_query             (GstPad *pad, GstQueryType type,
-                                                        GstFormat *format, gint64 *value);
-static const GstFormat* gst_dvdec_get_formats          (GstPad *pad);
-static gboolean        gst_dvdec_sink_convert          (GstPad *pad, GstFormat src_format, gint64 src_value,
-                                                        GstFormat *dest_format, gint64 *dest_value);
-static gboolean        gst_dvdec_src_convert           (GstPad *pad, GstFormat src_format, gint64 src_value,
-                                                        GstFormat *dest_format, gint64 *dest_value);
-
-static GstPadLinkReturn gst_dvdec_video_link (GstPad          *pad,
-                                              const GstCaps   *caps);
-static GstCaps*   gst_dvdec_video_getcaps   (GstPad       *pad);
-       
-static const GstEventMask*
-                       gst_dvdec_get_event_masks       (GstPad *pad);
-static gboolean        gst_dvdec_handle_src_event      (GstPad *pad, GstEvent *event);
-
-static void            gst_dvdec_loop                  (GstElement *element);
-
-static GstElementStateReturn   
-                       gst_dvdec_change_state          (GstElement *element);
-
-static void            gst_dvdec_set_property          (GObject *object, guint prop_id, 
-                                                        const GValue *value, GParamSpec *pspec);
-static void            gst_dvdec_get_property          (GObject *object, guint prop_id, 
-                                                        GValue *value, GParamSpec *pspec);
-
-/* The parent class pointer needs to be kept around for some object
- * operations.
- */
-static GstElementClass *parent_class = NULL;
+#define gst_dvdec_parent_class parent_class
+G_DEFINE_TYPE (GstDVDec, gst_dvdec, GST_TYPE_ELEMENT);
 
-/* This array holds the ids of the signals registered for this object.
- * The array indexes are based on the enum up above.
- */
-/*static guint gst_dvdec_signals[LAST_SIGNAL] = { 0 }; */
+static GstFlowReturn gst_dvdec_chain (GstPad * pad, GstObject * parent,
+    GstBuffer * buffer);
+static gboolean gst_dvdec_sink_event (GstPad * pad, GstObject * parent,
+    GstEvent * event);
 
-/* This function is used to register and subsequently return the type
- * identifier for this object class.  On first invocation, it will
- * register the type, providing the name of the class, struct sizes,
- * and pointers to the various functions that define the class.
- */
-GType
-gst_dvdec_get_type (void)
-{
-  static GType dvdec_type = 0;
-
-  if (!dvdec_type) {
-    static const GTypeInfo dvdec_info = {
-      sizeof (GstDVDecClass),      
-      gst_dvdec_base_init,      
-      NULL,
-      (GClassInitFunc) gst_dvdec_class_init,
-      NULL,
-      NULL,
-      sizeof (GstDVDec),
-      0,
-      (GInstanceInitFunc) gst_dvdec_init,
-    };
-    dvdec_type = g_type_register_static (GST_TYPE_ELEMENT, "GstDVDec", &dvdec_info, 0);
-  }
-  return dvdec_type;
-}
+static GstStateChangeReturn gst_dvdec_change_state (GstElement * element,
+    GstStateChange transition);
 
-static void
-gst_dvdec_base_init (gpointer g_class)
-{
-  GstElementClass *element_class = GST_ELEMENT_CLASS (g_class);
-
-  /* The pad templates can be easily generated from the factories above,
-   * and then added to the list of padtemplates for the elementfactory.
-   * Note that the generated padtemplates are stored in static global
-   * variables, for the gst_dvdec_init function to use later on.
-   */
-  gst_element_class_add_pad_template (element_class, gst_static_pad_template_get(&sink_temp));
-  gst_element_class_add_pad_template (element_class, gst_static_pad_template_get(&video_src_temp));
-  gst_element_class_add_pad_template (element_class, gst_static_pad_template_get(&audio_src_temp));
-
-  gst_element_class_set_details (element_class, &dvdec_details);
-}
+static void gst_dvdec_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec);
+static void gst_dvdec_get_property (GObject * object, guint prop_id,
+    GValue * value, GParamSpec * pspec);
 
-/* In order to create an instance of an object, the class must be
- * initialized by this function.  GObject will take care of running
- * it, based on the pointer to the function provided above.
- */
 static void
-gst_dvdec_class_init (GstDVDecClass *klass)
+gst_dvdec_class_init (GstDVDecClass * klass)
 {
-  /* Class pointers are needed to supply pointers to the private
-   * implementations of parent class methods.
-   */
   GObjectClass *gobject_class;
   GstElementClass *gstelement_class;
 
-  /* Since the dvdec class contains the parent classes, you can simply
-   * cast the pointer to get access to the parent classes.
-   */
-  gobject_class = (GObjectClass*) klass;
-  gstelement_class = (GstElementClass*) klass;
-
-  /* The parent class is needed for class method overrides. */
-  parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
-
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_CLAMP_LUMA,
-    g_param_spec_boolean ("clamp_luma", "Clamp luma", "Clamp luma",
-                          FALSE, G_PARAM_READWRITE));
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_CLAMP_CHROMA,
-    g_param_spec_boolean ("clamp_chroma", "Clamp chroma", "Clamp chroma",
-                          FALSE, G_PARAM_READWRITE));
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_QUALITY,
-    g_param_spec_flags ("quality", "Quality", "Decoding quality",
-                        GST_TYPE_DVDEC_QUALITY, DV_QUALITY_BEST, G_PARAM_READWRITE));
+  gobject_class = (GObjectClass *) klass;
+  gstelement_class = (GstElementClass *) klass;
 
   gobject_class->set_property = gst_dvdec_set_property;
   gobject_class->get_property = gst_dvdec_get_property;
-  
-  gstelement_class->change_state = gst_dvdec_change_state;
 
-  /* table initialization, only do once */
-  dv_init(0, 0);
+  g_object_class_install_property (G_OBJECT_CLASS (klass), PROP_CLAMP_LUMA,
+      g_param_spec_boolean ("clamp-luma", "Clamp luma", "Clamp luma",
+          FALSE, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), PROP_CLAMP_CHROMA,
+      g_param_spec_boolean ("clamp-chroma", "Clamp chroma", "Clamp chroma",
+          FALSE, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), PROP_QUALITY,
+      g_param_spec_enum ("quality", "Quality", "Decoding quality",
+          GST_TYPE_DVDEC_QUALITY, DV_DEFAULT_QUALITY,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), PROP_DECODE_NTH,
+      g_param_spec_int ("drop-factor", "Drop Factor", "Only decode Nth frame",
+          1, G_MAXINT, DV_DEFAULT_DECODE_NTH,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  gstelement_class->change_state = GST_DEBUG_FUNCPTR (gst_dvdec_change_state);
+
+  gst_element_class_add_pad_template (gstelement_class,
+      gst_static_pad_template_get (&sink_temp));
+  gst_element_class_add_pad_template (gstelement_class,
+      gst_static_pad_template_get (&src_temp));
+
+  gst_element_class_set_static_metadata (gstelement_class, "DV video decoder",
+      "Codec/Decoder/Video",
+      "Uses libdv to decode DV video (smpte314) (libdv.sourceforge.net)",
+      "Erik Walthinsen <omega@cse.ogi.edu>," "Wim Taymans <wim@fluendo.com>");
+
+  GST_DEBUG_CATEGORY_INIT (dvdec_debug, "dvdec", 0, "DV decoding element");
 }
 
-/* This function is responsible for initializing a specific instance of
- * the plugin.
- */
 static void
-gst_dvdec_init(GstDVDec *dvdec)
+gst_dvdec_init (GstDVDec * dvdec)
 {
-  gint i;
-  dvdec->found_header = FALSE;
-
-  dvdec->sinkpad = gst_pad_new_from_template (gst_static_pad_template_get (&sink_temp), "sink");
+  dvdec->sinkpad = gst_pad_new_from_static_template (&sink_temp, "sink");
+  gst_pad_set_chain_function (dvdec->sinkpad, gst_dvdec_chain);
+  gst_pad_set_event_function (dvdec->sinkpad, gst_dvdec_sink_event);
   gst_element_add_pad (GST_ELEMENT (dvdec), dvdec->sinkpad);
-  gst_pad_set_query_function (dvdec->sinkpad, NULL);
-  gst_pad_set_convert_function (dvdec->sinkpad, GST_DEBUG_FUNCPTR (gst_dvdec_sink_convert));
-  gst_pad_set_formats_function (dvdec->sinkpad, GST_DEBUG_FUNCPTR (gst_dvdec_get_formats));
-
-  dvdec->videosrcpad = gst_pad_new_from_template (gst_static_pad_template_get (&video_src_temp), "video");
-  gst_pad_set_query_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_src_query));
-  gst_pad_set_query_type_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_get_src_query_types));
-  gst_pad_set_event_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_handle_src_event));
-  gst_pad_set_event_mask_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_get_event_masks));
-  gst_pad_set_convert_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_src_convert));
-  gst_pad_set_formats_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_get_formats));
-  gst_pad_set_getcaps_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_video_getcaps));
-  gst_pad_set_link_function (dvdec->videosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_video_link));
-  gst_element_add_pad (GST_ELEMENT (dvdec), dvdec->videosrcpad);
-  
-  dvdec->audiosrcpad = gst_pad_new_from_template (gst_static_pad_template_get (&audio_src_temp), "audio");
-  gst_pad_set_query_function (dvdec->audiosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_src_query));
-  gst_pad_set_query_type_function (dvdec->audiosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_get_src_query_types));
-  gst_pad_set_event_function (dvdec->audiosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_handle_src_event));
-  gst_pad_set_event_mask_function (dvdec->audiosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_get_event_masks));
-  gst_pad_set_convert_function (dvdec->audiosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_src_convert));
-  gst_pad_set_formats_function (dvdec->audiosrcpad, GST_DEBUG_FUNCPTR (gst_dvdec_get_formats));
-  gst_pad_use_explicit_caps (dvdec->audiosrcpad);
-  gst_element_add_pad (GST_ELEMENT(dvdec), dvdec->audiosrcpad);
-
-  gst_element_set_loop_function (GST_ELEMENT (dvdec), gst_dvdec_loop);
-
-  dvdec->length = 0;
-  dvdec->next_ts = 0LL;
-  dvdec->end_position = -1LL;
-  dvdec->need_discont = FALSE;
-  dvdec->framerate = 0;
-  dvdec->height = 0;
-  dvdec->frequency = 0;
-  dvdec->channels = 0;
-  
+
+  dvdec->srcpad = gst_pad_new_from_static_template (&src_temp, "src");
+  gst_pad_use_fixed_caps (dvdec->srcpad);
+  gst_element_add_pad (GST_ELEMENT (dvdec), dvdec->srcpad);
+
+  dvdec->framerate_numerator = 0;
+  dvdec->framerate_denominator = 0;
+  dvdec->wide = FALSE;
+  dvdec->drop_factor = 1;
+
   dvdec->clamp_luma = FALSE;
   dvdec->clamp_chroma = FALSE;
-  dvdec->quality = DV_QUALITY_BEST;
-  dvdec->loop = FALSE;
-  
-  for (i = 0; i <4; i++) {
-    dvdec->audio_buffers[i] = (gint16 *)g_malloc (DV_AUDIO_MAX_SAMPLES * sizeof (gint16));
-  }
+  dvdec->quality = DV_DEFAULT_QUALITY;
 }
 
-static const GstFormat*
-gst_dvdec_get_formats (GstPad *pad)
-{
-  static const GstFormat src_formats[] = {
-    GST_FORMAT_BYTES,
-    GST_FORMAT_DEFAULT,
-    GST_FORMAT_TIME,
-    0
-  };
-  static const GstFormat sink_formats[] = {
-    GST_FORMAT_BYTES,
-    GST_FORMAT_TIME,
-    0
-  };
-            
-  return (GST_PAD_IS_SRC (pad) ? src_formats : sink_formats);
-} 
-
 static gboolean
-gst_dvdec_src_convert (GstPad *pad, GstFormat src_format, gint64 src_value,
-                       GstFormat *dest_format, gint64 *dest_value)
+gst_dvdec_sink_setcaps (GstDVDec * dvdec, GstCaps * caps)
 {
-  gboolean res = TRUE;
-  GstDVDec *dvdec;
-  gint scale = 1;
-        
-  dvdec = GST_DVDEC (gst_pad_get_parent (pad));
+  GstStructure *s;
+  const GValue *par = NULL, *rate = NULL;
+
+  /* first parse the caps */
+  s = gst_caps_get_structure (caps, 0);
+
+  /* we allow framerate and PAR to be overwritten. framerate is mandatory. */
+  if (!(rate = gst_structure_get_value (s, "framerate")))
+    goto no_framerate;
+  par = gst_structure_get_value (s, "pixel-aspect-ratio");
+
+  if (par) {
+    dvdec->par_x = gst_value_get_fraction_numerator (par);
+    dvdec->par_y = gst_value_get_fraction_denominator (par);
+    dvdec->need_par = FALSE;
+  } else {
+    dvdec->par_x = 0;
+    dvdec->par_y = 0;
+    dvdec->need_par = TRUE;
+  }
+  dvdec->framerate_numerator = gst_value_get_fraction_numerator (rate);
+  dvdec->framerate_denominator = gst_value_get_fraction_denominator (rate);
+  dvdec->sink_negotiated = TRUE;
+  dvdec->src_negotiated = FALSE;
 
-  if (dvdec->length == 0)
-    return FALSE;
+  return TRUE;
 
-  if (dvdec->decoder == NULL)
+  /* ERRORS */
+no_framerate:
+  {
+    GST_DEBUG_OBJECT (dvdec, "no framerate specified in caps");
     return FALSE;
-
-  switch (src_format) {
-    case GST_FORMAT_BYTES:
-      switch (*dest_format) {
-        case GST_FORMAT_TIME:
-        default:
-          res = FALSE;
-      }
-      break;
-    case GST_FORMAT_TIME:
-      switch (*dest_format) {
-        case GST_FORMAT_BYTES:
-         if (pad == dvdec->videosrcpad)
-           scale = 720 * dvdec->height * dvdec->bpp;
-         else if (pad == dvdec->audiosrcpad)
-           scale = dvdec->decoder->audio->num_channels * 2;
-         /* fallthrough */
-        case GST_FORMAT_DEFAULT:
-         if (pad == dvdec->videosrcpad)
-           *dest_value = src_value * dvdec->framerate * scale / GST_SECOND;
-         else if (pad == dvdec->audiosrcpad)
-           *dest_value = src_value * dvdec->decoder->audio->frequency * scale / GST_SECOND;
-         break;
-        default:
-          res = FALSE;
-      }
-      break;
-    default:
-      res = FALSE;
   }
-  return res;
 }
 
-static gboolean
-gst_dvdec_sink_convert (GstPad *pad, GstFormat src_format, gint64 src_value,
-                        GstFormat *dest_format, gint64 *dest_value)
+static void
+gst_dvdec_negotiate_pool (GstDVDec * dec, GstCaps * caps, GstVideoInfo * info)
 {
-  gboolean res = TRUE;
-  GstDVDec *dvdec;
-        
-  dvdec = GST_DVDEC (gst_pad_get_parent (pad));
+  GstQuery *query;
+  GstBufferPool *pool;
+  guint size, min, max;
+  GstStructure *config;
 
-  if (dvdec->length == 0)
-    return FALSE;
+  /* find a pool for the negotiated caps now */
+  query = gst_query_new_allocation (caps, TRUE);
 
-  switch (src_format) {
-    case GST_FORMAT_BYTES:
-      switch (*dest_format) {
-        case GST_FORMAT_TIME:
-       {
-          guint64 frame;
-         /* get frame number */
-          frame = src_value / dvdec->length;
-
-          *dest_value = (frame * GST_SECOND) / dvdec->framerate;
-          break;
-       }
-        default:
-          res = FALSE;
-      }
-      break;
-    case GST_FORMAT_TIME:
-      switch (*dest_format) {
-        case GST_FORMAT_BYTES:
-       {
-          guint64 frame;
-          /* calculate the frame */
-          frame = src_value * dvdec->framerate / GST_SECOND;
-          /* calculate the offset */
-          *dest_value = frame * dvdec->length;
-          break;
-       }
-        default:
-          res = FALSE;
-      }
-      break;
-    default:
-      res = FALSE;
+  if (!gst_pad_peer_query (dec->srcpad, query)) {
+    GST_DEBUG_OBJECT (dec, "didn't get downstream ALLOCATION hints");
   }
-  return res;
-}
 
-static const GstQueryType*
-gst_dvdec_get_src_query_types (GstPad *pad)
-{   
-  static const GstQueryType src_query_types[] = {
-    GST_QUERY_TOTAL,
-    GST_QUERY_POSITION,
-    0
-  };
-  return src_query_types;
-}
+  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);
+    size = MAX (size, info->size);
+  } else {
+    pool = NULL;
+    size = info->size;
+    min = max = 0;
+  }
 
-static gboolean
-gst_dvdec_src_query (GstPad *pad, GstQueryType type,
-                     GstFormat *format, gint64 *value)
-{
-  gboolean res = TRUE;
-  GstDVDec *dvdec;
+  if (pool == NULL) {
+    /* we did not get a pool, make one ourselves then */
+    pool = gst_video_buffer_pool_new ();
+  }
 
-  dvdec = GST_DVDEC (gst_pad_get_parent (pad));
-
-  switch (type) {
-    case GST_QUERY_TOTAL:
-      switch (*format) {
-        default:
-       {
-          guint64 len;
-         GstFormat tmp_format;
-
-         len = gst_bytestream_length (dvdec->bs);
-         tmp_format = GST_FORMAT_TIME;
-         if (len == -1 || !gst_pad_convert (dvdec->sinkpad, 
-                                            GST_FORMAT_BYTES, 
-                                            len, 
-                                            &tmp_format, value)) 
-         {
-           return FALSE;
-         }
-         if (!gst_pad_convert (pad, GST_FORMAT_TIME, *value, format, value)) {
-           return FALSE;
-         }
-          break;
-       }
-      }
-      break;
-    case GST_QUERY_POSITION:
-      switch (*format) {
-        default:
-         res = gst_pad_convert (pad, GST_FORMAT_TIME, dvdec->next_ts, format, value);
-          break;
-      }
-      break;
-    default:
-      res = FALSE;
-      break;
+  if (dec->pool) {
+    gst_buffer_pool_set_active (dec->pool, FALSE);
+    gst_object_unref (dec->pool);
   }
-  return res;
-} 
-
-static const GstEventMask*
-gst_dvdec_get_event_masks (GstPad *pad)
-{ 
-  static const GstEventMask src_event_masks[] = {
-    { GST_EVENT_SEEK, GST_SEEK_METHOD_SET |
-                      GST_SEEK_FLAG_FLUSH },
-    { 0, }
-  };
-  static const GstEventMask sink_event_masks[] = {
-    { GST_EVENT_EOS, 0 },
-    { GST_EVENT_DISCONTINUOUS, 0 },
-    { GST_EVENT_FLUSH, 0 },
-    { 0, }
-  };
-
-  return (GST_PAD_IS_SRC (pad) ? src_event_masks : sink_event_masks);
-} 
+  dec->pool = pool;
+
+  config = gst_buffer_pool_get_config (pool);
+  gst_buffer_pool_config_set_params (config, caps, size, min, max);
+
+  if (gst_query_find_allocation_meta (query, GST_VIDEO_META_API_TYPE, NULL)) {
+    /* just set the option, if the pool can support it we will transparently use
+     * it through the video info API. We could also see if the pool support this
+     * option and only activate it then. */
+    gst_buffer_pool_config_add_option (config,
+        GST_BUFFER_POOL_OPTION_VIDEO_META);
+  }
+  gst_buffer_pool_set_config (pool, config);
+
+  /* and activate */
+  gst_buffer_pool_set_active (pool, TRUE);
+
+  gst_query_unref (query);
+}
 
 static gboolean
-gst_dvdec_handle_sink_event (GstDVDec *dvdec)
+gst_dvdec_src_negotiate (GstDVDec * dvdec)
 {
-  guint32 remaining;
-  GstEvent *event;
-  GstEventType type;
-               
-  gst_bytestream_get_status (dvdec->bs, &remaining, &event);
-                 
-  type = event? GST_EVENT_TYPE (event) : GST_EVENT_UNKNOWN;
-                     
-  switch (type) {
-    case GST_EVENT_FLUSH:
-      break;
-    case GST_EVENT_DISCONTINUOUS:
-    {
-      gint i;
-      gboolean found = FALSE;
-      GstFormat format;
-
-      format = GST_FORMAT_TIME;
-      /* try to get a timestamp from the discont formats */
-      for (i = 0; i < GST_EVENT_DISCONT_OFFSET_LEN(event); i++) {
-       if (gst_pad_convert (dvdec->sinkpad, 
-                            GST_EVENT_DISCONT_OFFSET(event,i).format, 
-                            GST_EVENT_DISCONT_OFFSET(event,i).value,
-                            &format, &dvdec->next_ts)) 
-       {
-          found = TRUE;
-         break;
-       }
+  GstCaps *othercaps;
+
+  /* no PAR was specified in input, derive from encoded data */
+  if (dvdec->need_par) {
+    if (dvdec->PAL) {
+      if (dvdec->wide) {
+        dvdec->par_x = PAL_WIDE_PAR_X;
+        dvdec->par_y = PAL_WIDE_PAR_Y;
+      } else {
+        dvdec->par_x = PAL_NORMAL_PAR_X;
+        dvdec->par_y = PAL_NORMAL_PAR_Y;
       }
-      /* assume 0 then */
-      if (!found) {
-       dvdec->next_ts = 0LL;
+    } else {
+      if (dvdec->wide) {
+        dvdec->par_x = NTSC_WIDE_PAR_X;
+        dvdec->par_y = NTSC_WIDE_PAR_Y;
+      } else {
+        dvdec->par_x = NTSC_NORMAL_PAR_X;
+        dvdec->par_y = NTSC_NORMAL_PAR_Y;
       }
-      dvdec->need_discont = TRUE;
-      break;
     }
-    default:
-      return gst_pad_event_default (dvdec->sinkpad, event);
-      break;
+    GST_DEBUG_OBJECT (dvdec, "Inferred PAR %d/%d from video format",
+        dvdec->par_x, dvdec->par_y);
   }
-  gst_event_unref (event);
+
+  /* ignoring rgb, bgr0 for now */
+  dvdec->bpp = 2;
+
+  gst_video_info_set_format (&dvdec->vinfo, GST_VIDEO_FORMAT_YUY2,
+      720, dvdec->height);
+  dvdec->vinfo.fps_n = dvdec->framerate_numerator;
+  dvdec->vinfo.fps_d = dvdec->framerate_denominator;
+  dvdec->vinfo.par_n = dvdec->par_x;
+  dvdec->vinfo.par_d = dvdec->par_y;
+  if (dvdec->interlaced) {
+    dvdec->vinfo.interlace_mode = GST_VIDEO_INTERLACE_MODE_INTERLEAVED;
+  } else {
+    dvdec->vinfo.interlace_mode = GST_VIDEO_INTERLACE_MODE_PROGRESSIVE;
+  }
+
+  othercaps = gst_video_info_to_caps (&dvdec->vinfo);
+  gst_pad_set_caps (dvdec->srcpad, othercaps);
+
+  gst_dvdec_negotiate_pool (dvdec, othercaps, &dvdec->vinfo);
+  gst_caps_unref (othercaps);
+
+  dvdec->src_negotiated = TRUE;
+
   return TRUE;
 }
 
 static gboolean
-gst_dvdec_handle_src_event (GstPad *pad, GstEvent *event)
+gst_dvdec_sink_event (GstPad * pad, GstObject * parent, GstEvent * event)
 {
-  gboolean res = TRUE;
   GstDVDec *dvdec;
+  gboolean res = TRUE;
+
+  dvdec = GST_DVDEC (parent);
 
-  dvdec = GST_DVDEC (gst_pad_get_parent (pad));
   switch (GST_EVENT_TYPE (event)) {
-    case GST_EVENT_SEEK_SEGMENT:
-    {
-      gint64 position;
-      GstFormat format;
-
-      /* first bring the format to time */
-      format = GST_FORMAT_TIME;
-      if (!gst_pad_convert (pad, 
-                           GST_EVENT_SEEK_FORMAT (event), 
-                           GST_EVENT_SEEK_ENDOFFSET (event),
-                           &format, &position)) 
-      {
-        /* could not convert seek format to time offset */
-       res = FALSE;
-       break;
-      }
+    case GST_EVENT_FLUSH_STOP:
+      gst_segment_init (&dvdec->segment, GST_FORMAT_UNDEFINED);
+      dvdec->need_segment = FALSE;
+      break;
+    case GST_EVENT_SEGMENT:{
+      const GstSegment *segment;
 
-      dvdec->end_position = position;
-      dvdec->loop = GST_EVENT_SEEK_TYPE (event) & GST_SEEK_FLAG_SEGMENT_LOOP;
+      gst_event_parse_segment (event, &segment);
+
+      GST_DEBUG_OBJECT (dvdec, "Got SEGMENT %" GST_SEGMENT_FORMAT, &segment);
+
+      gst_segment_copy_into (segment, &dvdec->segment);
+      if (!gst_pad_has_current_caps (dvdec->srcpad)) {
+        dvdec->need_segment = TRUE;
+        gst_event_unref (event);
+        event = NULL;
+        res = TRUE;
+      } else {
+        dvdec->need_segment = FALSE;
+      }
+      break;
     }
-    case GST_EVENT_SEEK:
+    case GST_EVENT_CAPS:
     {
-      gint64 position;
-      GstFormat format;
-
-      /* first bring the format to time */
-      format = GST_FORMAT_TIME;
-      if (!gst_pad_convert (pad, 
-                           GST_EVENT_SEEK_FORMAT (event), 
-                           GST_EVENT_SEEK_OFFSET (event),
-                           &format, &position)) 
-      {
-        /* could not convert seek format to time offset */
-        res = FALSE;
-       break;
-      }
-      dvdec->next_ts = position;
-      /* then try to figure out the byteoffset for this time */
-      format = GST_FORMAT_BYTES;
-      if (!gst_pad_convert (dvdec->sinkpad, GST_FORMAT_TIME, position,
-                       &format, &position)) 
-      {
-       /* could not convert seek format to byte offset */
-       res = FALSE;
-       break;
-      }
-      /* seek to offset */
-      if (!gst_bytestream_seek (dvdec->bs, position, GST_SEEK_METHOD_SET)) {
-       res = FALSE;
-      }
-      if (GST_EVENT_TYPE (event) != GST_EVENT_SEEK_SEGMENT)
-        dvdec->end_position = -1;
+      GstCaps *caps;
+
+      gst_event_parse_caps (event, &caps);
+      res = gst_dvdec_sink_setcaps (dvdec, caps);
+      gst_event_unref (event);
+      event = NULL;
       break;
     }
+
     default:
-      res = FALSE;
       break;
   }
-  gst_event_unref (event);
-  return res;
-}
 
-static GstCaps*   
-gst_dvdec_video_getcaps (GstPad *pad)
-{
-  GstDVDec *dvdec;
-  GstCaps *caps;
-  GstPadTemplate *src_pad_template;
-  
-  dvdec = GST_DVDEC (gst_pad_get_parent (pad));
-  src_pad_template = gst_static_pad_template_get (&video_src_temp);
-  caps = gst_caps_copy(gst_pad_template_get_caps (src_pad_template));
-  
-  if (dvdec->found_header)
-  {
-    int i;
-    
-    /* set the height */
-    for (i = 0; i < gst_caps_get_size (caps); i++)
-    {
-       GstStructure *structure = gst_caps_get_structure (caps, i);
-       gst_structure_set(structure, 
-               "height", G_TYPE_INT, dvdec->height,
-              "framerate", G_TYPE_DOUBLE, dvdec->framerate, NULL
-             );
-    }
-  }
-  
-  return caps;
+  if (event)
+    res = gst_pad_push_event (dvdec->srcpad, event);
+
+  return res;
 }
 
-static GstPadLinkReturn 
-gst_dvdec_video_link (GstPad *pad, const GstCaps *caps)
+static GstFlowReturn
+gst_dvdec_chain (GstPad * pad, GstObject * parent, GstBuffer * buf)
 {
   GstDVDec *dvdec;
-  GstStructure *structure;
-  guint32 fourcc;
-  gint height;
-  gdouble framerate;
-  
-  dvdec = GST_DVDEC (gst_pad_get_parent (pad));
-
-  /* if we did not find a header yet, return delayed */
-  if (!dvdec->found_header)
-  {
-    return GST_PAD_LINK_DELAYED;
-  }
-  structure = gst_caps_get_structure (caps, 0);
-  
-  /* it worked, try to find what it was again */
-  if (!gst_structure_get_fourcc (structure, "format", &fourcc) ||
-      !gst_structure_get_int (structure, "height", &height) ||
-      !gst_structure_get_double (structure, "framerate", &framerate))
-    return GST_PAD_LINK_REFUSED;
-  
-  if ((height != dvdec->height) || (framerate != dvdec->framerate))
-    return GST_PAD_LINK_REFUSED;
-    
-  if (fourcc == GST_STR_FOURCC ("RGB ")) {
-    gint bpp;
-  
-    gst_structure_get_int (structure, "bpp", &bpp);
-    if (bpp == 24) {
-      dvdec->space = e_dv_color_rgb;
-      dvdec->bpp = 3;
-    }
-    else {
-      dvdec->space = e_dv_color_bgr0;
-      dvdec->bpp = 4;
-    }
-  }
-  else {
-    dvdec->space = e_dv_color_yuv;
-    dvdec->bpp = 2;
+  guint8 *inframe;
+  guint8 *outframe_ptrs[3];
+  gint outframe_pitches[3];
+  GstMapInfo map;
+  GstVideoFrame frame;
+  GstBuffer *outbuf;
+  GstFlowReturn ret = GST_FLOW_OK;
+  guint length;
+  guint64 cstart, cstop;
+  gboolean PAL, wide;
+
+  dvdec = GST_DVDEC (parent);
+
+  gst_buffer_map (buf, &map, GST_MAP_READ);
+  inframe = map.data;
+
+  /* buffer should be at least the size of one NTSC frame, this should
+   * be enough to decode the header. */
+  if (G_UNLIKELY (map.size < NTSC_BUFFER))
+    goto wrong_size;
+
+  /* preliminary dropping. unref and return if outside of configured segment */
+  if ((dvdec->segment.format == GST_FORMAT_TIME) &&
+      (!(gst_segment_clip (&dvdec->segment, GST_FORMAT_TIME,
+                  GST_BUFFER_TIMESTAMP (buf),
+                  GST_BUFFER_TIMESTAMP (buf) + GST_BUFFER_DURATION (buf),
+                  &cstart, &cstop))))
+    goto dropping;
+
+  if (G_UNLIKELY (dv_parse_header (dvdec->decoder, inframe) < 0))
+    goto parse_header_error;
+
+  /* get size */
+  PAL = dv_system_50_fields (dvdec->decoder);
+  wide = dv_format_wide (dvdec->decoder);
+
+  /* check the buffer is of right size after we know if we are
+   * dealing with PAL or NTSC */
+  length = (PAL ? PAL_BUFFER : NTSC_BUFFER);
+  if (G_UNLIKELY (map.size < length))
+    goto wrong_size;
+
+  dv_parse_packs (dvdec->decoder, inframe);
+
+  if (dvdec->video_offset % dvdec->drop_factor != 0)
+    goto skip;
+
+  /* renegotiate on change */
+  if (PAL != dvdec->PAL || wide != dvdec->wide) {
+    dvdec->src_negotiated = FALSE;
+    dvdec->PAL = PAL;
+    dvdec->wide = wide;
   }
-  
-  return GST_PAD_LINK_OK;
-}
 
-static void
-gst_dvdec_push (GstDVDec *dvdec, GstBuffer *outbuf, GstPad *pad, GstClockTime ts)
-{   
-  GST_BUFFER_TIMESTAMP (outbuf) = ts;
+  dvdec->height = (dvdec->PAL ? PAL_HEIGHT : NTSC_HEIGHT);
 
-  if (dvdec->need_discont) {
-    GstEvent *discont;
+  dvdec->interlaced = !dv_is_progressive (dvdec->decoder);
 
-    discont = gst_event_new_discontinuous (FALSE, GST_FORMAT_TIME, ts, NULL);
-    gst_pad_push (pad, GST_DATA (discont));
+  /* negotiate if not done yet */
+  if (!dvdec->src_negotiated) {
+    if (!gst_dvdec_src_negotiate (dvdec))
+      goto not_negotiated;
   }
 
-  gst_pad_push (pad, GST_DATA (outbuf));
+  if (gst_pad_check_reconfigure (dvdec->srcpad)) {
+    GstCaps *caps;
 
-  if ((dvdec->end_position != -1) &&
-      (dvdec->next_ts >= dvdec->end_position)) {
-    if (dvdec->loop) 
-      gst_pad_push (pad, GST_DATA(gst_event_new (GST_EVENT_SEGMENT_DONE)));
-    else
-      gst_pad_push (pad, GST_DATA(gst_event_new (GST_EVENT_EOS)));
+    caps = gst_pad_get_current_caps (dvdec->srcpad);
+    gst_dvdec_negotiate_pool (dvdec, caps, &dvdec->vinfo);
+    gst_caps_unref (caps);
   }
-}
 
-static void
-gst_dvdec_loop (GstElement *element)
-{   
-  GstDVDec *dvdec;
-  GstBuffer *buf, *outbuf;
-  guint8 *inframe;
-  gint height;
-  guint32 length, got_bytes;
-  GstFormat format;
-  guint64 ts;
-  gdouble fps;
-
-  dvdec = GST_DVDEC (element);
-
-  /* first read enough bytes to parse the header */
-  got_bytes = gst_bytestream_peek_bytes (dvdec->bs, &inframe, header_size);
-  if (got_bytes < header_size) {
-    gst_dvdec_handle_sink_event (dvdec);
-    return;
-  }
-  if (dv_parse_header (dvdec->decoder, inframe) < 0)
-  {
-    GST_ELEMENT_ERROR (dvdec, STREAM, DECODE, (NULL), (NULL));
-    return;
-  }
-  
-  /* after parsing the header we know the length of the data */
-  dvdec->PAL = dv_system_50_fields (dvdec->decoder);
-  dvdec->found_header = TRUE;
-  
-  fps = (dvdec->PAL ? PAL_FRAMERATE : NTSC_FRAMERATE);
-  height = (dvdec->PAL ? PAL_HEIGHT : NTSC_HEIGHT);
-  length = (dvdec->PAL ? PAL_BUFFER : NTSC_BUFFER);
-
-  if ((dvdec->framerate != fps) ||
-      (dvdec->height != height))
-  {
-    dvdec->height = height;
-    dvdec->framerate = fps;
-    
-    if (GST_PAD_LINK_FAILED (gst_pad_renegotiate (dvdec->videosrcpad)))
-    {
-      GST_ELEMENT_ERROR (dvdec, CORE, NEGOTIATION, (NULL), (NULL));
-      return;
-    }
-  }
-  
-  if (length != dvdec->length) {
-    dvdec->length = length;
-    gst_bytestream_size_hint (dvdec->bs, length);
+  if (dvdec->need_segment) {
+    gst_pad_push_event (dvdec->srcpad, gst_event_new_segment (&dvdec->segment));
+    dvdec->need_segment = FALSE;
   }
 
-  /* then read the read data */
-  got_bytes = gst_bytestream_read (dvdec->bs, &buf, length);
-  if (got_bytes < length) {
-    gst_dvdec_handle_sink_event (dvdec);
-    return;
+  ret = gst_buffer_pool_acquire_buffer (dvdec->pool, &outbuf, NULL);
+  if (G_UNLIKELY (ret != GST_FLOW_OK))
+    goto no_buffer;
+
+  gst_video_frame_map (&frame, &dvdec->vinfo, outbuf, GST_MAP_WRITE);
+
+  outframe_ptrs[0] = GST_VIDEO_FRAME_COMP_DATA (&frame, 0);
+  outframe_pitches[0] = GST_VIDEO_FRAME_COMP_STRIDE (&frame, 0);
+
+  /* the rest only matters for YUY2 */
+  if (dvdec->bpp < 3) {
+    outframe_ptrs[1] = GST_VIDEO_FRAME_COMP_DATA (&frame, 1);
+    outframe_ptrs[2] = GST_VIDEO_FRAME_COMP_DATA (&frame, 2);
+
+    outframe_pitches[1] = GST_VIDEO_FRAME_COMP_STRIDE (&frame, 1);
+    outframe_pitches[2] = GST_VIDEO_FRAME_COMP_STRIDE (&frame, 2);
   }
 
-  format = GST_FORMAT_TIME;
-  gst_pad_query (dvdec->videosrcpad, GST_QUERY_POSITION, &format, &ts);
+  GST_DEBUG_OBJECT (dvdec, "decoding and pushing buffer");
+  dv_decode_full_frame (dvdec->decoder, inframe,
+      e_dv_color_yuv, outframe_ptrs, outframe_pitches);
 
-  dvdec->next_ts += GST_SECOND / dvdec->framerate;
+  gst_video_frame_unmap (&frame);
 
-  dv_parse_packs (dvdec->decoder, GST_BUFFER_DATA (buf));
+  GST_BUFFER_FLAG_UNSET (outbuf, GST_VIDEO_BUFFER_FLAG_TFF);
 
-  if (GST_PAD_IS_LINKED (dvdec->audiosrcpad)) {
-    gint16 *a_ptr;
-    gint i, j;
+  GST_BUFFER_OFFSET (outbuf) = GST_BUFFER_OFFSET (buf);
+  GST_BUFFER_OFFSET_END (outbuf) = GST_BUFFER_OFFSET_END (buf);
+  GST_BUFFER_TIMESTAMP (outbuf) = cstart;
+  GST_BUFFER_DURATION (outbuf) = cstop - cstart;
 
-    dv_decode_full_audio (dvdec->decoder, GST_BUFFER_DATA (buf), dvdec->audio_buffers);
+  ret = gst_pad_push (dvdec->srcpad, outbuf);
 
-    if ((dvdec->decoder->audio->frequency != dvdec->frequency) ||
-        (dvdec->decoder->audio->num_channels != dvdec->channels))
-    {
-      if (!gst_pad_set_explicit_caps (dvdec->audiosrcpad,
-                      gst_caps_new_simple (
-                        "audio/x-raw-int", 
-                        "rate", G_TYPE_INT, dvdec->decoder->audio->frequency,
-                        "depth", G_TYPE_INT,  16,
-                        "width", G_TYPE_INT,  16,
-                        "signed", G_TYPE_BOOLEAN, TRUE,
-                        "channels", G_TYPE_INT, dvdec->decoder->audio->num_channels,
-                        "endianness", G_TYPE_INT, G_LITTLE_ENDIAN,
-                        NULL
-                      )))
-      {
-        gst_buffer_unref (buf);
-        GST_ELEMENT_ERROR (dvdec, CORE, NEGOTIATION, (NULL), (NULL));        
-        return;
-      }
-      
-      dvdec->frequency = dvdec->decoder->audio->frequency;
-      dvdec->channels = dvdec->decoder->audio->num_channels;
-    }
-      
-    outbuf = gst_buffer_new ();
-    GST_BUFFER_SIZE (outbuf) = dvdec->decoder->audio->samples_this_frame * 
-                        sizeof (gint16) * dvdec->decoder->audio->num_channels;
-    GST_BUFFER_DATA (outbuf) = g_malloc (GST_BUFFER_SIZE (outbuf));
-    
-    a_ptr = (gint16 *) GST_BUFFER_DATA (outbuf);
-    
-    for (i = 0; i < dvdec->decoder->audio->samples_this_frame; i++) {
-      for (j = 0; j < dvdec->decoder->audio->num_channels; j++) {
-        *(a_ptr++) = dvdec->audio_buffers[j][i];
-      }
-    }
-    gst_dvdec_push (dvdec, outbuf, dvdec->audiosrcpad, ts);
-  }
+skip:
+  dvdec->video_offset++;
 
-  if (GST_PAD_IS_LINKED (dvdec->videosrcpad)) {
-    guint8 *outframe;
-    guint8 *outframe_ptrs[3];
-    gint outframe_pitches[3];
-
-    outbuf = gst_buffer_new_and_alloc ((720 * height) * dvdec->bpp);
-    
-    outframe = GST_BUFFER_DATA (outbuf);
-
-    outframe_ptrs[0] = outframe;
-    outframe_pitches[0] = 720 * dvdec->bpp;
-
-    /* the rest only matters for YUY2 */
-    if (dvdec->bpp < 3) {
-      outframe_ptrs[1] = outframe_ptrs[0] + 720 * height;
-      outframe_ptrs[2] = outframe_ptrs[1] + 360 * height;
-  
-      outframe_pitches[1] = height / 2;
-      outframe_pitches[2] = outframe_pitches[1];
-    }
+done:
+  gst_buffer_unmap (buf, &map);
+  gst_buffer_unref (buf);
 
-    dv_decode_full_frame (dvdec->decoder, GST_BUFFER_DATA (buf), 
-                         dvdec->space, outframe_ptrs, outframe_pitches);
+  return ret;
 
-    gst_dvdec_push (dvdec, outbuf, dvdec->videosrcpad, ts);
+  /* ERRORS */
+wrong_size:
+  {
+    GST_ELEMENT_ERROR (dvdec, STREAM, DECODE,
+        (NULL), ("Input buffer too small"));
+    ret = GST_FLOW_ERROR;
+    goto done;
   }
-
-  if ((dvdec->end_position != -1) && 
-      (dvdec->next_ts >= dvdec->end_position) && 
-      !dvdec->loop) {
-    gst_element_set_eos (GST_ELEMENT (dvdec));
+parse_header_error:
+  {
+    GST_ELEMENT_ERROR (dvdec, STREAM, DECODE,
+        (NULL), ("Error parsing DV header"));
+    ret = GST_FLOW_ERROR;
+    goto done;
   }
-
-  if (dvdec->need_discont) {
-    dvdec->need_discont = FALSE;
+not_negotiated:
+  {
+    GST_DEBUG_OBJECT (dvdec, "could not negotiate output");
+    ret = GST_FLOW_NOT_NEGOTIATED;
+    goto done;
+  }
+no_buffer:
+  {
+    GST_DEBUG_OBJECT (dvdec, "could not allocate buffer");
+    goto done;
   }
 
-  gst_buffer_unref (buf);
+dropping:
+  {
+    GST_DEBUG_OBJECT (dvdec,
+        "dropping buffer since it's out of the configured segment");
+    goto done;
+  }
 }
 
-static GstElementStateReturn
-gst_dvdec_change_state (GstElement *element)
+static GstStateChangeReturn
+gst_dvdec_change_state (GstElement * element, GstStateChange transition)
 {
   GstDVDec *dvdec = GST_DVDEC (element);
-           
-  switch (GST_STATE_TRANSITION (element)) {
-    case GST_STATE_NULL_TO_READY:
+  GstStateChangeReturn ret;
+
+
+  switch (transition) {
+    case GST_STATE_CHANGE_NULL_TO_READY:
       break;
-    case GST_STATE_READY_TO_PAUSED:
-      dvdec->bs = gst_bytestream_new (dvdec->sinkpad);
-      dvdec->decoder = dv_decoder_new (0, dvdec->clamp_luma, dvdec->clamp_chroma);
-      dvdec->decoder->quality = dvdec->quality;
+    case GST_STATE_CHANGE_READY_TO_PAUSED:
+      dvdec->decoder =
+          dv_decoder_new (0, dvdec->clamp_luma, dvdec->clamp_chroma);
+      dvdec->decoder->quality = qualities[dvdec->quality];
+      dv_set_error_log (dvdec->decoder, NULL);
+      gst_video_info_init (&dvdec->vinfo);
+      gst_segment_init (&dvdec->segment, GST_FORMAT_UNDEFINED);
+      dvdec->src_negotiated = FALSE;
+      dvdec->sink_negotiated = FALSE;
+      dvdec->need_segment = FALSE;
+      /* 
+       * Enable this function call when libdv2 0.100 or higher is more
+       * common
+       */
+      /* dv_set_quality (dvdec->decoder, qualities [dvdec->quality]); */
       break;
-    case GST_STATE_PAUSED_TO_PLAYING:
+    case GST_STATE_CHANGE_PAUSED_TO_PLAYING:
+      break;
+    default:
       break;
-    case GST_STATE_PLAYING_TO_PAUSED:
+  }
+
+  ret = GST_ELEMENT_CLASS (parent_class)->change_state (element, transition);
+
+  switch (transition) {
+    case GST_STATE_CHANGE_PLAYING_TO_PAUSED:
       break;
-    case GST_STATE_PAUSED_TO_READY:
+    case GST_STATE_CHANGE_PAUSED_TO_READY:
       dv_decoder_free (dvdec->decoder);
       dvdec->decoder = NULL;
-      dvdec->found_header = FALSE;
-      gst_bytestream_destroy (dvdec->bs);
+      if (dvdec->pool) {
+        gst_buffer_pool_set_active (dvdec->pool, FALSE);
+        gst_object_unref (dvdec->pool);
+        dvdec->pool = NULL;
+      }
       break;
-    case GST_STATE_READY_TO_NULL:
+    case GST_STATE_CHANGE_READY_TO_NULL:
       break;
     default:
       break;
   }
-             
-  parent_class->change_state (element);
-               
-  return GST_STATE_SUCCESS;
+  return ret;
 }
 
-/* Arguments are part of the Gtk+ object system, and these functions
- * enable the element to respond to various arguments.
- */
 static void
-gst_dvdec_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec)
+gst_dvdec_set_property (GObject * object, guint prop_id, const GValue * value,
+    GParamSpec * pspec)
 {
-  GstDVDec *dvdec;
+  GstDVDec *dvdec = GST_DVDEC (object);
 
-  /* It's not null if we got it, but it might not be ours */
-  g_return_if_fail(GST_IS_DVDEC(object));
-
-  /* Get a pointer of the right type. */
-  dvdec = GST_DVDEC(object);
-
-  /* Check the argument id to see which argument we're setting. */
   switch (prop_id) {
-    case ARG_CLAMP_LUMA:
+    case PROP_CLAMP_LUMA:
       dvdec->clamp_luma = g_value_get_boolean (value);
       break;
-    case ARG_CLAMP_CHROMA:
+    case PROP_CLAMP_CHROMA:
       dvdec->clamp_chroma = g_value_get_boolean (value);
       break;
-    case ARG_QUALITY:
-      dvdec->quality = g_value_get_flags (value);
+    case PROP_QUALITY:
+      dvdec->quality = g_value_get_enum (value);
+      if ((dvdec->quality < 0) || (dvdec->quality > 5))
+        dvdec->quality = 0;
+      break;
+    case PROP_DECODE_NTH:
+      dvdec->drop_factor = g_value_get_int (value);
       break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
@@ -982,55 +648,27 @@ gst_dvdec_set_property (GObject *object, guint prop_id, const GValue *value, GPa
   }
 }
 
-/* The set function is simply the inverse of the get fuction. */
 static void
-gst_dvdec_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec)
+gst_dvdec_get_property (GObject * object, guint prop_id, GValue * value,
+    GParamSpec * pspec)
 {
-  GstDVDec *dvdec;
-
-  /* It's not null if we got it, but it might not be ours */
-  g_return_if_fail(GST_IS_DVDEC(object));
-  dvdec = GST_DVDEC(object);
+  GstDVDec *dvdec = GST_DVDEC (object);
 
   switch (prop_id) {
-    case ARG_CLAMP_LUMA:
+    case PROP_CLAMP_LUMA:
       g_value_set_boolean (value, dvdec->clamp_luma);
       break;
-    case ARG_CLAMP_CHROMA:
+    case PROP_CLAMP_CHROMA:
       g_value_set_boolean (value, dvdec->clamp_chroma);
       break;
-    case ARG_QUALITY:
-      g_value_set_flags (value, dvdec->quality);
+    case PROP_QUALITY:
+      g_value_set_enum (value, dvdec->quality);
+      break;
+    case PROP_DECODE_NTH:
+      g_value_set_int (value, dvdec->drop_factor);
       break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
   }
 }
-
-/* This is the entry into the plugin itself.  When the plugin loads,
- * this function is called to register everything that the plugin provides.
- */
-static gboolean
-plugin_init (GstPlugin *plugin)
-{
-  if (!gst_library_load ("gstbytestream"))
-    return FALSE;
-
-  if (!gst_element_register (plugin, "dvdec", GST_RANK_PRIMARY, gst_dvdec_get_type()))
-    return FALSE;
-
-  return TRUE;
-}
-
-GST_PLUGIN_DEFINE (
-  GST_VERSION_MAJOR,
-  GST_VERSION_MINOR,
-  "dvdec",
-  "Uses libdv to decode DV video (libdv.sourceforge.net)",
-  plugin_init,
-  VERSION,
-  "LGPL",
-  GST_PACKAGE,
-  GST_ORIGIN
-);