[Release 0.10.61] TSAM-6195 : Fix crash when stop camera
[platform/core/multimedia/libmm-camcorder.git] / src / mm_camcorder_platform.c
index 87f8bf8..5817538 100644 (file)
@@ -35,7 +35,6 @@
 /*---------------------------------------------------------------------------
 |    LOCAL VARIABLE DEFINITIONS for internal                                                           |
 ---------------------------------------------------------------------------*/
-#define        MM_CAMCORDER_ATTR_NONE  -1
 
 
 // Rule 1. 1:1 match except NONE.
@@ -151,6 +150,26 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_wdr_ctrl =
        "WDR"
 };
 
+static int __enum_conv_flip_ctrl[MM_CAMCORDER_FLIP_NUM];
+
+_MMCamcorderEnumConvert _mmcamcorder_enum_conv_flip_ctrl =
+{
+       MM_CAMCORDER_FLIP_NUM,
+       __enum_conv_flip_ctrl,
+       CONFIGURE_CATEGORY_CTRL_EFFECT,
+       "Flip"
+};
+
+static int __enum_conv_rotation_ctrl[MM_CAMCORDER_ROTATION_NUM];
+
+_MMCamcorderEnumConvert _mmcamcorder_enum_conv_rotation_ctrl =
+{
+       MM_CAMCORDER_ROTATION_NUM,
+       __enum_conv_rotation_ctrl,
+       CONFIGURE_CATEGORY_CTRL_EFFECT,
+       "Rotation"
+};
+
 
 static int __enum_conv_ahs[MM_CAMCORDER_AHS_NUM];
 
@@ -201,6 +220,18 @@ _MMCamcorderEnumConvert _mmcamcorder_enum_conv_detect_mode =
  * For detail information, refer below documents.
  *
  */
