diff options
| author | Raphael Assenat <raph@8d.com> | 2008-01-13 22:55:18 +0000 |
|---|---|---|
| committer | Richard Purdie <rpurdie@rpsys.net> | 2008-02-07 09:52:02 +0000 |
| commit | 2b7f1b8c8f4a8991dfeefc31844e15c642b6de2a (patch) | |
| tree | ac6a11ae2960b695ac1241888216b04646992440 | |
| parent | 92e015cb31c0a312bf2e0e5b96aef76a8c57e645 (diff) | |
| download | op-kernel-dev-2b7f1b8c8f4a8991dfeefc31844e15c642b6de2a.zip op-kernel-dev-2b7f1b8c8f4a8991dfeefc31844e15c642b6de2a.tar.gz | |
leds: Fix led-gpio active_low default brightness
When gpio_direction_output() is called, led_dat->active_low is used
as default value. This means that the led will always be off by
default. cdev.brightness should really have been set to LED_OFF
unconditionally to reflect this behavior.
Signed-off-by: Raphael Assenat <raph@8d.com>
Signed-off-by: Richard Purdie <rpurdie@rpsys.net>
| -rw-r--r-- | drivers/leds/leds-gpio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/leds/leds-gpio.c b/drivers/leds/leds-gpio.c index 99bc500..6c0a9c4 100644 --- a/drivers/leds/leds-gpio.c +++ b/drivers/leds/leds-gpio.c @@ -85,7 +85,7 @@ static int gpio_led_probe(struct platform_device *pdev) led_dat->can_sleep = gpio_cansleep(cur_led->gpio); led_dat->active_low = cur_led->active_low; led_dat->cdev.brightness_set = gpio_led_set; - led_dat->cdev.brightness = cur_led->active_low ? LED_FULL : LED_OFF; + led_dat->cdev.brightness = LED_OFF; ret = gpio_request(led_dat->gpio, led_dat->cdev.name); if (ret < 0) |
