* [PATCH] tools/power/x86/intel-speed-select: correct macro name in debug output
@ 2026-06-10 19:56 Ethan Nelson-Moore
2026-09-10 1:05 ` srinivas pandruvada
0 siblings, 1 reply; 2+ messages in thread
From: Ethan Nelson-Moore @ 2026-06-10 19:56 UTC (permalink / raw)
To: platform-driver-x86; +Cc: Ethan Nelson-Moore, Srinivas Pandruvada
A call to debug_printf() in isst-core-mbox.c incorrectly refers to
CONFIG_TDP_GET_TURBO_LIMIT instead of
CONFIG_TDP_GET_TURBO_LIMIT_RATIOS. Correct it.
Discovered while searching for CONFIG_* symbols referenced in code but
not defined.
Signed-off-by: Ethan Nelson-Moore <enelsonmoore@gmail.com>
---
tools/power/x86/intel-speed-select/isst-core-mbox.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
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 c81ecd602bcf..baadeb75376c 100644
--- a/tools/power/x86/intel-speed-select/isst-core-mbox.c
+++ b/tools/power/x86/intel-speed-select/isst-core-mbox.c
@@ -561,7 +561,7 @@ static int mbox_get_get_trl(struct isst_id *id, int level, int avx_level, int *t
if (ret)
return ret;
- debug_printf("cpu:%d CONFIG_TDP_GET_TURBO_LIMIT req:%x resp:%x\n", id->cpu,
+ debug_printf("cpu:%d CONFIG_TDP_GET_TURBO_LIMIT_RATIOS req:%x resp:%x\n", id->cpu,
req, resp);
trl[4] = resp & GENMASK(7, 0);
--
2.43.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] tools/power/x86/intel-speed-select: correct macro name in debug output
2026-06-10 19:56 [PATCH] tools/power/x86/intel-speed-select: correct macro name in debug output Ethan Nelson-Moore
@ 2026-09-10 1:05 ` srinivas pandruvada
0 siblings, 0 replies; 2+ messages in thread
From: srinivas pandruvada @ 2026-09-10 1:05 UTC (permalink / raw)
To: Ethan Nelson-Moore, platform-driver-x86
On Wed, 2026-06-10 at 12:56 -0700, Ethan Nelson-Moore wrote:
> A call to debug_printf() in isst-core-mbox.c incorrectly refers to
> CONFIG_TDP_GET_TURBO_LIMIT instead of
> CONFIG_TDP_GET_TURBO_LIMIT_RATIOS. Correct it.
>
> Discovered while searching for CONFIG_* symbols referenced in code
> but
> not defined.
>
> Signed-off-by: Ethan Nelson-Moore <enelsonmoore@gmail.com>
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/spandruvada/linux.git/log/?h=for-sst-v7.4.rc1
Thanks,
Srinivas
> ---
> tools/power/x86/intel-speed-select/isst-core-mbox.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> 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 c81ecd602bcf..baadeb75376c 100644
> --- a/tools/power/x86/intel-speed-select/isst-core-mbox.c
> +++ b/tools/power/x86/intel-speed-select/isst-core-mbox.c
> @@ -561,7 +561,7 @@ static int mbox_get_get_trl(struct isst_id *id,
> int level, int avx_level, int *t
> if (ret)
> return ret;
>
> - debug_printf("cpu:%d CONFIG_TDP_GET_TURBO_LIMIT req:%x
> resp:%x\n", id->cpu,
> + debug_printf("cpu:%d CONFIG_TDP_GET_TURBO_LIMIT_RATIOS
> req:%x resp:%x\n", id->cpu,
> req, resp);
>
> trl[4] = resp & GENMASK(7, 0);
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-09-10 1:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-10 19:56 [PATCH] tools/power/x86/intel-speed-select: correct macro name in debug output Ethan Nelson-Moore
2026-09-10 1:05 ` srinivas pandruvada
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.