display: Change get_var_display_config return type to const 77/288477/2
authorTaeminYeom <taemin.yeom@samsung.com>
Wed, 15 Feb 2023 08:22:26 +0000 (17:22 +0900)
committerTaeminYeom <taemin.yeom@samsung.com>
Fri, 17 Feb 2023 05:31:16 +0000 (14:31 +0900)
To prevent modify display config static struct in other file.
If it wanted to be modified later, setter will have to be added.

Change-Id: I477fc8476981f18d92509dfb6b2c0783cc7df734
Signed-off-by: TaeminYeom <taemin.yeom@samsung.com>
19 files changed:
plugins/iot-headed/display/core.c
plugins/iot-headed/display/device-interface.c
plugins/iot-headed/display/key-filter.c
plugins/mobile/display/core.c
plugins/mobile/display/device-interface.c
plugins/mobile/display/key-filter.c
plugins/tv/display/core.c
plugins/tv/display/device-interface.c
plugins/tv/display/key-filter.c
plugins/wearable/display/auto-brightness-sensorhub.c
plugins/wearable/display/core.c
plugins/wearable/display/device-interface.c
plugins/wearable/display/key-filter.c
src/display/auto-brightness.c
src/display/core.h
src/display/display-dbus.c
src/display/display-input.c
src/display/display-lock.c
src/display/setting.c

index 0a87e47..47e7e01 100644 (file)
@@ -210,7 +210,7 @@ struct display_function_info display_info = {
        .face_detection                 = NULL,
 };
 
-inline struct display_config* get_var_display_config()
+inline const struct display_config* get_var_display_config()
 {
        return &display_conf;
 }
