tools/power/x86/intel-speed-select: Remove isst_get_pbf_info_complete

isst_get_pbf_info_complete does nothing but just free the core_mask.
Remove the function and do free core_mask directly and free core mask in
the caller.

No functional changes are expected.

Signed-off-by: Zhang Rui <rui.zhang@intel.com>
[srinivas.pandruvada@linux.intel.com: changelog edits]
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>

diff --git a/tools/power/x86/intel-speed-select/isst-config.c b/tools/power/x86/intel-speed-select/isst-config.c
index 1f357d499c899477e7573417eb0904ba502e44dd..71e00d03062f8d5030f1836964396492c0415fb7 100644 (file)
--- a/tools/power/x86/intel-speed-select/isst-config.c
+++ b/tools/power/x86/intel-speed-select/isst-config.c
@@ -1331,7 +1331,7 @@ static void dump_pbf_config_for_cpu(struct isst_id *id, void *arg1, void *arg2,
                exit(1);
        } else {
                isst_pbf_display_information(id, outf, tdp_level, &pbf_info);
-               isst_get_pbf_info_complete(&pbf_info);
+               free_cpu_set(pbf_info.core_cpumask);
        }
 }
 

diff --git a/tools/power/x86/intel-speed-select/isst-core.c b/tools/power/x86/intel-speed-select/isst-core.c
index 030c0bd4e5f0455288b35dde3783eaa6e6152b76..41aa1adc202536e6882dd3a1cbad4f2065a456a7 100644 (file)
--- a/tools/power/x86/intel-speed-select/isst-core.c
+++ b/tools/power/x86/intel-speed-select/isst-core.c
@@ -390,11 +390,6 @@ int isst_get_pbf_info(struct isst_id *id, int level, struct isst_pbf_info *pbf_i
        return isst_ops->get_pbf_info(id, level, pbf_info);
 }
 
-void isst_get_pbf_info_complete(struct isst_pbf_info *pbf_info)
-{
-       free_cpu_set(pbf_info->core_cpumask);
-}
-
 int isst_set_pbf_fact_status(struct isst_id *id, int pbf, int enable)
 {
        struct isst_pkg_ctdp pkg_dev;

diff --git a/tools/power/x86/intel-speed-select/isst.h b/tools/power/x86/intel-speed-select/isst.h
index 18659506bf68b3bd179a752e9211ae7b4372af2b..06e089eaaa289a6619f0a75572d9a329749ccda9 100644 (file)
--- a/tools/power/x86/intel-speed-select/isst.h
+++ b/tools/power/x86/intel-speed-select/isst.h
@@ -253,7 +253,6 @@ extern int isst_set_tdp_level(struct isst_id *id, int tdp_level);
 extern int isst_set_pbf_fact_status(struct isst_id *id, int pbf, int enable);
 extern int isst_get_pbf_info(struct isst_id *id, int level,
                             struct isst_pbf_info *pbf_info);
-extern void isst_get_pbf_info_complete(struct isst_pbf_info *pbf_info);
 extern int isst_get_fact_info(struct isst_id *id, int level, int fact_bucket,
                              struct isst_fact_info *fact_info);
 extern int isst_get_fact_bucket_info(struct isst_id *id, int level,