diff options
| author | Len Brown <len.brown@intel.com> | 2026-05-04 15:32:12 -0400 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2026-05-24 10:26:22 -0400 |
| commit | f88e0542da4268bf6f7a5fa250aa6d61b2dce128 (patch) | |
| tree | e003306bd7d47aab38f76a0f1f285c2d89dd2b37 /tools/power/x86 | |
| parent | 56c811d5a583269015e72d7a821b93f3f44316df (diff) | |
| download | linux-next-f88e0542da4268bf6f7a5fa250aa6d61b2dce128.tar.gz linux-next-f88e0542da4268bf6f7a5fa250aa6d61b2dce128.zip | |
tools/power turbostat: Cleanup: bool force_load
It is confusing to increment what is effectively a boolean value.
Define force_load as boolean, and access it accordingly.
Assisted-by: Claude: Claude Sonnet 4.6
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'tools/power/x86')
| -rw-r--r-- | tools/power/x86/turbostat/turbostat.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c index ee0edabf5ac1..6819b70e0d56 100644 --- a/tools/power/x86/turbostat/turbostat.c +++ b/tools/power/x86/turbostat/turbostat.c @@ -523,7 +523,7 @@ unsigned int valid_rapl_msrs; unsigned int summary_only; unsigned int list_header_only; unsigned int dump_only; -unsigned int force_load; +bool force_load; unsigned int cpuid_has_aperf_mperf; unsigned int cpuid_has_hv; unsigned int has_aperf_access; @@ -1360,7 +1360,7 @@ void probe_platform_features(unsigned int family, unsigned int model) if (authentic_amd || hygon_genuine) { /* fallback to default features on unsupported models */ - force_load++; + force_load = true; if (max_extended_level >= 0x80000007) { unsigned int eax, ebx, ecx, edx; @@ -1383,7 +1383,7 @@ void probe_platform_features(unsigned int family, unsigned int model) } end: - if (force_load && !platform) { + if ((force_load == true) && !platform) { fprintf(outf, "Forced to run on unsupported platform!\n"); platform = &default_features; } @@ -11496,7 +11496,7 @@ void cmdline(int argc, char **argv) bic_lookup(&bic_enabled, optarg, SHOW_LIST); break; case 'f': - force_load++; + force_load = true; break; case 'd': debug++; |