gst-indent
[platform/upstream/gst-plugins-good.git] / sys / oss / gstosssink.c
index 59e1950..053eba4 100644 (file)
  * Boston, MA 02111-1307, USA.
  */
 
-
-#include <sys/types.h>
-#include <sys/stat.h>
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
 #include <sys/ioctl.h>
-#include <fcntl.h>
 #include <sys/soundcard.h>
-#include <unistd.h>
 #include <errno.h>
-
-#include <gstosssink.h>
-
-static GstElementDetails gst_osssink_details = {  
-  "Audio Sink (OSS)",
-  "Sink/Audio",
-  "Output to a sound card via OSS",
-  VERSION,
-  "Erik Walthinsen <omega@cse.ogi.edu>, "
-  "Wim Taymans <wim.taymans@chello.be>",
-  "(C) 1999",
-};
-
-static void                    gst_osssink_class_init          (GstOssSinkClass *klass);
-static void                    gst_osssink_init                (GstOssSink *osssink);
-static void                    gst_osssink_finalize            (GObject *object);
-
-static gboolean                gst_osssink_open_audio          (GstOssSink *sink);
-static void                    gst_osssink_close_audio         (GstOssSink *sink);
-static gboolean                gst_osssink_sync_parms          (GstOssSink *osssink);
-static GstElementStateReturn   gst_osssink_change_state        (GstElement *element);
-static void                    gst_osssink_set_clock           (GstElement *element, GstClock *clock);
-static GstClock*               gst_osssink_get_clock           (GstElement *element);
-
-static GstPadConnectReturn     gst_osssink_sinkconnect         (GstPad *pad, GstCaps *caps);
-
-static void                    gst_osssink_set_property        (GObject *object, guint prop_id, const GValue *value, 
-                                                                GParamSpec *pspec);
-static void                    gst_osssink_get_property        (GObject *object, guint prop_id, GValue *value, 
-                                                                GParamSpec *pspec);
-
-static void                    gst_osssink_chain               (GstPad *pad,GstBuffer *buf);
+#include <unistd.h>
+#include <string.h>
+
+#include "gstosssink.h"
+
+/* elementfactory information */
+static GstElementDetails gst_osssink_details =
+GST_ELEMENT_DETAILS ("Audio Sink (OSS)",
+    "Sink/Audio",
+    "Output to a sound card via OSS",
+    "Erik Walthinsen <omega@cse.ogi.edu>, "
+    "Wim Taymans <wim.taymans@chello.be>");
+
+static void gst_osssink_base_init (gpointer g_class);
+static void gst_osssink_class_init (GstOssSinkClass * klass);
+static void gst_osssink_init (GstOssSink * osssink);
+static void gst_osssink_dispose (GObject * object);
+
+static GstElementStateReturn gst_osssink_change_state (GstElement * element);
+static void gst_osssink_set_clock (GstElement * element, GstClock * clock);
+static GstClock *gst_osssink_get_clock (GstElement * element);
+static GstClockTime gst_osssink_get_time (GstClock * clock, gpointer data);
+
+static const GstFormat *gst_osssink_get_formats (GstPad * pad);
+static gboolean gst_osssink_convert (GstPad * pad, GstFormat src_format,
+    gint64 src_value, GstFormat * dest_format, gint64 * dest_value);
+static const GstQueryType *gst_osssink_get_query_types (GstPad * pad);
+static gboolean gst_osssink_query (GstElement * element, GstQueryType type,
+    GstFormat * format, gint64 * value);
+static gboolean gst_osssink_sink_query (GstPad * pad, GstQueryType type,
+    GstFormat * format, gint64 * value);
+
+static GstCaps *gst_osssink_sink_fixate (GstPad * pad, const GstCaps * caps);
+static GstPadLinkReturn gst_osssink_sinkconnect (GstPad * pad,
+    const GstCaps * caps);
+
+static void gst_osssink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec);
+static void gst_osssink_get_property (GObject * object, guint prop_id,
+    GValue * value, GParamSpec * pspec);
+
+static void gst_osssink_chain (GstPad * pad, GstData * _data);
 
 /* OssSink signals and args */
