commit e46f5534fa219a760822c8b08bc81a3267b5ff2b
author Andy Hung <hunga@google.com> Tue Mar 18 13:13:45 2014 -0700
committer Andy Hung <hunga@google.com> Tue Mar 18 14:30:17 2014 -0700
tree a17d2a3ca1865b2dcd2c09c60ffc93f9b082b647
parent 65b5ccd2a7a7c756db2cfa627ac75d17eb0484c2

Add PCM_32_BIT and PCM_8_24_BIT to format conversion

Change-Id: I533f0be84410117d1c31229bb80b8cabda430f99
Signed-off-by: Andy Hung <hunga@google.com>

audio_utils/format.c

diff --git a/audio_utils/format.c b/audio_utils/format.c
index fd02fb9..caa354a 100644
--- a/audio_utils/format.c
+++ b/audio_utils/format.c
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-//#define LOG_NDEBUG 0
+/* #define LOG_NDEBUG 0 */
 #define LOG_TAG "audio_utils_format"
 
 #include <cutils/log.h>
@@ -22,13 +22,21 @@
 #include <audio_utils/format.h>
 
 void memcpy_by_audio_format(void *dst, audio_format_t dst_format,
-        void *src, audio_format_t src_format, size_t count) {
-    if (dst_format == src_format
-            && (dst_format == AUDIO_FORMAT_PCM_16_BIT
-                    || dst_format == AUDIO_FORMAT_PCM_FLOAT
-                    || dst_format == AUDIO_FORMAT_PCM_24_BIT_PACKED)) {
-        memcpy(dst, src, count * audio_bytes_per_sample(dst_format));
-        return;
+        void *src, audio_format_t src_format, size_t count)
+{
+    /* default cases for error falls through to fatal log below. */
+    if (dst_format == src_format) {
+        switch (dst_format) {
+        case AUDIO_FORMAT_PCM_16_BIT:
+        case AUDIO_FORMAT_PCM_FLOAT:
+        case AUDIO_FORMAT_PCM_24_BIT_PACKED:
+        case AUDIO_FORMAT_PCM_32_BIT:
+        case AUDIO_FORMAT_PCM_8_24_BIT:
+            memcpy(dst, src, count * audio_bytes_per_sample(dst_format));
+            return;
+        default:
+            break;
+        }
     }
     switch (dst_format) {
     case AUDIO_FORMAT_PCM_16_BIT:
@@ -39,6 +47,12 @@
         case AUDIO_FORMAT_PCM_24_BIT_PACKED:
             memcpy_to_i16_from_p24((int16_t*)dst, (uint8_t*)src, count);
             return;
+        case AUDIO_FORMAT_PCM_32_BIT:
+            memcpy_to_i16_from_i32((int16_t*)dst, (int32_t*)src, count);
+            return;
+        case AUDIO_FORMAT_PCM_8_24_BIT:
+            memcpy_to_i16_from_q8_23((int16_t*)dst, (int32_t*)src, count);
+            return;
         default:
             break;
         }
@@ -51,6 +65,12 @@
         case AUDIO_FORMAT_PCM_24_BIT_PACKED:
             memcpy_to_float_from_p24((float*)dst, (uint8_t*)src, count);
             return;
+        case AUDIO_FORMAT_PCM_32_BIT:
+            memcpy_to_float_from_i32((float*)dst, (int32_t*)src, count);
+            return;
+        case AUDIO_FORMAT_PCM_8_24_BIT:
+            memcpy_to_float_from_q8_23((float*)dst, (int32_t*)src, count);
+            return;
         default:
             break;
         }
@@ -67,6 +87,30 @@
             break;
         }
         break;
+    case AUDIO_FORMAT_PCM_32_BIT:
+        switch (src_format) {
+        case AUDIO_FORMAT_PCM_16_BIT:
+            memcpy_to_i32_from_i16((int32_t*)dst, (int16_t*)src, count);
+            return;
+        case AUDIO_FORMAT_PCM_FLOAT:
+            memcpy_to_i32_from_float((int32_t*)dst, (float*)src, count);
+            return;
+        default:
+            break;
+        }
+        break;
+    case AUDIO_FORMAT_PCM_8_24_BIT:
+        switch (src_format) {
+        case AUDIO_FORMAT_PCM_16_BIT:
+            memcpy_to_q8_23_from_i16((int32_t*)dst, (int16_t*)src, count);
+            return;
+        case AUDIO_FORMAT_PCM_FLOAT:
+            memcpy_to_q8_23_from_float_with_clamp((int32_t*)dst, (float*)src, count);
+            return;
+        default:
+            break;
+        }
+        break;
     default:
         break;
     }