core: Remove unnecessary parameter for booting_done event 07/210607/1
authorDongwoo Lee <dwoo08.lee@samsung.com>
Thu, 18 Jul 2019 01:58:28 +0000 (10:58 +0900)
committerDongwoo Lee <dwoo08.lee@samsung.com>
Tue, 23 Jul 2019 02:17:14 +0000 (11:17 +0900)
Even the late activation process is removed since commit cdf2e4d1139c
("core: Remove late activation process"), the 'boot_done' state is
still passed as a parameter for INIT_DONE event. So, this patch
removes the unnecessary parameter.

Change-Id: I3b7bf1d6cba8080618fd486c7ab5c2db775ae7de
fixes: cdf2e4d1139c ("core: Remove late activation process")
Signed-off-by: Dongwoo Lee <dwoo08.lee@samsung.com>
src/core/main.c
src/pmqos/pmqos.c
src/thermal/thermal.c

index 03bc85ad936fc0b57bfe6791a0c2361325aabb06..f78e6c6f426b11205511612b344c3d6590fd6a37 100644 (file)
@@ -62,8 +62,7 @@ static int late_init(void)
        if (ret < 0)
                return ret;
 
-       ret = 1;
-       device_notify(DEVICE_NOTIFIER_INIT_DONE, &ret);
+       device_notify(DEVICE_NOTIFIER_INIT_DONE, NULL);
 
        return 0;
 }
index e527364fae6bfddb5fcf5be4c1c86e73358d5eee..87ae8179f57a80a27b79e0f9234ada008a94bee9 100644 (file)
@@ -107,12 +107,11 @@ static gboolean dbus_cb_pmqos_start(SystemPassPmqos *obj,
                GDBusMethodInvocation *invoc, gpointer user_data)
 {
        int ret = 0;
-       int booting_done = 1;
 
        if (g_pmqos)
                _I("PASS PMQoS is already started\n");
        else
-               ret = pmqos_init_done(&booting_done, NULL);
+               ret = pmqos_init_done(NULL, NULL);
 
        g_dbus_method_invocation_return_value(invoc, g_variant_new("(i)", ret));
 
@@ -471,20 +470,9 @@ err:
  */
 static int pmqos_init_done(void *data, void *user_data)
 {
-       int *booting_done;
        int ret;
        int i;
 
-       /*
-        * As a callback function for the booting-done event, it is notified
-        * with the result of the booting_finished() function by using the
-        * first argument, void *data. When it is successfully booted up, an int
-        * value, 1 is passed with the first argument.
-        */
-       booting_done = (int *)data;
-       if (!(*booting_done))
-               return -EINVAL;
-
        if (g_pmqos)
                return 0;
 
index fd91e17131c9e32af4b171189cde97241f12271e..0baecd3a20689e16daa8ccddadb9e7664860667e 100644 (file)
@@ -104,20 +104,9 @@ static void thermal_free(void)
  */
 static int thermal_init_done(void *data, void *user_data)
 {
-       int *booting_done;
        int ret;
        int i;
 
-       /*
-        * As a callback function for the booting-done event, it is notified
-        * with the result of the booting_finished() function by using the
-        * first argument, void *data. When it is successfully booted up, an int
-        * value, 1 is passed with the first argument.
-        */
-       booting_done = (int *)data;
-       if (!(*booting_done))
-               return -EINVAL;
-
        if (g_thermal)
                return 0;
 
@@ -166,12 +155,11 @@ static gboolean dbus_cb_thermal_start(SystemThermal *obj,
                GDBusMethodInvocation *invoc, gpointer user_data)
 {
        int ret = 0;
-       int booting_done = 1;
 
        if (g_thermal)
                _I("Thermal Monitor is already started\n");
        else
-               ret = thermal_init_done(&booting_done, NULL);
+               ret = thermal_init_done(NULL, NULL);
 
        g_dbus_method_invocation_return_value(invoc, g_variant_new("(i)", ret));