index ecb5eb2..188f583 100644 (file)
@@ -76,7 +76,7 @@ static int default_brightness;
 static int dpms_running_state = DPMS_SETTING_DONE;
 static bool display_dev_available = false;
 static guint release_timer;
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 
 inline struct _backlight_ops *get_var_backlight_ops(void)
 {
index db9a338..7b75ac7 100644 (file)
@@ -430,7 +430,7 @@ static int process_power_key(struct input_event *pinput)
        int ignore = true;
        static int value = KEY_RELEASED;
        unsigned int caps;
-       struct display_config *display_conf = get_var_display_config();
+       const struct display_config *display_conf = get_var_display_config();
        if (!display_conf) {
                _E("Failed to get display configuration variable.");
                return ignore;
index 7948278..4927f70 100644 (file)
@@ -212,7 +212,7 @@ struct display_function_info display_info = {
        .face_detection                 = NULL,
 };
 
-inline struct display_config* get_var_display_config()
+inline const struct display_config* get_var_display_config()
 {
        return &display_conf;
 }
index cef8002..d7d79db 100644 (file)
@@ -75,7 +75,7 @@ static int default_brightness;
 static int dpms_running_state = DPMS_SETTING_DONE;
 static bool display_dev_available = false;
 static guint release_timer;
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 
 inline struct _backlight_ops *get_var_backlight_ops(void)
 {
index 1e56db1..b8fb9ca 100644 (file)
@@ -449,7 +449,7 @@ static int process_power_key(struct input_event *pinput)
        int ignore = true;
        static int value = KEY_RELEASED;
        unsigned int caps;
-       struct display_config *display_conf = get_var_display_config();
+       const struct display_config *display_conf = get_var_display_config();
        if (!display_conf) {
                _E("Failed to get display configuration variable.");
                return ignore;
index f9655cd..a596b62 100644 (file)
@@ -210,7 +210,7 @@ struct display_function_info display_info = {
        .face_detection                 = NULL,
 };
 
-inline struct display_config* get_var_display_config()
+inline const struct display_config* get_var_display_config()
 {
        return &display_conf;
 }
index 8c1cd1c..128b637 100644 (file)
@@ -76,7 +76,7 @@ static int default_brightness;
 static int dpms_running_state = DPMS_SETTING_DONE;
 static bool display_dev_available = false;
 static guint release_timer;
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 
 inline struct _backlight_ops *get_var_backlight_ops(void)
 {
index 42851ce..e3b3cdf 100644 (file)
@@ -430,7 +430,7 @@ static int process_power_key(struct input_event *pinput)
        int ignore = true;
        static int value = KEY_RELEASED;
        unsigned int caps;
-       struct display_config *display_conf = get_var_display_config();
+       const struct display_config *display_conf = get_var_display_config();
        if(!display_conf) {
                _E("Failed to get display configuration variable.");
                return ignore;
index 9eda1a1..51950bf 100644 (file)
@@ -43,7 +43,7 @@ static bool lbm, hbm, hold_brt, lowdim;
 
 static void change_brightness_transit(int start, int end)
 {
-       struct display_config *display_conf = get_var_display_config();
+       const struct display_config *display_conf = get_var_display_config();
        if(!display_conf) {
                _E("Failed to get display configuration variable.");
                return ;
index 293206d..de4d504 100644 (file)
@@ -235,7 +235,7 @@ struct display_function_info display_info = {
        .face_detection                 = NULL,
 };
 
-inline struct display_config *get_var_display_config()
+inline const struct display_config *get_var_display_config()
 {
        return &display_conf;
 }
index 9689dd0..4683490 100644 (file)
@@ -86,7 +86,7 @@ static int aod_max_level = -1;
 static int aod_normal_level = -1;
 static int aod_min_level = -1;
 static int aod_charging_level = -1;
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 static struct battery_status *battery = NULL;
 static struct battery_status* (*fp_get_var_battery_status)(void);
 
index ab4b47c..65ba5ab 100644 (file)
@@ -426,7 +426,7 @@ static int process_power_key(struct input_event *pinput)
        int ignore = true;
        static int value = KEY_RELEASED;
        unsigned int caps;
-       struct display_config *display_conf = get_var_display_config();
+       const struct display_config *display_conf = get_var_display_config();
        if (!display_conf) {
                _E("Failed to get display configuration variable.");
                return ignore;
index 9b00ecf..d7a740d 100644 (file)
@@ -73,7 +73,7 @@ static char *min_brightness_name = 0;
 
 /* light sensor */
 static float lmax, lmin;
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 
 static bool update_working_position(void)
 {
index fb52ede..34b6c94 100644 (file)
@@ -127,7 +127,7 @@ struct display_config {
  * Global variables
  *   display_conf : configuration of display
  */
-struct display_config* get_var_display_config();
+const struct display_config* get_var_display_config();
 
 /*
  * @brief Display Extension features
index 5c92a43..19c30f2 100644 (file)
@@ -73,7 +73,7 @@
 
 static struct display_plugin *disp_plgn;
 static struct _backlight_ops *backlight_ops;
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 
 static GVariant *dbus_start(GDBusConnection *conn,
        const gchar *sender, const gchar *path, const gchar *iface, const gchar *name,
index 05fcde6..df63a54 100644 (file)
@@ -38,7 +38,7 @@ static void process_input(struct libinput_event *ev)
        struct libinput_event_keyboard *k;
        unsigned int time;
        int fd = 0;
-       struct display_config *display_conf = get_var_display_config();
+       const struct display_config *display_conf = get_var_display_config();
        if(!display_conf) {
                _E("Failed to get display configuration variable.");
                return;
index 265fd09..bfdd7e9 100644 (file)
@@ -42,7 +42,7 @@ static struct _backlight_ops *backlight_ops;
 static GList *cond_head[S_END];
 static int trans_condition;
 
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 
 bool check_lock_state(int state)
 {
index 0b0ebb2..cc695ff 100644 (file)
@@ -48,7 +48,7 @@ static int force_lcdtimeout = 0;
 static int custom_on_timeout = 0;
 static int custom_normal_timeout = 0;
 static int custom_dim_timeout = 0;
-static struct display_config *display_conf;
+static const struct display_config *display_conf;
 
 int (*update_pm_setting) (int key_idx, int val);