2023-07-27 06:35:51

by claudiu beznea

Subject: [PATCH 18/42] clk: at91: at91rm9200: fix checkpatch.pl check

Fix the following checkpatch.pl check:
drivers/clk/at91/at91rm9200.c:216: CHECK: Please use a blank line after function/struct/union/enum declarations

Signed-off-by: Claudiu Beznea <[email protected]>
---
drivers/clk/at91/at91rm9200.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/clk/at91/at91rm9200.c b/drivers/clk/at91/at91rm9200.c
index a862a0a18972..e0418ab5d65c 100644
--- a/drivers/clk/at91/at91rm9200.c
+++ b/drivers/clk/at91/at91rm9200.c
@@ -222,6 +222,7 @@ static void __init at91rm9200_pmc_setup(struct device_node *np)
err_free:
kfree(at91rm9200_pmc);
}
+
/*
* While the TCB can be used as the clocksource, the system timer is most likely
* to be used instead. However, the pinctrl driver doesn't support probe
--
2.39.2

2023-07-27 09:21:42

by Krzysztof Kozlowski

Subject: Re: [PATCH 18/42] clk: at91: at91rm9200: fix checkpatch.pl check

On 27/07/2023 07:31, Claudiu Beznea wrote:
> Fix the following checkpatch.pl check:
> drivers/clk/at91/at91rm9200.c:216: CHECK: Please use a blank line after function/struct/union/enum declarations
>
> Signed-off-by: Claudiu Beznea <[email protected]>
> ---
> drivers/clk/at91/at91rm9200.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/at91/at91rm9200.c b/drivers/clk/at91/at91rm9200.c
> index a862a0a18972..e0418ab5d65c 100644
> --- a/drivers/clk/at91/at91rm9200.c
> +++ b/drivers/clk/at91/at91rm9200.c
> @@ -222,6 +222,7 @@ static void __init at91rm9200_pmc_setup(struct device_node *np)
> err_free:
> kfree(at91rm9200_pmc);
> }
> +
> /*

Again, one commit per one blank line added. This is just ridiculous.

Best regards,
Krzysztof