Changeset 346643c in mainline for uspace/lib/drv/generic/remote_audio_pcm.c

Timestamp:

2012-07-11T12:05:30Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

039337e8

Parents:

94694a4

Message:

audio: Use enum for sample format.

File:

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

uspace/lib/drv/generic/remote_audio_pcm.c

@@ -118 +118 @@
 }
 
-int audio_pcm_start_playback(async_exch_t *exch, unsigned id,
-    unsigned parts, unsigned sample_rate, uint16_t sample_size,
-    uint8_t channels, bool sign)
+int audio_pcm_start_playback(async_exch_t *exch, unsigned id, unsigned parts,
+    unsigned channels, unsigned sample_rate, pcm_sample_format_t format)
 {
         if (!exch)
                 return EINVAL;
+        if (parts > UINT8_MAX || channels > UINT8_MAX)
+                return EINVAL;
+        assert((format & UINT16_MAX) == format);
         const sysarg_t packed =
-            (sample_size << 16) | (channels << 8) |
-            ((parts & 0x7f) << 1) | (sign ? 1 : 0);
+            (parts << 24) | (channels << 16) | (format & UINT16_MAX);
         return async_req_4_0(exch, DEV_IFACE_ID(AUDIO_PCM_BUFFER_IFACE),
             IPC_M_AUDIO_PCM_START_PLAYBACK, id, sample_rate, packed);
@@ -139 +140 @@
 }
 
-int audio_pcm_start_record(async_exch_t *exch, unsigned id,
-    unsigned parts, unsigned sample_rate, uint16_t sample_size,
-    uint8_t channels, bool sign)
+int audio_pcm_start_record(async_exch_t *exch, unsigned id, unsigned parts,
+    unsigned channels, unsigned sample_rate, pcm_sample_format_t format)
 {
         if (!exch)
                 return EINVAL;
-        sysarg_t packed =
-            (sample_size << 16) | (channels << 8) |
-            ((parts & 0x7f) << 1) | (sign ? 1 : 0);
+        if (parts > UINT8_MAX || channels > UINT8_MAX)
+                return EINVAL;
+        assert((format & UINT16_MAX) == format);
+        const sysarg_t packed =
+            (parts << 24) | (channels << 16) | (format & UINT16_MAX);
         return async_req_4_0(exch, DEV_IFACE_ID(AUDIO_PCM_BUFFER_IFACE),
             IPC_M_AUDIO_PCM_START_RECORD, id, sample_rate, packed);
@@ -309 +311 @@
         const unsigned id = DEV_IPC_GET_ARG1(*call);
         const unsigned rate = DEV_IPC_GET_ARG2(*call);
-        const unsigned size = DEV_IPC_GET_ARG3(*call) >> 16;
-        const unsigned channels = (DEV_IPC_GET_ARG3(*call) >> 8) & UINT8_MAX;
-        const unsigned parts = (DEV_IPC_GET_ARG3(*call) >> 1) & 0x7f;
-        const bool sign = (bool)(DEV_IPC_GET_ARG3(*call) & 1);
+        const unsigned parts = (DEV_IPC_GET_ARG3(*call) >> 24) & UINT8_MAX;
+        const unsigned channels = (DEV_IPC_GET_ARG3(*call) >> 16) & UINT8_MAX;
+        const pcm_sample_format_t format =DEV_IPC_GET_ARG3(*call) & UINT16_MAX;
 
         const int ret = pcm_iface->start_playback
-            ? pcm_iface->start_playback(fun, id, parts, rate, size, channels, sign)
+            ? pcm_iface->start_playback(fun, id, parts, channels, rate, format)
             : ENOTSUP;
         async_answer_0(callid, ret);
@@ -338 +339 @@
         const unsigned id = DEV_IPC_GET_ARG1(*call);
         const unsigned rate = DEV_IPC_GET_ARG2(*call);
-        const unsigned size = DEV_IPC_GET_ARG3(*call) >> 16;
-        const unsigned channels = (DEV_IPC_GET_ARG3(*call) >> 8) & UINT8_MAX;
-        const unsigned parts = (DEV_IPC_GET_ARG3(*call) >> 1) & 0x7f;
-        const bool sign = (bool)(DEV_IPC_GET_ARG3(*call) & 1);
+        const unsigned parts = (DEV_IPC_GET_ARG3(*call) >> 24) & UINT8_MAX;
+        const unsigned channels = (DEV_IPC_GET_ARG3(*call) >> 16) & UINT8_MAX;
+        const pcm_sample_format_t format =DEV_IPC_GET_ARG3(*call) & UINT16_MAX;
 
         const int ret = pcm_iface->start_record
-            ? pcm_iface->start_record(fun, id, parts, rate, size, channels, sign)
+            ? pcm_iface->start_record(fun, id, parts, channels, rate, format)
             : ENOTSUP;
         async_answer_0(callid, ret);