tools/power/x86/intel-speed-select: Simplify output for turbo-freq and base-freq
authorPrarit Bhargava <prarit@redhat.com>
Thu, 5 Sep 2019 12:03:06 +0000 (08:03 -0400)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Sat, 7 Sep 2019 18:19:13 +0000 (21:19 +0300)
The current output of 'intel-speed-select -c 53 perf-profile info -l 0'
shows

        speed-select-turbo-freq-support:1
        speed-select-base-freq-support:1
        speed-select-base-freq-enabled:0
        speed-select-turbo-freq-enabled:0

Simplify the output to single lines displaying status of disabled,
enabled, and unsupported.

Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: David Arcari <darcari@redhat.com>
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
tools/power/x86/intel-speed-select/isst-display.c

index 0d9a53a..3ae693e 100644 (file)
@@ -297,23 +297,25 @@ void isst_ctdp_display_information(int cpu, FILE *outf, int tdp_level,
                format_and_print(outf, base_level + 4, header, value);
 
                snprintf(header, sizeof(header),
-                        "speed-select-turbo-freq-support");
-               snprintf(value, sizeof(value), "%d", ctdp_level->fact_support);
+                        "speed-select-turbo-freq");
+               if (ctdp_level->fact_support) {
+                       if (ctdp_level->fact_enabled)
+                               snprintf(value, sizeof(value), "enabled");
+                       else
+                               snprintf(value, sizeof(value), "disabled");
+               } else
+                       snprintf(value, sizeof(value), "unsupported");
                format_and_print(outf, base_level + 4, header, value);
 
                snprintf(header, sizeof(header),
-                        "speed-select-base-freq-support");
-               snprintf(value, sizeof(value), "%d", ctdp_level->pbf_support);
-               format_and_print(outf, base_level + 4, header, value);
-
-               snprintf(header, sizeof(header),
-                        "speed-select-base-freq-enabled");
-               snprintf(value, sizeof(value), "%d", ctdp_level->pbf_enabled);
-               format_and_print(outf, base_level + 4, header, value);
-
-               snprintf(header, sizeof(header),
-                        "speed-select-turbo-freq-enabled");
-               snprintf(value, sizeof(value), "%d", ctdp_level->fact_enabled);
+                        "speed-select-base-freq");
+               if (ctdp_level->pbf_support) {
+                       if (ctdp_level->pbf_enabled)
+                               snprintf(value, sizeof(value), "enabled");
+                       else
+                               snprintf(value, sizeof(value), "disabled");
+               } else
+                       snprintf(value, sizeof(value), "unsupported");
                format_and_print(outf, base_level + 4, header, value);
 
                snprintf(header, sizeof(header), "thermal-design-power(W)");