-enum {
+enum
+{
   SIGNAL_HANDOFF,
   LAST_SIGNAL
 };
 
-enum {
+enum
+{
   ARG_0,
-  ARG_DEVICE,
   ARG_MUTE,
-  ARG_FORMAT,
-  ARG_CHANNELS,
-  ARG_FREQUENCY,
   ARG_FRAGMENT,
-  ARG_BUFFER_SIZE
+  ARG_BUFFER_SIZE,
+  ARG_SYNC,
+  ARG_CHUNK_SIZE,
   /* FILL ME */
 };
 
-GST_PAD_TEMPLATE_FACTORY (osssink_sink_factory,
-  "sink",
-  GST_PAD_SINK,
-  GST_PAD_ALWAYS,
-  GST_CAPS_NEW (
-    "osssink_sink",
-    "audio/raw",
-      "format",     GST_PROPS_STRING ("int"),   /* hack */
-      "law",        GST_PROPS_INT (0),
-      "endianness", GST_PROPS_INT (G_BYTE_ORDER),
-      "signed",     GST_PROPS_LIST (
-                     GST_PROPS_BOOLEAN (FALSE),
-                     GST_PROPS_BOOLEAN (TRUE)
-                   ),
-      "width",      GST_PROPS_LIST (
-                     GST_PROPS_INT (8),
-                     GST_PROPS_INT (16)
-                   ),
-      "depth",      GST_PROPS_LIST (
-                     GST_PROPS_INT (8),
-                     GST_PROPS_INT (16)
-                   ),
-      "rate",       GST_PROPS_INT_RANGE (1000, 48000),
-      "channels",   GST_PROPS_INT_RANGE (1, 2)
-  )
-);
-
-#define GST_TYPE_OSSSINK_CHANNELS (gst_osssink_channels_get_type())
-static GType 
-gst_osssink_channels_get_type(void) {
-  static GType osssink_channels_type = 0;
-  static GEnumValue osssink_channels[] = {
-    {0, "0", "Silence"},
-    {1, "1", "Mono"},
-    {2, "2", "Stereo"},
-    {0, NULL, NULL},
-  };
-  if (!osssink_channels_type) {
-    osssink_channels_type = g_enum_register_static("GstAudiosinkChannels", osssink_channels);
-  }
-  return osssink_channels_type;
-}
-
+static GstStaticPadTemplate osssink_sink_factory =
+GST_STATIC_PAD_TEMPLATE ("sink",
+    GST_PAD_SINK,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS ("audio/x-raw-int, "
+       "endianness = (int) BYTE_ORDER, "
+       "signed = (boolean) { TRUE, FALSE }, "
+       "width = (int) { 8, 16 }, "
+       "depth = (int) { 8, 16 }, "
+       "rate = (int) [ 1000, 48000 ], " "channels = (int) [ 1, 2 ]")
+    );
 
 static GstElementClass *parent_class = NULL;
 static guint gst_osssink_signals[LAST_SIGNAL] = { 0 };
 
 GType
-gst_osssink_get_type (void) 
+gst_osssink_get_type (void)
 {
   static GType osssink_type = 0;
 
   if (!osssink_type) {
     static const GTypeInfo osssink_info = {
-      sizeof(GstOssSinkClass),
-      NULL,
+      sizeof (GstOssSinkClass),
+      gst_osssink_base_init,
       NULL,
-      (GClassInitFunc)gst_osssink_class_init,
+      (GClassInitFunc) gst_osssink_class_init,
       NULL,
       NULL,
-      sizeof(GstOssSink),
+      sizeof (GstOssSink),
       0,
-      (GInstanceInitFunc)gst_osssink_init,
+      (GInstanceInitFunc) gst_osssink_init,
     };
-    osssink_type = g_type_register_static (GST_TYPE_ELEMENT, "GstOssSink", &osssink_info, 0);
+    osssink_type =
+       g_type_register_static (GST_TYPE_OSSELEMENT, "GstOssSink",
+       &osssink_info, 0);
   }
 
   return osssink_type;
 }
 
-static GstBufferPool*
-gst_osssink_get_bufferpool (GstPad *pad)
+static void
+gst_osssink_dispose (GObject * object)
 {
-  GstOssSink *oss;
-  
-  oss = GST_OSSSINK (gst_pad_get_parent(pad));
+  GstOssSink *osssink = (GstOssSink *) object;
+
+  gst_object_unparent (GST_OBJECT (osssink->provided_clock));
 
-  return oss->sinkpool;
+  G_OBJECT_CLASS (parent_class)->dispose (object);
 }
 
 static void
-gst_osssink_finalize (GObject *object)
+gst_osssink_base_init (gpointer g_class)
 {
-  GstOssSink *osssink = (GstOssSink *) object;
-
-  g_free (osssink->device);
+  GstElementClass *element_class = GST_ELEMENT_CLASS (g_class);
 
-  G_OBJECT_CLASS (parent_class)->finalize (object);
+  gst_element_class_set_details (element_class, &gst_osssink_details);
+  gst_element_class_add_pad_template (element_class,
+      gst_static_pad_template_get (&osssink_sink_factory));
 }
-
 static void
-gst_osssink_class_init (GstOssSinkClass *klass) 
+gst_osssink_class_init (GstOssSinkClass * klass)
 {
   GObjectClass *gobject_class;
   GstElementClass *gstelement_class;
 
-  gobject_class = (GObjectClass*)klass;
-  gstelement_class = (GstElementClass*)klass;
-
-  parent_class = g_type_class_ref(GST_TYPE_ELEMENT);
-
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_DEVICE,
-    g_param_spec_string("device","device","device",
-                        "/dev/dsp",G_PARAM_READWRITE)); /* CHECKME! */
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_MUTE,
-    g_param_spec_boolean("mute","mute","mute",
-                         TRUE,G_PARAM_READWRITE)); 
-
-  /* it would be nice to show format in symbolic form, oh well */
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FORMAT,
-    g_param_spec_int ("format","format","format",
-                      0, G_MAXINT, AFMT_S16_LE, G_PARAM_READWRITE)); 
-
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_CHANNELS,
-    g_param_spec_enum("channels","channels","channels",
-                      GST_TYPE_OSSSINK_CHANNELS,2,G_PARAM_READWRITE)); 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FREQUENCY,
-    g_param_spec_int("frequency","frequency","frequency",
-                     0,G_MAXINT,44100,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FRAGMENT,
-    g_param_spec_int("fragment","fragment","fragment",
-                     0,G_MAXINT,6,G_PARAM_READWRITE));
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BUFFER_SIZE,
-    g_param_spec_int("buffer_size","buffer_size","buffer_size",
-                     0,G_MAXINT,4096,G_PARAM_READWRITE));
+  gobject_class = (GObjectClass *) klass;
+  gstelement_class = (GstElementClass *) klass;
+
+  parent_class = g_type_class_ref (GST_TYPE_OSSELEMENT);
+
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_MUTE,
+      g_param_spec_boolean ("mute", "Mute", "Mute the audio",
+         FALSE, G_PARAM_READWRITE));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_SYNC,
+      g_param_spec_boolean ("sync", "Sync",
+         "If syncing on timestamps should be enabled", TRUE,
+         G_PARAM_READWRITE));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_FRAGMENT,
+      g_param_spec_int ("fragment", "Fragment",
+         "The fragment as 0xMMMMSSSS (MMMM = total fragments, 2^SSSS = fragment size)",
+         0, G_MAXINT, 6, G_PARAM_READWRITE));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_BUFFER_SIZE,
+      g_param_spec_uint ("buffer_size", "Buffer size",
+         "Size of buffers in osssink's bufferpool (bytes)", 0, G_MAXINT, 4096,
+         G_PARAM_READWRITE));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_CHUNK_SIZE,
+      g_param_spec_uint ("chunk_size", "Chunk size",
+         "Write data in chunk sized buffers", 0, G_MAXUINT, 4096,
+         G_PARAM_READWRITE));
 
   gst_osssink_signals[SIGNAL_HANDOFF] =
