diff options
author | Peter Ujfalusi <peter.ujfalusi@ti.com> | 2012-12-21 01:44:00 -0800 |
---|---|---|
committer | Bryan Wu <cooloney@gmail.com> | 2013-02-01 17:47:05 -0800 |
commit | 8a66a579083a20172a46e74d175a57621dccae0a (patch) | |
tree | e86f30b35852a0237ad64b7eed227f5edada1aa8 | |
parent | 261a5edd3ac77ecb4b33310a1dd1ed8d656f0569 (diff) | |
download | talos-op-linux-8a66a579083a20172a46e74d175a57621dccae0a.tar.gz talos-op-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; } |