cpuidle: Correct some typos in comments

[PATCH v1] cpuidle: Correct some typos in comments

Posted by Shen Lichuan 1 year, 4 months ago

Fixed some confusing typos that were currently identified with codespell,
the details are as follows:

-in the code comments:
drivers/cpuidle/cpuidle-arm.c:142: registeration ==> registration
drivers/cpuidle/cpuidle-qcom-spm.c:51: accidently ==> accidentally
drivers/cpuidle/cpuidle.c:409: dependant ==> dependent
drivers/cpuidle/driver.c:264: occuring ==> occurring
drivers/cpuidle/driver.c:299: occuring ==> occurring

Signed-off-by: Shen Lichuan <shenlichuan@vivo.com>
---
 drivers/cpuidle/cpuidle-arm.c      | 2 +-
 drivers/cpuidle/cpuidle-qcom-spm.c | 2 +-
 drivers/cpuidle/cpuidle.c          | 2 +-
 drivers/cpuidle/driver.c           | 4 ++--
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/cpuidle/cpuidle-arm.c b/drivers/cpuidle/cpuidle-arm.c
index 7cfb980a357d..caba6f4bb1b7 100644
--- a/drivers/cpuidle/cpuidle-arm.c
+++ b/drivers/cpuidle/cpuidle-arm.c
@@ -139,7 +139,7 @@ static int __init arm_idle_init_cpu(int cpu)
  *
  * Initializes arm cpuidle driver for all CPUs, if any CPU fails
  * to register cpuidle driver then rollback to cancel all CPUs