-    g_signal_new("handoff",G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,
-                   G_STRUCT_OFFSET(GstOssSinkClass,handoff), NULL, NULL,
-                   g_cclosure_marshal_VOID__VOID,G_TYPE_NONE,0);
-  
+      g_signal_new ("handoff", G_TYPE_FROM_CLASS (klass), G_SIGNAL_RUN_LAST,
+      G_STRUCT_OFFSET (GstOssSinkClass, handoff), NULL, NULL,
+      g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
+
   gobject_class->set_property = gst_osssink_set_property;
   gobject_class->get_property = gst_osssink_get_property;
-  gobject_class->finalize     = gst_osssink_finalize;
-  
+  gobject_class->dispose = gst_osssink_dispose;
+
   gstelement_class->change_state = GST_DEBUG_FUNCPTR (gst_osssink_change_state);
+  gstelement_class->query = GST_DEBUG_FUNCPTR (gst_osssink_query);
+  gstelement_class->set_clock = gst_osssink_set_clock;
+  gstelement_class->get_clock = gst_osssink_get_clock;
+
 }
 
-static GstClockTime 
-gst_osssink_get_time (GstClock *clock, gpointer data) 
+static void
+gst_osssink_init (GstOssSink * osssink)
 {
-  GstOssSink *osssink = GST_OSSSINK (data);
-  gint delay;
+  osssink->sinkpad =
+      gst_pad_new_from_template (gst_static_pad_template_get
+      (&osssink_sink_factory), "sink");
+  gst_element_add_pad (GST_ELEMENT (osssink), osssink->sinkpad);
+  gst_pad_set_link_function (osssink->sinkpad, gst_osssink_sinkconnect);
+  gst_pad_set_fixate_function (osssink->sinkpad, gst_osssink_sink_fixate);
+  gst_pad_set_convert_function (osssink->sinkpad, gst_osssink_convert);
+  gst_pad_set_query_function (osssink->sinkpad, gst_osssink_sink_query);
+  gst_pad_set_query_type_function (osssink->sinkpad,
+      gst_osssink_get_query_types);
+  gst_pad_set_formats_function (osssink->sinkpad, gst_osssink_get_formats);
 
-  if (!osssink->bps)
-    return 0;
+  gst_pad_set_chain_function (osssink->sinkpad, gst_osssink_chain);
 
-  ioctl (osssink->fd, SNDCTL_DSP_GETODELAY, &delay);
+  GST_DEBUG ("initializing osssink");
+  osssink->bufsize = 4096;
+  osssink->chunk_size = 4096;
+  osssink->mute = FALSE;
+  osssink->sync = TRUE;
+  osssink->resync = TRUE;
+  osssink->provided_clock =
+      gst_audio_clock_new ("ossclock", gst_osssink_get_time, osssink);
+  gst_object_set_parent (GST_OBJECT (osssink->provided_clock),
+      GST_OBJECT (osssink));
+  osssink->handled = 0;
 
-  return osssink->offset + (osssink->handled - delay) * 1000000LL / osssink->bps;
+  GST_FLAG_SET (osssink, GST_ELEMENT_THREAD_SUGGESTED);
+  GST_FLAG_SET (osssink, GST_ELEMENT_EVENT_AWARE);
 }
 
-static void 
-gst_osssink_init (GstOssSink *osssink) 
+static GstCaps *
+gst_osssink_sink_fixate (GstPad * pad, const GstCaps * caps)
 {
-  osssink->sinkpad = gst_pad_new_from_template (
-                 GST_PAD_TEMPLATE_GET (osssink_sink_factory), "sink");
-  gst_element_add_pad (GST_ELEMENT (osssink), osssink->sinkpad);
-  gst_pad_set_connect_function (osssink->sinkpad, gst_osssink_sinkconnect);
-  gst_pad_set_bufferpool_function (osssink->sinkpad, gst_osssink_get_bufferpool);
+  GstCaps *newcaps;
+  GstStructure *structure;
 
-  gst_pad_set_chain_function (osssink->sinkpad, gst_osssink_chain);
+  newcaps =
+      gst_caps_new_full (gst_structure_copy (gst_caps_get_structure (caps, 0)),
+      NULL);
+  structure = gst_caps_get_structure (newcaps, 0);
 
-  osssink->device = g_strdup ("/dev/dsp");
-  osssink->fd = -1;
-  osssink->channels = 1;
-  osssink->frequency = 11025;
-  osssink->fragment = 6;
-/* AFMT_*_BE not available on all OSS includes (e.g. FBSD) */
-#ifdef WORDS_BIGENDIAN
-  osssink->format = AFMT_S16_BE;
-#else
-  osssink->format = AFMT_S16_LE;
-#endif /* WORDS_BIGENDIAN */  
-  /* gst_clock_register (osssink->clock, GST_OBJECT (osssink)); */
-  osssink->bufsize = 4096;
-  osssink->offset = -1LL;
-  osssink->handled = 0LL;
-  /* 6 buffers per chunk by default */
-  osssink->sinkpool = gst_buffer_pool_get_default (osssink->bufsize, 6);
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "rate", 44100)) {
+    return newcaps;
+  }
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "depth", 16)) {
+    return newcaps;
+  }
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "width", 16)) {
+    return newcaps;
+  }
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "channels", 2)) {
+    return newcaps;
+  }
 
