tools/power/x86/intel-speed-select: Display AMX base frequency

AMX frequency is present in non TPMI platforms also. When platform
supports, the value is non zero. So, display AMX base frequency when
non zero, irrespective of platform API version.

Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>

diff --git a/tools/power/x86/intel-speed-select/isst-core-mbox.c b/tools/power/x86/intel-speed-select/isst-core-mbox.c
index 6eafef925d1ac9152e16196bdc0acf0886d514e5..6951a9c31dc55c2deff026257a724b52976f7cb4 100644 (file)
--- a/tools/power/x86/intel-speed-select/isst-core-mbox.c
+++ b/tools/power/x86/intel-speed-select/isst-core-mbox.c
@@ -383,10 +383,11 @@ static void _get_p1_info(struct isst_id *id, int config_index,
        ctdp_level->sse_p1 = resp & GENMASK(7, 0);
        ctdp_level->avx2_p1 = (resp & GENMASK(15, 8)) >> 8;
        ctdp_level->avx512_p1 = (resp & GENMASK(23, 16)) >> 16;
+       ctdp_level->amx_p1 = (resp & GENMASK(31, 24)) >> 24;
        debug_printf(
-               "cpu:%d ctdp:%d CONFIG_TDP_GET_P1_INFO resp:%x sse_p1:%d avx2_p1:%d avx512_p1:%d\n",
+               "cpu:%d ctdp:%d CONFIG_TDP_GET_P1_INFO resp:%x sse_p1:%d avx2_p1:%d avx512_p1:%d amx_p1:%d\n",
                id->cpu, config_index, resp, ctdp_level->sse_p1,
-               ctdp_level->avx2_p1, ctdp_level->avx512_p1);
+               ctdp_level->avx2_p1, ctdp_level->avx512_p1, ctdp_level->amx_p1);
 }
 
 static void _get_uncore_mem_freq(struct isst_id *id, int config_index,

diff --git a/tools/power/x86/intel-speed-select/isst-display.c b/tools/power/x86/intel-speed-select/isst-display.c
index 218fb317f7adac0fa6eda9174eb64f86cf5229f6..0e364b35f0933ce258460cb58b6e137025f58f84 100644 (file)
--- a/tools/power/x86/intel-speed-select/isst-display.c
+++ b/tools/power/x86/intel-speed-select/isst-display.c
@@ -424,7 +424,7 @@ void isst_ctdp_display_information(struct isst_id *id, FILE *outf, int tdp_level
                        format_and_print(outf, level + 2, header, value);
                }
 
-               if (api_version() > 1 && ctdp_level->amx_p1) {
+               if (ctdp_level->amx_p1) {
                        snprintf(header, sizeof(header), "base-frequency-amx(MHz)");
                        snprintf(value, sizeof(value), "%d",
                        ctdp_level->amx_p1 * isst_get_disp_freq_multiplier());