Enhance to check parameters and move volume_level_max array to audio_hal_volume_t

[Version] 0.1.6
[Profile] Common
[Issue Type] Enhancement

Change-Id: I33fd5b54e7ed9fd1ef4bebe66a0fa319ccca414c
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

diff --git a/packaging/audio-hal-emul.spec b/packaging/audio-hal-emul.spec
index cd4a853..2c8e12f 100644 (file)
--- a/packaging/audio-hal-emul.spec
+++ b/packaging/audio-hal-emul.spec
@@ -1,6 +1,6 @@
 Name:       audio-hal-emul
 Summary:    TIZEN Audio HAL for Emulator
-Version:    0.1.5
+Version:    0.1.6
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0

diff --git a/tizen-audio-internal.h b/tizen-audio-internal.h
index 9e8a6d1..423253a 100644 (file)
--- a/tizen-audio-internal.h
+++ b/tizen-audio-internal.h
@@ -181,7 +181,6 @@ typedef enum audio_gain {
 
 typedef struct audio_volume_value_table {
     double volume[AUDIO_VOLUME_TYPE_MAX][AUDIO_VOLUME_LEVEL_MAX];
-    uint32_t volume_level_max[AUDIO_VOLUME_LEVEL_MAX];
     double gain[AUDIO_GAIN_TYPE_MAX];
 } audio_volume_value_table_t;
 
@@ -191,6 +190,7 @@ enum {
 };
 
 typedef struct audio_hal_volume {
+    uint32_t volume_level_max[AUDIO_VOLUME_LEVEL_MAX];
     uint32_t volume_level[AUDIO_VOLUME_TYPE_MAX];
     audio_volume_value_table_t *volume_value_table;
 } audio_hal_volume_t;

diff --git a/tizen-audio-routing.c b/tizen-audio-routing.c
index 5be319e..29266fc 100644 (file)
--- a/tizen-audio-routing.c
+++ b/tizen-audio-routing.c
@@ -248,6 +248,7 @@ audio_return_t audio_update_route(void *audio_handle, audio_route_info_t *info)
 
     AUDIO_RETURN_VAL_IF_FAIL(ah, AUDIO_ERR_PARAMETER);
     AUDIO_RETURN_VAL_IF_FAIL(info, AUDIO_ERR_PARAMETER);
+    AUDIO_RETURN_VAL_IF_FAIL(info->role, AUDIO_ERR_PARAMETER);
 
     AUDIO_LOG_INFO("role:%s", info->role);
 

diff --git a/tizen-audio-stream.c b/tizen-audio-stream.c
index d07ed33..c60a443 100644 (file)
--- a/tizen-audio-stream.c
+++ b/tizen-audio-stream.c
@@ -44,6 +44,8 @@ audio_return_t audio_notify_stream_connection_changed(void *audio_handle, audio_
 
     AUDIO_RETURN_VAL_IF_FAIL(ah, AUDIO_ERR_PARAMETER);
     AUDIO_RETURN_VAL_IF_FAIL(info, AUDIO_ERR_PARAMETER);
+    AUDIO_RETURN_VAL_IF_FAIL(info->role, AUDIO_ERR_PARAMETER);
+    AUDIO_RETURN_VAL_IF_FAIL((info->direction <= AUDIO_DIRECTION_OUT), AUDIO_ERR_PARAMETER);
 
     AUDIO_LOG_INFO("role:%s, direction:%u, idx:%u, is_connected:%d", info->role, info->direction, info->idx, is_connected);
 

diff --git a/tizen-audio-volume.c b/tizen-audio-volume.c
index 1eb0ef1..48b551a 100644 (file)
--- a/tizen-audio-volume.c
+++ b/tizen-audio-volume.c
@@ -135,7 +135,7 @@ static void __dump_tb(audio_hal_t *ah)
         snprintf(dump_str_ptr, 8, "%6s:", vol_type_str);
         dump_str_ptr += strlen(dump_str_ptr);
 
-        for (vol_level_idx = 0; vol_level_idx < volume_value_table->volume_level_max[vol_type_idx]; vol_level_idx++) {
+        for (vol_level_idx = 0; vol_level_idx < ah->volume.volume_level_max[vol_type_idx]; vol_level_idx++) {
             snprintf(dump_str_ptr, 6, "%01.2f ", volume_value_table->volume[vol_type_idx][vol_level_idx]);
             dump_str_ptr += strlen(dump_str_ptr);
         }
@@ -198,7 +198,7 @@ static audio_return_t __load_volume_value_table_from_ini(audio_hal_t *ah)
     for (vol_type_idx = 0; vol_type_idx < AUDIO_VOLUME_TYPE_MAX; vol_type_idx++) {
         const char *vol_type_str = __get_volume_type_string_by_idx(vol_type_idx);
 
-        volume_value_table->volume_level_max[vol_type_idx] = 0;
+        ah->volume.volume_level_max[vol_type_idx] = 0;
         size = strlen(table_str) + strlen(vol_type_str) + 2;
         key = malloc(size);
         if (key) {
@@ -211,11 +211,11 @@ static audio_return_t __load_volume_value_table_from_ini(audio_hal_t *ah)
                     double vol_value = 0.0f;
                     if (strncmp(token, "0", strlen(token)))
                         vol_value = pow(10.0, (atof(token) - 100) / 20.0);
-                    volume_value_table->volume[vol_type_idx][volume_value_table->volume_level_max[vol_type_idx]++] = vol_value;
+                    volume_value_table->volume[vol_type_idx][ah->volume.volume_level_max[vol_type_idx]++] = vol_value;
                     token = strtok_r(NULL, delimiter, &ptr);
                 }
             } else {
-                volume_value_table->volume_level_max[vol_type_idx] = 1;
+                ah->volume.volume_level_max[vol_type_idx] = 1;
                 for (vol_level_idx = 0; vol_level_idx < AUDIO_VOLUME_LEVEL_MAX; vol_level_idx++) {
                     volume_value_table->volume[vol_type_idx][vol_level_idx] = VOLUME_VALUE_MAX;
                 }
@@ -305,16 +305,13 @@ audio_return_t _audio_volume_deinit(audio_hal_t *ah)
 audio_return_t audio_get_volume_level_max(void *audio_handle, audio_volume_info_t *info, uint32_t *level)
 {
     audio_hal_t *ah = (audio_hal_t *)audio_handle;
-    audio_volume_value_table_t *volume_value_table;
 
     AUDIO_RETURN_VAL_IF_FAIL(ah, AUDIO_ERR_PARAMETER);
     AUDIO_RETURN_VAL_IF_FAIL(info, AUDIO_ERR_PARAMETER);
     AUDIO_RETURN_VAL_IF_FAIL(level, AUDIO_ERR_PARAMETER);
-    AUDIO_RETURN_VAL_IF_FAIL(ah->volume.volume_value_table, AUDIO_ERR_PARAMETER);
 
     /* Get max volume level by device & type */
-    volume_value_table = ah->volume.volume_value_table;
-    *level = volume_value_table->volume_level_max[__get_volume_idx_by_string_type(info->type)];
+    *level = ah->volume.volume_level_max[__get_volume_idx_by_string_type(info->type)];
 
     AUDIO_LOG_DEBUG("get_[%s] volume_level_max: %d", info->type, *level);
 
@@ -349,7 +346,7 @@ audio_return_t audio_get_volume_value(void *audio_handle, audio_volume_info_t *i
 
     /* Get basic volume by device & type & level */
     volume_value_table = ah->volume.volume_value_table;
-    if (volume_value_table->volume_level_max[__get_volume_idx_by_string_type(info->type)] < level)
+    if (ah->volume.volume_level_max[__get_volume_idx_by_string_type(info->type)] < level)
         *value = VOLUME_VALUE_MAX;
     else
         *value = volume_value_table->volume[__get_volume_idx_by_string_type(info->type)][level];
@@ -367,6 +364,7 @@ audio_return_t audio_set_volume_level(void *audio_handle, audio_volume_info_t *i
 
     AUDIO_RETURN_VAL_IF_FAIL(ah, AUDIO_ERR_PARAMETER);
     AUDIO_RETURN_VAL_IF_FAIL(info, AUDIO_ERR_PARAMETER);
+    AUDIO_RETURN_VAL_IF_FAIL((ah->volume.volume_level_max[__get_volume_idx_by_string_type(info->type)] >= level), AUDIO_ERR_PARAMETER);
 
     /* Update volume level */
     ah->volume.volume_level[__get_volume_idx_by_string_type(info->type)] = level;
@@ -384,6 +382,8 @@ audio_return_t audio_get_volume_mute(void *audio_handle, audio_volume_info_t *in
     AUDIO_RETURN_VAL_IF_FAIL(info, AUDIO_ERR_PARAMETER);
     AUDIO_RETURN_VAL_IF_FAIL(mute, AUDIO_ERR_PARAMETER);
 
+    /* TODO. Not implemented */
+
     return audio_ret;
 }
 
@@ -395,5 +395,7 @@ audio_return_t audio_set_volume_mute(void *audio_handle, audio_volume_info_t *in
     AUDIO_RETURN_VAL_IF_FAIL(ah, AUDIO_ERR_PARAMETER);
     AUDIO_RETURN_VAL_IF_FAIL(info, AUDIO_ERR_PARAMETER);
 
+    /* TODO. Not implemented */
+
     return audio_ret;
 }