summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorUwe Kleine-König <uwe@kleine-koenig.org>2020-01-24 17:54:07 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2023-11-20 10:29:19 +0100
commit088c51a1067753ac74e3f1e1088863adaf99045c (patch)
tree2c5f337692e7d1bdcd414267699fd5138d9dca6e
parent58f31e4be996f43c30f1fd696e6b1a72ac8fdf4f (diff)
downloadlinux-stable-088c51a1067753ac74e3f1e1088863adaf99045c.tar.gz
linux-stable-088c51a1067753ac74e3f1e1088863adaf99045c.tar.bz2
linux-stable-088c51a1067753ac74e3f1e1088863adaf99045c.zip
leds: pwm: simplify if condition
[ Upstream commit b43a8f01fccbfdddbc7f9b2bbad11b7db3fda4e1 ] .pwm_period_ns is an unsigned integer. So when led->pwm_period_ns > 0 is false, we now assign 0 to a value that is already 0, so it doesn't hurt and we can skip checking the actual value. Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org> Tested-by: Jeff LaBundy <jeff@labundy.com> Signed-off-by: Pavel Machek <pavel@ucw.cz> Stable-dep-of: 76fe464c8e64 ("leds: pwm: Don't disable the PWM when the LED should be off") Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r--drivers/leds/leds-pwm.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/leds/leds-pwm.c b/drivers/leds/leds-pwm.c
index 5d3faae51d59..dc5c6100a419 100644
--- a/drivers/leds/leds-pwm.c
+++ b/drivers/leds/leds-pwm.c
@@ -117,7 +117,7 @@ static int led_pwm_add(struct device *dev, struct led_pwm_priv *priv,
pwm_get_args(led_data->pwm, &pargs);
led_data->period = pargs.period;
- if (!led_data->period && (led->pwm_period_ns > 0))
+ if (!led_data->period)
led_data->period = led->pwm_period_ns;
ret = led_classdev_register(dev, &led_data->cdev);