Messages in this thread Patch in this message |  | | From | Qianfeng Rong <> | | Subject | [PATCH v3 2/3] cpufreq: powernow: Use int type to store negative error codes | | Date | Tue, 2 Sep 2025 19:45:43 +0800 |
| |
In powernow_decode_bios(), the 'ret' variable stores either negative error codes or zero returned by get_ranges(), so it needs to be changed to int type. However, since the 'ret' variable is only used once and can be simplified to 'return get_ranges()', it is better to remove the 'ret' variable.
Signed-off-by: Qianfeng Rong <rongqianfeng@vivo.com> drivers/cpufreq/powernow-k7.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/cpufreq/powernow-k7.c b/drivers/cpufreq/powernow-k7.c index 31039330a3ba..72430c3c5500 100644 --- a/drivers/cpufreq/powernow-k7.c +++ b/drivers/cpufreq/powernow-k7.c @@ -451,7 +451,6 @@ static int powernow_decode_bios(int maxfid, int startvid) unsigned int i, j; unsigned char *p; unsigned int etuple; - unsigned int ret; etuple = cpuid_eax(0x80000001); @@ -500,8 +499,7 @@ static int powernow_decode_bios(int maxfid, int startvid) (startvid == pst->startvid)) { print_pst_entry(pst, j); p = (char *)pst + sizeof(*pst); - ret = get_ranges(p); - return ret; + return get_ranges(p); } else { unsigned int k; p = (char *)pst + sizeof(*pst); -- 2.34.1
|  |