pass: parser: Rename parser API for the readability 21/224721/7
authorChanwoo Choi <cw00.choi@samsung.com>
Tue, 11 Feb 2020 07:49:47 +0000 (16:49 +0900)
committerChanwoo Choi <cw00.choi@samsung.com>
Wed, 26 Feb 2020 10:22:19 +0000 (19:22 +0900)
Rename all parser API from "pass_*()" to "pass_parser_*()"
to improve the readability.

Change-Id: Iade47afa9463093698612bb2d81004ae918695f9
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
src/pass/pass-parser.c
src/pass/pass-parser.h
src/pass/pass.c
unittest/pass_haltests.cpp

index 26292a66a3863688483222fda68e995f7cd65593..acaca3c8054d0e8dcfd90df3716a83f71e9b3999 100644 (file)
@@ -961,7 +961,7 @@ static int parse_resource(struct parse_result *result, void *user_data)
  * @param      [in] path Path of configuration files (/etc/pass/pass.conf)
  * @return     @c 0 on success, otherwise error value
  */
-int pass_get_resource_config(struct pass *pass, char *path)
+int pass_parser_get_resource_config(struct pass *pass, char *path)
 {
        return config_parse(path, parse_resource, pass);
 }
@@ -971,7 +971,7 @@ int pass_get_resource_config(struct pass *pass, char *path)
  * @param      [in] pass Instance of 'struct pass' contains all information
  * @return     N/A
  */
-void pass_put_resource_config(struct pass *pass)
+void pass_parser_put_resource_config(struct pass *pass)
 {
        free(pass->res);
        pass->res = NULL;
@@ -984,7 +984,7 @@ void pass_put_resource_config(struct pass *pass)
  * @param      [in] path Path of configuration file for each h/w resource
  * @return     @c 0 on success, otherwise error value
  */
-int pass_get_each_resource_config(struct pass_resource *res, char *path)
+int pass_parser_get_each_resource_config(struct pass_resource *res, char *path)
 {
        struct pass_cpuhp *cpuhp;
        struct pass_pmqos *pmqos;
@@ -1068,7 +1068,7 @@ int pass_get_each_resource_config(struct pass_resource *res, char *path)
  * @param      [in] res Instance of h/w resource
  * @return     N/A
  */
-void pass_put_each_resource_config(struct pass_resource *res)
+void pass_parser_put_each_resource_config(struct pass_resource *res)
 {
        if (!res) {
                _E("cannot put pass table\n");
index bd554b5d072a9e6c2173b42de5867d29570f706e..538c6d730bfa1fe396adcc3971918ed32a4833e9 100644 (file)
 
 #include "pass.h"
 
-int pass_get_resource_config(struct pass *pass, char *path);
-void pass_put_resource_config(struct pass *pass);
+int pass_parser_get_resource_config(struct pass *pass, char *path);
+void pass_parser_put_resource_config(struct pass *pass);
 
-int pass_get_each_resource_config(struct pass_resource *res, char *path);
-void pass_put_each_resource_config(struct pass_resource *res);
+int pass_parser_get_each_resource_config(struct pass_resource *res, char *path);
+void pass_parser_put_each_resource_config(struct pass_resource *res);
 
 #endif /* __PASS_PARSER__ */
index 9038f25f1c7cbc1c6ae3f17f7c2a693d4da13936..764df8459e565af516a70d46f28f5f55b682d981 100644 (file)
@@ -205,7 +205,7 @@ static int pass_init_resource(struct pass_resource *res)
        int i;
 
        /* Get configuration of each resource from pass-resource*.conf */
-       ret = pass_get_each_resource_config(res, res->config_data.path_conf_file);
+       ret = pass_parser_get_each_resource_config(res, res->config_data.path_conf_file);
        if (ret < 0) {
                _E("cannot parse %s\n", res->config_data.path_conf_file);
                return -1;
@@ -300,7 +300,7 @@ static int pass_exit_resource(struct pass_resource *res)
        int ret;
 
        /* Put configuration of each resource from pass-resource*.conf */
-       pass_put_each_resource_config(res);
+       pass_parser_put_each_resource_config(res);
 
        if (is_supported_module(res, PASS_MODULE_CPUHP)) {
                ret = pass_cpuhp_exit(res);
@@ -361,7 +361,7 @@ static int pass_init_done(void *data, void *user_data)
        int i, ret;
 
        /* Get configuration of resource list from pass.conf */
-       ret = pass_get_resource_config(&g_pass, PASS_CONF_PATH);
+       ret = pass_parser_get_resource_config(&g_pass, PASS_CONF_PATH);
        if (ret < 0) {
                _E("cannot parse %s\n", PASS_CONF_PATH);
                return ret;
@@ -449,7 +449,7 @@ static void pass_exit_done(void)
        }
 
        /* Put configuration of resource list from pass.conf */
-       pass_put_resource_config(&g_pass);
+       pass_parser_put_resource_config(&g_pass);
        g_pass.state = PASS_OFF;
 }
 
index 4fbde96c2e7bdcd5a8d08332f8b6953605805675..de98ee00e8636869582d287338ef777bf242399d 100644 (file)
@@ -63,7 +63,7 @@ TEST_F(PowerMgntHalTest, GetResourceConfig_HandlesValidInput)
        ret = system("/bin/systemctl stop pass.service");
        ASSERT_EQ(ret, 0) << "PassServiceStop Failed";
 
-       ret = pass_get_resource_config(&g_pass, path);
+       ret = pass_parser_get_resource_config(&g_pass, path);
        ASSERT_EQ(ret, 0) << "GetResourceConfig Failed";
 
        for (i = 0; i < g_pass.num_resources; i++) {
@@ -491,7 +491,7 @@ TEST_F(PowerMgntHalTest, PutResourceConfig_HandlesValidInput)
 
                ret = pass_hal_put_resource(res);
                if (ret < 0) {
-                       pass_put_resource_config(&g_pass);
+                       pass_parser_put_resource_config(&g_pass);
                        ASSERT_EQ(ret, 0) << "PutResourceConfig Failed";
                }
        }