Add version of so files
[platform/core/multimedia/libmm-camcorder.git] / src / mm_camcorder_platform.c
index 1400985..10dc4fa 100644 (file)
@@ -44,7 +44,7 @@
 
 static int __enum_conv_whitebalance[MM_CAMCORDER_WHITE_BALANCE_NUM] ;
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_whitebalance = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_whitebalance = {
        MM_CAMCORDER_WHITE_BALANCE_NUM,
        __enum_conv_whitebalance,
        CONFIGURE_CATEGORY_CTRL_EFFECT,
@@ -54,7 +54,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_whitebalance = {
 
 static int __enum_conv_colortone[MM_CAMCORDER_COLOR_TONE_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_colortone = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_colortone = {
        MM_CAMCORDER_COLOR_TONE_NUM,
        __enum_conv_colortone,
        CONFIGURE_CATEGORY_CTRL_EFFECT,
@@ -64,7 +64,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_colortone = {
 
 static int __enum_conv_iso[MM_CAMCORDER_ISO_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_iso = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_iso = {
        MM_CAMCORDER_ISO_NUM,
        __enum_conv_iso,
        CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
@@ -74,7 +74,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_iso = {
 
 static int __enum_conv_prgrm[MM_CAMCORDER_SCENE_MODE_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_prgrm = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_prgrm = {
        MM_CAMCORDER_SCENE_MODE_NUM,
        __enum_conv_prgrm,
        CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
@@ -84,7 +84,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_prgrm = {
 
 static int __enum_conv_focus_mode[MM_CAMCORDER_FOCUS_MODE_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_focus_mode = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_focus_mode = {
        MM_CAMCORDER_FOCUS_MODE_NUM,
        __enum_conv_focus_mode,
        CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
@@ -94,7 +94,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_focus_mode = {
 
 static int __enum_conv_focus_type[MM_CAMCORDER_AUTO_FOCUS_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_focus_type = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_focus_type = {
        MM_CAMCORDER_AUTO_FOCUS_NUM,
        __enum_conv_focus_type,
        CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
@@ -104,7 +104,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_focus_type = {
 
 static int __enum_conv_ae_type[MM_CAMCORDER_AUTO_EXPOSURE_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_ae_type = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_ae_type = {
        MM_CAMCORDER_AUTO_EXPOSURE_NUM,
        __enum_conv_ae_type,
        CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
@@ -114,7 +114,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_ae_type = {
 
 static int __enum_conv_strobe_ctrl[MM_CAMCORDER_STROBE_CONTROL_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_strobe_ctrl = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_strobe_ctrl = {
        MM_CAMCORDER_STROBE_CONTROL_NUM,
        __enum_conv_strobe_ctrl,
        CONFIGURE_CATEGORY_CTRL_STROBE,
@@ -124,7 +124,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_strobe_ctrl = {
 
 static int __enum_conv_strobe_mode[MM_CAMCORDER_STROBE_MODE_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_strobe_mode = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_strobe_mode = {
        MM_CAMCORDER_STROBE_MODE_NUM,
        __enum_conv_strobe_mode,
        CONFIGURE_CATEGORY_CTRL_STROBE,
@@ -134,7 +134,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_strobe_mode = {
 
 static int __enum_conv_wdr_ctrl[MM_CAMCORDER_WDR_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_wdr_ctrl = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_wdr_ctrl = {
        MM_CAMCORDER_WDR_NUM,
        __enum_conv_wdr_ctrl,
        CONFIGURE_CATEGORY_CTRL_EFFECT,
@@ -143,7 +143,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_wdr_ctrl = {
 
 static int __enum_conv_flip_ctrl[MM_CAMCORDER_FLIP_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_flip_ctrl = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_flip_ctrl = {
        MM_CAMCORDER_FLIP_NUM,
        __enum_conv_flip_ctrl,
        CONFIGURE_CATEGORY_CTRL_EFFECT,
@@ -152,7 +152,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_flip_ctrl = {
 
 static int __enum_conv_rotation_ctrl[MM_CAMCORDER_ROTATION_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_rotation_ctrl = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_rotation_ctrl = {
        MM_CAMCORDER_ROTATION_NUM,
        __enum_conv_rotation_ctrl,
        CONFIGURE_CATEGORY_CTRL_EFFECT,
@@ -162,7 +162,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_rotation_ctrl = {
 
 static int __enum_conv_ahs[MM_CAMCORDER_AHS_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_ahs = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_ahs = {
        MM_CAMCORDER_AHS_NUM,
        __enum_conv_ahs,
        CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
@@ -172,7 +172,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_ahs = {
 
 static int __enum_conv_video_stabilization[MM_CAMCORDER_VIDEO_STABILIZATION_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_video_stabilization = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_video_stabilization = {
        MM_CAMCORDER_VIDEO_STABILIZATION_NUM,
        __enum_conv_video_stabilization,
        CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
@@ -182,7 +182,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_video_stabilization = {
 
 static int __enum_conv_hdr_capture[MM_CAMCORDER_HDR_CAPTURE_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_hdr_capture = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_hdr_capture = {
        MM_CAMCORDER_HDR_CAPTURE_NUM,
        __enum_conv_hdr_capture,
        CONFIGURE_CATEGORY_CTRL_CAPTURE,
@@ -191,7 +191,7 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_hdr_capture = {
 
 static int __enum_conv_detect_mode[MM_CAMCORDER_DETECT_MODE_NUM];
 
-_MMCamcorderEnumConvert _mmcamcorder_enum_conv_detect_mode = {
+static _MMCamcorderEnumConvert _mmcamcorder_enum_conv_detect_mode = {
        MM_CAMCORDER_DETECT_MODE_NUM,
        __enum_conv_detect_mode,
        CONFIGURE_CATEGORY_CTRL_DETECT,
@@ -652,6 +652,42 @@ static _MMCamcorderInfoConverting  g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                "VideoResolution",
                MM_CAMCONVERT_TYPE_INT_PAIR_ARRAY,
                NULL,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_STROBE,
+               MM_CAM_STROBE_BRIGHTNESS,
+               MM_CAMCORDER_ATTR_NONE,
+               "StrobeBrightness",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
+               NULL,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
+               MM_CAM_CAMERA_FOCUS_LEVEL,
+               MM_CAMCORDER_ATTR_NONE,
+               "FocusLevel",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
+               NULL,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_EFFECT,
+               MM_CAM_FILTER_WB_TEMPERATURE,
+               MM_CAMCORDER_ATTR_NONE,
+               "WhiteBalanceTemperature",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
+               NULL,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
+               MM_CAM_CAMERA_GAIN,
+               MM_CAMCORDER_ATTR_NONE,
+               "Gain",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
+               NULL,
        }
 };
 
@@ -679,8 +715,6 @@ int _mmcamcorder_convert_msl_to_sensor(MMHandleType handle, int attr_idx, int ms
 
        mmf_return_val_if_fail(hcamcorder, MM_ERROR_CAMCORDER_NOT_INITIALIZED);
 
-       /* _mmcam_dbg_log("attr_idx(%d), mslval(%d)", attr_idx, mslval); */
-
        info = hcamcorder->caminfo_convert;
 
        for (i = 0; i < size; i++) {
@@ -689,26 +723,31 @@ int _mmcamcorder_convert_msl_to_sensor(MMHandleType handle, int attr_idx, int ms
                        enum_convert = info[i].enum_convert;
 
                        if (enum_convert == NULL) {
-                               /* _mmcam_dbg_log("enum_convert is NULL. Just return the original value."); */
+                               MMCAM_LOG_VERBOSE("[attr:%d] NULL enum_convert, just return original[%d]",
+                                       attr_idx, mslval);
                                return mslval;
                        }
 
                        if (enum_convert->enum_arr == NULL) {
-                               _mmcam_dbg_warn("Unexpected error. Array pointer of enum_convert is NULL. Just return the original value.");
+                               MMCAM_LOG_WARNING("[attr:%d] NULL array pointer, just return original[%d]",
+                                       attr_idx, mslval);
                                return mslval;
                        }
 
                        if (enum_convert->total_enum_num > mslval && mslval >= 0) {
-                               /* _mmcam_dbg_log("original value(%d) -> converted value(%d)", mslval, enum_convert->enum_arr[mslval]); */
+                               MMCAM_LOG_VERBOSE("[attr:%d] original[%d] -> converted[%d]",
+                                       attr_idx, mslval, enum_convert->enum_arr[mslval]);
                                return enum_convert->enum_arr[mslval];
                        } else {
-                               _mmcam_dbg_warn("Input mslval[%d] is invalid(out of array[idx:%d,size:%d]), so can not convert. Just return the original value.", mslval, attr_idx, enum_convert->total_enum_num);
+                               MMCAM_LOG_WARNING("[attr:%d] invalid input mslval[%d][size:%d], just return original.",
+                                       attr_idx, mslval, enum_convert->total_enum_num);
                                return mslval;
                        }
                }
        }
 
-       _mmcam_dbg_warn("There is no category to match. Just return the original value.");
+       MMCAM_LOG_WARNING("[attr:%d] no category to match, just return original[%d]",
+               attr_idx, mslval);
 
        return mslval;
 }
@@ -718,24 +757,24 @@ int _mmcamcorder_get_fps_array_by_resolution(MMHandleType handle, int width, int
        MMCamAttrsInfo *infoW = NULL;
        MMCamAttrsInfo *infoH = NULL;
        int i = 0;
-       char nameFps[10] = {0,};
+       char nameFps[16] = {0,};
        bool valid_check = false;
 
        type_int_array *fps_array;
 
        mmf_camcorder_t *hcamcorder = MMF_CAMCORDER(handle);
 
-       /* _mmcam_dbg_log("prev resolution w:%d, h:%d", width, height); */
+       MMCAM_LOG_DEBUG("resolution %dx%d", width, height);
 
        infoW = (MMCamAttrsInfo*)calloc(1, sizeof(MMCamAttrsInfo));
        if (infoW == NULL) {
-               _mmcam_dbg_err("failed to alloc infoW");
+               MMCAM_LOG_ERROR("failed to alloc infoW");
                return MM_ERROR_CAMCORDER_LOW_MEMORY;
        }
 
        infoH = (MMCamAttrsInfo*)calloc(1, sizeof(MMCamAttrsInfo));
        if (infoH == NULL) {
-               _mmcam_dbg_err("failed to alloc infoH");
+               MMCAM_LOG_ERROR("failed to alloc infoH");
 
                free(infoW);
                infoW = NULL;
@@ -747,11 +786,11 @@ int _mmcamcorder_get_fps_array_by_resolution(MMHandleType handle, int width, int
        mm_camcorder_get_attribute_info(handle, MMCAM_CAMERA_HEIGHT, infoH);
 
        for (i = 0; i < infoW->int_array.count; i++) {
-               /* _mmcam_dbg_log("width :%d, height : %d\n", infoW->int_array.array[i], infoH->int_array.array[i]); */
+               MMCAM_LOG_VERBOSE("check resolution %dx%d", infoW->int_array.array[i], infoH->int_array.array[i]);
                if (infoW->int_array.array[i] == width && infoH->int_array.array[i] == height) {
                        valid_check = true;
-                       snprintf(nameFps, 10, "FPS%d", i);
-                       _mmcam_dbg_log("nameFps : %s!!!", nameFps);
+                       snprintf(nameFps, sizeof(nameFps), "FPS%d", i);
+                       MMCAM_LOG_INFO("nameFps[%s]", nameFps);
                        break;
                }
        }
@@ -762,12 +801,12 @@ int _mmcamcorder_get_fps_array_by_resolution(MMHandleType handle, int width, int
        infoH = NULL;
 
        if (!valid_check) {
-               _mmcam_dbg_err("FAILED : Can't find the valid resolution from attribute.");
+               MMCAM_LOG_ERROR("FAILED : Can't find the valid resolution from attribute.");
                return MM_ERROR_CAMCORDER_NOT_SUPPORTED;
        }
 
        if (!_mmcamcorder_conf_get_value_int_array(hcamcorder->conf_ctrl, CONFIGURE_CATEGORY_CTRL_CAMERA, nameFps, &fps_array)) {
-               _mmcam_dbg_err("FAILED : Can't find the valid FPS array.");
+               MMCAM_LOG_ERROR("FAILED : Can't find the valid FPS array.");
                return MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
        }
 
@@ -796,34 +835,40 @@ int _mmcamcorder_convert_sensor_to_msl(MMHandleType handle, int attr_idx, int se
                        enum_convert = info[i].enum_convert;
 
                        if (enum_convert == NULL) {
-                               /* _mmcam_dbg_log("enum_convert is NULL. Just return the original value."); */
+                               MMCAM_LOG_VERBOSE("[attr:%d] NULL enum_convert, just return original[%d]",
+                                       attr_idx, sensval);
                                return sensval;
                        }
 
                        if (enum_convert->enum_arr == NULL) {
-                               _mmcam_dbg_warn("Unexpected error. Array pointer of enum_convert is NULL. Just return the original value.");
+                               MMCAM_LOG_WARNING("[attr:%d] NULL array pointer, just return original[%d]",
+                                       attr_idx, sensval);
                                return sensval;
                        }
 
                        for (j = 0 ; j < enum_convert->total_enum_num ; j++) {
                                if (sensval == enum_convert->enum_arr[j]) {
-                                       /* _mmcam_dbg_log("original value(%d) -> converted value(%d)", sensval, j); */
+                                       MMCAM_LOG_VERBOSE("[attr:%d] original[%d] -> converted[%d]",
+                                               attr_idx, sensval, j);
                                        return j;
                                }
                        }
 
-                       _mmcam_dbg_warn("There is no sensor value matched with input param. Just return the original value.");
+                       MMCAM_LOG_WARNING("[attr:%d] no sensor value matched, just return original[%d]",
+                               attr_idx, sensval);
+
                        return sensval;
 
                }
        }
 
-       _mmcam_dbg_log("There is no category to match. Just return the original value.");
+       MMCAM_LOG_INFO("[attr:%d] no category to match, just return original[%d]",
+               attr_idx, sensval);
+
        return sensval;
 }
 
-static int
-__mmcamcorder_get_valid_array(int * original_array, int original_count, int ** valid_array, int * valid_default)
+static int __mmcamcorder_get_valid_array(int *original_array, int original_count, int **valid_array, int *valid_default)
 {
        int i = 0;
        int valid_count = 0;
@@ -842,12 +887,12 @@ __mmcamcorder_get_valid_array(int * original_array, int original_count, int ** v
                        for (i = 0; i < original_count; i++) {
                                if (original_array[i] != _MMCAMCORDER_SENSOR_ENUM_NONE) {
                                        (*valid_array)[valid_count++] = i;
-                                       /*_mmcam_dbg_log( "valid_array[%d] = %d", valid_count-1, (*valid_array)[valid_count-1] );*/
+                                       MMCAM_LOG_VERBOSE("valid_array[%d] = %d", valid_count-1, (*valid_array)[valid_count-1]);
 
                                        if (original_array[i] == *valid_default &&
                                            new_default == _MMCAMCORDER_SENSOR_ENUM_NONE) {
                                                new_default = i;
-                                               /*_mmcam_dbg_log( "converted MSL default[%d]", new_default );*/
+                                               MMCAM_LOG_VERBOSE("converted MSL default[%d]", new_default);
                                        }
                                }
                        }
@@ -873,7 +918,7 @@ int _mmcamcorder_init_attr_from_configure(MMHandleType handle, MMCamConvertingCa
 
        mmf_return_val_if_fail(hcamcorder, MM_ERROR_CAMCORDER_NOT_INITIALIZED);
 
-       _mmcam_dbg_log("category : %d", category);
+       MMCAM_LOG_INFO("category : %d", category);
 
        if (category & MM_CAMCONVERT_CATEGORY_CAMERA) {
                /* Initialize attribute related to camera control */
@@ -881,7 +926,7 @@ int _mmcamcorder_init_attr_from_configure(MMHandleType handle, MMCamConvertingCa
                table_size = sizeof(g_caminfo_convert) / sizeof(_MMCamcorderInfoConverting);
                ret = __mmcamcorder_set_info_to_attr(handle, info, table_size);
                if (ret != MM_ERROR_NONE) {
-                       _mmcam_dbg_err("camera info set error : 0x%x", ret);
+                       MMCAM_LOG_ERROR("camera info set error : 0x%x", ret);
                        return ret;
                }
        }
@@ -892,7 +937,7 @@ int _mmcamcorder_init_attr_from_configure(MMHandleType handle, MMCamConvertingCa
                table_size = sizeof(g_display_info) / sizeof(_MMCamcorderInfoConverting);
                ret = __mmcamcorder_set_info_to_attr(handle, info, table_size);
                if (ret != MM_ERROR_NONE) {
-                       _mmcam_dbg_err("display info set error : 0x%x", ret);
+                       MMCAM_LOG_ERROR("display info set error : 0x%x", ret);
                        return ret;
                }
        }
@@ -903,12 +948,12 @@ int _mmcamcorder_init_attr_from_configure(MMHandleType handle, MMCamConvertingCa
                table_size = sizeof(g_audio_info) / sizeof(_MMCamcorderInfoConverting);
                ret = __mmcamcorder_set_info_to_attr(handle, info, table_size);
                if (ret != MM_ERROR_NONE) {
-                       _mmcam_dbg_err("audio info set error : 0x%x", ret);
+                       MMCAM_LOG_ERROR("audio info set error : 0x%x", ret);
                        return ret;
                }
        }
 
-       _mmcam_dbg_log("done");
+       MMCAM_LOG_INFO("done");
 
        return ret;
 }
@@ -930,26 +975,21 @@ int __mmcamcorder_set_info_to_attr(MMHandleType handle, _MMCamcorderInfoConverti
        camera_conf *conf_info = NULL;
 
        for (i = 0; i < table_size; i++) {
-               /*
-               _mmcam_dbg_log("%d,%d,%d,%d,%s,%d",
+               MMCAM_LOG_VERBOSE("[type:%d][cat:%d][attr:%d][attr_pair:%d][key:%s][ctype:%d]",
                        info[i].type,
                        info[i].category,
                        info[i].attr_idx,
                        info[i].attr_idx_pair,
                        info[i].keyword,
                        info[i].conv_type);
-               */
 
                if (ret != MM_ERROR_NONE)
                        break;
 
-               if (info[i].type == CONFIGURE_TYPE_MAIN) {
+               if (info[i].type == CONFIGURE_TYPE_MAIN)
                        conf_info = hcamcorder->conf_main;
-                       /*_mmcam_dbg_log("MAIN configure [%s]", info[i].keyword);*/
-               } else {
+               else
                        conf_info = hcamcorder->conf_ctrl;
-                       /*_mmcam_dbg_log("CTRL configure [%s]", info[i].keyword);*/
-               }
 
                switch (info[i].conv_type) {
                case MM_CAMCONVERT_TYPE_INT:
@@ -985,15 +1025,10 @@ int __mmcamcorder_set_info_to_attr(MMHandleType handle, _MMCamcorderInfoConverti
                                }
 
                                if (iarray_size > 0) {
-                                       /*
-                                       _mmcam_dbg_log("INT Array. attr idx=%d array=%p, size=%d, default=%d",
-                                               info[i].attr_idx, iarray, iarray_size, idefault);
-                                       */
+                                       MMCAM_LOG_DEBUG("[%s] INT Array - size[%d], default[%d]",
+                                               info[i].keyword, iarray_size, idefault);
 
-                                       /* "mm_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
-                                       /* mm_attrs_set_valid_type(attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_ARRAY); */
                                        mm_attrs_set_valid_array(attrs, info[i].attr_idx, iarray, iarray_size, idefault);
-
                                        ret = mm_attrs_set_int(MMF_CAMCORDER_ATTRS(hcamcorder), info[i].attr_idx, idefault);
                                }
                        }
@@ -1011,11 +1046,10 @@ int __mmcamcorder_set_info_to_attr(MMHandleType handle, _MMCamcorderInfoConverti
                                break; /* skip to set, but not error */
 
                        if (irange) {
-                               /* _mmcam_dbg_log("INT Range. m:%d, s:%d, min=%d, max=%d", info[i].main_key, info[i].sub_key1, irange->min, irange->max); */
-                               /* "mm_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
-                               /* mm_attrs_set_valid_type (attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_RANGE); */
-                               mm_attrs_set_valid_range(attrs, info[i].attr_idx, irange->min, irange->max, irange->default_value);
+                               MMCAM_LOG_DEBUG("[%s] INT Range - min[%d], max[%d]",
+                                       info[i].keyword, irange->min, irange->max);
 
+                               mm_attrs_set_valid_range(attrs, info[i].attr_idx, irange->min, irange->max, irange->default_value);
                                ret = mm_attrs_set_int(MMF_CAMCORDER_ATTRS(hcamcorder), info[i].attr_idx, irange->default_value);
                        }
 
@@ -1024,46 +1058,36 @@ int __mmcamcorder_set_info_to_attr(MMHandleType handle, _MMCamcorderInfoConverti
                case MM_CAMCONVERT_TYPE_STRING:
                {
                        const char *cString = NULL;
-                       unsigned int iString_len = 0;
 
                        if (!_mmcamcorder_conf_get_value_string(handle, conf_info, info[i].category, info[i].keyword, &cString)) {
                                ret = MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
                                break; /* skip to set */
                        }
 
-                       /* _mmcam_dbg_log("String. m:%d, s:%d, cString=%s", info[i].main_key, info[i].sub_key1, cString); */
-                       /* strlen makes a crash when null pointer is passed. */
-                       if (cString)
-                               iString_len = strlen(cString);
-                       else
-                               iString_len = 0;
-
-                       ret = mm_attrs_set_string(attrs, info[i].attr_idx, cString, iString_len);
+                       MMCAM_LOG_DEBUG("[%s] String - string[%s]", info[i].keyword, cString);
+                       ret = mm_attrs_set_string(attrs, info[i].attr_idx, cString, cString ? strlen(cString) : 0);
                        break;
                }
                case MM_CAMCONVERT_TYPE_INT_PAIR_ARRAY:
                {
                        type_int_pair_array *pair_array = NULL;
 
-                       /*_mmcam_dbg_log("INT PAIR Array. type:%d, attr_idx:%d, attr_idx_pair:%d", info[i].type, info[i].attr_idx, info[i].attr_idx_pair);*/
-
                        if (!_mmcamcorder_conf_get_value_int_pair_array(conf_info, info[i].category, info[i].keyword, &pair_array))
                                break; /* skip to set, but not error */
 
                        if (pair_array && pair_array->count > 0) {
-                               /* "mm_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
-                               /* mm_attrs_set_valid_type(attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_ARRAY); */
                                mm_attrs_set_valid_array(attrs, info[i].attr_idx,
                                        pair_array->value[0],
                                        pair_array->count,
                                        pair_array->default_value[0]);
-                               /* "mm_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
-                               /* mm_attrs_set_valid_type(attrs, info[i].attr_idx_pair, MM_ATTRS_VALID_TYPE_INT_ARRAY); */
                                mm_attrs_set_valid_array(attrs, info[i].attr_idx_pair,
                                        pair_array->value[1],
                                        pair_array->count,
                                        pair_array->default_value[1]);
 
+                               MMCAM_LOG_DEBUG("[%s] INT PAIR Array - default[%d,%d]",
+                                       info[i].keyword, pair_array->default_value[0], pair_array->default_value[1]);
+
                                mm_attrs_set_int(MMF_CAMCORDER_ATTRS(hcamcorder), info[i].attr_idx, pair_array->default_value[0]);
                                mm_attrs_set_int(MMF_CAMCORDER_ATTRS(hcamcorder), info[i].attr_idx_pair, pair_array->default_value[1]);
                        }
@@ -1072,7 +1096,7 @@ int __mmcamcorder_set_info_to_attr(MMHandleType handle, _MMCamcorderInfoConverti
 
                case MM_CAMCONVERT_TYPE_USER:
                default:
-                       _mmcam_dbg_log("default : s:%d", info[i].attr_idx);
+                       MMCAM_LOG_WARNING("default - [attr:%d]", info[i].attr_idx);
                        break;
                }
        }
@@ -1198,10 +1222,9 @@ double _mmcamcorder_convert_volume(int mslVal)
                newVal = 1;
                break;
        default:
-               _mmcam_dbg_warn("out of range");
+               MMCAM_LOG_WARNING("out of range");
                break;
        }
 
        return newVal;
 }
-