-  osssink->provided_clock = NULL;
-  osssink->provided_clock = GST_CLOCK (gst_oss_clock_new ("ossclock", gst_osssink_get_time, osssink));
+  gst_caps_free (newcaps);
 
-  GST_ELEMENT (osssink)->setclockfunc   = gst_osssink_set_clock;
-  GST_ELEMENT (osssink)->getclockfunc   = gst_osssink_get_clock;
-  
-  GST_FLAG_SET (osssink, GST_ELEMENT_THREAD_SUGGESTED);
-  GST_FLAG_SET (osssink, GST_ELEMENT_EVENT_AWARE);
+  return NULL;
 }
 
-static GstPadConnectReturn 
-gst_osssink_sinkconnect (GstPad *pad, GstCaps *caps) 
+static GstPadLinkReturn
+gst_osssink_sinkconnect (GstPad * pad, const GstCaps * caps)
 {
-  gint law, endianness, width, depth;
-  gboolean sign;
-  gint format = -1;
   GstOssSink *osssink = GST_OSSSINK (gst_pad_get_parent (pad));
 
-  if (!GST_CAPS_IS_FIXED (caps))
-    return GST_PAD_CONNECT_DELAYED;
-  
-  gst_caps_get_int (caps, "width", &width);
-  gst_caps_get_int (caps, "depth", &depth);
-
-  if (width != depth) 
-    return GST_PAD_CONNECT_REFUSED;
-
-  /* laws 1 and 2 are 1 bps anyway */
-  osssink->bps = 1;
-
-  gst_caps_get_int (caps, "law", &law);
-  gst_caps_get_int (caps, "endianness", &endianness);
-  gst_caps_get_boolean (caps, "signed", &sign);
-
-  if (law == 0) {
-    if (width == 16) {
-      if (sign == TRUE) {
-        if (endianness == G_LITTLE_ENDIAN)
-         format = AFMT_S16_LE;
-        else if (endianness == G_BIG_ENDIAN)
-         format = AFMT_S16_BE;
-      }
-      else {
-        if (endianness == G_LITTLE_ENDIAN)
-         format = AFMT_U16_LE;
-        else if (endianness == G_BIG_ENDIAN)
-         format = AFMT_U16_BE;
-      }
-      osssink->bps = 2;
-    }
-    else if (width == 8) {
-      if (sign == TRUE) {
-       format = AFMT_S8;
-      }
-      else {
-        format = AFMT_U8;
-      }
-      osssink->bps = 1;
-    }
-  } else if (law == 1) {
-    format = AFMT_MU_LAW;
-  } else if (law == 2) {
-    format = AFMT_A_LAW;
-  } else {
-    g_critical ("unknown law");
-    return GST_PAD_CONNECT_REFUSED;
+  if (!gst_osselement_parse_caps (GST_OSSELEMENT (osssink), caps))
+    return GST_PAD_LINK_REFUSED;
+
+  if (!gst_osselement_sync_parms (GST_OSSELEMENT (osssink))) {
+    return GST_PAD_LINK_REFUSED;
   }
 
-  if (format == -1) 
-    return GST_PAD_CONNECT_REFUSED;
+  return GST_PAD_LINK_OK;
+}
+
+static inline gint
+gst_osssink_get_delay (GstOssSink * osssink)
+{
+  gint delay = 0;
+  gint ret;
 
-  osssink->format = format;
-  gst_caps_get_int (caps, "channels", &osssink->channels);
-  gst_caps_get_int (caps, "rate", &osssink->frequency);
+  if (GST_OSSELEMENT (osssink)->fd == -1)
+    return 0;
 
-  osssink->bps *= osssink->channels;
-  osssink->bps *= osssink->frequency;
+#ifdef SNDCTL_DSP_GETODELAY
+  ret = ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_GETODELAY, &delay);
+#else
+  ret = -1;
+#endif
+  if (ret < 0) {
+    audio_buf_info info;
 
-  if (!gst_osssink_sync_parms (osssink)) {
-    return GST_PAD_CONNECT_REFUSED;
+    if (ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_GETOSPACE, &info) < 0) {
+      delay = 0;
+    } else {
+      delay = (info.fragstotal * info.fragsize) - info.bytes;
+    }
   }
 
-  return GST_PAD_CONNECT_OK;
+  return delay;
 }
 
