linux-kernel - RE: [PATCH 2/2] clk: samsung: Fix typo error and extra space

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <003e01da381d$ae1b4240$0a51c6c0$@samsung.com>
Date: Tue, 26 Dec 2023 22:35:02 +0530
From: "Varada Pavani" <v.pavani@...sung.com>
To: "'Krzysztof Kozlowski'" <krzysztof.kozlowski@...aro.org>,
	<mturquette@...libre.com>, <sboyd@...nel.org>,
	<krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
	<linux-clk@...r.kernel.org>, <devicetree@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>, <s.nawrocki@...sung.com>,
	<tomasz.figa@...il.com>
Cc: <linux-samsung-soc@...r.kernel.org>, <alim.akhtar@...sung.com>,
	<aswani.reddy@...sung.com>, <pankaj.dubey@...sung.com>
Subject: RE: [PATCH 2/2] clk: samsung: Fix typo error and extra space

Hi Krzysztof,

-----Original Message-----
From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@...aro.org] 
Sent: 19 December 2023 17:36
To: Varada Pavani <v.pavani@...sung.com>; mturquette@...libre.com; sboyd@...nel.org; krzysztof.kozlowski+dt@...aro.org; conor+dt@...nel.org; linux-clk@...r.kernel.org; devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; s.nawrocki@...sung.com; tomasz.figa@...il.com
Cc: linux-samsung-soc@...r.kernel.org; alim.akhtar@...sung.com; aswani.reddy@...sung.com; pankaj.dubey@...sung.com
Subject: Re: [PATCH 2/2] clk: samsung: Fix typo error and extra space

On 19/12/2023 12:58, Varada Pavani wrote:
> Remove extra spaces and fix spelling mistakes in 'drivers/ 
> clk/samsung/clk-cpu.c' and 'drivers/clk/samsung/clk-cpu.h'
> 
> Signed-off-by: Varada Pavani <v.pavani@...sung.com>
> ---
>  drivers/clk/samsung/clk-cpu.c | 6 +++---  
> drivers/clk/samsung/clk-cpu.h | 2 +-
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/clk/samsung/clk-cpu.c 
> b/drivers/clk/samsung/clk-cpu.c index 3e62ade120c5..18568b8b1b9b 
> 100644
> --- a/drivers/clk/samsung/clk-cpu.c
> +++ b/drivers/clk/samsung/clk-cpu.c
> @@ -19,7 +19,7 @@
>   * clock and the corresponding rate changes of the auxillary clocks of the CPU
>   * domain. The platform clock driver provides a clock register configuration
>   * for each configurable rate which is then used to program the clock 
> hardware
> - * registers to acheive a fast co-oridinated rate change for all the 
> CPU domain
> + * registers to achieve a fast co-oridinated rate change for all the 
> + CPU domain
>   * clocks.
>   *
>   * On a rate change request for the CPU clock, the rate change is 
> propagated @@ -181,7 +181,7 @@ static int exynos_cpuclk_pre_rate_change(struct clk_notifier_data *ndata,
>  	 * If the old parent clock speed is less than the clock speed of
>  	 * the alternate parent, then it should be ensured that at no point
>  	 * the armclk speed is more than the old_prate until the dividers are
> -	 * set.  Also workaround the issue of the dividers being set to lower
> +	 * set. Also workaround the issue of the dividers being set to lower

Why? The double-space is correct.

Okay, I couldn’t see any double space at the end of the statement apart from this line and below one. So updated to have uniformity.


>  	 * values before the parent clock speed is set to new lower speed
>  	 * (this can result in too high speed of armclk output clocks).
>  	 */
> @@ -303,7 +303,7 @@ static int exynos5433_cpuclk_pre_rate_change(struct clk_notifier_data *ndata,
>  	 * If the old parent clock speed is less than the clock speed of
>  	 * the alternate parent, then it should be ensured that at no point
>  	 * the armclk speed is more than the old_prate until the dividers are
> -	 * set.  Also workaround the issue of the dividers being set to lower
> +	 * set. Also workaround the issue of the dividers being set to lower

Why?

Same as above.

>  	 * values before the parent clock speed is set to new lower speed
>  	 * (this can result in too high speed of armclk output clocks).
>  	 */
> diff --git a/drivers/clk/samsung/clk-cpu.h 
> b/drivers/clk/samsung/clk-cpu.h index fc9f67a3b22e..e0a1651174e6 
> 100644
> --- a/drivers/clk/samsung/clk-cpu.h
> +++ b/drivers/clk/samsung/clk-cpu.h
> @@ -33,7 +33,7 @@ struct exynos_cpuclk_cfg_data {
>   * @hw:	handle between CCF and CPU clock.
>   * @alt_parent: alternate parent clock to use when switching the speed
>   *	of the primary parent clock.
> - * @ctrl_base:	base address of the clock controller.
> + * @ctrl_base: base address of the clock controller.

Why only here and not in other places?

Will fix in V2 for this file.

Thanks for the review!

Best regards,
Krzysztof