+static _MMCamcorderInfoConverting      g_audio_info[] = {
+       {
+               CONFIGURE_TYPE_MAIN,
+               CONFIGURE_CATEGORY_MAIN_AUDIO_INPUT,
+               MM_CAM_AUDIO_DEVICE,
+               MM_CAMCORDER_ATTR_NONE,
+               "AudioDevice",
+               MM_CAMCONVERT_TYPE_INT_ARRAY,
+               NULL,
+       }
+};
+
 static _MMCamcorderInfoConverting      g_display_info[] = {
        {
                CONFIGURE_TYPE_MAIN,
@@ -262,23 +293,59 @@ static _MMCamcorderInfoConverting g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
        {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_CAMERA,
-               MM_CAM_CAMERA_FPS,
+               MM_CAM_CAMERA_FORMAT,
                MM_CAMCORDER_ATTR_NONE,
-               "FPS",
+               "PictureFormat",
                MM_CAMCONVERT_TYPE_INT_ARRAY,
                NULL,
        },
-       { /* 5 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_CAMERA,
-               MM_CAM_CAMERA_FORMAT,
+               MM_CAM_CAMERA_PAN_MECHA,
                MM_CAMCORDER_ATTR_NONE,
-               "PictureFormat",
-               MM_CAMCONVERT_TYPE_INT_ARRAY,
+               "PanMecha",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
+               NULL,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_CAMERA,
+               MM_CAM_CAMERA_PAN_ELEC,
+               MM_CAMCORDER_ATTR_NONE,
+               "PanElec",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
+               NULL,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_CAMERA,
+               MM_CAM_CAMERA_TILT_MECHA,
+               MM_CAMCORDER_ATTR_NONE,
+               "TiltMecha",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
                NULL,
        },
        {
                CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_CAMERA,
+               MM_CAM_CAMERA_TILT_ELEC,
+               MM_CAMCORDER_ATTR_NONE,
+               "TiltElec",
+               MM_CAMCONVERT_TYPE_INT_RANGE,
+               NULL,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_CAMERA,
+               MM_CAM_CAMERA_PTZ_TYPE,
+               MM_CAMCORDER_ATTR_NONE,
+               "PtzType",
+               MM_CAMCONVERT_TYPE_INT_ARRAY,
+               NULL,
+       },
+       {/* 10 */
+               CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_STROBE,
                MM_CAM_STROBE_CONTROL,
                MM_CAMCORDER_ATTR_NONE,
@@ -313,7 +380,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_RANGE,
                NULL,
        },
-       { /* 10 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_EFFECT,
                MM_CAM_FILTER_CONTRAST,
@@ -358,7 +425,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_ARRAY,
                &_mmcamcorder_enum_conv_whitebalance,
        },
-       { /* 15 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_EFFECT,
                MM_CAM_FILTER_COLOR_TONE,
@@ -367,7 +434,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_ARRAY,
                &_mmcamcorder_enum_conv_colortone,
        },
-       {
+       {/* 20 */
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_EFFECT,
                MM_CAM_CAMERA_WDR,
@@ -378,6 +445,24 @@ static _MMCamcorderInfoConverting  g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
        },
        {
                CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_EFFECT,
+               MM_CAM_CAMERA_FLIP,
+               MM_CAMCORDER_ATTR_NONE,
+               "Flip",
+               MM_CAMCONVERT_TYPE_INT_ARRAY,
+               &_mmcamcorder_enum_conv_flip_ctrl,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_EFFECT,
+               MM_CAM_CAMERA_ROTATION,
+               MM_CAMCORDER_ATTR_NONE,
+               "Rotation",
+               MM_CAMCONVERT_TYPE_INT_ARRAY,
+               &_mmcamcorder_enum_conv_rotation_ctrl,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
                MM_CAM_CAMERA_DIGITAL_ZOOM,
                MM_CAMCORDER_ATTR_NONE,
@@ -403,7 +488,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_ARRAY,
                &_mmcamcorder_enum_conv_focus_mode,
        },
-       { /* 20 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
                MM_CAM_CAMERA_AF_SCAN_RANGE,
@@ -439,7 +524,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_ARRAY,
                NULL,
        },
-       {
+       {/* 30 */
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
                MM_CAM_CAMERA_SHUTTER_SPEED,
@@ -448,7 +533,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_ARRAY,
                NULL,
        },
-       { /* 25 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
                MM_CAM_CAMERA_ISO,
@@ -493,7 +578,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_RANGE,
                NULL,
        },
-       { /* 30 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_CAPTURE,
                MM_CAM_CAPTURE_COUNT,
@@ -529,7 +614,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_RANGE,
                NULL,
        },
-       {
+       {/* 40 */
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_DETECT,
                MM_CAM_DETECT_FOCUS_SELECT,
@@ -538,7 +623,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_RANGE,
                NULL,
        },
-       { /* 35 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_DETECT,
                MM_CAM_DETECT_SELECT_NUMBER,
@@ -558,24 +643,6 @@ static _MMCamcorderInfoConverting  g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
        },
        {
                CONFIGURE_TYPE_CTRL,
-               CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
-               MM_CAM_CAMERA_FACE_ZOOM_MODE,
-               MM_CAMCORDER_ATTR_NONE,
-               "FaceZoomMode",
-               MM_CAMCONVERT_TYPE_INT_ARRAY,
-               NULL,
-       },
-       {
-               CONFIGURE_TYPE_CTRL,
-               CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
-               MM_CAM_CAMERA_FACE_ZOOM_LEVEL,
-               MM_CAMCORDER_ATTR_NONE,
-               "FaceZoomLevel",
-               MM_CAMCONVERT_TYPE_INT_RANGE,
-               NULL,
-       },
-       {
-               CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_CAMERA,
                MM_CAM_RECOMMEND_CAMERA_WIDTH,
                MM_CAM_RECOMMEND_CAMERA_HEIGHT,
@@ -583,7 +650,7 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                MM_CAMCONVERT_TYPE_INT_PAIR_ARRAY,
                NULL,
        },
-       { /* 40 */
+       {
                CONFIGURE_TYPE_CTRL,
                CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH,
                MM_CAM_CAMERA_VIDEO_STABILIZATION,
@@ -591,6 +658,15 @@ static _MMCamcorderInfoConverting  g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
                "VideoStabilization",
                MM_CAMCONVERT_TYPE_INT_ARRAY,
                &_mmcamcorder_enum_conv_video_stabilization,
+       },
+       {
+               CONFIGURE_TYPE_CTRL,
+               CONFIGURE_CATEGORY_CTRL_CAMERA,
+               MM_CAM_VIDEO_WIDTH,
+               MM_CAM_VIDEO_HEIGHT,
+               "VideoResolution",
+               MM_CAMCONVERT_TYPE_INT_PAIR_ARRAY,
+               NULL,
        }
 };
 
@@ -598,7 +674,6 @@ static _MMCamcorderInfoConverting   g_caminfo_convert[CAMINFO_CONVERT_NUM] = {
 |    LOCAL FUNCTION PROTOTYPES:                                                                                                |
 ---------------------------------------------------------------------------*/
 /* STATIC INTERNAL FUNCTION */
-static int  __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting *info, int table_size );
 static int  __mmcamcorder_get_valid_array(int * original_array, int original_count, int ** valid_array, int * valid_default );
 
 /*===========================================================================================
@@ -660,6 +735,71 @@ int _mmcamcorder_convert_msl_to_sensor(MMHandleType handle, int attr_idx, int ms
        return mslval;
 }
 
+int _mmcamcorder_get_fps_array_by_resolution(MMHandleType handle, int width, int height,  MMCamAttrsInfo* fps_info)
+{
+       MMCamAttrsInfo *infoW = NULL;
+       MMCamAttrsInfo *infoH = NULL;
+       int i = 0;
+       char nameFps[10] = {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);
+
+       infoW = (MMCamAttrsInfo*)calloc(1, sizeof(MMCamAttrsInfo));
+       if (infoW == NULL) {
+               _mmcam_dbg_err("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");
+
+               free(infoW);
+               infoW = NULL;
+
+               return MM_ERROR_CAMCORDER_LOW_MEMORY;
+       }
+
+       mm_camcorder_get_attribute_info(handle, MMCAM_CAMERA_WIDTH, infoW);
+       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]);
+               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);
+                       break;
+               }
+       }
+
+       free(infoW);
+       infoW = NULL;
+       free(infoH);
+       infoH = NULL;
+
+       if(!valid_check) {
+               _mmcam_dbg_err("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.");
+               return MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
+       }
+
+       fps_info->int_array.count = fps_array->count;
+       fps_info->int_array.array = fps_array->value;
+       fps_info->int_array.def = fps_array->default_value;
+
+       return MM_ERROR_NONE;
+}
+
 //convert sensor value to MSL value
 int _mmcamcorder_convert_sensor_to_msl(MMHandleType handle, int attr_idx, int sensval)
 {
@@ -671,7 +811,7 @@ int _mmcamcorder_convert_sensor_to_msl(MMHandleType handle, int attr_idx, int se
        mmf_return_val_if_fail(hcamcorder, MM_ERROR_CAMCORDER_NOT_INITIALIZED);
 
        info = hcamcorder->caminfo_convert;
-       
+
        for( i = 0 ; i < size ; i++ )
        {
                if( info[i].attr_idx == attr_idx )
@@ -716,36 +856,37 @@ __mmcamcorder_get_valid_array(int * original_array, int original_count, int ** v
        int i = 0;
        int valid_count = 0;
        int new_default = _MMCAMCORDER_SENSOR_ENUM_NONE;
-       
-       for (i = 0; i < original_count; i++)
-       {
-               if (original_array[i] != _MMCAMCORDER_SENSOR_ENUM_NONE)
+
+       for (i = 0; i < original_count; i++) {
+               if (original_array[i] != _MMCAMCORDER_SENSOR_ENUM_NONE) {
                        valid_count++;
+               }
        }
 
-       if (valid_count > 0)
-       {
+       if (valid_count > 0) {
                *valid_array = (int*)malloc(sizeof(int) * valid_count);
 
-               valid_count = 0;
-               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] );
-
-                               if( original_array[i] == *valid_default )
-                               {
-                                       new_default = i;
-                                       //_mmcam_dbg_log( "converted MSL default[%d]", new_default );
+               if (*valid_array) {
+                       valid_count = 0;
+                       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] );*/
+
+                                       if (original_array[i] == *valid_default &&
+                                           new_default == _MMCAMCORDER_SENSOR_ENUM_NONE) {
+                                               new_default = i;
+                                               /*_mmcam_dbg_log( "converted MSL default[%d]", new_default );*/
+                                       }
                                }
                        }