-static gboolean 
-gst_osssink_sync_parms (GstOssSink *osssink) 
+static GstClockTime
+gst_osssink_get_time (GstClock * clock, gpointer data)
 {
-  audio_buf_info ospace;
-  int frag;
-  gint target_format;
-  gint target_channels;
-  gint target_frequency;
-  GObject *object;
-
-  g_return_val_if_fail (osssink != NULL, FALSE);
-  g_return_val_if_fail (GST_IS_OSSSINK (osssink), FALSE);
-
-  if (osssink->fd == -1)
-    return FALSE;
-  
-  if (osssink->fragment >> 16)
-    frag = osssink->fragment;
-  else
-    frag = 0x7FFF0000 | osssink->fragment;
-  
-  GST_INFO (GST_CAT_PLUGIN_INFO, "osssink: trying to set sound card to %dHz %d bit %s (%08x fragment)",
-           osssink->frequency, osssink->format,
-           (osssink->channels == 2) ? "stereo" : "mono",frag);
-
-  ioctl (osssink->fd, SNDCTL_DSP_SETFRAGMENT, &frag);
-
-  ioctl (osssink->fd, SNDCTL_DSP_RESET, 0);
-
-  target_format = osssink->format;
-  target_channels = osssink->channels;
-  target_frequency = osssink->frequency;
-
-  ioctl (osssink->fd, SNDCTL_DSP_SETFMT, &osssink->format);
-  ioctl (osssink->fd, SNDCTL_DSP_CHANNELS, &osssink->channels);
-  ioctl (osssink->fd, SNDCTL_DSP_SPEED, &osssink->frequency);
-
-  ioctl (osssink->fd, SNDCTL_DSP_GETBLKSIZE, &osssink->fragment);
-  ioctl (osssink->fd, SNDCTL_DSP_GETOSPACE, &ospace);
-
-  GST_INFO (GST_CAT_PLUGIN_INFO, "osssink: set sound card to %dHz %d bit %s (%d bytes buffer, %08x fragment)",
-           osssink->frequency, osssink->format,
-           (osssink->channels == 2) ? "stereo" : "mono", ospace.bytes, osssink->fragment);
-
-  object = G_OBJECT (osssink);
-  g_object_freeze_notify (object);
-  g_object_notify (object, "channels");
-  g_object_notify (object, "frequency");
-  g_object_notify (object, "fragment");
-  g_object_notify (object, "format");
-  g_object_thaw_notify (object);
-
-  osssink->fragment_time = (1000000 * osssink->fragment) / osssink->bps;
-  GST_INFO (GST_CAT_PLUGIN_INFO, "fragment time %u %llu\n", osssink->bps, osssink->fragment_time);
-
-  if (target_format != osssink->format ||
-      target_channels != osssink->channels ||
-      target_frequency != osssink->frequency) 
-  {
-    g_warning ("could not configure oss with required parameters, enjoy the noise :)");
-    /* we could eventually return FALSE here, or just do some additional tests
-     * to see that the frequencies don't differ too much etc.. */
+  GstOssSink *osssink = GST_OSSSINK (data);
+  gint delay;
+  GstClockTimeDiff res;
+
+  if (!GST_OSSELEMENT (osssink)->bps)
+    return 0;
+
+  delay = gst_osssink_get_delay (osssink);
+
+  /* sometimes delay is bigger than the number of bytes sent to the device,
+   * which screws up this calculation, we assume that everything is still
+   * in the device then
+   * thomas: with proper handling of the return value, this doesn't seem to
+   * happen anymore, so remove the second code path after april 2004 */
+  if (delay > (gint64) osssink->handled) {
+    /*g_warning ("Delay %d > osssink->handled %" G_GUINT64_FORMAT
+       ", setting to osssink->handled",
+       delay, osssink->handled); */
+    delay = osssink->handled;
   }
-  return TRUE;
+  res =
+      ((gint64) osssink->handled -
+      delay) * GST_SECOND / GST_OSSELEMENT (osssink)->bps;
+  if (res < 0)
+    res = 0;
+
+  return res;
 }
 
-static GstClock*
-gst_osssink_get_clock (GstElement *element)
+static GstClock *
+gst_osssink_get_clock (GstElement * element)
 {
   GstOssSink *osssink;
-  
+
   osssink = GST_OSSSINK (element);
 
   return GST_CLOCK (osssink->provided_clock);
 }
 
 static void
-gst_osssink_set_clock (GstElement *element, GstClock *clock)
+gst_osssink_set_clock (GstElement * element, GstClock * clock)
 {
   GstOssSink *osssink;
-  
+
   osssink = GST_OSSSINK (element);
 
-  osssink->clock = clock;  
+  osssink->clock = clock;
 }
 
-static void 
-gst_osssink_chain (GstPad *pad, GstBuffer *buf) 
+static void
+gst_osssink_chain (GstPad * pad, GstData * _data)
 {
+  GstBuffer *buf = GST_BUFFER (_data);
   GstOssSink *osssink;
-  GstClockTime buftime;
+  GstClockTimeDiff buftime, soundtime, elementtime;
+  guchar *data;
+  guint to_write;
+  gint delay;
 
   /* this has to be an audio buffer */
   osssink = GST_OSSSINK (gst_pad_get_parent (pad));
 
   if (GST_IS_EVENT (buf)) {
-    switch (GST_EVENT_TYPE (buf)) {
+    GstEvent *event = GST_EVENT (buf);
+
+    switch (GST_EVENT_TYPE (event)) {
       case GST_EVENT_EOS:
-        ioctl (osssink->fd, SNDCTL_DSP_SYNC);
-       gst_pad_event_default (pad, GST_EVENT (buf));
-        return;
+       ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_SYNC, 0);
+       gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock),
+           FALSE);
+       gst_pad_event_default (pad, event);
+       return;
+      case GST_EVENT_DISCONTINUOUS:
+       osssink->resync = TRUE;
+       /* pass-through */
       default:
-        return;
+       gst_pad_event_default (pad, event);
+       return;
     }
+    g_assert_not_reached ();
   }
-  
-  buftime = GST_BUFFER_TIMESTAMP (buf);
 
