diff options
| author | Marc Zyngier <[email protected]> | 2025-04-13 10:11:42 +0000 |
|---|---|---|
| committer | Viresh Kumar <[email protected]> | 2025-04-16 08:07:44 +0000 |
| commit | 2b8e6b58889c672e1ae3601d9b2b070be4dc2fbc (patch) | |
| tree | a616a6867a71799e8fb01a735316f1b133432e91 /drivers/cpufreq/cppc_cpufreq.c | |
| parent | cpufreq: scpi: Fix null-ptr-deref in scpi_cpufreq_get_rate() (diff) | |
| download | kernel-2b8e6b58889c672e1ae3601d9b2b070be4dc2fbc.tar.gz kernel-2b8e6b58889c672e1ae3601d9b2b070be4dc2fbc.zip | |
cpufreq: cppc: Fix invalid return value in .get() callback
Returning a negative error code in a function with an unsigned
return type is a pretty bad idea. It is probably worse when the
justification for the change is "our static analisys tool found it".
Fixes: cf7de25878a1 ("cppc_cpufreq: Fix possible null pointer dereference")
Signed-off-by: Marc Zyngier <[email protected]>
Cc: "Rafael J. Wysocki" <[email protected]>
Cc: Viresh Kumar <[email protected]>
Reviewed-by: Lifeng Zheng <[email protected]>
Signed-off-by: Viresh Kumar <[email protected]>
Diffstat (limited to 'drivers/cpufreq/cppc_cpufreq.c')
| -rw-r--r-- | drivers/cpufreq/cppc_cpufreq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/cpufreq/cppc_cpufreq.c b/drivers/cpufreq/cppc_cpufreq.c index b3d74f9adcf0..cb93f00bafdb 100644 --- a/drivers/cpufreq/cppc_cpufreq.c +++ b/drivers/cpufreq/cppc_cpufreq.c @@ -747,7 +747,7 @@ static unsigned int cppc_cpufreq_get_rate(unsigned int cpu) int ret; if (!policy) - return -ENODEV; + return 0; cpu_data = policy->driver_data; |