Replace prefix from 'PASS' to 'HAL_POWER' 84/317584/2 accepted/tizen_9.0_unified tizen_9.0 accepted/tizen/9.0/unified/20250116.015842
authorUnsung Lee <unsung.lee@samsung.com>
Fri, 3 Jan 2025 10:32:12 +0000 (19:32 +0900)
committerUnsung Lee <unsung.lee@samsung.com>
Tue, 14 Jan 2025 08:55:27 +0000 (17:55 +0900)
Replace prefix of macro name from 'PASS' to 'HAL_POWER'.
This is because the pass module refers to macros defined in hal-api-power,
and the prefix of macros has been chaned in the hal-api-power.

Change-Id: Ifaf7b9b7d25708c988d00cfaefe89419dc305c15
Signed-off-by: Unsung Lee <unsung.lee@samsung.com>
src/pass/pass-hal.c
src/pass/pass-parser.c
src/pass/pass-rescon.c
src/pass/pass.c
src/pass/pass.h
tests/pass-unittests.cc
tests/unittest/pass-rescon/pass-rescon-unittests.cc
tests/unittest/pass-resmon/pass-resmon-unittests.cc

index 7b3840a74a89b14757de66f6c96e9187c4b240ec..6c21e749ceb1b80da8c8f9045ecc07af443d8d2a 100644 (file)
@@ -873,7 +873,7 @@ int pass_hal_save_initdata(struct pass_resource *res)
                return -EINVAL;
 
        switch (res->config_data.res_type) {
-       case PASS_RESOURCE_CPU_ID:
+       case HAL_POWER_RESOURCE_CPU_ID:
                ret = pass_hal_save_hotplug_initdata(res);
                if (ret < 0) {
                        _E("Failed to save hp initdata for '%s' resource",
@@ -881,8 +881,8 @@ int pass_hal_save_initdata(struct pass_resource *res)
                        return ret;
                }
                /* fall through */
-       case PASS_RESOURCE_BUS_ID:
-       case PASS_RESOURCE_GPU_ID:
+       case HAL_POWER_RESOURCE_BUS_ID:
+       case HAL_POWER_RESOURCE_GPU_ID:
                ret = pass_hal_save_dvfs_initdata(res);
                if (ret < 0) {
                        _E("Failed to save dvfs initdata for '%s' resource",
@@ -890,7 +890,7 @@ int pass_hal_save_initdata(struct pass_resource *res)
                        return ret;
                }
                /* fall through */
-       case PASS_RESOURCE_BATTERY_ID:
+       case HAL_POWER_RESOURCE_BATTERY_ID:
                ret = pass_hal_save_battery_initdata(res);
                if (ret < 0) {
                        _E("Failed to save battery initdata for '%s' resource",
@@ -905,7 +905,7 @@ int pass_hal_save_initdata(struct pass_resource *res)
                        return ret;
                }
                break;
-       case PASS_RESOURCE_MEMORY_ID:
+       case HAL_POWER_RESOURCE_MEMORY_ID:
                ret = pass_hal_save_memory_initdata(res);
                if (ret < 0) {
                        _E("Failed to save memory initdata for '%s' resource",
@@ -936,7 +936,7 @@ int pass_hal_restore_initdata(struct pass_resource *res)
                return -EINVAL;
 
        switch (res->config_data.res_type) {
-       case PASS_RESOURCE_CPU_ID:
+       case HAL_POWER_RESOURCE_CPU_ID:
                ret = pass_hal_restore_hotplug_initdata(res);
                if (ret < 0) {
                        _E("Failed to restore hp initdata for '%s' resource",
@@ -944,8 +944,8 @@ int pass_hal_restore_initdata(struct pass_resource *res)
                        return ret;
                }
                /* fall through */
-       case PASS_RESOURCE_BUS_ID:
-       case PASS_RESOURCE_GPU_ID:
+       case HAL_POWER_RESOURCE_BUS_ID:
+       case HAL_POWER_RESOURCE_GPU_ID:
                ret = pass_hal_restore_dvfs_initdata(res);
                if (ret < 0) {
                        _E("Failed to restore dvfs initdata for '%s' resource",
@@ -953,7 +953,7 @@ int pass_hal_restore_initdata(struct pass_resource *res)
                        return ret;
                }
                /* fall through */
-       case PASS_RESOURCE_BATTERY_ID:
+       case HAL_POWER_RESOURCE_BATTERY_ID:
                ret = pass_hal_restore_battery_initdata(res);
                if (ret < 0) {
                        _E("Failed to restore battery initdata for '%s' resource",
@@ -968,7 +968,7 @@ int pass_hal_restore_initdata(struct pass_resource *res)
                        return ret;
                }
                break;
-       case PASS_RESOURCE_MEMORY_ID:
+       case HAL_POWER_RESOURCE_MEMORY_ID:
                ret = pass_hal_restore_memory_initdata(res);
                if (ret < 0) {
                        _E("Failed to restore memory data for '%s' resource",
index d2542a4c4126758b3c3792243c0491cab5cf27ea..f57e34bd73905be79b7c70f096728a75b4ba0ba5 100644 (file)
@@ -680,7 +680,7 @@ static int parse_resource_data(struct pass *pass, int id, json_object *obj)
                config_data->res_thermal_priority = INT_MAX;
 
        switch (config_data->res_type) {
-       case PASS_RESOURCE_CPU_ID:
+       case HAL_POWER_RESOURCE_CPU_ID:
                if (config_data->num_cpus < 0 || config_data->cpu < 0) {
                        _E("Invalid 'number_of_cpus'(%d) or 'first_cpu'(%d) property\n",
                                config_data->num_cpus, config_data->cpu);
@@ -695,22 +695,22 @@ static int parse_resource_data(struct pass *pass, int id, json_object *obj)
                config_data->num_cpus = 0;
 
        /* Initialize config_data from property values of confiugartion file */
-       if (!strncmp(device_type, PASS_RESOURCE_CPU_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_CPU_ID;
-       else if (!strncmp(device_type, PASS_RESOURCE_GPU_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_GPU_ID;
-       else if (!strncmp(device_type, PASS_RESOURCE_BUS_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_BUS_ID;
-       else if (!strncmp(device_type, PASS_RESOURCE_MEMORY_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_MEMORY_ID;
-       else if (!strncmp(device_type, PASS_RESOURCE_BATTERY_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_BATTERY_ID;
-       else if (!strncmp(device_type, PASS_RESOURCE_DISPLAY_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_DISPLAY_ID;
-       else if (!strncmp(device_type, PASS_RESOURCE_DISK_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_DISK_ID;
-       else if (!strncmp(device_type, PASS_RESOURCE_NETWORK_NAME, strlen(device_type)))
-               config_data->res_type = PASS_RESOURCE_NETWORK_ID;
+       if (!strncmp(device_type, HAL_POWER_RESOURCE_CPU_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_CPU_ID;
+       else if (!strncmp(device_type, HAL_POWER_RESOURCE_GPU_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_GPU_ID;
+       else if (!strncmp(device_type, HAL_POWER_RESOURCE_BUS_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_BUS_ID;
+       else if (!strncmp(device_type, HAL_POWER_RESOURCE_MEMORY_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_MEMORY_ID;
+       else if (!strncmp(device_type, HAL_POWER_RESOURCE_BATTERY_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_BATTERY_ID;
+       else if (!strncmp(device_type, HAL_POWER_RESOURCE_DISPLAY_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_DISPLAY_ID;
+       else if (!strncmp(device_type, HAL_POWER_RESOURCE_DISK_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_DISK_ID;
+       else if (!strncmp(device_type, HAL_POWER_RESOURCE_NETWORK_NAME, strlen(device_type)))
+               config_data->res_type = HAL_POWER_RESOURCE_NETWORK_ID;
        else {
                _E("Unknown 'device_type' property value (%s)\n", device_type);
                return -EINVAL;
index 3e57f59ed699edc9bb8d89cf16fedda09273b76f..6c7aa519773c2a570d7e292c368f3b2bc8366030 100644 (file)
@@ -234,21 +234,21 @@ static int rescon_update(struct pass_resource *res)
        g_mutex_unlock(&rescon->scenario_level_mutex);
 
        switch (res_type) {
-       case PASS_RESOURCE_CPU_ID:
+       case HAL_POWER_RESOURCE_CPU_ID:
                limit_min_cpu = adjusted_level.limit_min_cpu;
                limit_max_cpu = adjusted_level.limit_max_cpu;
                /* fall through */
-       case PASS_RESOURCE_GPU_ID:
-       case PASS_RESOURCE_BUS_ID:
+       case HAL_POWER_RESOURCE_GPU_ID:
+       case HAL_POWER_RESOURCE_BUS_ID:
                limit_max_freq = adjusted_level.limit_max_freq;
                limit_min_freq = adjusted_level.limit_min_freq;
                /* fall through */
-       case PASS_RESOURCE_BATTERY_ID:
+       case HAL_POWER_RESOURCE_BATTERY_ID:
                cooling_device_state = adjusted_level.cooling_device_state;
                charging_status = adjusted_level.charging_status;
                charging_current_uA = adjusted_level.charging_current_uA;
                break;
-       case PASS_RESOURCE_MEMORY_ID:
+       case HAL_POWER_RESOURCE_MEMORY_ID:
                fault_around_bytes = adjusted_level.fault_around_bytes;
                break;
        default:
index 4f7b53dbe9aaadba388820e4abdc3895708daeb4..226210d25cf6692b6ea69b335db65a4eefb7fa5c 100644 (file)
@@ -191,15 +191,15 @@ static bool is_supported_module(struct pass_resource *res,
                                enum pass_module_type type)
 {
        switch (res->config_data.res_type) {
-       case PASS_RESOURCE_CPU_ID:
+       case HAL_POWER_RESOURCE_CPU_ID:
                return !!(PASS_RESOURCE_CPU_SUPPORTED_MODULES & type);
-       case PASS_RESOURCE_BUS_ID:
+       case HAL_POWER_RESOURCE_BUS_ID:
                return !!(PASS_RESOURCE_BUS_SUPPORTED_MODULES & type);
-       case PASS_RESOURCE_GPU_ID:
+       case HAL_POWER_RESOURCE_GPU_ID:
                return !!(PASS_RESOURCE_GPU_SUPPORTED_MODULES & type);
-       case PASS_RESOURCE_MEMORY_ID:
+       case HAL_POWER_RESOURCE_MEMORY_ID:
                return !!(PASS_RESOURCE_MEMORY_SUPPORTED_MODULES & type);
-       case PASS_RESOURCE_BATTERY_ID:
+       case HAL_POWER_RESOURCE_BATTERY_ID:
                return !!(PASS_RESOURCE_BATTERY_SUPPORTED_MODULES & type);
        default:
                return false;
index 2213e88dc0a2c75ed0013a6f0df6e362bc374555..e0509f553a8005aff58fb6fd4755de467b5cde92 100644 (file)
@@ -122,51 +122,51 @@ struct pass_level {
        /**
         * Limited maximum frequency
         * and this property is used for following resources:
-        * - PASS_RESOURCE_CPU_ID
-        * - PASS_RESOURCE_BUS_ID
-        * - PASS_RESOURCE_GPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_BUS_ID
+        * - HAL_POWER_RESOURCE_GPU_ID
         */
        int limit_max_freq;
        /**
         * Limited minimum frequency
         * and this property is used for following resources:
-        * - PASS_RESOURCE_CPU_ID
-        * - PASS_RESOURCE_BUS_ID
-        * - PASS_RESOURCE_GPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_BUS_ID
+        * - HAL_POWER_RESOURCE_GPU_ID
         */
        int limit_min_freq;
        /**
         * The maximum number of online CPU
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        int limit_max_cpu;
        /**
         * The minimum number of online CPU
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        int limit_min_cpu;
        /**
         * The number of bytes to be mapped around the fault
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_MEMORY_ID
+        * - HAL_POWER_RESOURCE_MEMORY_ID
         */
        int fault_around_bytes;
        /**
         * The current state of cooling device
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
-        * - PASS_RESOURCE_BUS_ID
-        * - PASS_RESOURCE_GPU_ID
-        * - PASS_RESOURCE_BATTERY_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_BUS_ID
+        * - HAL_POWER_RESOURCE_GPU_ID
+        * - HAL_POWER_RESOURCE_BATTERY_ID
         */
        int cooling_device_state;
 
        /**
         * The current state of battery charging device
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_BATTERY_ID
+        * - HAL_POWER_RESOURCE_BATTERY_ID
         */
        int charging_status;
        int charging_current_uA;
@@ -176,7 +176,7 @@ struct pass_level {
        /**
         * Period of timer for each pass_level
         * and this property is used for thefollowing resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        int gov_timeout;
 
@@ -185,50 +185,50 @@ struct pass_level {
        /**
         * The up condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        struct pass_level_condition up_cond[PASS_LEVEL_COND_MAX];
        /**
         * The down condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        struct pass_level_condition down_cond[PASS_LEVEL_COND_MAX];
        /**
         * The left condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        struct pass_level_condition left_cond[PASS_LEVEL_COND_MAX];
        /**
         * The right condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        struct pass_level_condition right_cond[PASS_LEVEL_COND_MAX];
 
        /**
         * The number of up condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        int num_up_cond;
        /**
         * The number of down condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        int num_down_cond;
        /**
         * The number of left condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        int num_left_cond;
        /**
         * The number of right condition to change between pass_level
         * and this property is used for the following resources:
-        * - PASS_RESOURCE_CPU_ID
+        * - HAL_POWER_RESOURCE_CPU_ID
         */
        int num_right_cond;
 };
@@ -387,10 +387,10 @@ struct pass_resource_config_data {
        /**
         * [mandatory] The unique id for each h/w resource. The id can have
         * the one value among following defined resource id.
-        * -  1: PASS_RESOURCE_CPU_ID
-        * -  2: PASS_RESOURCE_BUS_ID
-        * -  3: PASS_RESOURCE_GPU_ID
-        * -  4: PASS_RESOURCE_MEMORY_ID
+        * -  1: HAL_POWER_RESOURCE_CPU_ID
+        * -  2: HAL_POWER_RESOURCE_BUS_ID
+        * -  3: HAL_POWER_RESOURCE_GPU_ID
+        * -  4: HAL_POWER_RESOURCE_MEMORY_ID
         */
        unsigned int res_type;
        /**
@@ -431,17 +431,17 @@ struct pass_resource_config_data {
 
        /**
         * Path for load_table entry from kernel. If res_type is
-        * PASS_RESOURCE_CPU_ID, it is mandatory.
+        * HAL_POWER_RESOURCE_CPU_ID, it is mandatory.
         */
        char path_load_table[BUFF_MAX];
        /**
         * The number of supported CPU in the same cluster.
-        * If res_type is PASS_RESOURCE_CPU_ID, it is mandatory.
+        * If res_type is HAL_POWER_RESOURCE_CPU_ID, it is mandatory.
         */
        int num_cpus;
        /**
         * [optional] Index of first cpu in the same cluster.
-        * If res_type is PASS_RESOURCE_CPU_ID, it is mandatory.
+        * If res_type is HAL_POWER_RESOURCE_CPU_ID, it is mandatory.
         */
        int cpu;
 
@@ -546,10 +546,10 @@ struct pass_resource {
 
        /**
         * The hal instance of each h/w resource from pass_get_hal_info().
-        * - If res_type of cdata is PASS_RESOURCE_CPU_ID, hal.cpu will be used.
-        * - If res_type of cdata is PASS_RESOURCE_BUS_ID, hal.bus will be used.
-        * - If res_type of cdata is PASS_RESOURCE_GPU_ID, hal.gpu will be used.
-        * - If res_type of cdata is PASS_RESOURCE_MEMORY_ID,
+        * - If res_type of cdata is HAL_POWER_RESOURCE_CPU_ID, hal.cpu will be used.
+        * - If res_type of cdata is HAL_POWER_RESOURCE_BUS_ID, hal.bus will be used.
+        * - If res_type of cdata is HAL_POWER_RESOURCE_GPU_ID, hal.gpu will be used.
+        * - If res_type of cdata is HAL_POWER_RESOURCE_MEMORY_ID,
         *   hal.memory will be used.
         */
        union {
index 8ded14388c3c4b7f568fa640c22de991794aee57..f6ad86b1651da5f387cd8912e8fc1da456ce6fc4 100644 (file)
@@ -590,7 +590,7 @@ public:
 
 TEST(PassResconInitExitTest, pass_rescon_prepare_and_init_and_exit) {
        struct pass_resource res;
-       res.config_data.res_type = PASS_RESOURCE_CPU_ID;
+       res.config_data.res_type = HAL_POWER_RESOURCE_CPU_ID;
        res.rescon = NULL;
 
        int ret = pass_rescon_prepare(&res);
index cdea3c6c715e3b4ca31c871e0a303a23e6434d45..067de1c87ba6e9bb3f9490f01b5cf0ecc201a338 100644 (file)
@@ -47,7 +47,7 @@ static void init_pass_hal(void)
        gPassHalMock = new PassHalMock();
 
        g_resource = (struct pass_resource *)calloc(1, sizeof(struct pass_resource));
-       g_resource->config_data.res_type = PASS_RESOURCE_CPU_ID;
+       g_resource->config_data.res_type = HAL_POWER_RESOURCE_CPU_ID;
 
        EXPECT_CALL(*gPassHalMock, pass_hal_save_initdata(_)).WillRepeatedly(Return(0));
        EXPECT_CALL(*gPassHalMock, pass_hal_restore_initdata(_)).WillRepeatedly(Return(0));
index 9c1be4e1ce50844dc766ca285e9e735ac9f181f1..00fdab32c2e8e66bb1153d475f26b33b05aa8c0a 100644 (file)
@@ -48,7 +48,7 @@ static void init_pass_hal(void)
        gPassHalMock = new PassHalMock();
 
        g_resource = (struct pass_resource *)calloc(1, sizeof(struct pass_resource));
-       g_resource->config_data.res_type = PASS_RESOURCE_CPU_ID;
+       g_resource->config_data.res_type = HAL_POWER_RESOURCE_CPU_ID;
 
        EXPECT_CALL(*gPassHalMock, pass_hal_save_initdata(_)).WillRepeatedly(Return(0));
        EXPECT_CALL(*gPassHalMock, pass_hal_restore_initdata(_)).WillRepeatedly(Return(0));