-  if (!osssink->bps) {
+  if (!GST_OSSELEMENT (osssink)->bps) {
     gst_buffer_unref (buf);
-    gst_element_error (GST_ELEMENT (osssink), "capsnego was never performed, unknown data type");
+    GST_ELEMENT_ERROR (osssink, CORE, NEGOTIATION, (NULL),
+       ("format wasn't negotiated before chain function"));
+    return;
   }
 
-  if (osssink->fd >= 0) {
-    if (!osssink->mute) {
-      guchar *data = GST_BUFFER_DATA (buf);
-      gint size = GST_BUFFER_SIZE (buf);
-      /* gint frag = osssink->fragment;   <-- unused, for reason see above */
-
-      if (osssink->clock) {
-        /* FIXME, NEW_MEDIA/DISCONT?. Try to get our start point */
-        if (osssink->offset == -1LL && buftime != -1LL) {
-         /* g_print ("%lld  %lld %lld\n", osssink->offset, buftime, gst_clock_get_time (osssink->clock)); */
-         osssink->offset = buftime;
-         osssink->handled = 0;
-          gst_element_clock_wait (GST_ELEMENT (osssink), osssink->clock, buftime);
-        }
-       
-       
-       /* FIXME: reverted wtay's patch.
-          The way that's commented out isn't working on BE machines.
-          I guess it should be sample accurate and not capped inbetween.
-       while (size) {
-          gint tosend = MIN (size, frag);
-          write (osssink->fd, data, tosend);
-         data += tosend;
-         size -= tosend;
-          osssink->handled += tosend;
-       } */
-       write (osssink->fd, data, size);
+  data = GST_BUFFER_DATA (buf);
+  to_write = GST_BUFFER_SIZE (buf);
+  /* sync audio with buffers timestamp. elementtime is the *current* time.
+   * soundtime is the time if the soundcard has processed all queued data. */
+  elementtime = gst_element_get_time (GST_ELEMENT (osssink));
+  delay = gst_osssink_get_delay (osssink);
+  if (delay < 0)
+    delay = 0;
+  soundtime = elementtime + delay * GST_SECOND / GST_OSSELEMENT (osssink)->bps;
+  if (GST_BUFFER_TIMESTAMP_IS_VALID (buf)) {
+    buftime = GST_BUFFER_TIMESTAMP (buf);
+  } else {
+    buftime = soundtime;
+  }
+  GST_LOG_OBJECT (osssink,
+      "time: real %" G_GUINT64_FORMAT ", buffer: %" G_GUINT64_FORMAT, soundtime,
+      buftime);
+  if (MAX (buftime, soundtime) - MIN (buftime, soundtime) > (GST_SECOND / 10)) {
+    /* we need to adjust to the buffers here */
+    GST_INFO_OBJECT (osssink,
+       "need sync: real %" G_GUINT64_FORMAT ", buffer: %" G_GUINT64_FORMAT,
+       soundtime, buftime);
+    if (soundtime > buftime) {
+      /* do *not* throw frames out. It's useless. The next frame will come in
+       * too late. And the next one. And so on. We don't want to lose sound.
+       * This is a placeholder for what - some day - should become QoS, i.e.
+       * sending events upstream to drop buffers. */
+    } else {
+      guint64 to_handle =
+         (((buftime -
+                 soundtime) * GST_OSSELEMENT (osssink)->bps / GST_SECOND) /
+         ((GST_OSSELEMENT (osssink)->width / 8) *
+             GST_OSSELEMENT (osssink)->channels)) *
+         (GST_OSSELEMENT (osssink)->width / 8) *
+         GST_OSSELEMENT (osssink)->channels;
+
+      if (!osssink->resync) {
+       guint8 *buf = g_new (guint8, to_handle);
+
+       memset (buf, (GST_OSSELEMENT (osssink)->width == 8) ? 0 : 128,
+           to_handle);
+       while (to_handle > 0) {
+         gint done = write (GST_OSSELEMENT (osssink)->fd, buf,
+             MIN (to_handle, osssink->chunk_size));
+
+         if (done == -1 && errno != EINTR) {
+           break;
+         } else {
+           to_handle -= done;
+           osssink->handled += done;
+         }
+       }
+       g_free (buf);
+      } else {
+       /* Timestamps at start-of-stream (MPEG) or after seek (hey,
+        * again MPEG!) can be borken, therefore this hacklet. */
+       osssink->handled += to_handle;
       }
-      /* no clock, try to be as fast as possible */
-      else {
-        audio_buf_info ospace;
+    }
+  }
 
-        ioctl (osssink->fd, SNDCTL_DSP_GETOSPACE, &ospace);
+  if (GST_OSSELEMENT (osssink)->fd >= 0 && to_write > 0) {
+    if (!osssink->mute) {
 
-        if (ospace.bytes >= size) {
-          write (osssink->fd, data, size);
+      while (to_write > 0) {
+       gint done = write (GST_OSSELEMENT (osssink)->fd, data,
+           MIN (to_write, osssink->chunk_size));
+
+       if (done == -1) {
+         if (errno != EINTR)
+           break;
+       } else {
+         to_write -= done;
+         data += done;
+         osssink->handled += done;
        }
-        osssink->handled += size;
       }
+    } else {
+      g_warning ("muting osssinks unimplemented wrt clocks!");
     }
   }
+
+  gst_audio_clock_update_time ((GstAudioClock *) osssink->provided_clock,
+      gst_osssink_get_time (osssink->provided_clock, osssink));
+
   gst_buffer_unref (buf);
 }
 
-static void 
-gst_osssink_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec) 
+static const GstFormat *
+gst_osssink_get_formats (GstPad * pad)
+{
+  static const GstFormat formats[] = {
+    GST_FORMAT_TIME,
+    GST_FORMAT_DEFAULT,
+    GST_FORMAT_BYTES,
+    0
+  };
+  return formats;
+}
+
+static gboolean
+gst_osssink_convert (GstPad * pad, GstFormat src_format, gint64 src_value,
+    GstFormat * dest_format, gint64 * dest_value)
 {
   GstOssSink *osssink;
 
-  /* it's not null if we got it, but it might not be ours */
-  g_return_if_fail (GST_IS_OSSSINK (object));
-  
-  osssink = GST_OSSSINK (object);
+  osssink = GST_OSSSINK (gst_pad_get_parent (pad));
 
-  switch (prop_id) {
-    case ARG_DEVICE:
-      /* disallow changing the device while it is opened
-         get_property("device") should return the right one */
-      if (!GST_FLAG_IS_SET (osssink, GST_OSSSINK_OPEN))
-      {
-       g_free (osssink->device);
-        osssink->device = g_strdup (g_value_get_string (value));
-       g_object_notify (object, "device");
+  return gst_osselement_convert (GST_OSSELEMENT (osssink),
+      src_format, src_value, dest_format, dest_value);
+}
+
+static const GstQueryType *
+gst_osssink_get_query_types (GstPad * pad)
+{
+  static const GstQueryType query_types[] = {
+    GST_QUERY_LATENCY,
+    GST_QUERY_POSITION,
+    0,
+  };
+  return query_types;
+}
+
+static gboolean
+gst_osssink_sink_query (GstPad * pad, GstQueryType type, GstFormat * format,
+    gint64 * value)
+{
+  gboolean res = TRUE;
+  GstOssSink *osssink;
+
+  osssink = GST_OSSSINK (gst_pad_get_parent (pad));
+
+  switch (type) {
+    case GST_QUERY_LATENCY:
+      if (!gst_osssink_convert (pad,
+             GST_FORMAT_BYTES, gst_osssink_get_delay (osssink),
+             format, value)) {
+       res = FALSE;
       }
       break;
+    case GST_QUERY_POSITION:
+      if (!gst_osssink_convert (pad,
+             GST_FORMAT_TIME, gst_element_get_time (GST_ELEMENT (osssink)),
+             format, value)) {
+       res = FALSE;
+      }
+      break;
+    default:
+      res =
+         gst_pad_query (gst_pad_get_peer (osssink->sinkpad), type, format,
+         value);
+      break;
+  }
+
+  return res;
+}
+
+static gboolean
+gst_osssink_query (GstElement * element, GstQueryType type, GstFormat * format,
+    gint64 * value)
+{
+  GstOssSink *osssink = GST_OSSSINK (element);
+
+  return gst_osssink_sink_query (osssink->sinkpad, type, format, value);
+}
+
+static void
+gst_osssink_set_property (GObject * object, guint prop_id, const GValue * value,
+    GParamSpec * pspec)
+{
+  GstOssSink *osssink;
+
+  osssink = GST_OSSSINK (object);
+
+  switch (prop_id) {
     case ARG_MUTE:
       osssink->mute = g_value_get_boolean (value);
       g_object_notify (G_OBJECT (osssink), "mute");
       break;
-    case ARG_FORMAT:
-      osssink->format = g_value_get_int (value);
-      gst_osssink_sync_parms (osssink);
-      break;
-    case ARG_CHANNELS:
-      osssink->channels = g_value_get_enum (value);
-      gst_osssink_sync_parms (osssink);
-      break;
-    case ARG_FREQUENCY:
-      osssink->frequency = g_value_get_int (value);
-      gst_osssink_sync_parms (osssink);
-      break;
     case ARG_FRAGMENT:
-      osssink->fragment = g_value_get_int (value);
-      gst_osssink_sync_parms (osssink);
+      GST_OSSELEMENT (osssink)->fragment = g_value_get_int (value);
+      gst_osselement_sync_parms (GST_OSSELEMENT (osssink));
       break;
     case ARG_BUFFER_SIZE:
-      if (osssink->bufsize == g_value_get_int (value)) break;
-      osssink->bufsize = g_value_get_int (value);
-      osssink->sinkpool = gst_buffer_pool_get_default (osssink->bufsize, 6);
+      osssink->bufsize = g_value_get_uint (value);
       g_object_notify (object, "buffer_size");
       break;
+    case ARG_SYNC:
+      osssink->sync = g_value_get_boolean (value);
+      g_object_notify (G_OBJECT (osssink), "sync");
+      break;
+    case ARG_CHUNK_SIZE:
+      osssink->chunk_size = g_value_get_uint (value);
+      break;
     default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
   }
 }
 
-static void 
-gst_osssink_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec) 
+static void
+gst_osssink_get_property (GObject * object, guint prop_id, GValue * value,
+    GParamSpec * pspec)
 {
   GstOssSink *osssink;
 
-  /* it's not null if we got it, but it might not be ours */
-  g_return_if_fail (GST_IS_OSSSINK (object));
-  
   osssink = GST_OSSSINK (object);
 
   switch (prop_id) {
-    case ARG_DEVICE:
-      g_value_set_string (value, osssink->device);
-      break;
     case ARG_MUTE:
       g_value_set_boolean (value, osssink->mute);
       break;
-    case ARG_FORMAT:
-      g_value_set_int (value, osssink->format);
-      break;
-    case ARG_CHANNELS:
-      g_value_set_enum (value, osssink->channels);
-      break;
-    case ARG_FREQUENCY:
-      g_value_set_int (value, osssink->frequency);
-      break;
     case ARG_FRAGMENT:
-      g_value_set_int (value, osssink->fragment);
+      g_value_set_int (value, GST_OSSELEMENT (osssink)->fragment);
       break;
     case ARG_BUFFER_SIZE:
-      g_value_set_int (value, osssink->bufsize);
+      g_value_set_uint (value, osssink->bufsize);
+      break;
+    case ARG_SYNC:
+      g_value_set_boolean (value, osssink->sync);
+      break;
+    case ARG_CHUNK_SIZE:
+      g_value_set_uint (value, osssink->chunk_size);
       break;
     default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
   }
 }
 
-static gboolean
-gst_osssink_open_audio (GstOssSink *sink)
-{
-  gint caps;
-  g_return_val_if_fail (sink->fd == -1, FALSE);
-
-  GST_INFO (GST_CAT_PLUGIN_INFO, "osssink: attempting to open sound device");
-
-  /* first try to open the sound card */
-  sink->fd = open(sink->device, O_WRONLY | O_NONBLOCK);
-  if (errno == EBUSY) {
-    g_warning ("osssink: unable to open the sound device (in use ?)\n");
-    return FALSE;
-  }
-
-  /* re-open the sound device in blocking mode */
-  close(sink->fd);
-  sink->fd = open(sink->device, O_WRONLY);
-
-  /* if we have it, set the default parameters and go have fun */
-  if (sink->fd >= 0) {
-    /* set card state */
-    ioctl(sink->fd, SNDCTL_DSP_GETCAPS, &caps);
-
-    GST_INFO(GST_CAT_PLUGIN_INFO, "osssink: Capabilities %08x", caps);
-
-    if (caps & DSP_CAP_DUPLEX)         GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Full duplex");
-    if (caps & DSP_CAP_REALTIME)       GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Realtime");
-    if (caps & DSP_CAP_BATCH)          GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Batch");
-    if (caps & DSP_CAP_COPROC)         GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Has coprocessor");
-    if (caps & DSP_CAP_TRIGGER)        GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Trigger");
-    if (caps & DSP_CAP_MMAP)           GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Direct access");
-
-#ifdef DSP_CAP_MULTI
-    if (caps & DSP_CAP_MULTI)          GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Multiple open");
-#endif /* DSP_CAP_MULTI */
-
-#ifdef DSP_CAP_BIND
-    if (caps & DSP_CAP_BIND)           GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   Channel binding");
-#endif /* DSP_CAP_BIND */
-
-    ioctl(sink->fd, SNDCTL_DSP_GETFMTS, &caps);
-
-    GST_INFO (GST_CAT_PLUGIN_INFO, "osssink: Formats %08x", caps);
-    if (caps & AFMT_MU_LAW)            GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   MU_LAW");
-    if (caps & AFMT_A_LAW)             GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   A_LAW");
-    if (caps & AFMT_IMA_ADPCM)         GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   IMA_ADPCM");
-    if (caps & AFMT_U8)                GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   U8");
-    if (caps & AFMT_S16_LE)                    GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   S16_LE");
-    if (caps & AFMT_S16_BE)                    GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   S16_BE");
-    if (caps & AFMT_S8)                GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   S8");
-    if (caps & AFMT_U16_LE)                    GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   U16_LE");
-    if (caps & AFMT_U16_BE)                    GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   U16_BE");
-    if (caps & AFMT_MPEG)              GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   MPEG");
-#ifdef AFMT_AC3
-    if (caps & AFMT_AC3)               GST_INFO (GST_CAT_PLUGIN_INFO, "osssink:   AC3");
-#endif
-
-    GST_INFO (GST_CAT_PLUGIN_INFO, "osssink: opened audio (%s) with fd=%d", sink->device, sink->fd);
-    GST_FLAG_SET (sink, GST_OSSSINK_OPEN);
-
-    return TRUE;
-  }
-
-  return FALSE;
-}
-
-static void
-gst_osssink_close_audio (GstOssSink *sink)
-{
-  if (sink->fd < 0) return;
-
-  close(sink->fd);
-  sink->fd = -1;
-
-  GST_FLAG_UNSET (sink, GST_OSSSINK_OPEN);
-
-  GST_INFO (GST_CAT_PLUGIN_INFO, "osssink: closed sound device");
-}
-
-static GstElementStateReturn 
-gst_osssink_change_state (GstElement *element) 
+static GstElementStateReturn
+gst_osssink_change_state (GstElement * element)
 {
   GstOssSink *osssink;
 
-  g_return_val_if_fail (GST_IS_OSSSINK (element), FALSE);
-
   osssink = GST_OSSSINK (element);
 
   switch (GST_STATE_TRANSITION (element)) {
-    case GST_STATE_NULL_TO_READY:
-      if (!GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN)) {
-        if (!gst_osssink_open_audio (osssink)) {
-          return GST_STATE_FAILURE;
-        }
-      }
-      break;
     case GST_STATE_READY_TO_PAUSED:
-      osssink->offset = -1LL;
       break;
     case GST_STATE_PAUSED_TO_PLAYING:
-      /* gst_clock_adjust (osssink->clock, osssink->offset - gst_clock_get_time (osssink->clock)); */
+      gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock),
+         TRUE);
       break;
     case GST_STATE_PLAYING_TO_PAUSED:
-    {
-      if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN)) {
-       if (osssink->bps) {
-          GstClockTime time;
-          audio_buf_info ospace;
-          gint queued;
-
-          ioctl (osssink->fd, SNDCTL_DSP_GETOSPACE, &ospace);
-          ioctl (osssink->fd, SNDCTL_DSP_RESET, 0);
-
-          queued = (ospace.fragstotal * ospace.fragsize) - ospace.bytes;
-          time = osssink->offset + (osssink->handled - queued) * 1000000LL / osssink->bps;
-
-          //gst_clock_adjust (osssink->clock, time - gst_clock_get_time (osssink->clock));
-       }
-       else {
-          ioctl (osssink->fd, SNDCTL_DSP_RESET, 0);
-       }
-      }
-
+      if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN))
+       ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_RESET, 0);
+      gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock),
+         FALSE);
       break;
