Add get_lcd_power_node() for direct accessing the lcd power node 27/245327/4
authorHyotaek Shim <hyotaek.shim@samsung.com>
Wed, 7 Oct 2020 06:39:48 +0000 (15:39 +0900)
committerHyotaek Shim <hyotaek.shim@samsung.com>
Wed, 7 Oct 2020 08:35:59 +0000 (17:35 +0900)
Change-Id: Id47db38af56b2f6b6e4be0e6b46acf79b0564d75
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>
plugins/iot/display/device-interface.c
plugins/mobile/display/device-interface.c
plugins/tv/display/device-interface.c
plugins/wearable/display/device-interface.c

index eba4550..27fde01 100644 (file)
@@ -124,25 +124,22 @@ static int bl_brt(int brightness, int delay)
        return ret;
 }
 
-static int get_lcd_power(void)
+static int get_lcd_power_node(void)
 {
-       enum display_state val;
        int ret;
+       enum display_state val;
 
-       if (!display_dev || !display_dev->get_state) {
-               return dpms_get_state();
-       } else {
-               ret = display_dev->get_state(&val);
-               if (ret < 0)
-                       return ret;
+       ret = display_dev->get_state(&val);
+       if (ret < 0)
+               return ret;
 
-               if (val == DISPLAY_ON && ambient_get_state())
-                       return DPMS_OFF;
+       if (val == DISPLAY_ON && ambient_get_state())
+               return DPMS_OFF;
 
-               if (dpms_running_state != DPMS_SETTING_DONE)
-                       return dpms_running_state;
+       if (dpms_running_state != DPMS_SETTING_DONE)
+               return dpms_running_state;
 
-               switch (val) {
+       switch (val) {
                case DISPLAY_ON:
                        return DPMS_ON;
                case DISPLAY_STANDBY:
@@ -153,10 +150,17 @@ static int get_lcd_power(void)
                        return DPMS_OFF;
                default:
                        return -EINVAL;
-               }
        }
 }
 
+static int get_lcd_power(void)
+{
+       if (display_dev && display_dev->get_state)
+               return get_lcd_power_node();
+       else
+               return dpms_get_state();
+}
+
 bool display_dimstay_check(void)
 {
        if (get_pm_status_flag() & DIM_FLAG)
index 80b1d01..d9b5d3b 100644 (file)
@@ -123,25 +123,22 @@ static int bl_brt(int brightness, int delay)
        return ret;
 }
 
-static int get_lcd_power(void)
+static int get_lcd_power_node(void)
 {
-       enum display_state val;
        int ret;
+       enum display_state val;
 
-       if (!display_dev || !display_dev->get_state) {
-               return dpms_get_state();
-       } else {
-               ret = display_dev->get_state(&val);
-               if (ret < 0)
-                       return ret;
+       ret = display_dev->get_state(&val);
+       if (ret < 0)
+               return ret;
 
-               if (val == DISPLAY_ON && ambient_get_state())
-                       return DPMS_OFF;
+       if (val == DISPLAY_ON && ambient_get_state())
+               return DPMS_OFF;
 
-               if (dpms_running_state != DPMS_SETTING_DONE)
-                       return dpms_running_state;
+       if (dpms_running_state != DPMS_SETTING_DONE)
+               return dpms_running_state;
 
-               switch (val) {
+       switch (val) {
                case DISPLAY_ON:
                        return DPMS_ON;
                case DISPLAY_STANDBY:
@@ -152,10 +149,17 @@ static int get_lcd_power(void)
                        return DPMS_OFF;
                default:
                        return -EINVAL;
-               }
        }
 }
 
+static int get_lcd_power(void)
+{
+       if (display_dev && display_dev->get_state)
+               return get_lcd_power_node();
+       else
+               return dpms_get_state();
+}
+
 bool display_dimstay_check(void)
 {
        if (get_pm_status_flag() & DIM_FLAG)
index e7ab4df..e264290 100644 (file)
@@ -124,25 +124,22 @@ static int bl_brt(int brightness, int delay)
        return ret;
 }
 
-static int get_lcd_power(void)
+static int get_lcd_power_node(void)
 {
-       enum display_state val;
        int ret;
+       enum display_state val;
 
-       if (!display_dev || !display_dev->get_state) {
-               return dpms_get_state();
-       } else {
-               ret = display_dev->get_state(&val);
-               if (ret < 0)
-                       return ret;
+       ret = display_dev->get_state(&val);
+       if (ret < 0)
+               return ret;
 
-               if (val == DISPLAY_ON && ambient_get_state())
-                       return DPMS_OFF;
+       if (val == DISPLAY_ON && ambient_get_state())
+               return DPMS_OFF;
 
-               if (dpms_running_state != DPMS_SETTING_DONE)
-                       return dpms_running_state;
+       if (dpms_running_state != DPMS_SETTING_DONE)
+               return dpms_running_state;
 
-               switch (val) {
+       switch (val) {
                case DISPLAY_ON:
                        return DPMS_ON;
                case DISPLAY_STANDBY:
@@ -153,10 +150,17 @@ static int get_lcd_power(void)
                        return DPMS_OFF;
                default:
                        return -EINVAL;
-               }
        }
 }
 
+static int get_lcd_power(void)
+{
+       if (display_dev && display_dev->get_state)
+               return get_lcd_power_node();
+       else
+               return dpms_get_state();
+}
+
 bool display_dimstay_check(void)
 {
        if (get_pm_status_flag() & DIM_FLAG)
index 2e5351c..e95977b 100644 (file)
@@ -136,25 +136,22 @@ static int bl_brt(int brightness, int delay)
        return ret;
 }
 
-static int get_lcd_power(void)
+static int get_lcd_power_node(void)
 {
-       enum display_state val;
        int ret;
+       enum display_state val;
 
-       if (!display_dev || !display_dev->get_state) {
-               return dpms_get_state();
-       } else {
-               ret = display_dev->get_state(&val);
-               if (ret < 0)
-                       return ret;
+       ret = display_dev->get_state(&val);
+       if (ret < 0)
+               return ret;
 
-               if (val == DISPLAY_ON && ambient_get_state())
-                       return DPMS_OFF;
+       if (val == DISPLAY_ON && ambient_get_state())
+               return DPMS_OFF;
 
-               if (dpms_running_state != DPMS_SETTING_DONE)
-                       return dpms_running_state;
+       if (dpms_running_state != DPMS_SETTING_DONE)
+               return dpms_running_state;
 
-               switch (val) {
+       switch (val) {
                case DISPLAY_ON:
                        return DPMS_ON;
                case DISPLAY_STANDBY:
@@ -167,10 +164,17 @@ static int get_lcd_power(void)
                        return DPMS_DETACH;
                default:
                        return -EINVAL;
-               }
        }
 }
 
+static int get_lcd_power(void)
+{
+       if (display_dev && display_dev->get_state)
+               return get_lcd_power_node();
+       else
+               return dpms_get_state();
+}
+
 bool display_dimstay_check(void)
 {
        if (get_pm_status_flag() & DIM_FLAG)