Re: [PATCH] cpupower: Include stdint.h for int64_t in the CPPC helper

[Jeremy Linton]

Hi,

On 10/20/26 4:47 PM, Florian Fainelli wrote:
> cppc_show_perf_and_freq() uses int64_t without including the header that
> defines it. Building cpupower against musl fails with "unknown type name
> 'int64_t'" because the other included headers do not provide the type.
>
> Include stdint.h directly instead of relying on transitive declarations
> from other C library headers.
>
> Fixes: 68f34fad760b ("cpupower: Add generic CPPC performance display")
> Signed-off-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx>
> ---
>   tools/power/cpupower/utils/helpers/cppc.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/tools/power/cpupower/utils/helpers/cppc.c b/tools/power/cpupower/utils/helpers/cppc.c
> index 3493ce8551ea..e0e9784fe589 100644
> --- a/tools/power/cpupower/utils/helpers/cppc.c
> +++ b/tools/power/cpupower/utils/helpers/cppc.c
> @@ -1,6 +1,7 @@
>   // SPDX-License-Identifier: GPL-2.0
>   
>   #include <stdio.h>
> +#include <stdint.h>
>   #include <errno.h>
>   #include <stdlib.h>
>   #include <string.h>

Looks good, thanks

Reviewed-by: Jeremy Linton <jeremy.linton@xxxxxxx>