- * registeration.
+ * registration.
  */
 static int __init arm_idle_init(void)
 {
diff --git a/drivers/cpuidle/cpuidle-qcom-spm.c b/drivers/cpuidle/cpuidle-qcom-spm.c
index 1fc9968eae19..3ab240e0e122 100644
--- a/drivers/cpuidle/cpuidle-qcom-spm.c
+++ b/drivers/cpuidle/cpuidle-qcom-spm.c
@@ -48,7 +48,7 @@ static int qcom_cpu_spc(struct spm_driver_data *drv)
 	ret = cpu_suspend(0, qcom_pm_collapse);
 	/*
 	 * ARM common code executes WFI without calling into our driver and
-	 * if the SPM mode is not reset, then we may accidently power down the
+	 * if the SPM mode is not reset, then we may accidentally power down the
 	 * cpu when we intended only to gate the cpu clock.
 	 * Ensure the state is set to standby before returning.
 	 */
diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c
index 9e418aec1755..06ace16f9e71 100644
--- a/drivers/cpuidle/cpuidle.c
+++ b/drivers/cpuidle/cpuidle.c
@@ -406,7 +406,7 @@ void cpuidle_reflect(struct cpuidle_device *dev, int index)
  * Min polling interval of 10usec is a guess. It is assuming that
  * for most users, the time for a single ping-pong workload like
  * perf bench pipe would generally complete within 10usec but
- * this is hardware dependant. Actual time can be estimated with
+ * this is hardware dependent. Actual time can be estimated with
  *
  * perf bench sched pipe -l 10000
  *
diff --git a/drivers/cpuidle/driver.c b/drivers/cpuidle/driver.c
index cf5873cc45dc..9bbfa594c442 100644
--- a/drivers/cpuidle/driver.c
+++ b/drivers/cpuidle/driver.c
@@ -261,7 +261,7 @@ static void __cpuidle_unregister_driver(struct cpuidle_driver *drv)
  * @drv: a pointer to a valid struct cpuidle_driver
  *
  * Register the driver under a lock to prevent concurrent attempts to
- * [un]register the driver from occuring at the same time.
+ * [un]register the driver from occurring at the same time.
  *
  * Returns 0 on success, a negative error code (returned by
  * __cpuidle_register_driver()) otherwise.
@@ -296,7 +296,7 @@ EXPORT_SYMBOL_GPL(cpuidle_register_driver);
  * @drv: a pointer to a valid struct cpuidle_driver
  *
  * Unregisters the cpuidle driver under a lock to prevent concurrent attempts
- * to [un]register the driver from occuring at the same time.  @drv has to
+ * to [un]register the driver from occurring at the same time.  @drv has to
  * match the currently registered driver.
  */
 void cpuidle_unregister_driver(struct cpuidle_driver *drv)
-- 
2.17.1

Re: [PATCH v1] cpuidle: Correct some typos in comments

Posted by Rafael J. Wysocki 1 year, 3 months ago

On Fri, Sep 27, 2024 at 10:10 AM Shen Lichuan <shenlichuan@vivo.com> wrote:
>
> Fixed some confusing typos that were currently identified with codespell,
> the details are as follows:
>
> -in the code comments:
> drivers/cpuidle/cpuidle-arm.c:142: registeration ==> registration
> drivers/cpuidle/cpuidle-qcom-spm.c:51: accidently ==> accidentally
> drivers/cpuidle/cpuidle.c:409: dependant ==> dependent
> drivers/cpuidle/driver.c:264: occuring ==> occurring
> drivers/cpuidle/driver.c:299: occuring ==> occurring
>
> Signed-off-by: Shen Lichuan <shenlichuan@vivo.com>
> ---
>  drivers/cpuidle/cpuidle-arm.c      | 2 +-
>  drivers/cpuidle/cpuidle-qcom-spm.c | 2 +-
>  drivers/cpuidle/cpuidle.c          | 2 +-
>  drivers/cpuidle/driver.c           | 4 ++--
>  4 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/cpuidle/cpuidle-arm.c b/drivers/cpuidle/cpuidle-arm.c
> index 7cfb980a357d..caba6f4bb1b7 100644
> --- a/drivers/cpuidle/cpuidle-arm.c
> +++ b/drivers/cpuidle/cpuidle-arm.c
> @@ -139,7 +139,7 @@ static int __init arm_idle_init_cpu(int cpu)
>   *
>   * Initializes arm cpuidle driver for all CPUs, if any CPU fails
>   * to register cpuidle driver then rollback to cancel all CPUs
> - * registeration.
> + * registration.
>   */
>  static int __init arm_idle_init(void)
>  {
> diff --git a/drivers/cpuidle/cpuidle-qcom-spm.c b/drivers/cpuidle/cpuidle-qcom-spm.c
> index 1fc9968eae19..3ab240e0e122 100644
> --- a/drivers/cpuidle/cpuidle-qcom-spm.c
> +++ b/drivers/cpuidle/cpuidle-qcom-spm.c
> @@ -48,7 +48,7 @@ static int qcom_cpu_spc(struct spm_driver_data *drv)
>         ret = cpu_suspend(0, qcom_pm_collapse);
>         /*
>          * ARM common code executes WFI without calling into our driver and
> -        * if the SPM mode is not reset, then we may accidently power down the
> +        * if the SPM mode is not reset, then we may accidentally power down the
>          * cpu when we intended only to gate the cpu clock.
>          * Ensure the state is set to standby before returning.
>          */
> diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c
> index 9e418aec1755..06ace16f9e71 100644
> --- a/drivers/cpuidle/cpuidle.c
> +++ b/drivers/cpuidle/cpuidle.c
> @@ -406,7 +406,7 @@ void cpuidle_reflect(struct cpuidle_device *dev, int index)
>   * Min polling interval of 10usec is a guess. It is assuming that
>   * for most users, the time for a single ping-pong workload like
>   * perf bench pipe would generally complete within 10usec but
> - * this is hardware dependant. Actual time can be estimated with
> + * this is hardware dependent. Actual time can be estimated with
>   *
>   * perf bench sched pipe -l 10000
>   *
> diff --git a/drivers/cpuidle/driver.c b/drivers/cpuidle/driver.c
> index cf5873cc45dc..9bbfa594c442 100644
> --- a/drivers/cpuidle/driver.c
> +++ b/drivers/cpuidle/driver.c
> @@ -261,7 +261,7 @@ static void __cpuidle_unregister_driver(struct cpuidle_driver *drv)
>   * @drv: a pointer to a valid struct cpuidle_driver
>   *
>   * Register the driver under a lock to prevent concurrent attempts to
> - * [un]register the driver from occuring at the same time.
> + * [un]register the driver from occurring at the same time.
>   *
>   * Returns 0 on success, a negative error code (returned by
>   * __cpuidle_register_driver()) otherwise.
> @@ -296,7 +296,7 @@ EXPORT_SYMBOL_GPL(cpuidle_register_driver);
>   * @drv: a pointer to a valid struct cpuidle_driver
>   *
>   * Unregisters the cpuidle driver under a lock to prevent concurrent attempts
> - * to [un]register the driver from occuring at the same time.  @drv has to
> + * to [un]register the driver from occurring at the same time.  @drv has to
>   * match the currently registered driver.
>   */
>  void cpuidle_unregister_driver(struct cpuidle_driver *drv)
> --

Applied as 6.13 material, thanks!