diff mbox series

[2/2] pwm: sti: Remove unnecessary blank line

Message ID 20201111191449.409402-2-thierry.reding@gmail.com
State Accepted
Headers show
Series [1/2] pwm: sti: Avoid conditional gotos | expand

Commit Message

Thierry Reding Nov. 11, 2020, 7:14 p.m. UTC
A single blank line is enough to separate logical code blocks.

Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
 drivers/pwm/pwm-sti.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Lee Jones Nov. 12, 2020, 8:59 a.m. UTC | #1
On Wed, 11 Nov 2020, Thierry Reding wrote:

> A single blank line is enough to separate logical code blocks.
> 
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> ---
>  drivers/pwm/pwm-sti.c | 1 -
>  1 file changed, 1 deletion(-)

Acked-by: Lee Jones <lee.jones@linaro.org>
Uwe Kleine-König Nov. 12, 2020, 11:04 a.m. UTC | #2
[dropped Ajit Pal Singh from Cc:]

On Wed, Nov 11, 2020 at 08:14:49PM +0100, Thierry Reding wrote:
> A single blank line is enough to separate logical code blocks.
> 
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>

I would consider this too minor to create a patch for, but as you
already invested the time:

Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Thanks
Uwe
diff mbox series

Patch

diff --git a/drivers/pwm/pwm-sti.c b/drivers/pwm/pwm-sti.c
index c6c82724d327..ec667b31d7ec 100644
--- a/drivers/pwm/pwm-sti.c
+++ b/drivers/pwm/pwm-sti.c
@@ -505,7 +505,6 @@  static int sti_pwm_probe_dt(struct sti_pwm_chip *pc)
 	if (IS_ERR(pc->prescale_high))
 		return PTR_ERR(pc->prescale_high);
 
-
 	pc->pwm_out_en = devm_regmap_field_alloc(dev, pc->regmap,
 						 reg_fields[PWM_OUT_EN]);
 	if (IS_ERR(pc->pwm_out_en))