display: return raw brightness value 89/78789/2 accepted/tizen/common/20160804.174410 submit/tizen/20160803.081834
authortaeyoung <ty317.kim@samsung.com>
Thu, 7 Jul 2016 01:42:02 +0000 (10:42 +0900)
committerTaeyoung Kim <ty317.kim@samsung.com>
Wed, 3 Aug 2016 05:13:51 +0000 (22:13 -0700)
- The brightness value from driver is returned to device daemon.
  device daemon will calculate it for users

Change-Id: I8823352e0b3f85c3b563c75725e0e8b46aea7841
Signed-off-by: taeyoung <ty317.kim@samsung.com>
hw/display/display.c

index 47af6f3..0eb19cb 100755 (executable)
@@ -35,7 +35,7 @@
 
 static int brightness_temp;
 
-static int get_max_brightness(int *val)
+static int display_get_max_brightness(int *val)
 {
        static int max = -1;
        int r;
@@ -57,50 +57,43 @@ static int get_max_brightness(int *val)
 
 static int display_get_brightness(int *brightness)
 {
-       int r, v, max;
+       int r, v;
 
        if (!brightness) {
                _E("wrong parameter");
                return -EINVAL;
        }
 
-       r = get_max_brightness(&max);
-       if (r < 0) {
-               _E("fail to get max brightness (errno:%d)", r);
-               return r;
-       }
-
        r = sys_get_int(BACKLIGHT_PATH"/brightness", &v);
        if (r < 0) {
-               _E("fail to get brightness (errno:%d)", r);
+               _E("fail to get brightness : %d", r);
                v = brightness_temp;
 //             return r;
        }
 
-       *brightness = v * 100.f / max;
+       *brightness = v;
        return 0;
 }
 
 static int display_set_brightness(int brightness)
 {
-       int r, v, max;
+       int r, max;
 
-       if (brightness < 0 || brightness > 100) {
-               _E("wrong parameter");
-               return -EINVAL;
-       }
-
-       r = get_max_brightness(&max);
+       r = display_get_max_brightness(&max);
        if (r < 0) {
                _E("fail to get max brightness (errno:%d)", r);
                return r;
        }
 
-       v = brightness/100.f*max;
-       r = sys_set_int(BACKLIGHT_PATH"/brightness", v);
+       if (brightness < 0 || brightness > max) {
+               _E("wrong parameter");
+               return -EINVAL;
+       }
+
+       r = sys_set_int(BACKLIGHT_PATH"/brightness", brightness);
        if (r < 0) {
                _E("fail to set brightness (errno:%d)", r);
-               brightness_temp = v;
+               brightness_temp = brightness;
 //             return r;
        }
 
@@ -120,6 +113,7 @@ static int display_open(struct hw_info *info,
                return -ENOMEM;
 
        display_dev->common.info = info;
+       display_dev->get_max_brightness = display_get_max_brightness;
        display_dev->get_brightness = display_get_brightness;
        display_dev->set_brightness = display_set_brightness;