Changeset 00006e0 in mainline for uspace/lib/drv/generic/remote_audio_pcm_buffer.c

Timestamp:

2012-07-02T12:31:08Z (11 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

lfn, master, serial

Children:

2f7042e

Parents:

bd250b0f

Message:

audio: Allow client to specify buffer size.

File:

• uspace/lib/drv/generic/remote_audio_pcm_buffer.c (modified) (4 diffs)

uspace/lib/drv/generic/remote_audio_pcm_buffer.c

@@ -34 +34 @@
 #include <async.h>
 #include <errno.h>
-#include <assert.h>
 #include <str.h>
 #include <as.h>
@@ -85 +84 @@
     unsigned *id)
 {
-        if (!exch || !buffer || !size)
-                return EINVAL;
-        sysarg_t buffer_size, buffer_id;
-        const int ret = async_req_1_2(exch,
+        if (!exch || !buffer || !size || !id)
+                return EINVAL;
+
+        sysarg_t buffer_size = *size, buffer_id = 0;
+        const int ret = async_req_2_2(exch,
             DEV_IFACE_ID(AUDIO_PCM_BUFFER_IFACE), IPC_M_AUDIO_PCM_GET_BUFFER,
-            &buffer_size, &buffer_id);
+            (sysarg_t)buffer_size, &buffer_size, &buffer_id);
         if (ret == EOK) {
                 void *dst = NULL;
-                const int ret =
-                    async_share_in_start_0_0(exch, buffer_size, &dst);
+                // FIXME Do we need to know the flags?
+                const int ret = async_share_in_start_0_0(exch, buffer_size, &dst);
                 if (ret != EOK) {
                         return ret;
@@ -100 +100 @@
                 *buffer = dst;
                 *size = buffer_size;
-        }
-        if (ret == EOK && id)
                 *id = buffer_id;
+        }
         return ret;
 }
@@ -219 +218 @@
         }
         void *buffer = NULL;
-        size_t size = 0;
+        size_t size = DEV_IPC_GET_ARG1(*call);
         unsigned id = 0;
         const int ret = pcm_iface->get_buffer(fun, &buffer, &size, &id);