diff options
| author | Raphael Assenat <raph@8d.com> | 2008-01-13 17:55:18 -0500 |
|---|---|---|
| committer | Richard Purdie <rpurdie@rpsys.net> | 2008-02-07 04:52:02 -0500 |
| commit | 2b7f1b8c8f4a8991dfeefc31844e15c642b6de2a (patch) | |
| tree | ac6a11ae2960b695ac1241888216b04646992440 /drivers/leds | |
| parent | 92e015cb31c0a312bf2e0e5b96aef76a8c57e645 (diff) | |
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>
Diffstat (limited to 'drivers/leds')
| -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 99bc50059d35..6c0a9c4761ee 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) | |||
| 85 | led_dat->can_sleep = gpio_cansleep(cur_led->gpio); | 85 | led_dat->can_sleep = gpio_cansleep(cur_led->gpio); |
| 86 | led_dat->active_low = cur_led->active_low; | 86 | led_dat->active_low = cur_led->active_low; |
| 87 | led_dat->cdev.brightness_set = gpio_led_set; | 87 | led_dat->cdev.brightness_set = gpio_led_set; |
| 88 | led_dat->cdev.brightness = cur_led->active_low ? LED_FULL : LED_OFF; | 88 | led_dat->cdev.brightness = LED_OFF; |
| 89 | 89 | ||
| 90 | ret = gpio_request(led_dat->gpio, led_dat->cdev.name); | 90 | ret = gpio_request(led_dat->gpio, led_dat->cdev.name); |
| 91 | if (ret < 0) | 91 | if (ret < 0) |