update Brightness logic

[apps/core/preloaded/settings.git] / setting-display / src / setting-display-brightness.c

diff --git a/setting-display/src/setting-display-brightness.c b/setting-display/src/setting-display-brightness.c
index 92dd0d1..ce6fc04 100755 (executable)
--- a/setting-display/src/setting-display-brightness.c
+++ b/setting-display/src/setting-display-brightness.c
@@ -204,10 +204,6 @@ static int setting_display_brightness_create(void *cb)
                ad->data_br_sli->isIndicatorVisible = false;
                ad->data_br_sli->slider_min = 1;
 
-               int max_brt = 10;
-               device_get_max_brightness(BRIGHTNESS_DISPLAY, &max_brt);
-               SETTING_TRACE("******* max_brt = %d ", max_brt);
-               ad->data_br_sli->slider_max = max_brt;
                ad->data_br_sli->userdata = ad;
 
                if (auto_value) {
@@ -327,6 +323,8 @@ static void __toggle_automatic_chk(void* data, int chk_status)
                // TOGGLE OFF
                //to get value every time
                ad->low_bat = VCONFKEY_SYSMAN_BAT_NORMAL;
+
+               // in case of LOW BATTERY
                ret = vconf_get_int(VCONFKEY_SYSMAN_BATTERY_STATUS_LOW, &ad->low_bat);
                if (0 != ret) //add error handle,due to different target env..
                {
@@ -336,6 +334,7 @@ static void __toggle_automatic_chk(void* data, int chk_status)
                        return;
                }
 
+               // in case of CIRITICAL LOW BATTERY
                if (ad->low_bat <= VCONFKEY_SYSMAN_BAT_CRITICAL_LOW)
                {
                        setting_disable_genlist_item(ad->data_br_sli->item);
@@ -343,9 +342,13 @@ static void __toggle_automatic_chk(void* data, int chk_status)
                }
                else
                {
+                       // in case of NORMAL OP
+                       // 1. set vconf
+                       // 2. call brightness set API
                        int value = SETTING_BRIGHTNESS_LEVEL5;
                        //device_get_brightness(BRIGHTNESS_DISPLAY, &value);
-                       if (ad->data_br_sli) value = elm_slider_value_get(ad->data_br_sli->eo_check);
+                       if (ad->data_br_sli)
+                               value = elm_slider_value_get(ad->data_br_sli->eo_check);
                        SETTING_TRACE(" value of ad->data_br_sli->eo_check: %d ", value);
                        setting_enable_genlist_item(ad->data_br_sli->item);
                        ret = setting_set_int_slp_key(INT_SLP_SETTING_LCD_BRIGHTNESS, value, &err);
@@ -356,6 +359,10 @@ static void __toggle_automatic_chk(void* data, int chk_status)
                                                        NULL, POPUP_INTERVAL, FALSE, FALSE);
                                return;
                        }
+
+                       /* call brightness API */
+                       ret = device_set_brightness(BRIGHTNESS_DISPLAY, value);
+                       /* TODO error handling here */
                }
        }
 }