-    }
     case GST_STATE_PAUSED_TO_READY:
       if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN))
-        ioctl (osssink->fd, SNDCTL_DSP_RESET, 0);
+       ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_RESET, 0);
+      gst_osselement_reset (GST_OSSELEMENT (osssink));
+      osssink->handled = 0;
+      osssink->resync = TRUE;
       break;
-    case GST_STATE_READY_TO_NULL:
-      if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN))
-        gst_osssink_close_audio (osssink);
+    default:
       break;
   }
-      
+
   if (GST_ELEMENT_CLASS (parent_class)->change_state)
     return GST_ELEMENT_CLASS (parent_class)->change_state (element);
 
   return GST_STATE_SUCCESS;
 }
-
-gboolean 
-gst_osssink_factory_init (GstPlugin *plugin) 
-{ 
-  GstElementFactory *factory;
-
-  factory = gst_element_factory_new ("osssink", GST_TYPE_OSSSINK, &gst_osssink_details);
-  g_return_val_if_fail (factory != NULL, FALSE);
-
-  gst_element_factory_add_pad_template (factory, GST_PAD_TEMPLATE_GET (osssink_sink_factory));
-
-  gst_plugin_add_feature (plugin, GST_PLUGIN_FEATURE (factory));
-
-  return TRUE;
-}
-