Changes in uspace/drv/audio/hdaudio/pcm_iface.c [4a5ae542:57a2208] in mainline

File:

• uspace/drv/audio/hdaudio/pcm_iface.c (modified) (5 diffs)

uspace/drv/audio/hdaudio/pcm_iface.c

@@ -100 +100 @@
 static unsigned hda_query_cap(ddf_fun_t *fun, audio_cap_t cap)
 {
-        hda_t *hda = fun_to_hda(fun);
-
         ddf_msg(LVL_NOTE, "hda_query_cap(%d)", cap);
         switch (cap) {
         case AUDIO_CAP_PLAYBACK:
         case AUDIO_CAP_INTERRUPT:
-                /* XXX Only if we have an output converter */
                 return 1;
+        case AUDIO_CAP_BUFFER_POS:
         case AUDIO_CAP_CAPTURE:
-                /* Yes if we have an input converter */
-                return hda->ctl->codec->in_aw >= 0;
-        case AUDIO_CAP_BUFFER_POS:
                 return 0;
         case AUDIO_CAP_MAX_BUFFER:
@@ -153 +148 @@
 {
         hda_t *hda = fun_to_hda(fun);
-        int rc;
 
         hda_lock(hda);
 
         ddf_msg(LVL_NOTE, "hda_get_buffer(): hda=%p", hda);
-        if (hda->pcm_buffers != NULL) {
-                hda_unlock(hda);
-                return EBUSY;
-        }
-
-        ddf_msg(LVL_NOTE, "hda_get_buffer() - allocate stream buffers");
-        rc = hda_stream_buffers_alloc(hda, &hda->pcm_buffers);
-        if (rc != EOK) {
-                assert(rc == ENOMEM);
-                hda_unlock(hda);
-                return ENOMEM;
-        }
-
-        ddf_msg(LVL_NOTE, "hda_get_buffer() - fill info");
-        /* XXX This is only one buffer */
-        *buffer = hda->pcm_buffers->buf[0];
-        *size = hda->pcm_buffers->bufsize * hda->pcm_buffers->nbuffers;
-
-        ddf_msg(LVL_NOTE, "hda_get_buffer() returing EOK, buffer=%p, size=%zu",
-            *buffer, *size);
-
-        hda_unlock(hda);
-        return EOK;
-}
-
-static int hda_get_buffer_position(ddf_fun_t *fun, size_t *pos)
-{
-        ddf_msg(LVL_NOTE, "hda_get_buffer_position()");
-        return ENOTSUP;
-}
-
-static int hda_set_event_session(ddf_fun_t *fun, async_sess_t *sess)
-{
-        hda_t *hda = fun_to_hda(fun);
-
-        ddf_msg(LVL_NOTE, "hda_set_event_session()");
-        hda_lock(hda);
-        hda->ev_sess = sess;
-        hda_unlock(hda);
-
-        return EOK;
-}
-
-static async_sess_t *hda_get_event_session(ddf_fun_t *fun)
-{
-        hda_t *hda = fun_to_hda(fun);
-        async_sess_t *sess;
-
-        ddf_msg(LVL_NOTE, "hda_get_event_session()");
-
-        hda_lock(hda);
-        sess = hda->ev_sess;
-        hda_unlock(hda);
-
-        return sess;
-}
-
-static int hda_release_buffer(ddf_fun_t *fun)
-{
-        hda_t *hda = fun_to_hda(fun);
-
-        hda_lock(hda);
-
-        ddf_msg(LVL_NOTE, "hda_release_buffer()");
-        if (hda->pcm_buffers == NULL) {
-                hda_unlock(hda);
-                return EINVAL;
-        }
-
-        hda_stream_buffers_free(hda->pcm_buffers);
-        hda->pcm_buffers = NULL;
-
-        hda_unlock(hda);
-        return EOK;
-}
-
-static int hda_start_playback(ddf_fun_t *fun, unsigned frames,
-    unsigned channels, unsigned rate, pcm_sample_format_t format)
-{
-        hda_t *hda = fun_to_hda(fun);
-        int rc;
-
-        ddf_msg(LVL_NOTE, "hda_start_playback()");
-        hda_lock(hda);
-
         if (hda->pcm_stream != NULL) {
                 hda_unlock(hda);
@@ -253 +162 @@
         fmt = (fmt_base_44khz << fmt_base) | (fmt_bits_16 << fmt_bits_l) | 1;
 
-        ddf_msg(LVL_NOTE, "hda_start_playback() - create output stream");
-        hda->pcm_stream = hda_stream_create(hda, sdir_output, hda->pcm_buffers,
-            fmt);
+        ddf_msg(LVL_NOTE, "hda_get_buffer() - create stream");
+        hda->pcm_stream = hda_stream_create(hda, sdir_output, fmt);
         if (hda->pcm_stream == NULL) {
                 hda_unlock(hda);
                 return EIO;
         }
+
+        ddf_msg(LVL_NOTE, "hda_get_buffer() - fill info");
+        /* XXX This is only one buffer */
+        *buffer = hda->pcm_stream->buf[0];
+        *size = hda->pcm_stream->bufsize * hda->pcm_stream->nbuffers;
+
+        ddf_msg(LVL_NOTE, "hda_get_buffer() retturing EOK, buffer=%p, size=%zu",
+            *buffer, *size);
+
+        hda_unlock(hda);
+        return EOK;
+}
+
+static int hda_get_buffer_position(ddf_fun_t *fun, size_t *pos)
+{
+        ddf_msg(LVL_NOTE, "hda_get_buffer_position()");
+        return ENOTSUP;
+}
+
+static int hda_set_event_session(ddf_fun_t *fun, async_sess_t *sess)
+{
+        hda_t *hda = fun_to_hda(fun);
+
+        ddf_msg(LVL_NOTE, "hda_set_event_session()");
+        hda_lock(hda);
+        hda->ev_sess = sess;
+        hda_unlock(hda);
+
+        return EOK;
+}
+
+static async_sess_t *hda_get_event_session(ddf_fun_t *fun)
+{
+        hda_t *hda = fun_to_hda(fun);
+        async_sess_t *sess;
+
+        ddf_msg(LVL_NOTE, "hda_get_event_session()");
+
+        hda_lock(hda);
+        sess = hda->ev_sess;
+        hda_unlock(hda);
+
+        return sess;
+}
+
+static int hda_release_buffer(ddf_fun_t *fun)
+{
+        hda_t *hda = fun_to_hda(fun);
+
+        hda_lock(hda);
+
+        ddf_msg(LVL_NOTE, "hda_release_buffer()");
+        if (hda->pcm_stream == NULL) {
+                hda_unlock(hda);
+                return EINVAL;
+        }
+
+        hda_stream_destroy(hda->pcm_stream);
+        hda->pcm_stream = NULL;
+
+        hda_unlock(hda);
+        return EOK;
+}
+
+static int hda_start_playback(ddf_fun_t *fun, unsigned frames,
+    unsigned channels, unsigned rate, pcm_sample_format_t format)
+{
+        hda_t *hda = fun_to_hda(fun);
+        int rc;
+
+        ddf_msg(LVL_NOTE, "hda_start_playback()");
+        hda_lock(hda);
 
         rc = hda_out_converter_setup(hda->ctl->codec, hda->pcm_stream);
         if (rc != EOK) {
-                hda_stream_destroy(hda->pcm_stream);
-                hda->pcm_stream = NULL;
                 hda_unlock(hda);
                 return rc;
         }
 
+        hda_stream_start(hda->pcm_stream);
         hda->playing = true;
-        hda_stream_start(hda->pcm_stream);
         hda_unlock(hda);
         return EOK;
@@ -284 +262 @@
         hda_stream_reset(hda->pcm_stream);
         hda->playing = false;
-        hda_stream_destroy(hda->pcm_stream);
-        hda->pcm_stream = NULL;
-
         hda_unlock(hda);
 
@@ -296 +271 @@
     unsigned rate, pcm_sample_format_t format)
 {
-        hda_t *hda = fun_to_hda(fun);
-        int rc;
-
         ddf_msg(LVL_NOTE, "hda_start_capture()");
-        hda_lock(hda);
-
-        if (hda->pcm_stream != NULL) {
-                hda_unlock(hda);
-                return EBUSY;
-        }
-
-        /* XXX Choose appropriate parameters */
-        uint32_t fmt;
-        /* 48 kHz, 16-bits, 1 channel */
-        fmt = (fmt_base_44khz << fmt_base) | (fmt_bits_16 << fmt_bits_l) | 1;
-
-        ddf_msg(LVL_NOTE, "hda_start_capture() - create input stream");
-        hda->pcm_stream = hda_stream_create(hda, sdir_input, hda->pcm_buffers,
-            fmt);
-        if (hda->pcm_stream == NULL) {
-                hda_unlock(hda);
-                return EIO;
-        }
-
-        rc = hda_in_converter_setup(hda->ctl->codec, hda->pcm_stream);
-        if (rc != EOK) {
-                hda_stream_destroy(hda->pcm_stream);
-                hda->pcm_stream = NULL;
-                hda_unlock(hda);
-                return rc;
-        }
-
-        hda->capturing = true;
-        hda_stream_start(hda->pcm_stream);
-        hda_unlock(hda);
-        return EOK;
+        return ENOTSUP;
 }
 
 static int hda_stop_capture(ddf_fun_t *fun, bool immediate)
 {
-        hda_t *hda = fun_to_hda(fun);
-
         ddf_msg(LVL_NOTE, "hda_stop_capture()");
-        hda_lock(hda);
-        hda_stream_stop(hda->pcm_stream);
-        hda_stream_reset(hda->pcm_stream);
-        hda->capturing = false;
-        hda_stream_destroy(hda->pcm_stream);
-        hda->pcm_stream = NULL;
-        hda_unlock(hda);
-
-        hda_pcm_event(hda, PCM_EVENT_CAPTURE_TERMINATED);
-        return EOK;
+        return ENOTSUP;
 }