-               }               
+               }
+               else {
+                       valid_count = 0;
+               }
        }
 
-       if( new_default != _MMCAMCORDER_SENSOR_ENUM_NONE )
-       {
+       if (new_default != _MMCAMCORDER_SENSOR_ENUM_NONE) {
                *valid_default = new_default;
        }
 
@@ -753,7 +894,7 @@ __mmcamcorder_get_valid_array(int * original_array, int original_count, int ** v
 }
 
 
-int _mmcamcorder_init_attr_from_configure(MMHandleType handle)
+int _mmcamcorder_init_attr_from_configure(MMHandleType handle, MMCamConvertingCategory category)
 {
        mmf_camcorder_t *hcamcorder = MMF_CAMCORDER(handle);
        _MMCamcorderInfoConverting *info = NULL;
@@ -763,34 +904,54 @@ int _mmcamcorder_init_attr_from_configure(MMHandleType handle)
 
        mmf_return_val_if_fail(hcamcorder, MM_ERROR_CAMCORDER_NOT_INITIALIZED);
 
-       _mmcam_dbg_log("");
+       _mmcam_dbg_log("category : %d", category);
 
-       /* Initialize attribute related to camera control */
-       info = hcamcorder->caminfo_convert;
-       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( "ret : %x", ret );
-               return ret;
+       if (category & MM_CAMCONVERT_CATEGORY_CAMERA) {
+               /* Initialize attribute related to camera control */
+               info = hcamcorder->caminfo_convert;
+               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);
+                       return ret;
+               }
+       }
+
+       if (category & MM_CAMCONVERT_CATEGORY_DISPLAY) {
+               /* Initialize attribute related to display */
+               info = g_display_info;
+               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);
+                       return ret;
+               }
+       }
+
+       if (category & MM_CAMCONVERT_CATEGORY_AUDIO) {
+               /* Initialize attribute related to audio */
+               info = g_audio_info;
+               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);
+                       return ret;
+               }
        }
 
