diff options
| author | Peter Ujfalusi <peter.ujfalusi@ti.com> | 2012-12-21 10:44:00 +0100 |
|---|---|---|
| committer | Bryan Wu <cooloney@gmail.com> | 2013-02-02 02:47:05 +0100 |
| commit | 8a66a579083a20172a46e74d175a57621dccae0a (patch) | |
| tree | e86f30b35852a0237ad64b7eed227f5edada1aa8 | |
| parent | pwm: Add devm_of_pwm_get() as exported API for users (diff) | |
| download | linux-8a66a579083a20172a46e74d175a57621dccae0a.tar.xz linux-8a66a579083a20172a46e74d175a57621dccae0a.zip | |
leds: leds-pwm: Simplify cleanup code
The code looks more nicer if we use:
while (i--)
instead:
if (i > 0)
for (i = i - 1; i >= 0; i--)
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
| -rw-r--r-- | drivers/leds/leds-pwm.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/leds/leds-pwm.c b/drivers/leds/leds-pwm.c index c767837522fe..46f4e44c6c6d 100644 --- a/drivers/leds/leds-pwm.c +++ b/drivers/leds/leds-pwm.c @@ -104,10 +104,8 @@ static int led_pwm_probe(struct platform_device *pdev) return 0; err: - if (i > 0) { - for (i = i - 1; i >= 0; i--) - led_classdev_unregister(&priv->leds[i].cdev); - } + while (i--) + led_classdev_unregister(&priv->leds[i].cdev); return ret; } |