| Message ID | 20191003213502.102110-1-mka@chromium.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | backlight: pwm_bl: Add missing curly branches in else branch | expand |
On Thu, Oct 03, 2019 at 02:35:02PM -0700, Matthias Kaehlcke wrote: > Add curly braces to an 'else' branch in pwm_backlight_update_status() > to match the corresponding 'if' branch. > > Signed-off-by: Matthias Kaehlcke <mka@chromium.org> Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org> > --- > > drivers/video/backlight/pwm_bl.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c > index 746eebc411df..130abff2705f 100644 > --- a/drivers/video/backlight/pwm_bl.c > +++ b/drivers/video/backlight/pwm_bl.c > @@ -125,8 +125,9 @@ static int pwm_backlight_update_status(struct backlight_device *bl) > state.duty_cycle = compute_duty_cycle(pb, brightness); > pwm_apply_state(pb->pwm, &state); > pwm_backlight_power_on(pb); > - } else > + } else { > pwm_backlight_power_off(pb); > + } > > if (pb->notify_after) > pb->notify_after(pb->dev, brightness); > -- > 2.23.0.444.g18eeb5a265-goog >
diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index 746eebc411df..130abff2705f 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -125,8 +125,9 @@ static int pwm_backlight_update_status(struct backlight_device *bl) state.duty_cycle = compute_duty_cycle(pb, brightness); pwm_apply_state(pb->pwm, &state); pwm_backlight_power_on(pb); - } else + } else { pwm_backlight_power_off(pb); + } if (pb->notify_after) pb->notify_after(pb->dev, brightness);
Add curly braces to an 'else' branch in pwm_backlight_update_status() to match the corresponding 'if' branch. Signed-off-by: Matthias Kaehlcke <mka@chromium.org> --- drivers/video/backlight/pwm_bl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)