2022-06-24 07:11:23

by Xiang wangx

Subject: [PATCH] cpupower: Fix typo in comment

Delete the redundant word 'cpu'.

Signed-off-by: Xiang wangx <[email protected]>
---
tools/power/cpupower/bench/system.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/power/cpupower/bench/system.c b/tools/power/cpupower/bench/system.c
index 40f3679e70b5..eb79dd9ac670 100644
--- a/tools/power/cpupower/bench/system.c
+++ b/tools/power/cpupower/bench/system.c
@@ -37,7 +37,7 @@ long long int get_time()
* sets the cpufreq governor
*
* @param governor cpufreq governor name
- * @param cpu cpu for which the governor should be set
+ * @param cpu for which the governor should be set
*
* @retval 0 on success
* @retval -1 when failed
--
2.36.1

2022-06-24 14:35:46

by Shuah Khan

Subject: Re: [PATCH] cpupower: Fix typo in comment

On 6/24/22 12:55 AM, Xiang wangx wrote:
> Delete the redundant word 'cpu'.
>
> Signed-off-by: Xiang wangx <[email protected]>
> ---
> tools/power/cpupower/bench/system.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/power/cpupower/bench/system.c b/tools/power/cpupower/bench/system.c
> index 40f3679e70b5..eb79dd9ac670 100644
> --- a/tools/power/cpupower/bench/system.c
> +++ b/tools/power/cpupower/bench/system.c
> @@ -37,7 +37,7 @@ long long int get_time()
> * sets the cpufreq governor
> *
> * @param governor cpufreq governor name
> - * @param cpu cpu for which the governor should be set
> + * @param cpu for which the governor should be set

This is correct the way it is. It specifies the name of the parameter
and describes what it is.

> *
> * @retval 0 on success
> * @retval -1 when failed
>

There is nothing to fix here.

thanks,
-- Shuah