* [PATCH 2/2] clk: samsung: Fix typo error and extra space
2023-12-19 11:58 ` [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml' Varada Pavani
@ 2023-12-19 11:58 ` Varada Pavani
2023-12-19 12:06 ` Krzysztof Kozlowski
2023-12-19 12:07 ` [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml' Krzysztof Kozlowski
` (2 subsequent siblings)
3 siblings, 1 reply; 10+ messages in thread
From: Varada Pavani @ 2023-12-19 11:58 UTC (permalink / raw)
To: mturquette, sboyd, krzysztof.kozlowski+dt, conor+dt, linux-clk,
devicetree, linux-kernel, s.nawrocki, tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey,
Varada Pavani
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@samsung.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
* 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
* 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.
* @lock: cpu clock domain register access lock.
* @cfg: cpu clock rate configuration data.
* @num_cfgs: number of array elements in @cfg array.
--
2.17.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH 2/2] clk: samsung: Fix typo error and extra space
2023-12-19 11:58 ` [PATCH 2/2] clk: samsung: Fix typo error and extra space Varada Pavani
@ 2023-12-19 12:06 ` Krzysztof Kozlowski
2023-12-26 17:05 ` Varada Pavani
0 siblings, 1 reply; 10+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-19 12:06 UTC (permalink / raw)
To: Varada Pavani, mturquette, sboyd, krzysztof.kozlowski+dt,
conor+dt, linux-clk, devicetree, linux-kernel, s.nawrocki,
tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
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@samsung.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.
> * 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?
> * 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?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread* RE: [PATCH 2/2] clk: samsung: Fix typo error and extra space
2023-12-19 12:06 ` Krzysztof Kozlowski
@ 2023-12-26 17:05 ` Varada Pavani
2023-12-27 11:40 ` Krzysztof Kozlowski
0 siblings, 1 reply; 10+ messages in thread
From: Varada Pavani @ 2023-12-26 17:05 UTC (permalink / raw)
To: 'Krzysztof Kozlowski', mturquette, sboyd,
krzysztof.kozlowski+dt, conor+dt, linux-clk, devicetree,
linux-kernel, s.nawrocki, tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
Hi Krzysztof,
-----Original Message-----
From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
Sent: 19 December 2023 17:36
To: Varada Pavani <v.pavani@samsung.com>; mturquette@baylibre.com; sboyd@kernel.org; krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; linux-clk@vger.kernel.org; devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; s.nawrocki@samsung.com; tomasz.figa@gmail.com
Cc: linux-samsung-soc@vger.kernel.org; alim.akhtar@samsung.com; aswani.reddy@samsung.com; pankaj.dubey@samsung.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@samsung.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
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 2/2] clk: samsung: Fix typo error and extra space
2023-12-26 17:05 ` Varada Pavani
@ 2023-12-27 11:40 ` Krzysztof Kozlowski
0 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-27 11:40 UTC (permalink / raw)
To: Varada Pavani, mturquette, sboyd, krzysztof.kozlowski+dt,
conor+dt, linux-clk, devicetree, linux-kernel, s.nawrocki,
tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
On 26/12/2023 18:05, Varada Pavani wrote:
>> 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.
Wait, there are only two statements here, so where do you expect double
space? There is no "one space at the end of the statement".
Your quoting is still not correct. Which part of above is quote of my
message, which yours?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml'
2023-12-19 11:58 ` [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml' Varada Pavani
2023-12-19 11:58 ` [PATCH 2/2] clk: samsung: Fix typo error and extra space Varada Pavani
@ 2023-12-19 12:07 ` Krzysztof Kozlowski
2023-12-26 16:57 ` Varada Pavani
2024-01-23 8:21 ` Krzysztof Kozlowski
2024-01-23 9:54 ` (subset) " Krzysztof Kozlowski
3 siblings, 1 reply; 10+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-19 12:07 UTC (permalink / raw)
To: Varada Pavani, mturquette, sboyd, krzysztof.kozlowski+dt,
conor+dt, linux-clk, devicetree, linux-kernel, s.nawrocki,
tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
On 19/12/2023 12:58, Varada Pavani wrote:
> Fix typo 'inteernal' to 'internal' in 'Documentation/devicetree/
> bindings/clock/tesla,fsd-clock.yaml'.
>
> Signed-off-by: Varada Pavani <v.pavani@samsung.com>
> ---
> Documentation/devicetree/bindings/clock/tesla,fsd-clock.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Are there any other typos in that file or in other Samsung clock bindings?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml'
2023-12-19 12:07 ` [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml' Krzysztof Kozlowski
@ 2023-12-26 16:57 ` Varada Pavani
2023-12-26 18:57 ` Krzysztof Kozlowski
0 siblings, 1 reply; 10+ messages in thread
From: Varada Pavani @ 2023-12-26 16:57 UTC (permalink / raw)
To: 'Krzysztof Kozlowski', mturquette, sboyd,
krzysztof.kozlowski+dt, conor+dt, linux-clk, devicetree,
linux-kernel, s.nawrocki, tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
Hi Krzysztof,
-----Original Message-----
From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
Sent: 19 December 2023 17:37
To: Varada Pavani <v.pavani@samsung.com>; mturquette@baylibre.com; sboyd@kernel.org; krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; linux-clk@vger.kernel.org; devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; s.nawrocki@samsung.com; tomasz.figa@gmail.com
Cc: linux-samsung-soc@vger.kernel.org; alim.akhtar@samsung.com; aswani.reddy@samsung.com; pankaj.dubey@samsung.com
Subject: Re: [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml'
On 19/12/2023 12:58, Varada Pavani wrote:
> Fix typo 'inteernal' to 'internal' in 'Documentation/devicetree/
> bindings/clock/tesla,fsd-clock.yaml'.
>
> Signed-off-by: Varada Pavani <v.pavani@samsung.com>
> ---
> Documentation/devicetree/bindings/clock/tesla,fsd-clock.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Are there any other typos in that file or in other Samsung clock bindings?
I couldn’t find any other typos in current file and also in other Samsung clock bindings.
Thanks for the review!
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml'
2023-12-26 16:57 ` Varada Pavani
@ 2023-12-26 18:57 ` Krzysztof Kozlowski
0 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-26 18:57 UTC (permalink / raw)
To: Varada Pavani, mturquette, sboyd, krzysztof.kozlowski+dt,
conor+dt, linux-clk, devicetree, linux-kernel, s.nawrocki,
tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
On 26/12/2023 17:57, Varada Pavani wrote:
>
> Are there any other typos in that file or in other Samsung clock bindings?
>
> I couldn’t find any other typos in current file and also in other Samsung clock bindings.
OK, that's fine. I will take it after the merge window.
You need to fix your email program, so it will properly mark quotes.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml'
2023-12-19 11:58 ` [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml' Varada Pavani
2023-12-19 11:58 ` [PATCH 2/2] clk: samsung: Fix typo error and extra space Varada Pavani
2023-12-19 12:07 ` [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml' Krzysztof Kozlowski
@ 2024-01-23 8:21 ` Krzysztof Kozlowski
2024-01-23 9:54 ` (subset) " Krzysztof Kozlowski
3 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-23 8:21 UTC (permalink / raw)
To: Varada Pavani, mturquette, sboyd, krzysztof.kozlowski+dt,
conor+dt, linux-clk, devicetree, linux-kernel, s.nawrocki,
tomasz.figa
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
On 19/12/2023 12:58, Varada Pavani wrote:
> Fix typo 'inteernal' to 'internal' in 'Documentation/devicetree/
> bindings/clock/tesla,fsd-clock.yaml'.
>
> Signed-off-by: Varada Pavani <v.pavani@samsung.com>
> ---
> Documentation/devicetree/bindings/clock/tesla,fsd-clock.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied with subject fixed.
Please use subject prefixes matching the subsystem. You can get them for
example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
your patch is touching.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: (subset) [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml'
2023-12-19 11:58 ` [PATCH 1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml' Varada Pavani
` (2 preceding siblings ...)
2024-01-23 8:21 ` Krzysztof Kozlowski
@ 2024-01-23 9:54 ` Krzysztof Kozlowski
3 siblings, 0 replies; 10+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-23 9:54 UTC (permalink / raw)
To: mturquette, sboyd, krzysztof.kozlowski+dt, conor+dt, linux-clk,
devicetree, linux-kernel, s.nawrocki, tomasz.figa, Varada Pavani
Cc: linux-samsung-soc, alim.akhtar, aswani.reddy, pankaj.dubey
On Tue, 19 Dec 2023 17:28:33 +0530, Varada Pavani wrote:
> Fix typo 'inteernal' to 'internal' in 'Documentation/devicetree/
> bindings/clock/tesla,fsd-clock.yaml'.
>
>
Applied, thanks!
[1/2] dt-bindings: clock: Fix spelling mistake in 'tesla,fsd-clock.yaml'
https://git.kernel.org/krzk/linux/c/5b62ea0b85ea8e11945f862fd2eee16ac3a45d4b
Best regards,
--
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
^ permalink raw reply [flat|nested] 10+ messages in thread