-       /* Initialize attribute related to display */
-       info = g_display_info;
-       table_size = sizeof(g_display_info) / sizeof(_MMCamcorderInfoConverting);
-       ret = __mmcamcorder_set_info_to_attr( handle, info, table_size );
-       _mmcam_dbg_log( "result: %x", ret );
+       _mmcam_dbg_log("done");
 
        return ret;
 }
 
-static int
-__mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting *info, int table_size )
+
+int __mmcamcorder_set_info_to_attr(MMHandleType handle, _MMCamcorderInfoConverting *info, int table_size)
 {
        mmf_camcorder_t *hcamcorder = MMF_CAMCORDER(handle);
        MMHandleType     attrs      = 0;
 
-       int i;
+       int i = 0;
+       int ret = MM_ERROR_NONE;
 
        mmf_return_val_if_fail(hcamcorder, MM_ERROR_CAMCORDER_NOT_INITIALIZED);
 
@@ -801,14 +962,12 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
 
        for (i = 0; i < table_size ; i++ )
        {
-               int ret = MM_ERROR_NONE;
-
                /*
                _mmcam_dbg_log( "%d,%d,%d,%d,%s,%d",
                                info[i].type,
                                info[i].category,
                                info[i].attr_idx,
-                               info[i].attr_idx_sub,
+                               info[i].attr_idx_pair,
                                info[i].keyword,
                                info[i].conv_type );
                */
@@ -816,29 +975,27 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
                if( info[i].type == CONFIGURE_TYPE_MAIN )
                {
                        conf_info = hcamcorder->conf_main;
-                       _mmcam_dbg_log( "MAIN configure [%s]", info[i].keyword );
+                       /*_mmcam_dbg_log( "MAIN configure [%s]", info[i].keyword );*/
                }
                else
                {
                        conf_info = hcamcorder->conf_ctrl;
-                       _mmcam_dbg_log( "CTRL configure [%s]", info[i].keyword );
+                       /*_mmcam_dbg_log( "CTRL configure [%s]", info[i].keyword );*/
                }
 
                switch(info[i].conv_type)
                {
                        case MM_CAMCONVERT_TYPE_INT:
                        {
-                               int iarray = (int)NULL;
+                               int ivalue = 0;
 
-                               if (!_mmcamcorder_conf_get_value_int( conf_info, info[i].category, info[i].keyword, &iarray ))
+                               if (!_mmcamcorder_conf_get_value_int(handle, conf_info, info[i].category, info[i].keyword, &ivalue))
                                {
                                        ret = MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
                                        break;          //skip to set
                                }
 
-                               ret = mm_attrs_set_int(MMF_CAMCORDER_ATTRS(hcamcorder), info[i].attr_idx, iarray);
-
-                               //_mmcam_dbg_log("INT . m:%d, s:%d, arr=%d", info[i].main_key, info[i].sub_key1, iarray);
+                               ret = mm_attrs_set_int(MMF_CAMCORDER_ATTRS(hcamcorder), info[i].attr_idx, ivalue);
                                break;
                        }
                        case MM_CAMCONVERT_TYPE_INT_ARRAY:
@@ -857,7 +1014,7 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
                                if( tarray )
                                {
                                        idefault = tarray->default_value;
-                                       
+
                                        if( info[i].enum_convert )
                                        {
                                                iarray_size = __mmcamcorder_get_valid_array(tarray->value, tarray->count, &iarray, &idefault);
@@ -871,15 +1028,16 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
                                        if( iarray_size > 0 )
                                        {
                                                /*
-                                               _mmcam_dbg_log("INT Array. m:%d, s:%d, arr=%x, size=%d, default=%d", 
-                                                               info[i].main_key, info[i].sub_key1, iarray, iarray_size, idefault);
+                                               _mmcam_dbg_log("INT Array. attr idx=%d array=%p, size=%d, default=%d",
+                                                              info[i].attr_idx, iarray, iarray_size, idefault);
                                                */
-                                               
-                                               mmf_attrs_set_valid_type(attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_ARRAY);
-                                               mmf_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);
+                                               /* "mmf_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
+                                               //mmf_attrs_set_valid_type(attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_ARRAY);
+                                               mmf_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);
+                                       }
                                }
 
                                if (iarray && iarray != tarray->value )
@@ -900,8 +1058,8 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
                                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);
-                               
-                                       mmf_attrs_set_valid_type (attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_RANGE);
+                                       /* "mmf_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
+                                       //mmf_attrs_set_valid_type (attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_RANGE);
                                        mmf_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);
@@ -911,10 +1069,10 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
                        }
                        case MM_CAMCONVERT_TYPE_STRING:
                        {
-                               char* cString = NULL;
+                               const char *cString = NULL;
                                unsigned int iString_len = 0;
 
-                               if (!_mmcamcorder_conf_get_value_string(conf_info, info[i].category, info[i].keyword, &cString ))
+                               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
@@ -934,7 +1092,7 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
                        {
                                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);
+                               /*_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))
                                {
@@ -944,12 +1102,14 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
 
                                if( pair_array && pair_array->count > 0 )
                                {
-                                       mmf_attrs_set_valid_type(attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_ARRAY);
+                                       /* "mmf_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
+                                       //mmf_attrs_set_valid_type(attrs, info[i].attr_idx, MM_ATTRS_VALID_TYPE_INT_ARRAY);
                                        mmf_attrs_set_valid_array(attrs, info[i].attr_idx,
                                                                  pair_array->value[0],
                                                                  pair_array->count,
                                                                  pair_array->default_value[0]);
-                                       mmf_attrs_set_valid_type(attrs, info[i].attr_idx_pair, MM_ATTRS_VALID_TYPE_INT_ARRAY);
+                                       /* "mmf_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
+                                       //mmf_attrs_set_valid_type(attrs, info[i].attr_idx_pair, MM_ATTRS_VALID_TYPE_INT_ARRAY);
                                        mmf_attrs_set_valid_array(attrs, info[i].attr_idx_pair,
                                                                  pair_array->value[1],
                                                                  pair_array->count,
@@ -966,16 +1126,11 @@ __mmcamcorder_set_info_to_attr( MMHandleType handle, _MMCamcorderInfoConverting
                                _mmcam_dbg_log("default : s:%d", info[i].attr_idx);
                                break;
                }
-               
-               if (ret != MM_ERROR_NONE)
-               {
-                       _mmcam_dbg_warn("Setting error. ( s:%d, ret:%x)", info[i].attr_idx, ret);
-               }
        }
 
-       if (mmf_attrs_commit(attrs) == -1)
+       if (ret != MM_ERROR_NONE || mmf_attrs_commit(attrs) == -1)
                return MM_ERROR_CAMCORDER_INVALID_ARGUMENT;
-       else    
+       else
                return MM_ERROR_NONE;
 }
 
@@ -1019,6 +1174,8 @@ int _mmcamcorder_init_convert_table(MMHandleType handle)
        memcpy(&(hcamcorder->enum_conv[ENUM_CONVERT_EXPOSURE_MODE]), &_mmcamcorder_enum_conv_ae_type, enum_conv_size);
        memcpy(&(hcamcorder->enum_conv[ENUM_CONVERT_STROBE_MODE]), &_mmcamcorder_enum_conv_strobe_mode, enum_conv_size);
        memcpy(&(hcamcorder->enum_conv[ENUM_CONVERT_WDR]), &_mmcamcorder_enum_conv_wdr_ctrl, enum_conv_size);
+       memcpy(&(hcamcorder->enum_conv[ENUM_CONVERT_FLIP]), &_mmcamcorder_enum_conv_flip_ctrl, enum_conv_size);
+       memcpy(&(hcamcorder->enum_conv[ENUM_CONVERT_ROTATION]), &_mmcamcorder_enum_conv_rotation_ctrl, enum_conv_size);
        memcpy(&(hcamcorder->enum_conv[ENUM_CONVERT_ANTI_HAND_SHAKE]), &_mmcamcorder_enum_conv_ahs, enum_conv_size);
        memcpy(&(hcamcorder->enum_conv[ENUM_CONVERT_VIDEO_STABILIZATION]), &_mmcamcorder_enum_conv_video_stabilization, enum_conv_size);
 
@@ -1032,6 +1189,8 @@ int _mmcamcorder_init_convert_table(MMHandleType handle)
        _mmcamcorder_set_converted_value(handle, &(hcamcorder->enum_conv[ENUM_CONVERT_EXPOSURE_MODE]));
        _mmcamcorder_set_converted_value(handle, &(hcamcorder->enum_conv[ENUM_CONVERT_STROBE_MODE]));
        _mmcamcorder_set_converted_value(handle, &(hcamcorder->enum_conv[ENUM_CONVERT_WDR]));
+       _mmcamcorder_set_converted_value(handle, &(hcamcorder->enum_conv[ENUM_CONVERT_FLIP]));
+       _mmcamcorder_set_converted_value(handle, &(hcamcorder->enum_conv[ENUM_CONVERT_ROTATION]));
        _mmcamcorder_set_converted_value(handle, &(hcamcorder->enum_conv[ENUM_CONVERT_ANTI_HAND_SHAKE]));
        _mmcamcorder_set_converted_value(handle, &(hcamcorder->enum_conv[ENUM_CONVERT_VIDEO_STABILIZATION]));
 
@@ -1051,6 +1210,10 @@ int _mmcamcorder_init_convert_table(MMHandleType handle)
                                        hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_COLOR_TONE]);
                                } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "WDR")) {
                                        hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_WDR]);
+                               } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "Flip")) {
+                                       hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_FLIP]);
+                               } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "Rotation")) {
+                                       hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_ROTATION]);
                                }
                                break;
                        case CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH: