diff options
51 files changed, 2376 insertions, 413 deletions
diff --git a/Documentation/leds/00-INDEX b/Documentation/leds/00-INDEX index 29f481df32c7..5fefe374892f 100644 --- a/Documentation/leds/00-INDEX +++ b/Documentation/leds/00-INDEX | |||
@@ -6,3 +6,5 @@ leds-lp5521.txt | |||
6 | - notes on how to use the leds-lp5521 driver. | 6 | - notes on how to use the leds-lp5521 driver. |
7 | leds-lp5523.txt | 7 | leds-lp5523.txt |
8 | - notes on how to use the leds-lp5523 driver. | 8 | - notes on how to use the leds-lp5523 driver. |
9 | leds-lm3556.txt | ||
10 | - notes on how to use the leds-lm3556 driver. | ||
diff --git a/Documentation/leds/leds-blinkm.txt b/Documentation/leds/leds-blinkm.txt new file mode 100644 index 000000000000..9dd92f4cf4e1 --- /dev/null +++ b/Documentation/leds/leds-blinkm.txt | |||
@@ -0,0 +1,80 @@ | |||
1 | The leds-blinkm driver supports the devices of the BlinkM family. | ||
2 | |||
3 | They are RGB-LED modules driven by a (AT)tiny microcontroller and | ||
4 | communicate through I2C. The default address of these modules is | ||
5 | 0x09 but this can be changed through a command. By this you could | ||
6 | dasy-chain up to 127 BlinkMs on an I2C bus. | ||
7 | |||
8 | The device accepts RGB and HSB color values through separate commands. | ||
9 | Also you can store blinking sequences as "scripts" in | ||
10 | the controller and run them. Also fading is an option. | ||
11 | |||
12 | The interface this driver provides is 2-fold: | ||
13 | |||
14 | a) LED class interface for use with triggers | ||
15 | ############################################ | ||
16 | |||
17 | The registration follows the scheme: | ||
18 | blinkm-<i2c-bus-nr>-<i2c-device-nr>-<color> | ||
19 | |||
20 | $ ls -h /sys/class/leds/blinkm-6-* | ||
21 | /sys/class/leds/blinkm-6-9-blue: | ||
22 | brightness device max_brightness power subsystem trigger uevent | ||
23 | |||
24 | /sys/class/leds/blinkm-6-9-green: | ||
25 | brightness device max_brightness power subsystem trigger uevent | ||
26 | |||
27 | /sys/class/leds/blinkm-6-9-red: | ||
28 | brightness device max_brightness power subsystem trigger uevent | ||
29 | |||
30 | (same is /sys/bus/i2c/devices/6-0009/leds) | ||
31 | |||
32 | We can control the colors separated into red, green and blue and | ||
33 | assign triggers on each color. | ||
34 | |||
35 | E.g.: | ||
36 | |||
37 | $ cat blinkm-6-9-blue/brightness | ||
38 | 05 | ||
39 | |||
40 | $ echo 200 > blinkm-6-9-blue/brightness | ||
41 | $ | ||
42 | |||
43 | $ modprobe ledtrig-heartbeat | ||
44 | $ echo heartbeat > blinkm-6-9-green/trigger | ||
45 | $ | ||
46 | |||
47 | |||
48 | b) Sysfs group to control rgb, fade, hsb, scripts ... | ||
49 | ##################################################### | ||
50 | |||
51 | This extended interface is available as folder blinkm | ||
52 | in the sysfs folder of the I2C device. | ||
53 | E.g. below /sys/bus/i2c/devices/6-0009/blinkm | ||
54 | |||
55 | $ ls -h /sys/bus/i2c/devices/6-0009/blinkm/ | ||
56 | blue green red test | ||
57 | |||
58 | Currently supported is just setting red, green, blue | ||
59 | and a test sequence. | ||
60 | |||
61 | E.g.: | ||
62 | |||
63 | $ cat * | ||
64 | 00 | ||
65 | 00 | ||
66 | 00 | ||
67 | #Write into test to start test sequence!# | ||
68 | |||
69 | $ echo 1 > test | ||
70 | $ | ||
71 | |||
72 | $ echo 255 > red | ||
73 | $ | ||
74 | |||
75 | |||
76 | |||
77 | as of 6/2012 | ||
78 | |||
79 | dl9pf <at> gmx <dot> de | ||
80 | |||
diff --git a/Documentation/leds/leds-lm3556.txt b/Documentation/leds/leds-lm3556.txt new file mode 100644 index 000000000000..d9eb91b51913 --- /dev/null +++ b/Documentation/leds/leds-lm3556.txt | |||
@@ -0,0 +1,85 @@ | |||
1 | Kernel driver for lm3556 | ||
2 | ======================== | ||
3 | |||
4 | *Texas Instrument: | ||
5 | 1.5 A Synchronous Boost LED Flash Driver w/ High-Side Current Source | ||
6 | * Datasheet: http://www.national.com/ds/LM/LM3556.pdf | ||
7 | |||
8 | Authors: | ||
9 | Daniel Jeong | ||
10 | Contact:Daniel Jeong(daniel.jeong-at-ti.com, gshark.jeong-at-gmail.com) | ||
11 | |||
12 | Description | ||
13 | ----------- | ||
14 | There are 3 functions in LM3556, Flash, Torch and Indicator. | ||
15 | |||
16 | FLASH MODE | ||
17 | In Flash Mode, the LED current source(LED) provides 16 target current levels | ||
18 | from 93.75 mA to 1500 mA.The Flash currents are adjusted via the CURRENT | ||
19 | CONTROL REGISTER(0x09).Flash mode is activated by the ENABLE REGISTER(0x0A), | ||
20 | or by pulling the STROBE pin HIGH. | ||
21 | LM3556 Flash can be controlled through sys/class/leds/flash/brightness file | ||
22 | * if STROBE pin is enabled, below example control brightness only, and | ||
23 | ON / OFF will be controlled by STROBE pin. | ||
24 | |||
25 | Flash Example: | ||
26 | OFF : #echo 0 > sys/class/leds/flash/brightness | ||
27 | 93.75 mA: #echo 1 > sys/class/leds/flash/brightness | ||
28 | ... ..... | ||
29 | 1500 mA: #echo 16 > sys/class/leds/flash/brightness | ||
30 | |||
31 | TORCH MODE | ||
32 | In Torch Mode, the current source(LED) is programmed via the CURRENT CONTROL | ||
33 | REGISTER(0x09).Torch Mode is activated by the ENABLE REGISTER(0x0A) or by the | ||
34 | hardware TORCH input. | ||
35 | LM3556 torch can be controlled through sys/class/leds/torch/brightness file. | ||
36 | * if TORCH pin is enabled, below example control brightness only, | ||
37 | and ON / OFF will be controlled by TORCH pin. | ||
38 | |||
39 | Torch Example: | ||
40 | OFF : #echo 0 > sys/class/leds/torch/brightness | ||
41 | 46.88 mA: #echo 1 > sys/class/leds/torch/brightness | ||
42 | ... ..... | ||
43 | 375 mA : #echo 8 > sys/class/leds/torch/brightness | ||
44 | |||
45 | INDICATOR MODE | ||
46 | Indicator pattern can be set through sys/class/leds/indicator/pattern file, | ||
47 | and 4 patterns are pre-defined in indicator_pattern array. | ||
48 | According to N-lank, Pulse time and N Period values, different pattern wiill | ||
49 | be generated.If you want new patterns for your own device, change | ||
50 | indicator_pattern array with your own values and INDIC_PATTERN_SIZE. | ||
51 | Please refer datasheet for more detail about N-Blank, Pulse time and N Period. | ||
52 | |||
53 | Indicator pattern example: | ||
54 | pattern 0: #echo 0 > sys/class/leds/indicator/pattern | ||
55 | .... | ||
56 | pattern 3: #echo 3 > sys/class/leds/indicator/pattern | ||
57 | |||
58 | Indicator brightness can be controlled through | ||
59 | sys/class/leds/indicator/brightness file. | ||
60 | |||
61 | Example: | ||
62 | OFF : #echo 0 > sys/class/leds/indicator/brightness | ||
63 | 5.86 mA : #echo 1 > sys/class/leds/indicator/brightness | ||
64 | ........ | ||
65 | 46.875mA : #echo 8 > sys/class/leds/indicator/brightness | ||
66 | |||
67 | Notes | ||
68 | ----- | ||
69 | Driver expects it is registered using the i2c_board_info mechanism. | ||
70 | To register the chip at address 0x63 on specific adapter, set the platform data | ||
71 | according to include/linux/platform_data/leds-lm3556.h, set the i2c board info | ||
72 | |||
73 | Example: | ||
74 | static struct i2c_board_info __initdata board_i2c_ch4[] = { | ||
75 | { | ||
76 | I2C_BOARD_INFO(LM3556_NAME, 0x63), | ||
77 | .platform_data = &lm3556_pdata, | ||
78 | }, | ||
79 | }; | ||
80 | |||
81 | and register it in the platform init function | ||
82 | |||
83 | Example: | ||
84 | board_register_i2c_bus(4, 400, | ||
85 | board_i2c_ch4, ARRAY_SIZE(board_i2c_ch4)); | ||
diff --git a/Documentation/leds/ledtrig-oneshot.txt b/Documentation/leds/ledtrig-oneshot.txt new file mode 100644 index 000000000000..07cd1fa41a3a --- /dev/null +++ b/Documentation/leds/ledtrig-oneshot.txt | |||
@@ -0,0 +1,59 @@ | |||
1 | One-shot LED Trigger | ||
2 | ==================== | ||
3 | |||
4 | This is a LED trigger useful for signaling the user of an event where there are | ||
5 | no clear trap points to put standard led-on and led-off settings. Using this | ||
6 | trigger, the application needs only to signal the trigger when an event has | ||
7 | happened, than the trigger turns the LED on and than keeps it off for a | ||
8 | specified amount of time. | ||
9 | |||
10 | This trigger is meant to be usable both for sporadic and dense events. In the | ||
11 | first case, the trigger produces a clear single controlled blink for each | ||
12 | event, while in the latter it keeps blinking at constant rate, as to signal | ||
13 | that the events are arriving continuously. | ||
14 | |||
15 | A one-shot LED only stays in a constant state when there are no events. An | ||
16 | additional "invert" property specifies if the LED has to stay off (normal) or | ||
17 | on (inverted) when not rearmed. | ||
18 | |||
19 | The trigger can be activated from user space on led class devices as shown | ||
20 | below: | ||
21 | |||
22 | echo oneshot > trigger | ||
23 | |||
24 | This adds the following sysfs attributes to the LED: | ||
25 | |||
26 | delay_on - specifies for how many milliseconds the LED has to stay at | ||
27 | LED_FULL brightness after it has been armed. | ||
28 | Default to 100 ms. | ||
29 | |||
30 | delay_off - specifies for how many milliseconds the LED has to stay at | ||
31 | LED_OFF brightness after it has been armed. | ||
32 | Default to 100 ms. | ||
33 | |||
34 | invert - reverse the blink logic. If set to 0 (default) blink on for delay_on | ||
35 | ms, then blink off for delay_off ms, leaving the LED normally off. If | ||
36 | set to 1, blink off for delay_off ms, then blink on for delay_on ms, | ||
37 | leaving the LED normally on. | ||
38 | Setting this value also immediately change the LED state. | ||
39 | |||
40 | shot - write any non-empty string to signal an events, this starts a blink | ||
41 | sequence if not already running. | ||
42 | |||
43 | Example use-case: network devices, initialization: | ||
44 | |||
45 | echo oneshot > trigger # set trigger for this led | ||
46 | echo 33 > delay_on # blink at 1 / (33 + 33) Hz on continuous traffic | ||
47 | echo 33 > delay_off | ||
48 | |||
49 | interface goes up: | ||
50 | |||
51 | echo 1 > invert # set led as normally-on, turn the led on | ||
52 | |||
53 | packet received/transmitted: | ||
54 | |||
55 | echo 1 > shot # led starts blinking, ignored if already blinking | ||
56 | |||
57 | interface goes down | ||
58 | |||
59 | echo 0 > invert # set led as normally-off, turn the led off | ||
diff --git a/MAINTAINERS b/MAINTAINERS index 8f363b005226..3bc49687b653 100644 --- a/MAINTAINERS +++ b/MAINTAINERS | |||
@@ -1529,6 +1529,11 @@ W: http://blackfin.uclinux.org/ | |||
1529 | S: Supported | 1529 | S: Supported |
1530 | F: drivers/i2c/busses/i2c-bfin-twi.c | 1530 | F: drivers/i2c/busses/i2c-bfin-twi.c |
1531 | 1531 | ||
1532 | BLINKM RGB LED DRIVER | ||
1533 | M: Jan-Simon Moeller <jansimon.moeller@gmx.de> | ||
1534 | S: Maintained | ||
1535 | F: drivers/leds/leds-blinkm.c | ||
1536 | |||
1532 | BLOCK LAYER | 1537 | BLOCK LAYER |
1533 | M: Jens Axboe <axboe@kernel.dk> | 1538 | M: Jens Axboe <axboe@kernel.dk> |
1534 | T: git git://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git | 1539 | T: git git://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git |
diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig index 12b2b55c519e..c96bbaadeebd 100644 --- a/drivers/leds/Kconfig +++ b/drivers/leds/Kconfig | |||
@@ -200,6 +200,13 @@ config LEDS_LP5523 | |||
200 | Driver provides direct control via LED class and interface for | 200 | Driver provides direct control via LED class and interface for |
201 | programming the engines. | 201 | programming the engines. |
202 | 202 | ||
203 | config LEDS_LP8788 | ||
204 | tristate "LED support for the TI LP8788 PMIC" | ||
205 | depends on LEDS_CLASS | ||
206 | depends on MFD_LP8788 | ||
207 | help | ||
208 | This option enables support for the Keyboard LEDs on the LP8788 PMIC. | ||
209 | |||
203 | config LEDS_CLEVO_MAIL | 210 | config LEDS_CLEVO_MAIL |
204 | tristate "Mail LED on Clevo notebook" | 211 | tristate "Mail LED on Clevo notebook" |
205 | depends on LEDS_CLASS | 212 | depends on LEDS_CLASS |
@@ -415,6 +422,14 @@ config LEDS_MAX8997 | |||
415 | This option enables support for on-chip LED drivers on | 422 | This option enables support for on-chip LED drivers on |
416 | MAXIM MAX8997 PMIC. | 423 | MAXIM MAX8997 PMIC. |
417 | 424 | ||
425 | config LEDS_LM3556 | ||
426 | tristate "LED support for LM3556 Chip" | ||
427 | depends on LEDS_CLASS && I2C | ||
428 | select REGMAP_I2C | ||
429 | help | ||
430 | This option enables support for LEDs connected to LM3556. | ||
431 | LM3556 includes Torch, Flash and Indicator functions. | ||
432 | |||
418 | config LEDS_OT200 | 433 | config LEDS_OT200 |
419 | tristate "LED support for the Bachmann OT200" | 434 | tristate "LED support for the Bachmann OT200" |
420 | depends on LEDS_CLASS && HAS_IOMEM | 435 | depends on LEDS_CLASS && HAS_IOMEM |
@@ -422,6 +437,14 @@ config LEDS_OT200 | |||
422 | This option enables support for the LEDs on the Bachmann OT200. | 437 | This option enables support for the LEDs on the Bachmann OT200. |
423 | Say Y to enable LEDs on the Bachmann OT200. | 438 | Say Y to enable LEDs on the Bachmann OT200. |
424 | 439 | ||
440 | config LEDS_BLINKM | ||
441 | tristate "LED support for the BlinkM I2C RGB LED" | ||
442 | depends on LEDS_CLASS | ||
443 | depends on I2C | ||
444 | help | ||
445 | This option enables support for the BlinkM RGB LED connected | ||
446 | through I2C. Say Y to enable support for the BlinkM LED. | ||
447 | |||
425 | config LEDS_TRIGGERS | 448 | config LEDS_TRIGGERS |
426 | bool "LED Trigger support" | 449 | bool "LED Trigger support" |
427 | depends on LEDS_CLASS | 450 | depends on LEDS_CLASS |
@@ -443,6 +466,20 @@ config LEDS_TRIGGER_TIMER | |||
443 | 466 | ||
444 | If unsure, say Y. | 467 | If unsure, say Y. |
445 | 468 | ||
469 | config LEDS_TRIGGER_ONESHOT | ||
470 | tristate "LED One-shot Trigger" | ||
471 | depends on LEDS_TRIGGERS | ||
472 | help | ||
473 | This allows LEDs to blink in one-shot pulses with parameters | ||
474 | controlled via sysfs. It's useful to notify the user on | ||
475 | sporadic events, when there are no clear begin and end trap points, | ||
476 | or on dense events, where this blinks the LED at constant rate if | ||
477 | rearmed continuously. | ||
478 | |||
479 | It also shows how to use the led_blink_set_oneshot() function. | ||
480 | |||
481 | If unsure, say Y. | ||
482 | |||
446 | config LEDS_TRIGGER_IDE_DISK | 483 | config LEDS_TRIGGER_IDE_DISK |
447 | bool "LED IDE Disk Trigger" | 484 | bool "LED IDE Disk Trigger" |
448 | depends on IDE_GD_ATA | 485 | depends on IDE_GD_ATA |
@@ -497,7 +534,7 @@ config LEDS_TRIGGER_TRANSIENT | |||
497 | depends on LEDS_TRIGGERS | 534 | depends on LEDS_TRIGGERS |
498 | help | 535 | help |
499 | This allows one time activation of a transient state on | 536 | This allows one time activation of a transient state on |
500 | GPIO/PWM based hadrware. | 537 | GPIO/PWM based hardware. |
501 | If unsure, say Y. | 538 | If unsure, say Y. |
502 | 539 | ||
503 | endif # NEW_LEDS | 540 | endif # NEW_LEDS |
diff --git a/drivers/leds/Makefile b/drivers/leds/Makefile index f8958cd6cf6e..a4429a9217bc 100644 --- a/drivers/leds/Makefile +++ b/drivers/leds/Makefile | |||
@@ -24,6 +24,7 @@ obj-$(CONFIG_LEDS_GPIO) += leds-gpio.o | |||
24 | obj-$(CONFIG_LEDS_LP3944) += leds-lp3944.o | 24 | obj-$(CONFIG_LEDS_LP3944) += leds-lp3944.o |
25 | obj-$(CONFIG_LEDS_LP5521) += leds-lp5521.o | 25 | obj-$(CONFIG_LEDS_LP5521) += leds-lp5521.o |
26 | obj-$(CONFIG_LEDS_LP5523) += leds-lp5523.o | 26 | obj-$(CONFIG_LEDS_LP5523) += leds-lp5523.o |
27 | obj-$(CONFIG_LEDS_LP8788) += leds-lp8788.o | ||
27 | obj-$(CONFIG_LEDS_TCA6507) += leds-tca6507.o | 28 | obj-$(CONFIG_LEDS_TCA6507) += leds-tca6507.o |
28 | obj-$(CONFIG_LEDS_CLEVO_MAIL) += leds-clevo-mail.o | 29 | obj-$(CONFIG_LEDS_CLEVO_MAIL) += leds-clevo-mail.o |
29 | obj-$(CONFIG_LEDS_HP6XX) += leds-hp6xx.o | 30 | obj-$(CONFIG_LEDS_HP6XX) += leds-hp6xx.o |
@@ -47,12 +48,15 @@ obj-$(CONFIG_LEDS_NETXBIG) += leds-netxbig.o | |||
47 | obj-$(CONFIG_LEDS_ASIC3) += leds-asic3.o | 48 | obj-$(CONFIG_LEDS_ASIC3) += leds-asic3.o |
48 | obj-$(CONFIG_LEDS_RENESAS_TPU) += leds-renesas-tpu.o | 49 | obj-$(CONFIG_LEDS_RENESAS_TPU) += leds-renesas-tpu.o |
49 | obj-$(CONFIG_LEDS_MAX8997) += leds-max8997.o | 50 | obj-$(CONFIG_LEDS_MAX8997) += leds-max8997.o |
51 | obj-$(CONFIG_LEDS_LM3556) += leds-lm3556.o | ||
52 | obj-$(CONFIG_LEDS_BLINKM) += leds-blinkm.o | ||
50 | 53 | ||
51 | # LED SPI Drivers | 54 | # LED SPI Drivers |
52 | obj-$(CONFIG_LEDS_DAC124S085) += leds-dac124s085.o | 55 | obj-$(CONFIG_LEDS_DAC124S085) += leds-dac124s085.o |
53 | 56 | ||
54 | # LED Triggers | 57 | # LED Triggers |
55 | obj-$(CONFIG_LEDS_TRIGGER_TIMER) += ledtrig-timer.o | 58 | obj-$(CONFIG_LEDS_TRIGGER_TIMER) += ledtrig-timer.o |
59 | obj-$(CONFIG_LEDS_TRIGGER_ONESHOT) += ledtrig-oneshot.o | ||
56 | obj-$(CONFIG_LEDS_TRIGGER_IDE_DISK) += ledtrig-ide-disk.o | 60 | obj-$(CONFIG_LEDS_TRIGGER_IDE_DISK) += ledtrig-ide-disk.o |
57 | obj-$(CONFIG_LEDS_TRIGGER_HEARTBEAT) += ledtrig-heartbeat.o | 61 | obj-$(CONFIG_LEDS_TRIGGER_HEARTBEAT) += ledtrig-heartbeat.o |
58 | obj-$(CONFIG_LEDS_TRIGGER_BACKLIGHT) += ledtrig-backlight.o | 62 | obj-$(CONFIG_LEDS_TRIGGER_BACKLIGHT) += ledtrig-backlight.o |
diff --git a/drivers/leds/led-class.c b/drivers/leds/led-class.c index e663e6f413e9..c599095bc005 100644 --- a/drivers/leds/led-class.c +++ b/drivers/leds/led-class.c | |||
@@ -53,7 +53,7 @@ static ssize_t led_brightness_store(struct device *dev, | |||
53 | 53 | ||
54 | if (state == LED_OFF) | 54 | if (state == LED_OFF) |
55 | led_trigger_remove(led_cdev); | 55 | led_trigger_remove(led_cdev); |
56 | led_set_brightness(led_cdev, state); | 56 | __led_set_brightness(led_cdev, state); |
57 | 57 | ||
58 | return size; | 58 | return size; |
59 | } | 59 | } |
@@ -82,7 +82,12 @@ static void led_timer_function(unsigned long data) | |||
82 | unsigned long delay; | 82 | unsigned long delay; |
83 | 83 | ||
84 | if (!led_cdev->blink_delay_on || !led_cdev->blink_delay_off) { | 84 | if (!led_cdev->blink_delay_on || !led_cdev->blink_delay_off) { |
85 | led_set_brightness(led_cdev, LED_OFF); | 85 | __led_set_brightness(led_cdev, LED_OFF); |
86 | return; | ||
87 | } | ||
88 | |||
89 | if (led_cdev->flags & LED_BLINK_ONESHOT_STOP) { | ||
90 | led_cdev->flags &= ~LED_BLINK_ONESHOT_STOP; | ||
86 | return; | 91 | return; |
87 | } | 92 | } |
88 | 93 | ||
@@ -100,7 +105,21 @@ static void led_timer_function(unsigned long data) | |||
100 | delay = led_cdev->blink_delay_off; | 105 | delay = led_cdev->blink_delay_off; |
101 | } | 106 | } |
102 | 107 | ||
103 | led_set_brightness(led_cdev, brightness); | 108 | __led_set_brightness(led_cdev, brightness); |
109 | |||
110 | /* Return in next iteration if led is in one-shot mode and we are in | ||
111 | * the final blink state so that the led is toggled each delay_on + | ||
112 | * delay_off milliseconds in worst case. | ||
113 | */ | ||
114 | if (led_cdev->flags & LED_BLINK_ONESHOT) { | ||
115 | if (led_cdev->flags & LED_BLINK_INVERT) { | ||
116 | if (brightness) | ||
117 | led_cdev->flags |= LED_BLINK_ONESHOT_STOP; | ||
118 | } else { | ||
119 | if (!brightness) | ||
120 | led_cdev->flags |= LED_BLINK_ONESHOT_STOP; | ||
121 | } | ||
122 | } | ||
104 | 123 | ||
105 | mod_timer(&led_cdev->blink_timer, jiffies + msecs_to_jiffies(delay)); | 124 | mod_timer(&led_cdev->blink_timer, jiffies + msecs_to_jiffies(delay)); |
106 | } | 125 | } |
@@ -203,7 +222,7 @@ void led_classdev_unregister(struct led_classdev *led_cdev) | |||
203 | #endif | 222 | #endif |
204 | 223 | ||
205 | /* Stop blinking */ | 224 | /* Stop blinking */ |
206 | led_brightness_set(led_cdev, LED_OFF); | 225 | led_set_brightness(led_cdev, LED_OFF); |
207 | 226 | ||
208 | device_unregister(led_cdev->dev); | 227 | device_unregister(led_cdev->dev); |
209 | 228 | ||
diff --git a/drivers/leds/led-core.c b/drivers/leds/led-core.c index d65353d8d3fc..2ab05af3de31 100644 --- a/drivers/leds/led-core.c +++ b/drivers/leds/led-core.c | |||
@@ -24,14 +24,6 @@ EXPORT_SYMBOL_GPL(leds_list_lock); | |||
24 | LIST_HEAD(leds_list); | 24 | LIST_HEAD(leds_list); |
25 | EXPORT_SYMBOL_GPL(leds_list); | 25 | EXPORT_SYMBOL_GPL(leds_list); |
26 | 26 | ||
27 | static void led_stop_software_blink(struct led_classdev *led_cdev) | ||
28 | { | ||
29 | /* deactivate previous settings */ | ||
30 | del_timer_sync(&led_cdev->blink_timer); | ||
31 | led_cdev->blink_delay_on = 0; | ||
32 | led_cdev->blink_delay_off = 0; | ||
33 | } | ||
34 | |||
35 | static void led_set_software_blink(struct led_classdev *led_cdev, | 27 | static void led_set_software_blink(struct led_classdev *led_cdev, |
36 | unsigned long delay_on, | 28 | unsigned long delay_on, |
37 | unsigned long delay_off) | 29 | unsigned long delay_off) |
@@ -53,7 +45,7 @@ static void led_set_software_blink(struct led_classdev *led_cdev, | |||
53 | 45 | ||
54 | /* never off - just set to brightness */ | 46 | /* never off - just set to brightness */ |
55 | if (!delay_off) { | 47 | if (!delay_off) { |
56 | led_set_brightness(led_cdev, led_cdev->blink_brightness); | 48 | __led_set_brightness(led_cdev, led_cdev->blink_brightness); |
57 | return; | 49 | return; |
58 | } | 50 | } |
59 | 51 | ||
@@ -61,13 +53,12 @@ static void led_set_software_blink(struct led_classdev *led_cdev, | |||
61 | } | 53 | } |
62 | 54 | ||
63 | 55 | ||
64 | void led_blink_set(struct led_classdev *led_cdev, | 56 | static void led_blink_setup(struct led_classdev *led_cdev, |
65 | unsigned long *delay_on, | 57 | unsigned long *delay_on, |
66 | unsigned long *delay_off) | 58 | unsigned long *delay_off) |
67 | { | 59 | { |
68 | del_timer_sync(&led_cdev->blink_timer); | 60 | if (!(led_cdev->flags & LED_BLINK_ONESHOT) && |
69 | 61 | led_cdev->blink_set && | |
70 | if (led_cdev->blink_set && | ||
71 | !led_cdev->blink_set(led_cdev, delay_on, delay_off)) | 62 | !led_cdev->blink_set(led_cdev, delay_on, delay_off)) |
72 | return; | 63 | return; |
73 | 64 | ||
@@ -77,12 +68,49 @@ void led_blink_set(struct led_classdev *led_cdev, | |||
77 | 68 | ||
78 | led_set_software_blink(led_cdev, *delay_on, *delay_off); | 69 | led_set_software_blink(led_cdev, *delay_on, *delay_off); |
79 | } | 70 | } |
71 | |||
72 | void led_blink_set(struct led_classdev *led_cdev, | ||
73 | unsigned long *delay_on, | ||
74 | unsigned long *delay_off) | ||
75 | { | ||
76 | del_timer_sync(&led_cdev->blink_timer); | ||
77 | |||
78 | led_cdev->flags &= ~LED_BLINK_ONESHOT; | ||
79 | led_cdev->flags &= ~LED_BLINK_ONESHOT_STOP; | ||
80 | |||
81 | led_blink_setup(led_cdev, delay_on, delay_off); | ||
82 | } | ||
80 | EXPORT_SYMBOL(led_blink_set); | 83 | EXPORT_SYMBOL(led_blink_set); |
81 | 84 | ||
82 | void led_brightness_set(struct led_classdev *led_cdev, | 85 | void led_blink_set_oneshot(struct led_classdev *led_cdev, |
86 | unsigned long *delay_on, | ||
87 | unsigned long *delay_off, | ||
88 | int invert) | ||
89 | { | ||
90 | if ((led_cdev->flags & LED_BLINK_ONESHOT) && | ||
91 | timer_pending(&led_cdev->blink_timer)) | ||
92 | return; | ||
93 | |||
94 | led_cdev->flags |= LED_BLINK_ONESHOT; | ||
95 | led_cdev->flags &= ~LED_BLINK_ONESHOT_STOP; | ||
96 | |||
97 | if (invert) | ||
98 | led_cdev->flags |= LED_BLINK_INVERT; | ||
99 | else | ||
100 | led_cdev->flags &= ~LED_BLINK_INVERT; | ||
101 | |||
102 | led_blink_setup(led_cdev, delay_on, delay_off); | ||
103 | } | ||
104 | EXPORT_SYMBOL(led_blink_set_oneshot); | ||
105 | |||
106 | void led_set_brightness(struct led_classdev *led_cdev, | ||
83 | enum led_brightness brightness) | 107 | enum led_brightness brightness) |
84 | { | 108 | { |
85 | led_stop_software_blink(led_cdev); | 109 | /* stop and clear soft-blink timer */ |
86 | led_cdev->brightness_set(led_cdev, brightness); | 110 | del_timer_sync(&led_cdev->blink_timer); |
111 | led_cdev->blink_delay_on = 0; | ||
112 | led_cdev->blink_delay_off = 0; | ||
113 | |||
114 | __led_set_brightness(led_cdev, brightness); | ||
87 | } | 115 | } |
88 | EXPORT_SYMBOL(led_brightness_set); | 116 | EXPORT_SYMBOL(led_set_brightness); |
diff --git a/drivers/leds/led-triggers.c b/drivers/leds/led-triggers.c index 46b4c766335d..6157cbbf4113 100644 --- a/drivers/leds/led-triggers.c +++ b/drivers/leds/led-triggers.c | |||
@@ -99,7 +99,7 @@ ssize_t led_trigger_show(struct device *dev, struct device_attribute *attr, | |||
99 | EXPORT_SYMBOL_GPL(led_trigger_show); | 99 | EXPORT_SYMBOL_GPL(led_trigger_show); |
100 | 100 | ||
101 | /* Caller must ensure led_cdev->trigger_lock held */ | 101 | /* Caller must ensure led_cdev->trigger_lock held */ |
102 | void led_trigger_set(struct led_classdev *led_cdev, struct led_trigger *trigger) | 102 | void led_trigger_set(struct led_classdev *led_cdev, struct led_trigger *trig) |
103 | { | 103 | { |
104 | unsigned long flags; | 104 | unsigned long flags; |
105 | 105 | ||
@@ -112,15 +112,15 @@ void led_trigger_set(struct led_classdev *led_cdev, struct led_trigger *trigger) | |||
112 | if (led_cdev->trigger->deactivate) | 112 | if (led_cdev->trigger->deactivate) |
113 | led_cdev->trigger->deactivate(led_cdev); | 113 | led_cdev->trigger->deactivate(led_cdev); |
114 | led_cdev->trigger = NULL; | 114 | led_cdev->trigger = NULL; |
115 | led_brightness_set(led_cdev, LED_OFF); | 115 | led_set_brightness(led_cdev, LED_OFF); |
116 | } | 116 | } |
117 | if (trigger) { | 117 | if (trig) { |
118 | write_lock_irqsave(&trigger->leddev_list_lock, flags); | 118 | write_lock_irqsave(&trig->leddev_list_lock, flags); |
119 | list_add_tail(&led_cdev->trig_list, &trigger->led_cdevs); | 119 | list_add_tail(&led_cdev->trig_list, &trig->led_cdevs); |
120 | write_unlock_irqrestore(&trigger->leddev_list_lock, flags); | 120 | write_unlock_irqrestore(&trig->leddev_list_lock, flags); |
121 | led_cdev->trigger = trigger; | 121 | led_cdev->trigger = trig; |
122 | if (trigger->activate) | 122 | if (trig->activate) |
123 | trigger->activate(led_cdev); | 123 | trig->activate(led_cdev); |
124 | } | 124 | } |
125 | } | 125 | } |
126 | EXPORT_SYMBOL_GPL(led_trigger_set); | 126 | EXPORT_SYMBOL_GPL(led_trigger_set); |
@@ -153,24 +153,24 @@ EXPORT_SYMBOL_GPL(led_trigger_set_default); | |||
153 | 153 | ||
154 | /* LED Trigger Interface */ | 154 | /* LED Trigger Interface */ |
155 | 155 | ||
156 | int led_trigger_register(struct led_trigger *trigger) | 156 | int led_trigger_register(struct led_trigger *trig) |
157 | { | 157 | { |
158 | struct led_classdev *led_cdev; | 158 | struct led_classdev *led_cdev; |
159 | struct led_trigger *trig; | 159 | struct led_trigger *_trig; |
160 | 160 | ||
161 | rwlock_init(&trigger->leddev_list_lock); | 161 | rwlock_init(&trig->leddev_list_lock); |
162 | INIT_LIST_HEAD(&trigger->led_cdevs); | 162 | INIT_LIST_HEAD(&trig->led_cdevs); |
163 | 163 | ||
164 | down_write(&triggers_list_lock); | 164 | down_write(&triggers_list_lock); |
165 | /* Make sure the trigger's name isn't already in use */ | 165 | /* Make sure the trigger's name isn't already in use */ |
166 | list_for_each_entry(trig, &trigger_list, next_trig) { | 166 | list_for_each_entry(_trig, &trigger_list, next_trig) { |
167 | if (!strcmp(trig->name, trigger->name)) { | 167 | if (!strcmp(_trig->name, trig->name)) { |
168 | up_write(&triggers_list_lock); | 168 | up_write(&triggers_list_lock); |
169 | return -EEXIST; | 169 | return -EEXIST; |
170 | } | 170 | } |
171 | } | 171 | } |
172 | /* Add to the list of led triggers */ | 172 | /* Add to the list of led triggers */ |
173 | list_add_tail(&trigger->next_trig, &trigger_list); | 173 | list_add_tail(&trig->next_trig, &trigger_list); |
174 | up_write(&triggers_list_lock); | 174 | up_write(&triggers_list_lock); |
175 | 175 | ||
176 | /* Register with any LEDs that have this as a default trigger */ | 176 | /* Register with any LEDs that have this as a default trigger */ |
@@ -178,8 +178,8 @@ int led_trigger_register(struct led_trigger *trigger) | |||
178 | list_for_each_entry(led_cdev, &leds_list, node) { | 178 | list_for_each_entry(led_cdev, &leds_list, node) { |
179 | down_write(&led_cdev->trigger_lock); | 179 | down_write(&led_cdev->trigger_lock); |
180 | if (!led_cdev->trigger && led_cdev->default_trigger && | 180 | if (!led_cdev->trigger && led_cdev->default_trigger && |
181 | !strcmp(led_cdev->default_trigger, trigger->name)) | 181 | !strcmp(led_cdev->default_trigger, trig->name)) |
182 | led_trigger_set(led_cdev, trigger); | 182 | led_trigger_set(led_cdev, trig); |
183 | up_write(&led_cdev->trigger_lock); | 183 | up_write(&led_cdev->trigger_lock); |
184 | } | 184 | } |
185 | up_read(&leds_list_lock); | 185 | up_read(&leds_list_lock); |
@@ -188,20 +188,20 @@ int led_trigger_register(struct led_trigger *trigger) | |||
188 | } | 188 | } |
189 | EXPORT_SYMBOL_GPL(led_trigger_register); | 189 | EXPORT_SYMBOL_GPL(led_trigger_register); |
190 | 190 | ||
191 | void led_trigger_unregister(struct led_trigger *trigger) | 191 | void led_trigger_unregister(struct led_trigger *trig) |
192 | { | 192 | { |
193 | struct led_classdev *led_cdev; | 193 | struct led_classdev *led_cdev; |
194 | 194 | ||
195 | /* Remove from the list of led triggers */ | 195 | /* Remove from the list of led triggers */ |
196 | down_write(&triggers_list_lock); | 196 | down_write(&triggers_list_lock); |
197 | list_del(&trigger->next_trig); | 197 | list_del(&trig->next_trig); |
198 | up_write(&triggers_list_lock); | 198 | up_write(&triggers_list_lock); |
199 | 199 | ||
200 | /* Remove anyone actively using this trigger */ | 200 | /* Remove anyone actively using this trigger */ |
201 | down_read(&leds_list_lock); | 201 | down_read(&leds_list_lock); |
202 | list_for_each_entry(led_cdev, &leds_list, node) { | 202 | list_for_each_entry(led_cdev, &leds_list, node) { |
203 | down_write(&led_cdev->trigger_lock); | 203 | down_write(&led_cdev->trigger_lock); |
204 | if (led_cdev->trigger == trigger) | 204 | if (led_cdev->trigger == trig) |
205 | led_trigger_set(led_cdev, NULL); | 205 | led_trigger_set(led_cdev, NULL); |
206 | up_write(&led_cdev->trigger_lock); | 206 | up_write(&led_cdev->trigger_lock); |
207 | } | 207 | } |
@@ -211,58 +211,80 @@ EXPORT_SYMBOL_GPL(led_trigger_unregister); | |||
211 | 211 | ||
212 | /* Simple LED Tigger Interface */ | 212 | /* Simple LED Tigger Interface */ |
213 | 213 | ||
214 | void led_trigger_event(struct led_trigger *trigger, | 214 | void led_trigger_event(struct led_trigger *trig, |
215 | enum led_brightness brightness) | 215 | enum led_brightness brightness) |
216 | { | 216 | { |
217 | struct list_head *entry; | 217 | struct list_head *entry; |
218 | 218 | ||
219 | if (!trigger) | 219 | if (!trig) |
220 | return; | 220 | return; |
221 | 221 | ||
222 | read_lock(&trigger->leddev_list_lock); | 222 | read_lock(&trig->leddev_list_lock); |
223 | list_for_each(entry, &trigger->led_cdevs) { | 223 | list_for_each(entry, &trig->led_cdevs) { |
224 | struct led_classdev *led_cdev; | 224 | struct led_classdev *led_cdev; |
225 | 225 | ||
226 | led_cdev = list_entry(entry, struct led_classdev, trig_list); | 226 | led_cdev = list_entry(entry, struct led_classdev, trig_list); |
227 | led_set_brightness(led_cdev, brightness); | 227 | led_set_brightness(led_cdev, brightness); |
228 | } | 228 | } |
229 | read_unlock(&trigger->leddev_list_lock); | 229 | read_unlock(&trig->leddev_list_lock); |
230 | } | 230 | } |
231 | EXPORT_SYMBOL_GPL(led_trigger_event); | 231 | EXPORT_SYMBOL_GPL(led_trigger_event); |
232 | 232 | ||
233 | void led_trigger_blink(struct led_trigger *trigger, | 233 | static void led_trigger_blink_setup(struct led_trigger *trig, |
234 | unsigned long *delay_on, | 234 | unsigned long *delay_on, |
235 | unsigned long *delay_off) | 235 | unsigned long *delay_off, |
236 | int oneshot, | ||
237 | int invert) | ||
236 | { | 238 | { |
237 | struct list_head *entry; | 239 | struct list_head *entry; |
238 | 240 | ||
239 | if (!trigger) | 241 | if (!trig) |
240 | return; | 242 | return; |
241 | 243 | ||
242 | read_lock(&trigger->leddev_list_lock); | 244 | read_lock(&trig->leddev_list_lock); |
243 | list_for_each(entry, &trigger->led_cdevs) { | 245 | list_for_each(entry, &trig->led_cdevs) { |
244 | struct led_classdev *led_cdev; | 246 | struct led_classdev *led_cdev; |
245 | 247 | ||
246 | led_cdev = list_entry(entry, struct led_classdev, trig_list); | 248 | led_cdev = list_entry(entry, struct led_classdev, trig_list); |
247 | led_blink_set(led_cdev, delay_on, delay_off); | 249 | if (oneshot) |
250 | led_blink_set_oneshot(led_cdev, delay_on, delay_off, | ||
251 | invert); | ||
252 | else | ||
253 | led_blink_set(led_cdev, delay_on, delay_off); | ||
248 | } | 254 | } |
249 | read_unlock(&trigger->leddev_list_lock); | 255 | read_unlock(&trig->leddev_list_lock); |
256 | } | ||
257 | |||
258 | void led_trigger_blink(struct led_trigger *trig, | ||
259 | unsigned long *delay_on, | ||
260 | unsigned long *delay_off) | ||
261 | { | ||
262 | led_trigger_blink_setup(trig, delay_on, delay_off, 0, 0); | ||
250 | } | 263 | } |
251 | EXPORT_SYMBOL_GPL(led_trigger_blink); | 264 | EXPORT_SYMBOL_GPL(led_trigger_blink); |
252 | 265 | ||
266 | void led_trigger_blink_oneshot(struct led_trigger *trig, | ||
267 | unsigned long *delay_on, | ||
268 | unsigned long *delay_off, | ||
269 | int invert) | ||
270 | { | ||
271 | led_trigger_blink_setup(trig, delay_on, delay_off, 1, invert); | ||
272 | } | ||
273 | EXPORT_SYMBOL_GPL(led_trigger_blink_oneshot); | ||
274 | |||
253 | void led_trigger_register_simple(const char *name, struct led_trigger **tp) | 275 | void led_trigger_register_simple(const char *name, struct led_trigger **tp) |
254 | { | 276 | { |
255 | struct led_trigger *trigger; | 277 | struct led_trigger *trig; |
256 | int err; | 278 | int err; |
257 | 279 | ||
258 | trigger = kzalloc(sizeof(struct led_trigger), GFP_KERNEL); | 280 | trig = kzalloc(sizeof(struct led_trigger), GFP_KERNEL); |
259 | 281 | ||
260 | if (trigger) { | 282 | if (trig) { |
261 | trigger->name = name; | 283 | trig->name = name; |
262 | err = led_trigger_register(trigger); | 284 | err = led_trigger_register(trig); |
263 | if (err < 0) { | 285 | if (err < 0) { |
264 | kfree(trigger); | 286 | kfree(trig); |
265 | trigger = NULL; | 287 | trig = NULL; |
266 | printk(KERN_WARNING "LED trigger %s failed to register" | 288 | printk(KERN_WARNING "LED trigger %s failed to register" |
267 | " (%d)\n", name, err); | 289 | " (%d)\n", name, err); |
268 | } | 290 | } |
@@ -270,15 +292,15 @@ void led_trigger_register_simple(const char *name, struct led_trigger **tp) | |||
270 | printk(KERN_WARNING "LED trigger %s failed to register" | 292 | printk(KERN_WARNING "LED trigger %s failed to register" |
271 | " (no memory)\n", name); | 293 | " (no memory)\n", name); |
272 | 294 | ||
273 | *tp = trigger; | 295 | *tp = trig; |
274 | } | 296 | } |
275 | EXPORT_SYMBOL_GPL(led_trigger_register_simple); | 297 | EXPORT_SYMBOL_GPL(led_trigger_register_simple); |
276 | 298 | ||
277 | void led_trigger_unregister_simple(struct led_trigger *trigger) | 299 | void led_trigger_unregister_simple(struct led_trigger *trig) |
278 | { | 300 | { |
279 | if (trigger) | 301 | if (trig) |
280 | led_trigger_unregister(trigger); | 302 | led_trigger_unregister(trig); |
281 | kfree(trigger); | 303 | kfree(trig); |
282 | } | 304 | } |
283 | EXPORT_SYMBOL_GPL(led_trigger_unregister_simple); | 305 | EXPORT_SYMBOL_GPL(led_trigger_unregister_simple); |
284 | 306 | ||
diff --git a/drivers/leds/leds-88pm860x.c b/drivers/leds/leds-88pm860x.c index 5b61aaf7ac0f..61897cfeeda6 100644 --- a/drivers/leds/leds-88pm860x.c +++ b/drivers/leds/leds-88pm860x.c | |||
@@ -209,7 +209,7 @@ static int pm860x_led_probe(struct platform_device *pdev) | |||
209 | return -EINVAL; | 209 | return -EINVAL; |
210 | } | 210 | } |
211 | 211 | ||
212 | data = kzalloc(sizeof(struct pm860x_led), GFP_KERNEL); | 212 | data = devm_kzalloc(&pdev->dev, sizeof(struct pm860x_led), GFP_KERNEL); |
213 | if (data == NULL) | 213 | if (data == NULL) |
214 | return -ENOMEM; | 214 | return -ENOMEM; |
215 | strncpy(data->name, res->name, MFD_NAME_SIZE - 1); | 215 | strncpy(data->name, res->name, MFD_NAME_SIZE - 1); |
@@ -220,7 +220,6 @@ static int pm860x_led_probe(struct platform_device *pdev) | |||
220 | data->port = pdata->flags; | 220 | data->port = pdata->flags; |
221 | if (data->port < 0) { | 221 | if (data->port < 0) { |
222 | dev_err(&pdev->dev, "check device failed\n"); | 222 | dev_err(&pdev->dev, "check device failed\n"); |
223 | kfree(data); | ||
224 | return -EINVAL; | 223 | return -EINVAL; |
225 | } | 224 | } |
226 | 225 | ||
@@ -233,13 +232,10 @@ static int pm860x_led_probe(struct platform_device *pdev) | |||
233 | ret = led_classdev_register(chip->dev, &data->cdev); | 232 | ret = led_classdev_register(chip->dev, &data->cdev); |
234 | if (ret < 0) { | 233 | if (ret < 0) { |
235 | dev_err(&pdev->dev, "Failed to register LED: %d\n", ret); | 234 | dev_err(&pdev->dev, "Failed to register LED: %d\n", ret); |
236 | goto out; | 235 | return ret; |
237 | } | 236 | } |
238 | pm860x_led_set(&data->cdev, 0); | 237 | pm860x_led_set(&data->cdev, 0); |
239 | return 0; | 238 | return 0; |
240 | out: | ||
241 | kfree(data); | ||
242 | return ret; | ||
243 | } | 239 | } |
244 | 240 | ||
245 | static int pm860x_led_remove(struct platform_device *pdev) | 241 | static int pm860x_led_remove(struct platform_device *pdev) |
@@ -247,7 +243,6 @@ static int pm860x_led_remove(struct platform_device *pdev) | |||
247 | struct pm860x_led *data = platform_get_drvdata(pdev); | 243 | struct pm860x_led *data = platform_get_drvdata(pdev); |
248 | 244 | ||
249 | led_classdev_unregister(&data->cdev); | 245 | led_classdev_unregister(&data->cdev); |
250 | kfree(data); | ||
251 | 246 | ||
252 | return 0; | 247 | return 0; |
253 | } | 248 | } |
diff --git a/drivers/leds/leds-adp5520.c b/drivers/leds/leds-adp5520.c index b1400db3f839..aa56a867693a 100644 --- a/drivers/leds/leds-adp5520.c +++ b/drivers/leds/leds-adp5520.c | |||
@@ -119,7 +119,8 @@ static int __devinit adp5520_led_probe(struct platform_device *pdev) | |||
119 | return -EFAULT; | 119 | return -EFAULT; |
120 | } | 120 | } |
121 | 121 | ||
122 | led = kzalloc(sizeof(*led) * pdata->num_leds, GFP_KERNEL); | 122 | led = devm_kzalloc(&pdev->dev, sizeof(*led) * pdata->num_leds, |
123 | GFP_KERNEL); | ||
123 | if (led == NULL) { | 124 | if (led == NULL) { |
124 | dev_err(&pdev->dev, "failed to alloc memory\n"); | 125 | dev_err(&pdev->dev, "failed to alloc memory\n"); |
125 | return -ENOMEM; | 126 | return -ENOMEM; |
@@ -129,7 +130,7 @@ static int __devinit adp5520_led_probe(struct platform_device *pdev) | |||
129 | 130 | ||
130 | if (ret) { | 131 | if (ret) { |
131 | dev_err(&pdev->dev, "failed to write\n"); | 132 | dev_err(&pdev->dev, "failed to write\n"); |
132 | goto err_free; | 133 | return ret; |
133 | } | 134 | } |
134 | 135 | ||
135 | for (i = 0; i < pdata->num_leds; ++i) { | 136 | for (i = 0; i < pdata->num_leds; ++i) { |
@@ -179,8 +180,6 @@ err: | |||
179 | } | 180 | } |
180 | } | 181 | } |
181 | 182 | ||
182 | err_free: | ||
183 | kfree(led); | ||
184 | return ret; | 183 | return ret; |
185 | } | 184 | } |
186 | 185 | ||
@@ -200,7 +199,6 @@ static int __devexit adp5520_led_remove(struct platform_device *pdev) | |||
200 | cancel_work_sync(&led[i].work); | 199 | cancel_work_sync(&led[i].work); |
201 | } | 200 | } |
202 | 201 | ||
203 | kfree(led); | ||
204 | return 0; | 202 | return 0; |
205 | } | 203 | } |
206 | 204 | ||
diff --git a/drivers/leds/leds-asic3.c b/drivers/leds/leds-asic3.c index 525a92492837..5de74ff90dcf 100644 --- a/drivers/leds/leds-asic3.c +++ b/drivers/leds/leds-asic3.c | |||
@@ -99,12 +99,13 @@ static int __devinit asic3_led_probe(struct platform_device *pdev) | |||
99 | 99 | ||
100 | ret = mfd_cell_enable(pdev); | 100 | ret = mfd_cell_enable(pdev); |
101 | if (ret < 0) | 101 | if (ret < 0) |
102 | goto ret0; | 102 | return ret; |
103 | 103 | ||
104 | led->cdev = kzalloc(sizeof(struct led_classdev), GFP_KERNEL); | 104 | led->cdev = devm_kzalloc(&pdev->dev, sizeof(struct led_classdev), |
105 | GFP_KERNEL); | ||
105 | if (!led->cdev) { | 106 | if (!led->cdev) { |
106 | ret = -ENOMEM; | 107 | ret = -ENOMEM; |
107 | goto ret1; | 108 | goto out; |
108 | } | 109 | } |
109 | 110 | ||
110 | led->cdev->name = led->name; | 111 | led->cdev->name = led->name; |
@@ -115,15 +116,12 @@ static int __devinit asic3_led_probe(struct platform_device *pdev) | |||
115 | 116 | ||
116 | ret = led_classdev_register(&pdev->dev, led->cdev); | 117 | ret = led_classdev_register(&pdev->dev, led->cdev); |
117 | if (ret < 0) | 118 | if (ret < 0) |
118 | goto ret2; | 119 | goto out; |
119 | 120 | ||
120 | return 0; | 121 | return 0; |
121 | 122 | ||
122 | ret2: | 123 | out: |
123 | kfree(led->cdev); | ||
124 | ret1: | ||
125 | (void) mfd_cell_disable(pdev); | 124 | (void) mfd_cell_disable(pdev); |
126 | ret0: | ||
127 | return ret; | 125 | return ret; |
128 | } | 126 | } |
129 | 127 | ||
@@ -133,8 +131,6 @@ static int __devexit asic3_led_remove(struct platform_device *pdev) | |||
133 | 131 | ||
134 | led_classdev_unregister(led->cdev); | 132 | led_classdev_unregister(led->cdev); |
135 | 133 | ||
136 | kfree(led->cdev); | ||
137 | |||
138 | return mfd_cell_disable(pdev); | 134 | return mfd_cell_disable(pdev); |
139 | } | 135 | } |
140 | 136 | ||
diff --git a/drivers/leds/leds-atmel-pwm.c b/drivers/leds/leds-atmel-pwm.c index 64ad702a2ecc..45430632faab 100644 --- a/drivers/leds/leds-atmel-pwm.c +++ b/drivers/leds/leds-atmel-pwm.c | |||
@@ -46,7 +46,8 @@ static int __devinit pwmled_probe(struct platform_device *pdev) | |||
46 | if (!pdata || pdata->num_leds < 1) | 46 | if (!pdata || pdata->num_leds < 1) |
47 | return -ENODEV; | 47 | return -ENODEV; |
48 | 48 | ||
49 | leds = kcalloc(pdata->num_leds, sizeof(*leds), GFP_KERNEL); | 49 | leds = devm_kzalloc(&pdev->dev, pdata->num_leds * sizeof(*leds), |
50 | GFP_KERNEL); | ||
50 | if (!leds) | 51 | if (!leds) |
51 | return -ENOMEM; | 52 | return -ENOMEM; |
52 | 53 | ||
@@ -108,7 +109,6 @@ err: | |||
108 | pwm_channel_free(&leds[i].pwmc); | 109 | pwm_channel_free(&leds[i].pwmc); |
109 | } | 110 | } |
110 | } | 111 | } |
111 | kfree(leds); | ||
112 | 112 | ||
113 | return status; | 113 | return status; |
114 | } | 114 | } |
@@ -129,7 +129,6 @@ static int __exit pwmled_remove(struct platform_device *pdev) | |||
129 | pwm_channel_free(&led->pwmc); | 129 | pwm_channel_free(&led->pwmc); |
130 | } | 130 | } |
131 | 131 | ||
132 | kfree(leds); | ||
133 | platform_set_drvdata(pdev, NULL); | 132 | platform_set_drvdata(pdev, NULL); |
134 | return 0; | 133 | return 0; |
135 | } | 134 | } |
diff --git a/drivers/leds/leds-bd2802.c b/drivers/leds/leds-bd2802.c index 591cbdf5a046..89ca6a2a19d1 100644 --- a/drivers/leds/leds-bd2802.c +++ b/drivers/leds/leds-bd2802.c | |||
@@ -677,7 +677,7 @@ static int __devinit bd2802_probe(struct i2c_client *client, | |||
677 | struct bd2802_led_platform_data *pdata; | 677 | struct bd2802_led_platform_data *pdata; |
678 | int ret, i; | 678 | int ret, i; |
679 | 679 | ||
680 | led = kzalloc(sizeof(struct bd2802_led), GFP_KERNEL); | 680 | led = devm_kzalloc(&client->dev, sizeof(struct bd2802_led), GFP_KERNEL); |
681 | if (!led) { | 681 | if (!led) { |
682 | dev_err(&client->dev, "failed to allocate driver data\n"); | 682 | dev_err(&client->dev, "failed to allocate driver data\n"); |
683 | return -ENOMEM; | 683 | return -ENOMEM; |
@@ -697,7 +697,7 @@ static int __devinit bd2802_probe(struct i2c_client *client, | |||
697 | ret = bd2802_write_byte(client, BD2802_REG_CLKSETUP, 0x00); | 697 | ret = bd2802_write_byte(client, BD2802_REG_CLKSETUP, 0x00); |
698 | if (ret < 0) { | 698 | if (ret < 0) { |
699 | dev_err(&client->dev, "failed to detect device\n"); | 699 | dev_err(&client->dev, "failed to detect device\n"); |
700 | goto failed_free; | 700 | return ret; |
701 | } else | 701 | } else |
702 | dev_info(&client->dev, "return 0x%02x\n", ret); | 702 | dev_info(&client->dev, "return 0x%02x\n", ret); |
703 | 703 | ||
@@ -729,9 +729,6 @@ static int __devinit bd2802_probe(struct i2c_client *client, | |||
729 | failed_unregister_dev_file: | 729 | failed_unregister_dev_file: |
730 | for (i--; i >= 0; i--) | 730 | for (i--; i >= 0; i--) |
731 | device_remove_file(&led->client->dev, bd2802_attributes[i]); | 731 | device_remove_file(&led->client->dev, bd2802_attributes[i]); |
732 | failed_free: | ||
733 | kfree(led); | ||
734 | |||
735 | return ret; | 732 | return ret; |
736 | } | 733 | } |
737 | 734 | ||
@@ -746,7 +743,6 @@ static int __exit bd2802_remove(struct i2c_client *client) | |||
746 | bd2802_disable_adv_conf(led); | 743 | bd2802_disable_adv_conf(led); |
747 | for (i = 0; i < ARRAY_SIZE(bd2802_attributes); i++) | 744 | for (i = 0; i < ARRAY_SIZE(bd2802_attributes); i++) |
748 | device_remove_file(&led->client->dev, bd2802_attributes[i]); | 745 | device_remove_file(&led->client->dev, bd2802_attributes[i]); |
749 | kfree(led); | ||
750 | 746 | ||
751 | return 0; | 747 | return 0; |
752 | } | 748 | } |
diff --git a/drivers/leds/leds-blinkm.c b/drivers/leds/leds-blinkm.c new file mode 100644 index 000000000000..f7c3d7f1ec52 --- /dev/null +++ b/drivers/leds/leds-blinkm.c | |||
@@ -0,0 +1,815 @@ | |||
1 | /* | ||
2 | * leds-blinkm.c | ||
3 | * (c) Jan-Simon Möller (dl9pf@gmx.de) | ||
4 | * | ||
5 | * This program is free software; you can redistribute it and/or modify | ||
6 | * it under the terms of the GNU General Public License as published by | ||
7 | * the Free Software Foundation; either version 2 of the License, or | ||
8 | * (at your option) any later version. | ||
9 | * | ||
10 | * This program is distributed in the hope that it will be useful, | ||
11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
13 | * GNU General Public License for more details. | ||
14 | * | ||
15 | * You should have received a copy of the GNU General Public License | ||
16 | * along with this program; if not, write to the Free Software | ||
17 | * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. | ||
18 | */ | ||
19 | |||
20 | #include <linux/module.h> | ||
21 | #include <linux/init.h> | ||
22 | #include <linux/slab.h> | ||
23 | #include <linux/jiffies.h> | ||
24 | #include <linux/i2c.h> | ||
25 | #include <linux/err.h> | ||
26 | #include <linux/mutex.h> | ||
27 | #include <linux/sysfs.h> | ||
28 | #include <linux/printk.h> | ||
29 | #include <linux/pm_runtime.h> | ||
30 | #include <linux/leds.h> | ||
31 | #include <linux/delay.h> | ||
32 | |||
33 | /* Addresses to scan - BlinkM is on 0x09 by default*/ | ||
34 | static const unsigned short normal_i2c[] = { 0x09, I2C_CLIENT_END }; | ||
35 | |||
36 | static int blinkm_transfer_hw(struct i2c_client *client, int cmd); | ||
37 | static int blinkm_test_run(struct i2c_client *client); | ||
38 | |||
39 | struct blinkm_led { | ||
40 | struct i2c_client *i2c_client; | ||
41 | struct led_classdev led_cdev; | ||
42 | int id; | ||
43 | atomic_t active; | ||
44 | }; | ||
45 | |||
46 | struct blinkm_work { | ||
47 | struct blinkm_led *blinkm_led; | ||
48 | struct work_struct work; | ||
49 | }; | ||
50 | |||
51 | #define cdev_to_blmled(c) container_of(c, struct blinkm_led, led_cdev) | ||
52 | #define work_to_blmwork(c) container_of(c, struct blinkm_work, work) | ||
53 | |||
54 | struct blinkm_data { | ||
55 | struct i2c_client *i2c_client; | ||
56 | struct mutex update_lock; | ||
57 | /* used for led class interface */ | ||
58 | struct blinkm_led blinkm_leds[3]; | ||
59 | /* used for "blinkm" sysfs interface */ | ||
60 | u8 red; /* color red */ | ||
61 | u8 green; /* color green */ | ||
62 | u8 blue; /* color blue */ | ||
63 | /* next values to use for transfer */ | ||
64 | u8 next_red; /* color red */ | ||
65 | u8 next_green; /* color green */ | ||
66 | u8 next_blue; /* color blue */ | ||
67 | /* internal use */ | ||
68 | u8 args[7]; /* set of args for transmission */ | ||
69 | u8 i2c_addr; /* i2c addr */ | ||
70 | u8 fw_ver; /* firmware version */ | ||
71 | /* used, but not from userspace */ | ||
72 | u8 hue; /* HSB hue */ | ||
73 | u8 saturation; /* HSB saturation */ | ||
74 | u8 brightness; /* HSB brightness */ | ||
75 | u8 next_hue; /* HSB hue */ | ||
76 | u8 next_saturation; /* HSB saturation */ | ||
77 | u8 next_brightness; /* HSB brightness */ | ||
78 | /* currently unused / todo */ | ||
79 | u8 fade_speed; /* fade speed 1 - 255 */ | ||
80 | s8 time_adjust; /* time adjust -128 - 127 */ | ||
81 | u8 fade:1; /* fade on = 1, off = 0 */ | ||
82 | u8 rand:1; /* rand fade mode on = 1 */ | ||
83 | u8 script_id; /* script ID */ | ||
84 | u8 script_repeats; /* repeats of script */ | ||
85 | u8 script_startline; /* line to start */ | ||
86 | }; | ||
87 | |||
88 | /* Colors */ | ||
89 | #define RED 0 | ||
90 | #define GREEN 1 | ||
91 | #define BLUE 2 | ||
92 | |||
93 | /* mapping command names to cmd chars - see datasheet */ | ||
94 | #define BLM_GO_RGB 0 | ||
95 | #define BLM_FADE_RGB 1 | ||
96 | #define BLM_FADE_HSB 2 | ||
97 | #define BLM_FADE_RAND_RGB 3 | ||
98 | #define BLM_FADE_RAND_HSB 4 | ||
99 | #define BLM_PLAY_SCRIPT 5 | ||
100 | #define BLM_STOP_SCRIPT 6 | ||
101 | #define BLM_SET_FADE_SPEED 7 | ||
102 | #define BLM_SET_TIME_ADJ 8 | ||
103 | #define BLM_GET_CUR_RGB 9 | ||
104 | #define BLM_WRITE_SCRIPT_LINE 10 | ||
105 | #define BLM_READ_SCRIPT_LINE 11 | ||
106 | #define BLM_SET_SCRIPT_LR 12 /* Length & Repeats */ | ||
107 | #define BLM_SET_ADDR 13 | ||
108 | #define BLM_GET_ADDR 14 | ||
109 | #define BLM_GET_FW_VER 15 | ||
110 | #define BLM_SET_STARTUP_PARAM 16 | ||
111 | |||
112 | /* BlinkM Commands | ||
113 | * as extracted out of the datasheet: | ||
114 | * | ||
115 | * cmdchar = command (ascii) | ||
116 | * cmdbyte = command in hex | ||
117 | * nr_args = number of arguments (to send) | ||
118 | * nr_ret = number of return values (to read) | ||
119 | * dir = direction (0 = read, 1 = write, 2 = both) | ||
120 | * | ||
121 | */ | ||
122 | static const struct { | ||
123 | char cmdchar; | ||
124 | u8 cmdbyte; | ||
125 | u8 nr_args; | ||
126 | u8 nr_ret; | ||
127 | u8 dir:2; | ||
128 | } blinkm_cmds[17] = { | ||
129 | /* cmdchar, cmdbyte, nr_args, nr_ret, dir */ | ||
130 | { 'n', 0x6e, 3, 0, 1}, | ||
131 | { 'c', 0x63, 3, 0, 1}, | ||
132 | { 'h', 0x68, 3, 0, 1}, | ||
133 | { 'C', 0x43, 3, 0, 1}, | ||
134 | { 'H', 0x48, 3, 0, 1}, | ||
135 | { 'p', 0x70, 3, 0, 1}, | ||
136 | { 'o', 0x6f, 0, 0, 1}, | ||
137 | { 'f', 0x66, 1, 0, 1}, | ||
138 | { 't', 0x74, 1, 0, 1}, | ||
139 | { 'g', 0x67, 0, 3, 0}, | ||
140 | { 'W', 0x57, 7, 0, 1}, | ||
141 | { 'R', 0x52, 2, 5, 2}, | ||
142 | { 'L', 0x4c, 3, 0, 1}, | ||
143 | { 'A', 0x41, 4, 0, 1}, | ||
144 | { 'a', 0x61, 0, 1, 0}, | ||
145 | { 'Z', 0x5a, 0, 1, 0}, | ||
146 | { 'B', 0x42, 5, 0, 1}, | ||
147 | }; | ||
148 | |||
149 | static ssize_t show_color_common(struct device *dev, char *buf, int color) | ||
150 | { | ||
151 | struct i2c_client *client; | ||
152 | struct blinkm_data *data; | ||
153 | int ret; | ||
154 | |||
155 | client = to_i2c_client(dev); | ||
156 | data = i2c_get_clientdata(client); | ||
157 | |||
158 | ret = blinkm_transfer_hw(client, BLM_GET_CUR_RGB); | ||
159 | if (ret < 0) | ||
160 | return ret; | ||
161 | switch (color) { | ||
162 | case RED: | ||
163 | return scnprintf(buf, PAGE_SIZE, "%02X\n", data->red); | ||
164 | break; | ||
165 | case GREEN: | ||
166 | return scnprintf(buf, PAGE_SIZE, "%02X\n", data->green); | ||
167 | break; | ||
168 | case BLUE: | ||
169 | return scnprintf(buf, PAGE_SIZE, "%02X\n", data->blue); | ||
170 | break; | ||
171 | default: | ||
172 | return -EINVAL; | ||
173 | } | ||
174 | return -EINVAL; | ||
175 | } | ||
176 | |||
177 | static int store_color_common(struct device *dev, const char *buf, int color) | ||
178 | { | ||
179 | struct i2c_client *client; | ||
180 | struct blinkm_data *data; | ||
181 | int ret; | ||
182 | u8 value; | ||
183 | |||
184 | client = to_i2c_client(dev); | ||
185 | data = i2c_get_clientdata(client); | ||
186 | |||
187 | ret = kstrtou8(buf, 10, &value); | ||
188 | if (ret < 0) { | ||
189 | dev_err(dev, "BlinkM: value too large!\n"); | ||
190 | return ret; | ||
191 | } | ||
192 | |||
193 | switch (color) { | ||
194 | case RED: | ||
195 | data->next_red = value; | ||
196 | break; | ||
197 | case GREEN: | ||
198 | data->next_green = value; | ||
199 | break; | ||
200 | case BLUE: | ||
201 | data->next_blue = value; | ||
202 | break; | ||
203 | default: | ||
204 | return -EINVAL; | ||
205 | } | ||
206 | |||
207 | dev_dbg(dev, "next_red = %d, next_green = %d, next_blue = %d\n", | ||
208 | data->next_red, data->next_green, data->next_blue); | ||
209 | |||
210 | /* if mode ... */ | ||
211 | ret = blinkm_transfer_hw(client, BLM_GO_RGB); | ||
212 | if (ret < 0) { | ||
213 | dev_err(dev, "BlinkM: can't set RGB\n"); | ||
214 | return ret; | ||
215 | } | ||
216 | return 0; | ||
217 | } | ||
218 | |||
219 | static ssize_t show_red(struct device *dev, struct device_attribute *attr, | ||
220 | char *buf) | ||
221 | { | ||
222 | return show_color_common(dev, buf, RED); | ||
223 | } | ||
224 | |||
225 | static ssize_t store_red(struct device *dev, struct device_attribute *attr, | ||
226 | const char *buf, size_t count) | ||
227 | { | ||
228 | int ret; | ||
229 | |||
230 | ret = store_color_common(dev, buf, RED); | ||
231 | if (ret < 0) | ||
232 | return ret; | ||
233 | return count; | ||
234 | } | ||
235 | |||
236 | static DEVICE_ATTR(red, S_IRUGO | S_IWUSR, show_red, store_red); | ||
237 | |||
238 | static ssize_t show_green(struct device *dev, struct device_attribute *attr, | ||
239 | char *buf) | ||
240 | { | ||
241 | return show_color_common(dev, buf, GREEN); | ||
242 | } | ||
243 | |||
244 | static ssize_t store_green(struct device *dev, struct device_attribute *attr, | ||
245 | const char *buf, size_t count) | ||
246 | { | ||
247 | |||
248 | int ret; | ||
249 | |||
250 | ret = store_color_common(dev, buf, GREEN); | ||
251 | if (ret < 0) | ||
252 | return ret; | ||
253 | return count; | ||
254 | } | ||
255 | |||
256 | static DEVICE_ATTR(green, S_IRUGO | S_IWUSR, show_green, store_green); | ||
257 | |||
258 | static ssize_t show_blue(struct device *dev, struct device_attribute *attr, | ||
259 | char *buf) | ||
260 | { | ||
261 | return show_color_common(dev, buf, BLUE); | ||
262 | } | ||
263 | |||
264 | static ssize_t store_blue(struct device *dev, struct device_attribute *attr, | ||
265 | const char *buf, size_t count) | ||
266 | { | ||
267 | int ret; | ||
268 | |||
269 | ret = store_color_common(dev, buf, BLUE); | ||
270 | if (ret < 0) | ||
271 | return ret; | ||
272 | return count; | ||
273 | } | ||
274 | |||
275 | static DEVICE_ATTR(blue, S_IRUGO | S_IWUSR, show_blue, store_blue); | ||
276 | |||
277 | static ssize_t show_test(struct device *dev, struct device_attribute *attr, | ||
278 | char *buf) | ||
279 | { | ||
280 | return scnprintf(buf, PAGE_SIZE, | ||
281 | "#Write into test to start test sequence!#\n"); | ||
282 | } | ||
283 | |||
284 | static ssize_t store_test(struct device *dev, struct device_attribute *attr, | ||
285 | const char *buf, size_t count) | ||
286 | { | ||
287 | |||
288 | struct i2c_client *client; | ||
289 | int ret; | ||
290 | client = to_i2c_client(dev); | ||
291 | |||
292 | /*test */ | ||
293 | ret = blinkm_test_run(client); | ||
294 | if (ret < 0) | ||
295 | return ret; | ||
296 | |||
297 | return count; | ||
298 | } | ||
299 | |||
300 | static DEVICE_ATTR(test, S_IRUGO | S_IWUSR, show_test, store_test); | ||
301 | |||
302 | /* TODO: HSB, fade, timeadj, script ... */ | ||
303 | |||
304 | static struct attribute *blinkm_attrs[] = { | ||
305 | &dev_attr_red.attr, | ||
306 | &dev_attr_green.attr, | ||
307 | &dev_attr_blue.attr, | ||
308 | &dev_attr_test.attr, | ||
309 | NULL, | ||
310 | }; | ||
311 | |||
312 | static struct attribute_group blinkm_group = { | ||
313 | .name = "blinkm", | ||
314 | .attrs = blinkm_attrs, | ||
315 | }; | ||
316 | |||
317 | static int blinkm_write(struct i2c_client *client, int cmd, u8 *arg) | ||
318 | { | ||
319 | int result; | ||
320 | int i; | ||
321 | int arglen = blinkm_cmds[cmd].nr_args; | ||
322 | /* write out cmd to blinkm - always / default step */ | ||
323 | result = i2c_smbus_write_byte(client, blinkm_cmds[cmd].cmdbyte); | ||
324 | if (result < 0) | ||
325 | return result; | ||
326 | /* no args to write out */ | ||
327 | if (arglen == 0) | ||
328 | return 0; | ||
329 | |||
330 | for (i = 0; i < arglen; i++) { | ||
331 | /* repeat for arglen */ | ||
332 | result = i2c_smbus_write_byte(client, arg[i]); | ||
333 | if (result < 0) | ||
334 | return result; | ||
335 | } | ||
336 | return 0; | ||
337 | } | ||
338 | |||
339 | static int blinkm_read(struct i2c_client *client, int cmd, u8 *arg) | ||
340 | { | ||
341 | int result; | ||
342 | int i; | ||
343 | int retlen = blinkm_cmds[cmd].nr_ret; | ||
344 | for (i = 0; i < retlen; i++) { | ||
345 | /* repeat for retlen */ | ||
346 | result = i2c_smbus_read_byte(client); | ||
347 | if (result < 0) | ||
348 | return result; | ||
349 | arg[i] = result; | ||
350 | } | ||
351 | |||
352 | return 0; | ||
353 | } | ||
354 | |||
355 | static int blinkm_transfer_hw(struct i2c_client *client, int cmd) | ||
356 | { | ||
357 | /* the protocol is simple but non-standard: | ||
358 | * e.g. cmd 'g' (= 0x67) for "get device address" | ||
359 | * - which defaults to 0x09 - would be the sequence: | ||
360 | * a) write 0x67 to the device (byte write) | ||
361 | * b) read the value (0x09) back right after (byte read) | ||
362 | * | ||
363 | * Watch out for "unfinished" sequences (i.e. not enough reads | ||
364 | * or writes after a command. It will make the blinkM misbehave. | ||
365 | * Sequence is key here. | ||
366 | */ | ||
367 | |||
368 | /* args / return are in private data struct */ | ||
369 | struct blinkm_data *data = i2c_get_clientdata(client); | ||
370 | |||
371 | /* We start hardware transfers which are not to be | ||
372 | * mixed with other commands. Aquire a lock now. */ | ||
373 | if (mutex_lock_interruptible(&data->update_lock) < 0) | ||
374 | return -EAGAIN; | ||
375 | |||
376 | /* switch cmd - usually write before reads */ | ||
377 | switch (cmd) { | ||
378 | case BLM_FADE_RAND_RGB: | ||
379 | case BLM_GO_RGB: | ||
380 | case BLM_FADE_RGB: | ||
381 | data->args[0] = data->next_red; | ||
382 | data->args[1] = data->next_green; | ||
383 | data->args[2] = data->next_blue; | ||
384 | blinkm_write(client, cmd, data->args); | ||
385 | data->red = data->args[0]; | ||
386 | data->green = data->args[1]; | ||
387 | data->blue = data->args[2]; | ||
388 | break; | ||
389 | case BLM_FADE_HSB: | ||
390 | case BLM_FADE_RAND_HSB: | ||
391 | data->args[0] = data->next_hue; | ||
392 | data->args[1] = data->next_saturation; | ||
393 | data->args[2] = data->next_brightness; | ||
394 | blinkm_write(client, cmd, data->args); | ||
395 | data->hue = data->next_hue; | ||
396 | data->saturation = data->next_saturation; | ||
397 | data->brightness = data->next_brightness; | ||
398 | break; | ||
399 | case BLM_PLAY_SCRIPT: | ||
400 | data->args[0] = data->script_id; | ||
401 | data->args[1] = data->script_repeats; | ||
402 | data->args[2] = data->script_startline; | ||
403 | blinkm_write(client, cmd, data->args); | ||
404 | break; | ||
405 | case BLM_STOP_SCRIPT: | ||
406 | blinkm_write(client, cmd, NULL); | ||
407 | break; | ||
408 | case BLM_GET_CUR_RGB: | ||
409 | data->args[0] = data->red; | ||
410 | data->args[1] = data->green; | ||
411 | data->args[2] = data->blue; | ||
412 | blinkm_write(client, cmd, NULL); | ||
413 | blinkm_read(client, cmd, data->args); | ||
414 | data->red = data->args[0]; | ||
415 | data->green = data->args[1]; | ||
416 | data->blue = data->args[2]; | ||
417 | break; | ||
418 | case BLM_GET_ADDR: | ||
419 | data->args[0] = data->i2c_addr; | ||
420 | blinkm_write(client, cmd, NULL); | ||
421 | blinkm_read(client, cmd, data->args); | ||
422 | data->i2c_addr = data->args[0]; | ||
423 | break; | ||
424 | case BLM_SET_TIME_ADJ: | ||
425 | case BLM_SET_FADE_SPEED: | ||
426 | case BLM_READ_SCRIPT_LINE: | ||
427 | case BLM_WRITE_SCRIPT_LINE: | ||
428 | case BLM_SET_SCRIPT_LR: | ||
429 | case BLM_SET_ADDR: | ||
430 | case BLM_GET_FW_VER: | ||
431 | case BLM_SET_STARTUP_PARAM: | ||
432 | dev_err(&client->dev, | ||
433 | "BlinkM: cmd %d not implemented yet.\n", cmd); | ||
434 | break; | ||
435 | default: | ||
436 | dev_err(&client->dev, "BlinkM: unknown command %d\n", cmd); | ||
437 | mutex_unlock(&data->update_lock); | ||
438 | return -EINVAL; | ||
439 | } /* end switch(cmd) */ | ||
440 | |||
441 | /* transfers done, unlock */ | ||
442 | mutex_unlock(&data->update_lock); | ||
443 | return 0; | ||
444 | } | ||
445 | |||
446 | static void led_work(struct work_struct *work) | ||
447 | { | ||
448 | int ret; | ||
449 | struct blinkm_led *led; | ||
450 | struct blinkm_data *data ; | ||
451 | struct blinkm_work *blm_work = work_to_blmwork(work); | ||
452 | |||
453 | led = blm_work->blinkm_led; | ||
454 | data = i2c_get_clientdata(led->i2c_client); | ||
455 | ret = blinkm_transfer_hw(led->i2c_client, BLM_GO_RGB); | ||
456 | atomic_dec(&led->active); | ||
457 | dev_dbg(&led->i2c_client->dev, | ||
458 | "# DONE # next_red = %d, next_green = %d," | ||
459 | " next_blue = %d, active = %d\n", | ||
460 | data->next_red, data->next_green, | ||
461 | data->next_blue, atomic_read(&led->active)); | ||
462 | kfree(blm_work); | ||
463 | } | ||
464 | |||
465 | static int blinkm_led_common_set(struct led_classdev *led_cdev, | ||
466 | enum led_brightness value, int color) | ||
467 | { | ||
468 | /* led_brightness is 0, 127 or 255 - we just use it here as-is */ | ||
469 | struct blinkm_led *led = cdev_to_blmled(led_cdev); | ||
470 | struct blinkm_data *data = i2c_get_clientdata(led->i2c_client); | ||
471 | struct blinkm_work *bl_work; | ||
472 | |||
473 | switch (color) { | ||
474 | case RED: | ||
475 | /* bail out if there's no change */ | ||
476 | if (data->next_red == (u8) value) | ||
477 | return 0; | ||
478 | /* we assume a quite fast sequence here ([off]->on->off) | ||
479 | * think of network led trigger - we cannot blink that fast, so | ||
480 | * in case we already have a off->on->off transition queued up, | ||
481 | * we refuse to queue up more. | ||
482 | * Revisit: fast-changing brightness. */ | ||
483 | if (atomic_read(&led->active) > 1) | ||
484 | return 0; | ||
485 | data->next_red = (u8) value; | ||
486 | break; | ||
487 | case GREEN: | ||
488 | /* bail out if there's no change */ | ||
489 | if (data->next_green == (u8) value) | ||
490 | return 0; | ||
491 | /* we assume a quite fast sequence here ([off]->on->off) | ||
492 | * Revisit: fast-changing brightness. */ | ||
493 | if (atomic_read(&led->active) > 1) | ||
494 | return 0; | ||
495 | data->next_green = (u8) value; | ||
496 | break; | ||
497 | case BLUE: | ||
498 | /* bail out if there's no change */ | ||
499 | if (data->next_blue == (u8) value) | ||
500 | return 0; | ||
501 | /* we assume a quite fast sequence here ([off]->on->off) | ||
502 | * Revisit: fast-changing brightness. */ | ||
503 | if (atomic_read(&led->active) > 1) | ||
504 | return 0; | ||
505 | data->next_blue = (u8) value; | ||
506 | break; | ||
507 | |||
508 | default: | ||
509 | dev_err(&led->i2c_client->dev, "BlinkM: unknown color.\n"); | ||
510 | return -EINVAL; | ||
511 | } | ||
512 | |||
513 | bl_work = kzalloc(sizeof(*bl_work), GFP_ATOMIC); | ||
514 | if (!bl_work) | ||
515 | return -ENOMEM; | ||
516 | |||
517 | atomic_inc(&led->active); | ||
518 | dev_dbg(&led->i2c_client->dev, | ||
519 | "#TO_SCHED# next_red = %d, next_green = %d," | ||
520 | " next_blue = %d, active = %d\n", | ||
521 | data->next_red, data->next_green, | ||
522 | data->next_blue, atomic_read(&led->active)); | ||
523 | |||
524 | /* a fresh work _item_ for each change */ | ||
525 | bl_work->blinkm_led = led; | ||
526 | INIT_WORK(&bl_work->work, led_work); | ||
527 | /* queue work in own queue for easy sync on exit*/ | ||
528 | schedule_work(&bl_work->work); | ||
529 | |||
530 | return 0; | ||
531 | } | ||
532 | |||
533 | static void blinkm_led_red_set(struct led_classdev *led_cdev, | ||
534 | enum led_brightness value) | ||
535 | { | ||
536 | blinkm_led_common_set(led_cdev, value, RED); | ||
537 | } | ||
538 | |||
539 | static void blinkm_led_green_set(struct led_classdev *led_cdev, | ||
540 | enum led_brightness value) | ||
541 | { | ||
542 | blinkm_led_common_set(led_cdev, value, GREEN); | ||
543 | } | ||
544 | |||
545 | static void blinkm_led_blue_set(struct led_classdev *led_cdev, | ||
546 | enum led_brightness value) | ||
547 | { | ||
548 | blinkm_led_common_set(led_cdev, value, BLUE); | ||
549 | } | ||
550 | |||
551 | static void blinkm_init_hw(struct i2c_client *client) | ||
552 | { | ||
553 | int ret; | ||
554 | ret = blinkm_transfer_hw(client, BLM_STOP_SCRIPT); | ||
555 | ret = blinkm_transfer_hw(client, BLM_GO_RGB); | ||
556 | } | ||
557 | |||
558 | static int blinkm_test_run(struct i2c_client *client) | ||
559 | { | ||
560 | int ret; | ||
561 | struct blinkm_data *data = i2c_get_clientdata(client); | ||
562 | |||
563 | data->next_red = 0x01; | ||
564 | data->next_green = 0x05; | ||
565 | data->next_blue = 0x10; | ||
566 | ret = blinkm_transfer_hw(client, BLM_GO_RGB); | ||
567 | if (ret < 0) | ||
568 | return ret; | ||
569 | msleep(2000); | ||
570 | |||
571 | data->next_red = 0x25; | ||
572 | data->next_green = 0x10; | ||
573 | data->next_blue = 0x31; | ||
574 | ret = blinkm_transfer_hw(client, BLM_FADE_RGB); | ||
575 | if (ret < 0) | ||
576 | return ret; | ||
577 | msleep(2000); | ||
578 | |||
579 | data->next_hue = 0x50; | ||
580 | data->next_saturation = 0x10; | ||
581 | data->next_brightness = 0x20; | ||
582 | ret = blinkm_transfer_hw(client, BLM_FADE_HSB); | ||
583 | if (ret < 0) | ||
584 | return ret; | ||
585 | msleep(2000); | ||
586 | |||
587 | return 0; | ||
588 | } | ||
589 | |||
590 | /* Return 0 if detection is successful, -ENODEV otherwise */ | ||
591 | static int blinkm_detect(struct i2c_client *client, struct i2c_board_info *info) | ||
592 | { | ||
593 | struct i2c_adapter *adapter = client->adapter; | ||
594 | int ret; | ||
595 | int count = 99; | ||
596 | u8 tmpargs[7]; | ||
597 | |||
598 | if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA | ||
599 | | I2C_FUNC_SMBUS_WORD_DATA | ||
600 | | I2C_FUNC_SMBUS_WRITE_BYTE)) | ||
601 | return -ENODEV; | ||
602 | |||
603 | /* Now, we do the remaining detection. Simple for now. */ | ||
604 | /* We might need more guards to protect other i2c slaves */ | ||
605 | |||
606 | /* make sure the blinkM is balanced (read/writes) */ | ||
607 | while (count > 0) { | ||
608 | ret = blinkm_write(client, BLM_GET_ADDR, NULL); | ||
609 | usleep_range(5000, 10000); | ||
610 | ret = blinkm_read(client, BLM_GET_ADDR, tmpargs); | ||
611 | usleep_range(5000, 10000); | ||
612 | if (tmpargs[0] == 0x09) | ||
613 | count = 0; | ||
614 | count--; | ||
615 | } | ||
616 | |||
617 | /* Step 1: Read BlinkM address back - cmd_char 'a' */ | ||
618 | ret = blinkm_write(client, BLM_GET_ADDR, NULL); | ||
619 | if (ret < 0) | ||
620 | return ret; | ||
621 | usleep_range(20000, 30000); /* allow a small delay */ | ||
622 | ret = blinkm_read(client, BLM_GET_ADDR, tmpargs); | ||
623 | if (ret < 0) | ||
624 | return ret; | ||
625 | |||
626 | if (tmpargs[0] != 0x09) { | ||
627 | dev_err(&client->dev, "enodev DEV ADDR = 0x%02X\n", tmpargs[0]); | ||
628 | return -ENODEV; | ||
629 | } | ||
630 | |||
631 | strlcpy(info->type, "blinkm", I2C_NAME_SIZE); | ||
632 | return 0; | ||
633 | } | ||
634 | |||
635 | static int __devinit blinkm_probe(struct i2c_client *client, | ||
636 | const struct i2c_device_id *id) | ||
637 | { | ||
638 | struct blinkm_data *data; | ||
639 | struct blinkm_led *led[3]; | ||
640 | int err, i; | ||
641 | char blinkm_led_name[28]; | ||
642 | |||
643 | data = devm_kzalloc(&client->dev, | ||
644 | sizeof(struct blinkm_data), GFP_KERNEL); | ||
645 | if (!data) { | ||
646 | err = -ENOMEM; | ||
647 | goto exit; | ||
648 | } | ||
649 | |||
650 | data->i2c_addr = 0x09; | ||
651 | data->i2c_addr = 0x08; | ||
652 | /* i2c addr - use fake addr of 0x08 initially (real is 0x09) */ | ||
653 | data->fw_ver = 0xfe; | ||
654 | /* firmware version - use fake until we read real value | ||
655 | * (currently broken - BlinkM confused!) */ | ||
656 | data->script_id = 0x01; | ||
657 | data->i2c_client = client; | ||
658 | |||
659 | i2c_set_clientdata(client, data); | ||
660 | mutex_init(&data->update_lock); | ||
661 | |||
662 | /* Register sysfs hooks */ | ||
663 | err = sysfs_create_group(&client->dev.kobj, &blinkm_group); | ||
664 | if (err < 0) { | ||
665 | dev_err(&client->dev, "couldn't register sysfs group\n"); | ||
666 | goto exit; | ||
667 | } | ||
668 | |||
669 | for (i = 0; i < 3; i++) { | ||
670 | /* RED = 0, GREEN = 1, BLUE = 2 */ | ||
671 | led[i] = &data->blinkm_leds[i]; | ||
672 | led[i]->i2c_client = client; | ||
673 | led[i]->id = i; | ||
674 | led[i]->led_cdev.max_brightness = 255; | ||
675 | led[i]->led_cdev.flags = LED_CORE_SUSPENDRESUME; | ||
676 | atomic_set(&led[i]->active, 0); | ||
677 | switch (i) { | ||
678 | case RED: | ||
679 | snprintf(blinkm_led_name, sizeof(blinkm_led_name), | ||
680 | "blinkm-%d-%d-red", | ||
681 | client->adapter->nr, | ||
682 | client->addr); | ||
683 | led[i]->led_cdev.name = blinkm_led_name; | ||
684 | led[i]->led_cdev.brightness_set = blinkm_led_red_set; | ||
685 | err = led_classdev_register(&client->dev, | ||
686 | &led[i]->led_cdev); | ||
687 | if (err < 0) { | ||
688 | dev_err(&client->dev, | ||
689 | "couldn't register LED %s\n", | ||
690 | led[i]->led_cdev.name); | ||
691 | goto failred; | ||
692 | } | ||
693 | break; | ||
694 | case GREEN: | ||
695 | snprintf(blinkm_led_name, sizeof(blinkm_led_name), | ||
696 | "blinkm-%d-%d-green", | ||
697 | client->adapter->nr, | ||
698 | client->addr); | ||
699 | led[i]->led_cdev.name = blinkm_led_name; | ||
700 | led[i]->led_cdev.brightness_set = blinkm_led_green_set; | ||
701 | err = led_classdev_register(&client->dev, | ||
702 | &led[i]->led_cdev); | ||
703 | if (err < 0) { | ||
704 | dev_err(&client->dev, | ||
705 | "couldn't register LED %s\n", | ||
706 | led[i]->led_cdev.name); | ||
707 | goto failgreen; | ||
708 | } | ||
709 | break; | ||
710 | case BLUE: | ||
711 | snprintf(blinkm_led_name, sizeof(blinkm_led_name), | ||
712 | "blinkm-%d-%d-blue", | ||
713 | client->adapter->nr, | ||
714 | client->addr); | ||
715 | led[i]->led_cdev.name = blinkm_led_name; | ||
716 | led[i]->led_cdev.brightness_set = blinkm_led_blue_set; | ||
717 | err = led_classdev_register(&client->dev, | ||
718 | &led[i]->led_cdev); | ||
719 | if (err < 0) { | ||
720 | dev_err(&client->dev, | ||
721 | "couldn't register LED %s\n", | ||
722 | led[i]->led_cdev.name); | ||
723 | goto failblue; | ||
724 | } | ||
725 | break; | ||
726 | } /* end switch */ | ||
727 | } /* end for */ | ||
728 | |||
729 | /* Initialize the blinkm */ | ||
730 | blinkm_init_hw(client); | ||
731 | |||
732 | return 0; | ||
733 | |||
734 | failblue: | ||
735 | led_classdev_unregister(&led[GREEN]->led_cdev); | ||
736 | |||
737 | failgreen: | ||
738 | led_classdev_unregister(&led[RED]->led_cdev); | ||
739 | |||
740 | failred: | ||
741 | sysfs_remove_group(&client->dev.kobj, &blinkm_group); | ||
742 | exit: | ||
743 | return err; | ||
744 | } | ||
745 | |||
746 | static int __devexit blinkm_remove(struct i2c_client *client) | ||
747 | { | ||
748 | struct blinkm_data *data = i2c_get_clientdata(client); | ||
749 | int ret = 0; | ||
750 | int i; | ||
751 | |||
752 | /* make sure no workqueue entries are pending */ | ||
753 | for (i = 0; i < 3; i++) { | ||
754 | flush_scheduled_work(); | ||
755 | led_classdev_unregister(&data->blinkm_leds[i].led_cdev); | ||
756 | } | ||
757 | |||
758 | /* reset rgb */ | ||
759 | data->next_red = 0x00; | ||
760 | data->next_green = 0x00; | ||
761 | data->next_blue = 0x00; | ||
762 | ret = blinkm_transfer_hw(client, BLM_FADE_RGB); | ||
763 | if (ret < 0) | ||
764 | dev_err(&client->dev, "Failure in blinkm_remove ignored. Continuing.\n"); | ||
765 | |||
766 | /* reset hsb */ | ||
767 | data->next_hue = 0x00; | ||
768 | data->next_saturation = 0x00; | ||
769 | data->next_brightness = 0x00; | ||
770 | ret = blinkm_transfer_hw(client, BLM_FADE_HSB); | ||
771 | if (ret < 0) | ||
772 | dev_err(&client->dev, "Failure in blinkm_remove ignored. Continuing.\n"); | ||
773 | |||
774 | /* red fade to off */ | ||
775 | data->next_red = 0xff; | ||
776 | ret = blinkm_transfer_hw(client, BLM_GO_RGB); | ||
777 | if (ret < 0) | ||
778 | dev_err(&client->dev, "Failure in blinkm_remove ignored. Continuing.\n"); | ||
779 | |||
780 | /* off */ | ||
781 | data->next_red = 0x00; | ||
782 | ret = blinkm_transfer_hw(client, BLM_FADE_RGB); | ||
783 | if (ret < 0) | ||
784 | dev_err(&client->dev, "Failure in blinkm_remove ignored. Continuing.\n"); | ||
785 | |||
786 | sysfs_remove_group(&client->dev.kobj, &blinkm_group); | ||
787 | return 0; | ||
788 | } | ||
789 | |||
790 | static const struct i2c_device_id blinkm_id[] = { | ||
791 | {"blinkm", 0}, | ||
792 | {} | ||
793 | }; | ||
794 | |||
795 | MODULE_DEVICE_TABLE(i2c, blinkm_id); | ||
796 | |||
797 | /* This is the driver that will be inserted */ | ||
798 | static struct i2c_driver blinkm_driver = { | ||
799 | .class = I2C_CLASS_HWMON, | ||
800 | .driver = { | ||
801 | .name = "blinkm", | ||
802 | }, | ||
803 | .probe = blinkm_probe, | ||
804 | .remove = __devexit_p(blinkm_remove), | ||
805 | .id_table = blinkm_id, | ||
806 | .detect = blinkm_detect, | ||
807 | .address_list = normal_i2c, | ||
808 | }; | ||
809 | |||
810 | module_i2c_driver(blinkm_driver); | ||
811 | |||
812 | MODULE_AUTHOR("Jan-Simon Moeller <dl9pf@gmx.de>"); | ||
813 | MODULE_DESCRIPTION("BlinkM RGB LED driver"); | ||
814 | MODULE_LICENSE("GPL"); | ||
815 | |||
diff --git a/drivers/leds/leds-da903x.c b/drivers/leds/leds-da903x.c index d9cd73ebd6c4..cc77c9d92615 100644 --- a/drivers/leds/leds-da903x.c +++ b/drivers/leds/leds-da903x.c | |||
@@ -108,7 +108,7 @@ static int __devinit da903x_led_probe(struct platform_device *pdev) | |||
108 | return -EINVAL; | 108 | return -EINVAL; |
109 | } | 109 | } |
110 | 110 | ||
111 | led = kzalloc(sizeof(struct da903x_led), GFP_KERNEL); | 111 | led = devm_kzalloc(&pdev->dev, sizeof(struct da903x_led), GFP_KERNEL); |
112 | if (led == NULL) { | 112 | if (led == NULL) { |
113 | dev_err(&pdev->dev, "failed to alloc memory for LED%d\n", id); | 113 | dev_err(&pdev->dev, "failed to alloc memory for LED%d\n", id); |
114 | return -ENOMEM; | 114 | return -ENOMEM; |
@@ -129,15 +129,11 @@ static int __devinit da903x_led_probe(struct platform_device *pdev) | |||
129 | ret = led_classdev_register(led->master, &led->cdev); | 129 | ret = led_classdev_register(led->master, &led->cdev); |
130 | if (ret) { | 130 | if (ret) { |
131 | dev_err(&pdev->dev, "failed to register LED %d\n", id); | 131 | dev_err(&pdev->dev, "failed to register LED %d\n", id); |
132 | goto err; | 132 | return ret; |
133 | } | 133 | } |
134 | 134 | ||
135 | platform_set_drvdata(pdev, led); | 135 | platform_set_drvdata(pdev, led); |
136 | return 0; | 136 | return 0; |
137 | |||
138 | err: | ||
139 | kfree(led); | ||
140 | return ret; | ||
141 | } | 137 | } |
142 | 138 | ||
143 | static int __devexit da903x_led_remove(struct platform_device *pdev) | 139 | static int __devexit da903x_led_remove(struct platform_device *pdev) |
@@ -145,7 +141,6 @@ static int __devexit da903x_led_remove(struct platform_device *pdev) | |||
145 | struct da903x_led *led = platform_get_drvdata(pdev); | 141 | struct da903x_led *led = platform_get_drvdata(pdev); |
146 | 142 | ||
147 | led_classdev_unregister(&led->cdev); | 143 | led_classdev_unregister(&led->cdev); |
148 | kfree(led); | ||
149 | return 0; | 144 | return 0; |
150 | } | 145 | } |
151 | 146 | ||
diff --git a/drivers/leds/leds-dac124s085.c b/drivers/leds/leds-dac124s085.c index d56c14269ff0..1f9d8e62d37e 100644 --- a/drivers/leds/leds-dac124s085.c +++ b/drivers/leds/leds-dac124s085.c | |||
@@ -69,7 +69,7 @@ static int dac124s085_probe(struct spi_device *spi) | |||
69 | struct dac124s085_led *led; | 69 | struct dac124s085_led *led; |
70 | int i, ret; | 70 | int i, ret; |
71 | 71 | ||
72 | dac = kzalloc(sizeof(*dac), GFP_KERNEL); | 72 | dac = devm_kzalloc(&spi->dev, sizeof(*dac), GFP_KERNEL); |
73 | if (!dac) | 73 | if (!dac) |
74 | return -ENOMEM; | 74 | return -ENOMEM; |
75 | 75 | ||
@@ -102,7 +102,6 @@ eledcr: | |||
102 | led_classdev_unregister(&dac->leds[i].ldev); | 102 | led_classdev_unregister(&dac->leds[i].ldev); |
103 | 103 | ||
104 | spi_set_drvdata(spi, NULL); | 104 | spi_set_drvdata(spi, NULL); |
105 | kfree(dac); | ||
106 | return ret; | 105 | return ret; |
107 | } | 106 | } |
108 | 107 | ||
@@ -117,7 +116,6 @@ static int dac124s085_remove(struct spi_device *spi) | |||
117 | } | 116 | } |
118 | 117 | ||
119 | spi_set_drvdata(spi, NULL); | 118 | spi_set_drvdata(spi, NULL); |
120 | kfree(dac); | ||
121 | 119 | ||
122 | return 0; | 120 | return 0; |
123 | } | 121 | } |
diff --git a/drivers/leds/leds-gpio.c b/drivers/leds/leds-gpio.c index f4c470a3bc8d..c032b2180340 100644 --- a/drivers/leds/leds-gpio.c +++ b/drivers/leds/leds-gpio.c | |||
@@ -178,7 +178,8 @@ static struct gpio_leds_priv * __devinit gpio_leds_create_of(struct platform_dev | |||
178 | if (!count) | 178 | if (!count) |
179 | return NULL; | 179 | return NULL; |
180 | 180 | ||
181 | priv = kzalloc(sizeof_gpio_leds_priv(count), GFP_KERNEL); | 181 | priv = devm_kzalloc(&pdev->dev, sizeof_gpio_leds_priv(count), |
182 | GFP_KERNEL); | ||
182 | if (!priv) | 183 | if (!priv) |
183 | return NULL; | 184 | return NULL; |
184 | 185 | ||
@@ -215,7 +216,6 @@ static struct gpio_leds_priv * __devinit gpio_leds_create_of(struct platform_dev | |||
215 | err: | 216 | err: |
216 | for (count = priv->num_leds - 2; count >= 0; count--) | 217 | for (count = priv->num_leds - 2; count >= 0; count--) |
217 | delete_gpio_led(&priv->leds[count]); | 218 | delete_gpio_led(&priv->leds[count]); |
218 | kfree(priv); | ||
219 | return NULL; | 219 | return NULL; |
220 | } | 220 | } |
221 | 221 | ||
@@ -239,8 +239,9 @@ static int __devinit gpio_led_probe(struct platform_device *pdev) | |||
239 | int i, ret = 0; | 239 | int i, ret = 0; |
240 | 240 | ||
241 | if (pdata && pdata->num_leds) { | 241 | if (pdata && pdata->num_leds) { |
242 | priv = kzalloc(sizeof_gpio_leds_priv(pdata->num_leds), | 242 | priv = devm_kzalloc(&pdev->dev, |
243 | GFP_KERNEL); | 243 | sizeof_gpio_leds_priv(pdata->num_leds), |
244 | GFP_KERNEL); | ||
244 | if (!priv) | 245 | if (!priv) |
245 | return -ENOMEM; | 246 | return -ENOMEM; |
246 | 247 | ||
@@ -253,7 +254,6 @@ static int __devinit gpio_led_probe(struct platform_device *pdev) | |||
253 | /* On failure: unwind the led creations */ | 254 | /* On failure: unwind the led creations */ |
254 | for (i = i - 1; i >= 0; i--) | 255 | for (i = i - 1; i >= 0; i--) |
255 | delete_gpio_led(&priv->leds[i]); | 256 | delete_gpio_led(&priv->leds[i]); |
256 | kfree(priv); | ||
257 | return ret; | 257 | return ret; |
258 | } | 258 | } |
259 | } | 259 | } |
@@ -277,7 +277,6 @@ static int __devexit gpio_led_remove(struct platform_device *pdev) | |||
277 | delete_gpio_led(&priv->leds[i]); | 277 | delete_gpio_led(&priv->leds[i]); |
278 | 278 | ||
279 | dev_set_drvdata(&pdev->dev, NULL); | 279 | dev_set_drvdata(&pdev->dev, NULL); |
280 | kfree(priv); | ||
281 | 280 | ||
282 | return 0; | 281 | return 0; |
283 | } | 282 | } |
diff --git a/drivers/leds/leds-lm3530.c b/drivers/leds/leds-lm3530.c index 84ba6de8039c..23637bdb275d 100644 --- a/drivers/leds/leds-lm3530.c +++ b/drivers/leds/leds-lm3530.c | |||
@@ -386,28 +386,24 @@ static int __devinit lm3530_probe(struct i2c_client *client, | |||
386 | 386 | ||
387 | if (pdata == NULL) { | 387 | if (pdata == NULL) { |
388 | dev_err(&client->dev, "platform data required\n"); | 388 | dev_err(&client->dev, "platform data required\n"); |
389 | err = -ENODEV; | 389 | return -ENODEV; |
390 | goto err_out; | ||
391 | } | 390 | } |
392 | 391 | ||
393 | /* BL mode */ | 392 | /* BL mode */ |
394 | if (pdata->mode > LM3530_BL_MODE_PWM) { | 393 | if (pdata->mode > LM3530_BL_MODE_PWM) { |
395 | dev_err(&client->dev, "Illegal Mode request\n"); | 394 | dev_err(&client->dev, "Illegal Mode request\n"); |
396 | err = -EINVAL; | 395 | return -EINVAL; |
397 | goto err_out; | ||
398 | } | 396 | } |
399 | 397 | ||
400 | if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) { | 398 | if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) { |
401 | dev_err(&client->dev, "I2C_FUNC_I2C not supported\n"); | 399 | dev_err(&client->dev, "I2C_FUNC_I2C not supported\n"); |
402 | err = -EIO; | 400 | return -EIO; |
403 | goto err_out; | ||
404 | } | 401 | } |
405 | 402 | ||
406 | drvdata = kzalloc(sizeof(struct lm3530_data), GFP_KERNEL); | 403 | drvdata = devm_kzalloc(&client->dev, sizeof(struct lm3530_data), |
407 | if (drvdata == NULL) { | 404 | GFP_KERNEL); |
408 | err = -ENOMEM; | 405 | if (drvdata == NULL) |
409 | goto err_out; | 406 | return -ENOMEM; |
410 | } | ||
411 | 407 | ||
412 | drvdata->mode = pdata->mode; | 408 | drvdata->mode = pdata->mode; |
413 | drvdata->client = client; | 409 | drvdata->client = client; |
@@ -425,7 +421,7 @@ static int __devinit lm3530_probe(struct i2c_client *client, | |||
425 | dev_err(&client->dev, "regulator get failed\n"); | 421 | dev_err(&client->dev, "regulator get failed\n"); |
426 | err = PTR_ERR(drvdata->regulator); | 422 | err = PTR_ERR(drvdata->regulator); |
427 | drvdata->regulator = NULL; | 423 | drvdata->regulator = NULL; |
428 | goto err_regulator_get; | 424 | return err; |
429 | } | 425 | } |
430 | 426 | ||
431 | if (drvdata->pdata->brt_val) { | 427 | if (drvdata->pdata->brt_val) { |
@@ -458,9 +454,6 @@ err_create_file: | |||
458 | err_class_register: | 454 | err_class_register: |
459 | err_reg_init: | 455 | err_reg_init: |
460 | regulator_put(drvdata->regulator); | 456 | regulator_put(drvdata->regulator); |
461 | err_regulator_get: | ||
462 | kfree(drvdata); | ||
463 | err_out: | ||
464 | return err; | 457 | return err; |
465 | } | 458 | } |
466 | 459 | ||
@@ -474,7 +467,6 @@ static int __devexit lm3530_remove(struct i2c_client *client) | |||
474 | regulator_disable(drvdata->regulator); | 467 | regulator_disable(drvdata->regulator); |
475 | regulator_put(drvdata->regulator); | 468 | regulator_put(drvdata->regulator); |
476 | led_classdev_unregister(&drvdata->led_dev); | 469 | led_classdev_unregister(&drvdata->led_dev); |
477 | kfree(drvdata); | ||
478 | return 0; | 470 | return 0; |
479 | } | 471 | } |
480 | 472 | ||
diff --git a/drivers/leds/leds-lm3556.c b/drivers/leds/leds-lm3556.c new file mode 100644 index 000000000000..3062abd9a532 --- /dev/null +++ b/drivers/leds/leds-lm3556.c | |||
@@ -0,0 +1,512 @@ | |||
1 | /* | ||
2 | * Simple driver for Texas Instruments LM3556 LED Flash driver chip (Rev0x03) | ||
3 | * Copyright (C) 2012 Texas Instruments | ||
4 | * | ||
5 | * This program is free software; you can redistribute it and/or modify | ||
6 | * it under the terms of the GNU General Public License version 2 as | ||
7 | * published by the Free Software Foundation. | ||
8 | * | ||
9 | * Please refer Documentation/leds/leds-lm3556.txt file. | ||
10 | */ | ||
11 | #include <linux/module.h> | ||
12 | #include <linux/delay.h> | ||
13 | #include <linux/i2c.h> | ||
14 | #include <linux/leds.h> | ||
15 | #include <linux/slab.h> | ||
16 | #include <linux/platform_device.h> | ||
17 | #include <linux/fs.h> | ||
18 | #include <linux/regmap.h> | ||
19 | #include <linux/platform_data/leds-lm3556.h> | ||
20 | |||
21 | #define REG_FILT_TIME (0x0) | ||
22 | #define REG_IVFM_MODE (0x1) | ||
23 | #define REG_NTC (0x2) | ||
24 | #define REG_INDIC_TIME (0x3) | ||
25 | #define REG_INDIC_BLINK (0x4) | ||
26 | #define REG_INDIC_PERIOD (0x5) | ||
27 | #define REG_TORCH_TIME (0x6) | ||
28 | #define REG_CONF (0x7) | ||
29 | #define REG_FLASH (0x8) | ||
30 | #define REG_I_CTRL (0x9) | ||
31 | #define REG_ENABLE (0xA) | ||
32 | #define REG_FLAG (0xB) | ||
33 | #define REG_MAX (0xB) | ||
34 | |||
35 | #define IVFM_FILTER_TIME_SHIFT (3) | ||
36 | #define UVLO_EN_SHIFT (7) | ||
37 | #define HYSTERSIS_SHIFT (5) | ||
38 | #define IVM_D_TH_SHIFT (2) | ||
39 | #define IVFM_ADJ_MODE_SHIFT (0) | ||
40 | #define NTC_EVENT_LVL_SHIFT (5) | ||
41 | #define NTC_TRIP_TH_SHIFT (2) | ||
42 | #define NTC_BIAS_I_LVL_SHIFT (0) | ||
43 | #define INDIC_RAMP_UP_TIME_SHIFT (3) | ||
44 | #define INDIC_RAMP_DN_TIME_SHIFT (0) | ||
45 | #define INDIC_N_BLANK_SHIFT (4) | ||
46 | #define INDIC_PULSE_TIME_SHIFT (0) | ||
47 | #define INDIC_N_PERIOD_SHIFT (0) | ||
48 | #define TORCH_RAMP_UP_TIME_SHIFT (3) | ||
49 | #define TORCH_RAMP_DN_TIME_SHIFT (0) | ||
50 | #define STROBE_USUAGE_SHIFT (7) | ||
51 | #define STROBE_PIN_POLARITY_SHIFT (6) | ||
52 | #define TORCH_PIN_POLARITY_SHIFT (5) | ||
53 | #define TX_PIN_POLARITY_SHIFT (4) | ||
54 | #define TX_EVENT_LVL_SHIFT (3) | ||
55 | #define IVFM_EN_SHIFT (2) | ||
56 | #define NTC_MODE_SHIFT (1) | ||
57 | #define INDIC_MODE_SHIFT (0) | ||
58 | #define INDUCTOR_I_LIMIT_SHIFT (6) | ||
59 | #define FLASH_RAMP_TIME_SHIFT (3) | ||
60 | #define FLASH_TOUT_TIME_SHIFT (0) | ||
61 | #define TORCH_I_SHIFT (4) | ||
62 | #define FLASH_I_SHIFT (0) | ||
63 | #define NTC_EN_SHIFT (7) | ||
64 | #define TX_PIN_EN_SHIFT (6) | ||
65 | #define STROBE_PIN_EN_SHIFT (5) | ||
66 | #define TORCH_PIN_EN_SHIFT (4) | ||
67 | #define PRECHG_MODE_EN_SHIFT (3) | ||
68 | #define PASS_MODE_ONLY_EN_SHIFT (2) | ||
69 | #define MODE_BITS_SHIFT (0) | ||
70 | |||
71 | #define IVFM_FILTER_TIME_MASK (0x3) | ||
72 | #define UVLO_EN_MASK (0x1) | ||
73 | #define HYSTERSIS_MASK (0x3) | ||
74 | #define IVM_D_TH_MASK (0x7) | ||
75 | #define IVFM_ADJ_MODE_MASK (0x3) | ||
76 | #define NTC_EVENT_LVL_MASK (0x1) | ||
77 | #define NTC_TRIP_TH_MASK (0x7) | ||
78 | #define NTC_BIAS_I_LVL_MASK (0x3) | ||
79 | #define INDIC_RAMP_UP_TIME_MASK (0x7) | ||
80 | #define INDIC_RAMP_DN_TIME_MASK (0x7) | ||
81 | #define INDIC_N_BLANK_MASK (0x7) | ||
82 | #define INDIC_PULSE_TIME_MASK (0x7) | ||
83 | #define INDIC_N_PERIOD_MASK (0x7) | ||
84 | #define TORCH_RAMP_UP_TIME_MASK (0x7) | ||
85 | #define TORCH_RAMP_DN_TIME_MASK (0x7) | ||
86 | #define STROBE_USUAGE_MASK (0x1) | ||
87 | #define STROBE_PIN_POLARITY_MASK (0x1) | ||
88 | #define TORCH_PIN_POLARITY_MASK (0x1) | ||
89 | #define TX_PIN_POLARITY_MASK (0x1) | ||
90 | #define TX_EVENT_LVL_MASK (0x1) | ||
91 | #define IVFM_EN_MASK (0x1) | ||
92 | #define NTC_MODE_MASK (0x1) | ||
93 | #define INDIC_MODE_MASK (0x1) | ||
94 | #define INDUCTOR_I_LIMIT_MASK (0x3) | ||
95 | #define FLASH_RAMP_TIME_MASK (0x7) | ||
96 | #define FLASH_TOUT_TIME_MASK (0x7) | ||
97 | #define TORCH_I_MASK (0x7) | ||
98 | #define FLASH_I_MASK (0xF) | ||
99 | #define NTC_EN_MASK (0x1) | ||
100 | #define TX_PIN_EN_MASK (0x1) | ||
101 | #define STROBE_PIN_EN_MASK (0x1) | ||
102 | #define TORCH_PIN_EN_MASK (0x1) | ||
103 | #define PRECHG_MODE_EN_MASK (0x1) | ||
104 | #define PASS_MODE_ONLY_EN_MASK (0x1) | ||
105 | #define MODE_BITS_MASK (0x13) | ||
106 | #define EX_PIN_CONTROL_MASK (0xF1) | ||
107 | #define EX_PIN_ENABLE_MASK (0x70) | ||
108 | |||
109 | enum lm3556_indic_pulse_time { | ||
110 | PULSE_TIME_0_MS = 0, | ||
111 | PULSE_TIME_32_MS, | ||
112 | PULSE_TIME_64_MS, | ||
113 | PULSE_TIME_92_MS, | ||
114 | PULSE_TIME_128_MS, | ||
115 | PULSE_TIME_160_MS, | ||
116 | PULSE_TIME_196_MS, | ||
117 | PULSE_TIME_224_MS, | ||
118 | PULSE_TIME_256_MS, | ||
119 | PULSE_TIME_288_MS, | ||
120 | PULSE_TIME_320_MS, | ||
121 | PULSE_TIME_352_MS, | ||
122 | PULSE_TIME_384_MS, | ||
123 | PULSE_TIME_416_MS, | ||
124 | PULSE_TIME_448_MS, | ||
125 | PULSE_TIME_480_MS, | ||
126 | }; | ||
127 | |||
128 | enum lm3556_indic_n_blank { | ||
129 | INDIC_N_BLANK_0 = 0, | ||
130 | INDIC_N_BLANK_1, | ||
131 | INDIC_N_BLANK_2, | ||
132 | INDIC_N_BLANK_3, | ||
133 | INDIC_N_BLANK_4, | ||
134 | INDIC_N_BLANK_5, | ||
135 | INDIC_N_BLANK_6, | ||
136 | INDIC_N_BLANK_7, | ||
137 | INDIC_N_BLANK_8, | ||
138 | INDIC_N_BLANK_9, | ||
139 | INDIC_N_BLANK_10, | ||
140 | INDIC_N_BLANK_11, | ||
141 | INDIC_N_BLANK_12, | ||
142 | INDIC_N_BLANK_13, | ||
143 | INDIC_N_BLANK_14, | ||
144 | INDIC_N_BLANK_15, | ||
145 | }; | ||
146 | |||
147 | enum lm3556_indic_period { | ||
148 | INDIC_PERIOD_0 = 0, | ||
149 | INDIC_PERIOD_1, | ||
150 | INDIC_PERIOD_2, | ||
151 | INDIC_PERIOD_3, | ||
152 | INDIC_PERIOD_4, | ||
153 | INDIC_PERIOD_5, | ||
154 | INDIC_PERIOD_6, | ||
155 | INDIC_PERIOD_7, | ||
156 | }; | ||
157 | |||
158 | enum lm3556_mode { | ||
159 | MODES_STASNDBY = 0, | ||
160 | MODES_INDIC, | ||
161 | MODES_TORCH, | ||
162 | MODES_FLASH | ||
163 | }; | ||
164 | |||
165 | #define INDIC_PATTERN_SIZE 4 | ||
166 | |||
167 | struct indicator { | ||
168 | u8 blinking; | ||
169 | u8 period_cnt; | ||
170 | }; | ||
171 | |||
172 | struct lm3556_chip_data { | ||
173 | struct device *dev; | ||
174 | |||
175 | struct led_classdev cdev_flash; | ||
176 | struct led_classdev cdev_torch; | ||
177 | struct led_classdev cdev_indicator; | ||
178 | |||
179 | struct lm3556_platform_data *pdata; | ||
180 | struct regmap *regmap; | ||
181 | struct mutex lock; | ||
182 | |||
183 | unsigned int last_flag; | ||
184 | }; | ||
185 | |||
186 | /* indicator pattern */ | ||
187 | static struct indicator indicator_pattern[INDIC_PATTERN_SIZE] = { | ||
188 | [0] = {(INDIC_N_BLANK_1 << INDIC_N_BLANK_SHIFT) | ||
189 | | PULSE_TIME_32_MS, INDIC_PERIOD_1}, | ||
190 | [1] = {(INDIC_N_BLANK_15 << INDIC_N_BLANK_SHIFT) | ||
191 | | PULSE_TIME_32_MS, INDIC_PERIOD_2}, | ||
192 | [2] = {(INDIC_N_BLANK_10 << INDIC_N_BLANK_SHIFT) | ||
193 | | PULSE_TIME_32_MS, INDIC_PERIOD_4}, | ||
194 | [3] = {(INDIC_N_BLANK_5 << INDIC_N_BLANK_SHIFT) | ||
195 | | PULSE_TIME_32_MS, INDIC_PERIOD_7}, | ||
196 | }; | ||
197 | |||
198 | /* chip initialize */ | ||
199 | static int __devinit lm3556_chip_init(struct lm3556_chip_data *chip) | ||
200 | { | ||
201 | unsigned int reg_val; | ||
202 | int ret; | ||
203 | struct lm3556_platform_data *pdata = chip->pdata; | ||
204 | |||
205 | /* set config register */ | ||
206 | ret = regmap_read(chip->regmap, REG_CONF, ®_val); | ||
207 | if (ret < 0) { | ||
208 | dev_err(chip->dev, "Failed to read REG_CONF Register\n"); | ||
209 | goto out; | ||
210 | } | ||
211 | |||
212 | reg_val &= (~EX_PIN_CONTROL_MASK); | ||
213 | reg_val |= ((pdata->torch_pin_polarity & 0x01) | ||
214 | << TORCH_PIN_POLARITY_SHIFT); | ||
215 | reg_val |= ((pdata->strobe_usuage & 0x01) << STROBE_USUAGE_SHIFT); | ||
216 | reg_val |= ((pdata->strobe_pin_polarity & 0x01) | ||
217 | << STROBE_PIN_POLARITY_SHIFT); | ||
218 | reg_val |= ((pdata->tx_pin_polarity & 0x01) << TX_PIN_POLARITY_SHIFT); | ||
219 | reg_val |= ((pdata->indicator_mode & 0x01) << INDIC_MODE_SHIFT); | ||
220 | |||
221 | ret = regmap_write(chip->regmap, REG_CONF, reg_val); | ||
222 | if (ret < 0) { | ||
223 | dev_err(chip->dev, "Failed to write REG_CONF Regisgter\n"); | ||
224 | goto out; | ||
225 | } | ||
226 | |||
227 | /* set enable register */ | ||
228 | ret = regmap_read(chip->regmap, REG_ENABLE, ®_val); | ||
229 | if (ret < 0) { | ||
230 | dev_err(chip->dev, "Failed to read REG_ENABLE Register\n"); | ||
231 | goto out; | ||
232 | } | ||
233 | |||
234 | reg_val &= (~EX_PIN_ENABLE_MASK); | ||
235 | reg_val |= ((pdata->torch_pin_en & 0x01) << TORCH_PIN_EN_SHIFT); | ||
236 | reg_val |= ((pdata->strobe_pin_en & 0x01) << STROBE_PIN_EN_SHIFT); | ||
237 | reg_val |= ((pdata->tx_pin_en & 0x01) << TX_PIN_EN_SHIFT); | ||
238 | |||
239 | ret = regmap_write(chip->regmap, REG_ENABLE, reg_val); | ||
240 | if (ret < 0) { | ||
241 | dev_err(chip->dev, "Failed to write REG_ENABLE Regisgter\n"); | ||
242 | goto out; | ||
243 | } | ||
244 | |||
245 | out: | ||
246 | return ret; | ||
247 | } | ||
248 | |||
249 | /* chip control */ | ||
250 | static int lm3556_control(struct lm3556_chip_data *chip, | ||
251 | u8 brightness, enum lm3556_mode opmode) | ||
252 | { | ||
253 | int ret; | ||
254 | struct lm3556_platform_data *pdata = chip->pdata; | ||
255 | |||
256 | ret = regmap_read(chip->regmap, REG_FLAG, &chip->last_flag); | ||
257 | if (ret < 0) { | ||
258 | dev_err(chip->dev, "Failed to read REG_FLAG Register\n"); | ||
259 | goto out; | ||
260 | } | ||
261 | |||
262 | if (chip->last_flag) | ||
263 | dev_info(chip->dev, "Last FLAG is 0x%x\n", chip->last_flag); | ||
264 | |||
265 | /* brightness 0 means off-state */ | ||
266 | if (!brightness) | ||
267 | opmode = MODES_STASNDBY; | ||
268 | |||
269 | switch (opmode) { | ||
270 | case MODES_TORCH: | ||
271 | ret = regmap_update_bits(chip->regmap, REG_I_CTRL, | ||
272 | TORCH_I_MASK << TORCH_I_SHIFT, | ||
273 | (brightness - 1) << TORCH_I_SHIFT); | ||
274 | |||
275 | if (pdata->torch_pin_en) | ||
276 | opmode |= (TORCH_PIN_EN_MASK << TORCH_PIN_EN_SHIFT); | ||
277 | break; | ||
278 | |||
279 | case MODES_FLASH: | ||
280 | ret = regmap_update_bits(chip->regmap, REG_I_CTRL, | ||
281 | FLASH_I_MASK << FLASH_I_SHIFT, | ||
282 | (brightness - 1) << FLASH_I_SHIFT); | ||
283 | break; | ||
284 | |||
285 | case MODES_INDIC: | ||
286 | ret = regmap_update_bits(chip->regmap, REG_I_CTRL, | ||
287 | TORCH_I_MASK << TORCH_I_SHIFT, | ||
288 | (brightness - 1) << TORCH_I_SHIFT); | ||
289 | break; | ||
290 | |||
291 | case MODES_STASNDBY: | ||
292 | if (pdata->torch_pin_en) | ||
293 | opmode |= (TORCH_PIN_EN_MASK << TORCH_PIN_EN_SHIFT); | ||
294 | break; | ||
295 | |||
296 | default: | ||
297 | return ret; | ||
298 | } | ||
299 | if (ret < 0) { | ||
300 | dev_err(chip->dev, "Failed to write REG_I_CTRL Register\n"); | ||
301 | goto out; | ||
302 | } | ||
303 | ret = regmap_update_bits(chip->regmap, REG_ENABLE, | ||
304 | MODE_BITS_MASK << MODE_BITS_SHIFT, | ||
305 | opmode << MODE_BITS_SHIFT); | ||
306 | |||
307 | out: | ||
308 | return ret; | ||
309 | } | ||
310 | |||
311 | /* torch */ | ||
312 | static void lm3556_torch_brightness_set(struct led_classdev *cdev, | ||
313 | enum led_brightness brightness) | ||
314 | { | ||
315 | struct lm3556_chip_data *chip = | ||
316 | container_of(cdev, struct lm3556_chip_data, cdev_torch); | ||
317 | |||
318 | mutex_lock(&chip->lock); | ||
319 | lm3556_control(chip, brightness, MODES_TORCH); | ||
320 | mutex_unlock(&chip->lock); | ||
321 | } | ||
322 | |||
323 | /* flash */ | ||
324 | static void lm3556_strobe_brightness_set(struct led_classdev *cdev, | ||
325 | enum led_brightness brightness) | ||
326 | { | ||
327 | struct lm3556_chip_data *chip = | ||
328 | container_of(cdev, struct lm3556_chip_data, cdev_flash); | ||
329 | |||
330 | mutex_lock(&chip->lock); | ||
331 | lm3556_control(chip, brightness, MODES_FLASH); | ||
332 | mutex_unlock(&chip->lock); | ||
333 | } | ||
334 | |||
335 | /* indicator */ | ||
336 | static void lm3556_indicator_brightness_set(struct led_classdev *cdev, | ||
337 | enum led_brightness brightness) | ||
338 | { | ||
339 | struct lm3556_chip_data *chip = | ||
340 | container_of(cdev, struct lm3556_chip_data, cdev_indicator); | ||
341 | |||
342 | mutex_lock(&chip->lock); | ||
343 | lm3556_control(chip, brightness, MODES_INDIC); | ||
344 | mutex_unlock(&chip->lock); | ||
345 | } | ||
346 | |||
347 | /* indicator pattern */ | ||
348 | static ssize_t lm3556_indicator_pattern_store(struct device *dev, | ||
349 | struct device_attribute *devAttr, | ||
350 | const char *buf, size_t size) | ||
351 | { | ||
352 | ssize_t ret; | ||
353 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
354 | struct lm3556_chip_data *chip = | ||
355 | container_of(led_cdev, struct lm3556_chip_data, cdev_indicator); | ||
356 | unsigned int state; | ||
357 | |||
358 | ret = kstrtouint(buf, 10, &state); | ||
359 | if (ret) | ||
360 | goto out; | ||
361 | if (state > INDIC_PATTERN_SIZE - 1) | ||
362 | state = INDIC_PATTERN_SIZE - 1; | ||
363 | |||
364 | ret = regmap_write(chip->regmap, REG_INDIC_BLINK, | ||
365 | indicator_pattern[state].blinking); | ||
366 | if (ret < 0) { | ||
367 | dev_err(chip->dev, "Failed to write REG_ENABLE Regisgter\n"); | ||
368 | goto out; | ||
369 | } | ||
370 | |||
371 | ret = regmap_write(chip->regmap, REG_INDIC_PERIOD, | ||
372 | indicator_pattern[state].period_cnt); | ||
373 | if (ret < 0) { | ||
374 | dev_err(chip->dev, "Failed to write REG_ENABLE Regisgter\n"); | ||
375 | goto out; | ||
376 | } | ||
377 | |||
378 | return size; | ||
379 | out: | ||
380 | dev_err(chip->dev, "Indicator pattern doesn't saved\n"); | ||
381 | return size; | ||
382 | } | ||
383 | |||
384 | static DEVICE_ATTR(pattern, 0666, NULL, lm3556_indicator_pattern_store); | ||
385 | |||
386 | static const struct regmap_config lm3556_regmap = { | ||
387 | .reg_bits = 8, | ||
388 | .val_bits = 8, | ||
389 | .max_register = REG_MAX, | ||
390 | }; | ||
391 | |||
392 | /* module initialize */ | ||
393 | static int __devinit lm3556_probe(struct i2c_client *client, | ||
394 | const struct i2c_device_id *id) | ||
395 | { | ||
396 | struct lm3556_platform_data *pdata = client->dev.platform_data; | ||
397 | struct lm3556_chip_data *chip; | ||
398 | |||
399 | int err; | ||
400 | |||
401 | if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) { | ||
402 | dev_err(&client->dev, "i2c functionality check fail.\n"); | ||
403 | return -EOPNOTSUPP; | ||
404 | } | ||
405 | |||
406 | if (pdata == NULL) { | ||
407 | dev_err(&client->dev, "Needs Platform Data.\n"); | ||
408 | return -ENODATA; | ||
409 | } | ||
410 | |||
411 | chip = | ||
412 | devm_kzalloc(&client->dev, sizeof(struct lm3556_chip_data), | ||
413 | GFP_KERNEL); | ||
414 | if (!chip) | ||
415 | return -ENOMEM; | ||
416 | |||
417 | chip->dev = &client->dev; | ||
418 | chip->pdata = pdata; | ||
419 | |||
420 | chip->regmap = devm_regmap_init_i2c(client, &lm3556_regmap); | ||
421 | if (IS_ERR(chip->regmap)) { | ||
422 | err = PTR_ERR(chip->regmap); | ||
423 | dev_err(&client->dev, "Failed to allocate register map: %d\n", | ||
424 | err); | ||
425 | return err; | ||
426 | } | ||
427 | |||
428 | mutex_init(&chip->lock); | ||
429 | i2c_set_clientdata(client, chip); | ||
430 | |||
431 | err = lm3556_chip_init(chip); | ||
432 | if (err < 0) | ||
433 | goto err_out; | ||
434 | |||
435 | /* flash */ | ||
436 | chip->cdev_flash.name = "flash"; | ||
437 | chip->cdev_flash.max_brightness = 16; | ||
438 | chip->cdev_flash.brightness_set = lm3556_strobe_brightness_set; | ||
439 | err = led_classdev_register((struct device *) | ||
440 | &client->dev, &chip->cdev_flash); | ||
441 | if (err < 0) | ||
442 | goto err_out; | ||
443 | /* torch */ | ||
444 | chip->cdev_torch.name = "torch"; | ||
445 | chip->cdev_torch.max_brightness = 8; | ||
446 | chip->cdev_torch.brightness_set = lm3556_torch_brightness_set; | ||
447 | err = led_classdev_register((struct device *) | ||
448 | &client->dev, &chip->cdev_torch); | ||
449 | if (err < 0) | ||
450 | goto err_create_torch_file; | ||
451 | /* indicator */ | ||
452 | chip->cdev_indicator.name = "indicator"; | ||
453 | chip->cdev_indicator.max_brightness = 8; | ||
454 | chip->cdev_indicator.brightness_set = lm3556_indicator_brightness_set; | ||
455 | err = led_classdev_register((struct device *) | ||
456 | &client->dev, &chip->cdev_indicator); | ||
457 | if (err < 0) | ||
458 | goto err_create_indicator_file; | ||
459 | |||
460 | err = device_create_file(chip->cdev_indicator.dev, &dev_attr_pattern); | ||
461 | if (err < 0) | ||
462 | goto err_create_pattern_file; | ||
463 | |||
464 | dev_info(&client->dev, "LM3556 is initialized\n"); | ||
465 | return 0; | ||
466 | |||
467 | err_create_pattern_file: | ||
468 | led_classdev_unregister(&chip->cdev_indicator); | ||
469 | err_create_indicator_file: | ||
470 | led_classdev_unregister(&chip->cdev_torch); | ||
471 | err_create_torch_file: | ||
472 | led_classdev_unregister(&chip->cdev_flash); | ||
473 | err_out: | ||
474 | return err; | ||
475 | } | ||
476 | |||
477 | static int __devexit lm3556_remove(struct i2c_client *client) | ||
478 | { | ||
479 | struct lm3556_chip_data *chip = i2c_get_clientdata(client); | ||
480 | |||
481 | device_remove_file(chip->cdev_indicator.dev, &dev_attr_pattern); | ||
482 | led_classdev_unregister(&chip->cdev_indicator); | ||
483 | led_classdev_unregister(&chip->cdev_torch); | ||
484 | led_classdev_unregister(&chip->cdev_flash); | ||
485 | regmap_write(chip->regmap, REG_ENABLE, 0); | ||
486 | return 0; | ||
487 | } | ||
488 | |||
489 | static const struct i2c_device_id lm3556_id[] = { | ||
490 | {LM3556_NAME, 0}, | ||
491 | {} | ||
492 | }; | ||
493 | |||
494 | MODULE_DEVICE_TABLE(i2c, lm3556_id); | ||
495 | |||
496 | static struct i2c_driver lm3556_i2c_driver = { | ||
497 | .driver = { | ||
498 | .name = LM3556_NAME, | ||
499 | .owner = THIS_MODULE, | ||
500 | .pm = NULL, | ||
501 | }, | ||
502 | .probe = lm3556_probe, | ||
503 | .remove = __devexit_p(lm3556_remove), | ||
504 | .id_table = lm3556_id, | ||
505 | }; | ||
506 | |||
507 | module_i2c_driver(lm3556_i2c_driver); | ||
508 | |||
509 | MODULE_DESCRIPTION("Texas Instruments Flash Lighting driver for LM3556"); | ||
510 | MODULE_AUTHOR("Daniel Jeong <daniel.jeong@ti.com>"); | ||
511 | MODULE_AUTHOR("G.Shark Jeong <gshark.jeong@gmail.com>"); | ||
512 | MODULE_LICENSE("GPL v2"); | ||
diff --git a/drivers/leds/leds-lp3944.c b/drivers/leds/leds-lp3944.c index b8f9f0a5d431..c298f7d9f535 100644 --- a/drivers/leds/leds-lp3944.c +++ b/drivers/leds/leds-lp3944.c | |||
@@ -393,7 +393,8 @@ static int __devinit lp3944_probe(struct i2c_client *client, | |||
393 | return -ENODEV; | 393 | return -ENODEV; |
394 | } | 394 | } |
395 | 395 | ||
396 | data = kzalloc(sizeof(struct lp3944_data), GFP_KERNEL); | 396 | data = devm_kzalloc(&client->dev, sizeof(struct lp3944_data), |
397 | GFP_KERNEL); | ||
397 | if (!data) | 398 | if (!data) |
398 | return -ENOMEM; | 399 | return -ENOMEM; |
399 | 400 | ||
@@ -403,10 +404,8 @@ static int __devinit lp3944_probe(struct i2c_client *client, | |||
403 | mutex_init(&data->lock); | 404 | mutex_init(&data->lock); |
404 | 405 | ||
405 | err = lp3944_configure(client, data, lp3944_pdata); | 406 | err = lp3944_configure(client, data, lp3944_pdata); |
406 | if (err < 0) { | 407 | if (err < 0) |
407 | kfree(data); | ||
408 | return err; | 408 | return err; |
409 | } | ||
410 | 409 | ||
411 | dev_info(&client->dev, "lp3944 enabled\n"); | 410 | dev_info(&client->dev, "lp3944 enabled\n"); |
412 | return 0; | 411 | return 0; |
@@ -431,8 +430,6 @@ static int __devexit lp3944_remove(struct i2c_client *client) | |||
431 | break; | 430 | break; |
432 | } | 431 | } |
433 | 432 | ||
434 | kfree(data); | ||
435 | |||
436 | return 0; | 433 | return 0; |
437 | } | 434 | } |
438 | 435 | ||
diff --git a/drivers/leds/leds-lp5521.c b/drivers/leds/leds-lp5521.c index 23815624f35e..2064aefedc07 100644 --- a/drivers/leds/leds-lp5521.c +++ b/drivers/leds/leds-lp5521.c | |||
@@ -744,7 +744,7 @@ static int __devinit lp5521_probe(struct i2c_client *client, | |||
744 | int ret, i, led; | 744 | int ret, i, led; |
745 | u8 buf; | 745 | u8 buf; |
746 | 746 | ||
747 | chip = kzalloc(sizeof(*chip), GFP_KERNEL); | 747 | chip = devm_kzalloc(&client->dev, sizeof(*chip), GFP_KERNEL); |
748 | if (!chip) | 748 | if (!chip) |
749 | return -ENOMEM; | 749 | return -ENOMEM; |
750 | 750 | ||
@@ -755,8 +755,7 @@ static int __devinit lp5521_probe(struct i2c_client *client, | |||
755 | 755 | ||
756 | if (!pdata) { | 756 | if (!pdata) { |
757 | dev_err(&client->dev, "no platform data\n"); | 757 | dev_err(&client->dev, "no platform data\n"); |
758 | ret = -EINVAL; | 758 | return -EINVAL; |
759 | goto fail1; | ||
760 | } | 759 | } |
761 | 760 | ||
762 | mutex_init(&chip->lock); | 761 | mutex_init(&chip->lock); |
@@ -766,7 +765,7 @@ static int __devinit lp5521_probe(struct i2c_client *client, | |||
766 | if (pdata->setup_resources) { | 765 | if (pdata->setup_resources) { |
767 | ret = pdata->setup_resources(); | 766 | ret = pdata->setup_resources(); |
768 | if (ret < 0) | 767 | if (ret < 0) |
769 | goto fail1; | 768 | return ret; |
770 | } | 769 | } |
771 | 770 | ||
772 | if (pdata->enable) { | 771 | if (pdata->enable) { |
@@ -807,7 +806,7 @@ static int __devinit lp5521_probe(struct i2c_client *client, | |||
807 | ret = lp5521_configure(client); | 806 | ret = lp5521_configure(client); |
808 | if (ret < 0) { | 807 | if (ret < 0) { |
809 | dev_err(&client->dev, "error configuring chip\n"); | 808 | dev_err(&client->dev, "error configuring chip\n"); |
810 | goto fail2; | 809 | goto fail1; |
811 | } | 810 | } |
812 | 811 | ||
813 | /* Initialize leds */ | 812 | /* Initialize leds */ |
@@ -822,7 +821,7 @@ static int __devinit lp5521_probe(struct i2c_client *client, | |||
822 | ret = lp5521_init_led(&chip->leds[led], client, i, pdata); | 821 | ret = lp5521_init_led(&chip->leds[led], client, i, pdata); |
823 | if (ret) { | 822 | if (ret) { |
824 | dev_err(&client->dev, "error initializing leds\n"); | 823 | dev_err(&client->dev, "error initializing leds\n"); |
825 | goto fail3; | 824 | goto fail2; |
826 | } | 825 | } |
827 | chip->num_leds++; | 826 | chip->num_leds++; |
828 | 827 | ||
@@ -840,21 +839,19 @@ static int __devinit lp5521_probe(struct i2c_client *client, | |||
840 | ret = lp5521_register_sysfs(client); | 839 | ret = lp5521_register_sysfs(client); |
841 | if (ret) { | 840 | if (ret) { |
842 | dev_err(&client->dev, "registering sysfs failed\n"); | 841 | dev_err(&client->dev, "registering sysfs failed\n"); |
843 | goto fail3; | 842 | goto fail2; |
844 | } | 843 | } |
845 | return ret; | 844 | return ret; |
846 | fail3: | 845 | fail2: |
847 | for (i = 0; i < chip->num_leds; i++) { | 846 | for (i = 0; i < chip->num_leds; i++) { |
848 | led_classdev_unregister(&chip->leds[i].cdev); | 847 | led_classdev_unregister(&chip->leds[i].cdev); |
849 | cancel_work_sync(&chip->leds[i].brightness_work); | 848 | cancel_work_sync(&chip->leds[i].brightness_work); |
850 | } | 849 | } |
851 | fail2: | 850 | fail1: |
852 | if (pdata->enable) | 851 | if (pdata->enable) |
853 | pdata->enable(0); | 852 | pdata->enable(0); |
854 | if (pdata->release_resources) | 853 | if (pdata->release_resources) |
855 | pdata->release_resources(); | 854 | pdata->release_resources(); |
856 | fail1: | ||
857 | kfree(chip); | ||
858 | return ret; | 855 | return ret; |
859 | } | 856 | } |
860 | 857 | ||
@@ -875,7 +872,6 @@ static int __devexit lp5521_remove(struct i2c_client *client) | |||
875 | chip->pdata->enable(0); | 872 | chip->pdata->enable(0); |
876 | if (chip->pdata->release_resources) | 873 | if (chip->pdata->release_resources) |
877 | chip->pdata->release_resources(); | 874 | chip->pdata->release_resources(); |
878 | kfree(chip); | ||
879 | return 0; | 875 | return 0; |
880 | } | 876 | } |
881 | 877 | ||
diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c index 857a3e15f2dd..fbc12acada95 100644 --- a/drivers/leds/leds-lp5523.c +++ b/drivers/leds/leds-lp5523.c | |||
@@ -877,7 +877,7 @@ static int __devinit lp5523_probe(struct i2c_client *client, | |||
877 | struct lp5523_platform_data *pdata; | 877 | struct lp5523_platform_data *pdata; |
878 | int ret, i, led; | 878 | int ret, i, led; |
879 | 879 | ||
880 | chip = kzalloc(sizeof(*chip), GFP_KERNEL); | 880 | chip = devm_kzalloc(&client->dev, sizeof(*chip), GFP_KERNEL); |
881 | if (!chip) | 881 | if (!chip) |
882 | return -ENOMEM; | 882 | return -ENOMEM; |
883 | 883 | ||
@@ -888,8 +888,7 @@ static int __devinit lp5523_probe(struct i2c_client *client, | |||
888 | 888 | ||
889 | if (!pdata) { | 889 | if (!pdata) { |
890 | dev_err(&client->dev, "no platform data\n"); | 890 | dev_err(&client->dev, "no platform data\n"); |
891 | ret = -EINVAL; | 891 | return -EINVAL; |
892 | goto fail1; | ||
893 | } | 892 | } |
894 | 893 | ||
895 | mutex_init(&chip->lock); | 894 | mutex_init(&chip->lock); |
@@ -899,7 +898,7 @@ static int __devinit lp5523_probe(struct i2c_client *client, | |||
899 | if (pdata->setup_resources) { | 898 | if (pdata->setup_resources) { |
900 | ret = pdata->setup_resources(); | 899 | ret = pdata->setup_resources(); |
901 | if (ret < 0) | 900 | if (ret < 0) |
902 | goto fail1; | 901 | return ret; |
903 | } | 902 | } |
904 | 903 | ||
905 | if (pdata->enable) { | 904 | if (pdata->enable) { |
@@ -916,7 +915,7 @@ static int __devinit lp5523_probe(struct i2c_client *client, | |||
916 | */ | 915 | */ |
917 | ret = lp5523_detect(client); | 916 | ret = lp5523_detect(client); |
918 | if (ret) | 917 | if (ret) |
919 | goto fail2; | 918 | goto fail1; |
920 | 919 | ||
921 | dev_info(&client->dev, "LP5523 Programmable led chip found\n"); | 920 | dev_info(&client->dev, "LP5523 Programmable led chip found\n"); |
922 | 921 | ||
@@ -925,13 +924,13 @@ static int __devinit lp5523_probe(struct i2c_client *client, | |||
925 | ret = lp5523_init_engine(&chip->engines[i], i + 1); | 924 | ret = lp5523_init_engine(&chip->engines[i], i + 1); |
926 | if (ret) { | 925 | if (ret) { |
927 | dev_err(&client->dev, "error initializing engine\n"); | 926 | dev_err(&client->dev, "error initializing engine\n"); |
928 | goto fail2; | 927 | goto fail1; |
929 | } | 928 | } |
930 | } | 929 | } |
931 | ret = lp5523_configure(client); | 930 | ret = lp5523_configure(client); |
932 | if (ret < 0) { | 931 | if (ret < 0) { |
933 | dev_err(&client->dev, "error configuring chip\n"); | 932 | dev_err(&client->dev, "error configuring chip\n"); |
934 | goto fail2; | 933 | goto fail1; |
935 | } | 934 | } |
936 | 935 | ||
937 | /* Initialize leds */ | 936 | /* Initialize leds */ |
@@ -943,10 +942,13 @@ static int __devinit lp5523_probe(struct i2c_client *client, | |||
943 | if (pdata->led_config[i].led_current == 0) | 942 | if (pdata->led_config[i].led_current == 0) |
944 | continue; | 943 | continue; |
945 | 944 | ||
945 | INIT_WORK(&chip->leds[led].brightness_work, | ||
946 | lp5523_led_brightness_work); | ||
947 | |||
946 | ret = lp5523_init_led(&chip->leds[led], &client->dev, i, pdata); | 948 | ret = lp5523_init_led(&chip->leds[led], &client->dev, i, pdata); |
947 | if (ret) { | 949 | if (ret) { |
948 | dev_err(&client->dev, "error initializing leds\n"); | 950 | dev_err(&client->dev, "error initializing leds\n"); |
949 | goto fail3; | 951 | goto fail2; |
950 | } | 952 | } |
951 | chip->num_leds++; | 953 | chip->num_leds++; |
952 | 954 | ||
@@ -956,30 +958,25 @@ static int __devinit lp5523_probe(struct i2c_client *client, | |||
956 | LP5523_REG_LED_CURRENT_BASE + chip->leds[led].chan_nr, | 958 | LP5523_REG_LED_CURRENT_BASE + chip->leds[led].chan_nr, |
957 | chip->leds[led].led_current); | 959 | chip->leds[led].led_current); |
958 | 960 | ||
959 | INIT_WORK(&(chip->leds[led].brightness_work), | ||
960 | lp5523_led_brightness_work); | ||
961 | |||
962 | led++; | 961 | led++; |
963 | } | 962 | } |
964 | 963 | ||
965 | ret = lp5523_register_sysfs(client); | 964 | ret = lp5523_register_sysfs(client); |
966 | if (ret) { | 965 | if (ret) { |
967 | dev_err(&client->dev, "registering sysfs failed\n"); | 966 | dev_err(&client->dev, "registering sysfs failed\n"); |
968 | goto fail3; | 967 | goto fail2; |
969 | } | 968 | } |
970 | return ret; | 969 | return ret; |
971 | fail3: | 970 | fail2: |
972 | for (i = 0; i < chip->num_leds; i++) { | 971 | for (i = 0; i < chip->num_leds; i++) { |
973 | led_classdev_unregister(&chip->leds[i].cdev); | 972 | led_classdev_unregister(&chip->leds[i].cdev); |
974 | cancel_work_sync(&chip->leds[i].brightness_work); | 973 | cancel_work_sync(&chip->leds[i].brightness_work); |
975 | } | 974 | } |
976 | fail2: | 975 | fail1: |
977 | if (pdata->enable) | 976 | if (pdata->enable) |
978 | pdata->enable(0); | 977 | pdata->enable(0); |
979 | if (pdata->release_resources) | 978 | if (pdata->release_resources) |
980 | pdata->release_resources(); | 979 | pdata->release_resources(); |
981 | fail1: | ||
982 | kfree(chip); | ||
983 | return ret; | 980 | return ret; |
984 | } | 981 | } |
985 | 982 | ||
@@ -999,7 +996,6 @@ static int lp5523_remove(struct i2c_client *client) | |||
999 | chip->pdata->enable(0); | 996 | chip->pdata->enable(0); |
1000 | if (chip->pdata->release_resources) | 997 | if (chip->pdata->release_resources) |
1001 | chip->pdata->release_resources(); | 998 | chip->pdata->release_resources(); |
1002 | kfree(chip); | ||
1003 | return 0; | 999 | return 0; |
1004 | } | 1000 | } |
1005 | 1001 | ||
diff --git a/drivers/leds/leds-lp8788.c b/drivers/leds/leds-lp8788.c new file mode 100644 index 000000000000..53bd136f1ef0 --- /dev/null +++ b/drivers/leds/leds-lp8788.c | |||
@@ -0,0 +1,193 @@ | |||
1 | /* | ||
2 | * TI LP8788 MFD - keyled driver | ||
3 | * | ||
4 | * Copyright 2012 Texas Instruments | ||
5 | * | ||
6 | * Author: Milo(Woogyom) Kim <milo.kim@ti.com> | ||
7 | * | ||
8 | * This program is free software; you can redistribute it and/or modify | ||
9 | * it under the terms of the GNU General Public License version 2 as | ||
10 | * published by the Free Software Foundation. | ||
11 | * | ||
12 | */ | ||
13 | |||
14 | #include <linux/module.h> | ||
15 | #include <linux/slab.h> | ||
16 | #include <linux/err.h> | ||
17 | #include <linux/platform_device.h> | ||
18 | #include <linux/leds.h> | ||
19 | #include <linux/mutex.h> | ||
20 | #include <linux/mfd/lp8788.h> | ||
21 | #include <linux/mfd/lp8788-isink.h> | ||
22 | |||
23 | #define MAX_BRIGHTNESS LP8788_ISINK_MAX_PWM | ||
24 | #define DEFAULT_LED_NAME "keyboard-backlight" | ||
25 | |||
26 | struct lp8788_led { | ||
27 | struct lp8788 *lp; | ||
28 | struct mutex lock; | ||
29 | struct work_struct work; | ||
30 | struct led_classdev led_dev; | ||
31 | enum lp8788_isink_number isink_num; | ||
32 | enum led_brightness brightness; | ||
33 | int on; | ||
34 | }; | ||
35 | |||
36 | struct lp8788_led_config { | ||
37 | enum lp8788_isink_scale scale; | ||
38 | enum lp8788_isink_number num; | ||
39 | int iout; | ||
40 | }; | ||
41 | |||
42 | static struct lp8788_led_config default_led_config = { | ||
43 | .scale = LP8788_ISINK_SCALE_100mA, | ||
44 | .num = LP8788_ISINK_3, | ||
45 | .iout = 0, | ||
46 | }; | ||
47 | |||
48 | static int lp8788_led_init_device(struct lp8788_led *led, | ||
49 | struct lp8788_led_platform_data *pdata) | ||
50 | { | ||
51 | struct lp8788_led_config *cfg = &default_led_config; | ||
52 | u8 addr, mask, val; | ||
53 | int ret; | ||
54 | |||
55 | if (pdata) { | ||
56 | cfg->scale = pdata->scale; | ||
57 | cfg->num = pdata->num; | ||
58 | cfg->iout = pdata->iout_code; | ||
59 | } | ||
60 | |||
61 | led->isink_num = cfg->num; | ||
62 | |||
63 | /* scale configuration */ | ||
64 | addr = LP8788_ISINK_CTRL; | ||
65 | mask = 1 << (cfg->num + LP8788_ISINK_SCALE_OFFSET); | ||
66 | val = cfg->scale << cfg->num; | ||
67 | ret = lp8788_update_bits(led->lp, addr, mask, val); | ||
68 | if (ret) | ||
69 | return ret; | ||
70 | |||
71 | /* current configuration */ | ||
72 | addr = lp8788_iout_addr[cfg->num]; | ||
73 | mask = lp8788_iout_mask[cfg->num]; | ||
74 | val = cfg->iout; | ||
75 | |||
76 | return lp8788_update_bits(led->lp, addr, mask, val); | ||
77 | } | ||
78 | |||
79 | static void lp8788_led_enable(struct lp8788_led *led, | ||
80 | enum lp8788_isink_number num, int on) | ||
81 | { | ||
82 | u8 mask = 1 << num; | ||
83 | u8 val = on << num; | ||
84 | |||
85 | if (lp8788_update_bits(led->lp, LP8788_ISINK_CTRL, mask, val)) | ||
86 | return; | ||
87 | |||
88 | led->on = on; | ||
89 | } | ||
90 | |||
91 | static void lp8788_led_work(struct work_struct *work) | ||
92 | { | ||
93 | struct lp8788_led *led = container_of(work, struct lp8788_led, work); | ||
94 | enum lp8788_isink_number num = led->isink_num; | ||
95 | int enable; | ||
96 | u8 val = led->brightness; | ||
97 | |||
98 | mutex_lock(&led->lock); | ||
99 | |||
100 | switch (num) { | ||
101 | case LP8788_ISINK_1: | ||
102 | case LP8788_ISINK_2: | ||
103 | case LP8788_ISINK_3: | ||
104 | lp8788_write_byte(led->lp, lp8788_pwm_addr[num], val); | ||
105 | break; | ||
106 | default: | ||
107 | mutex_unlock(&led->lock); | ||
108 | return; | ||
109 | } | ||
110 | |||
111 | enable = (val > 0) ? 1 : 0; | ||
112 | if (enable != led->on) | ||
113 | lp8788_led_enable(led, num, enable); | ||
114 | |||
115 | mutex_unlock(&led->lock); | ||
116 | } | ||
117 | |||
118 | static void lp8788_brightness_set(struct led_classdev *led_cdev, | ||
119 | enum led_brightness brt_val) | ||
120 | { | ||
121 | struct lp8788_led *led = | ||
122 | container_of(led_cdev, struct lp8788_led, led_dev); | ||
123 | |||
124 | led->brightness = brt_val; | ||
125 | schedule_work(&led->work); | ||
126 | } | ||
127 | |||
128 | static __devinit int lp8788_led_probe(struct platform_device *pdev) | ||
129 | { | ||
130 | struct lp8788 *lp = dev_get_drvdata(pdev->dev.parent); | ||
131 | struct lp8788_led_platform_data *led_pdata; | ||
132 | struct lp8788_led *led; | ||
133 | int ret; | ||
134 | |||
135 | led = devm_kzalloc(lp->dev, sizeof(struct lp8788_led), GFP_KERNEL); | ||
136 | if (!led) | ||
137 | return -ENOMEM; | ||
138 | |||
139 | led->lp = lp; | ||
140 | led->led_dev.max_brightness = MAX_BRIGHTNESS; | ||
141 | led->led_dev.brightness_set = lp8788_brightness_set; | ||
142 | |||
143 | led_pdata = lp->pdata ? lp->pdata->led_pdata : NULL; | ||
144 | |||
145 | if (!led_pdata || !led_pdata->name) | ||
146 | led->led_dev.name = DEFAULT_LED_NAME; | ||
147 | else | ||
148 | led->led_dev.name = led_pdata->name; | ||
149 | |||
150 | mutex_init(&led->lock); | ||
151 | INIT_WORK(&led->work, lp8788_led_work); | ||
152 | |||
153 | platform_set_drvdata(pdev, led); | ||
154 | |||
155 | ret = lp8788_led_init_device(led, led_pdata); | ||
156 | if (ret) { | ||
157 | dev_err(lp->dev, "led init device err: %d\n", ret); | ||
158 | return ret; | ||
159 | } | ||
160 | |||
161 | ret = led_classdev_register(lp->dev, &led->led_dev); | ||
162 | if (ret) { | ||
163 | dev_err(lp->dev, "led register err: %d\n", ret); | ||
164 | return ret; | ||
165 | } | ||
166 | |||
167 | return 0; | ||
168 | } | ||
169 | |||
170 | static int __devexit lp8788_led_remove(struct platform_device *pdev) | ||
171 | { | ||
172 | struct lp8788_led *led = platform_get_drvdata(pdev); | ||
173 | |||
174 | led_classdev_unregister(&led->led_dev); | ||
175 | flush_work_sync(&led->work); | ||
176 | |||
177 | return 0; | ||
178 | } | ||
179 | |||
180 | static struct platform_driver lp8788_led_driver = { | ||
181 | .probe = lp8788_led_probe, | ||
182 | .remove = __devexit_p(lp8788_led_remove), | ||
183 | .driver = { | ||
184 | .name = LP8788_DEV_KEYLED, | ||
185 | .owner = THIS_MODULE, | ||
186 | }, | ||
187 | }; | ||
188 | module_platform_driver(lp8788_led_driver); | ||
189 | |||
190 | MODULE_DESCRIPTION("Texas Instruments LP8788 Keyboard LED Driver"); | ||
191 | MODULE_AUTHOR("Milo Kim"); | ||
192 | MODULE_LICENSE("GPL"); | ||
193 | MODULE_ALIAS("platform:lp8788-keyled"); | ||
diff --git a/drivers/leds/leds-lt3593.c b/drivers/leds/leds-lt3593.c index e311a96c4469..09a732217f6d 100644 --- a/drivers/leds/leds-lt3593.c +++ b/drivers/leds/leds-lt3593.c | |||
@@ -149,8 +149,9 @@ static int __devinit lt3593_led_probe(struct platform_device *pdev) | |||
149 | if (!pdata) | 149 | if (!pdata) |
150 | return -EBUSY; | 150 | return -EBUSY; |
151 | 151 | ||
152 | leds_data = kzalloc(sizeof(struct lt3593_led_data) * pdata->num_leds, | 152 | leds_data = devm_kzalloc(&pdev->dev, |
153 | GFP_KERNEL); | 153 | sizeof(struct lt3593_led_data) * pdata->num_leds, |
154 | GFP_KERNEL); | ||
154 | if (!leds_data) | 155 | if (!leds_data) |
155 | return -ENOMEM; | 156 | return -ENOMEM; |
156 | 157 | ||
@@ -169,8 +170,6 @@ err: | |||
169 | for (i = i - 1; i >= 0; i--) | 170 | for (i = i - 1; i >= 0; i--) |
170 | delete_lt3593_led(&leds_data[i]); | 171 | delete_lt3593_led(&leds_data[i]); |
171 | 172 | ||
172 | kfree(leds_data); | ||
173 | |||
174 | return ret; | 173 | return ret; |
175 | } | 174 | } |
176 | 175 | ||
@@ -185,8 +184,6 @@ static int __devexit lt3593_led_remove(struct platform_device *pdev) | |||
185 | for (i = 0; i < pdata->num_leds; i++) | 184 | for (i = 0; i < pdata->num_leds; i++) |
186 | delete_lt3593_led(&leds_data[i]); | 185 | delete_lt3593_led(&leds_data[i]); |
187 | 186 | ||
188 | kfree(leds_data); | ||
189 | |||
190 | return 0; | 187 | return 0; |
191 | } | 188 | } |
192 | 189 | ||
diff --git a/drivers/leds/leds-max8997.c b/drivers/leds/leds-max8997.c index f4c0e37fad1e..569e36de37df 100644 --- a/drivers/leds/leds-max8997.c +++ b/drivers/leds/leds-max8997.c | |||
@@ -49,71 +49,37 @@ struct max8997_led { | |||
49 | struct mutex mutex; | 49 | struct mutex mutex; |
50 | }; | 50 | }; |
51 | 51 | ||
52 | static void max8997_led_clear_mode(struct max8997_led *led, | ||
53 | enum max8997_led_mode mode) | ||
54 | { | ||
55 | struct i2c_client *client = led->iodev->i2c; | ||
56 | u8 val = 0, mask = 0; | ||
57 | int ret; | ||
58 | |||
59 | switch (mode) { | ||
60 | case MAX8997_FLASH_MODE: | ||
61 | mask = led->id ? | ||
62 | MAX8997_LED1_FLASH_MASK : MAX8997_LED0_FLASH_MASK; | ||
63 | break; | ||
64 | case MAX8997_MOVIE_MODE: | ||
65 | mask = led->id ? | ||
66 | MAX8997_LED1_MOVIE_MASK : MAX8997_LED0_MOVIE_MASK; | ||
67 | break; | ||
68 | case MAX8997_FLASH_PIN_CONTROL_MODE: | ||
69 | mask = led->id ? | ||
70 | MAX8997_LED1_FLASH_PIN_MASK : MAX8997_LED0_FLASH_PIN_MASK; | ||
71 | break; | ||
72 | case MAX8997_MOVIE_PIN_CONTROL_MODE: | ||
73 | mask = led->id ? | ||
74 | MAX8997_LED1_MOVIE_PIN_MASK : MAX8997_LED0_MOVIE_PIN_MASK; | ||
75 | break; | ||
76 | default: | ||
77 | break; | ||
78 | } | ||
79 | |||
80 | if (mask) { | ||
81 | ret = max8997_update_reg(client, | ||
82 | MAX8997_REG_LEN_CNTL, val, mask); | ||
83 | if (ret) | ||
84 | dev_err(led->iodev->dev, | ||
85 | "failed to update register(%d)\n", ret); | ||
86 | } | ||
87 | } | ||
88 | |||
89 | static void max8997_led_set_mode(struct max8997_led *led, | 52 | static void max8997_led_set_mode(struct max8997_led *led, |
90 | enum max8997_led_mode mode) | 53 | enum max8997_led_mode mode) |
91 | { | 54 | { |
92 | int ret; | 55 | int ret; |
93 | struct i2c_client *client = led->iodev->i2c; | 56 | struct i2c_client *client = led->iodev->i2c; |
94 | u8 mask = 0; | 57 | u8 mask = 0, val; |
95 | |||
96 | /* First, clear the previous mode */ | ||
97 | max8997_led_clear_mode(led, led->led_mode); | ||
98 | 58 | ||
99 | switch (mode) { | 59 | switch (mode) { |
100 | case MAX8997_FLASH_MODE: | 60 | case MAX8997_FLASH_MODE: |
101 | mask = led->id ? | 61 | mask = MAX8997_LED1_FLASH_MASK | MAX8997_LED0_FLASH_MASK; |
62 | val = led->id ? | ||
102 | MAX8997_LED1_FLASH_MASK : MAX8997_LED0_FLASH_MASK; | 63 | MAX8997_LED1_FLASH_MASK : MAX8997_LED0_FLASH_MASK; |
103 | led->cdev.max_brightness = MAX8997_LED_FLASH_MAX_BRIGHTNESS; | 64 | led->cdev.max_brightness = MAX8997_LED_FLASH_MAX_BRIGHTNESS; |
104 | break; | 65 | break; |
105 | case MAX8997_MOVIE_MODE: | 66 | case MAX8997_MOVIE_MODE: |
106 | mask = led->id ? | 67 | mask = MAX8997_LED1_MOVIE_MASK | MAX8997_LED0_MOVIE_MASK; |
68 | val = led->id ? | ||
107 | MAX8997_LED1_MOVIE_MASK : MAX8997_LED0_MOVIE_MASK; | 69 | MAX8997_LED1_MOVIE_MASK : MAX8997_LED0_MOVIE_MASK; |
108 | led->cdev.max_brightness = MAX8997_LED_MOVIE_MAX_BRIGHTNESS; | 70 | led->cdev.max_brightness = MAX8997_LED_MOVIE_MAX_BRIGHTNESS; |
109 | break; | 71 | break; |
110 | case MAX8997_FLASH_PIN_CONTROL_MODE: | 72 | case MAX8997_FLASH_PIN_CONTROL_MODE: |
111 | mask = led->id ? | 73 | mask = MAX8997_LED1_FLASH_PIN_MASK | |
74 | MAX8997_LED0_FLASH_PIN_MASK; | ||
75 | val = led->id ? | ||
112 | MAX8997_LED1_FLASH_PIN_MASK : MAX8997_LED0_FLASH_PIN_MASK; | 76 | MAX8997_LED1_FLASH_PIN_MASK : MAX8997_LED0_FLASH_PIN_MASK; |
113 | led->cdev.max_brightness = MAX8997_LED_FLASH_MAX_BRIGHTNESS; | 77 | led->cdev.max_brightness = MAX8997_LED_FLASH_MAX_BRIGHTNESS; |
114 | break; | 78 | break; |
115 | case MAX8997_MOVIE_PIN_CONTROL_MODE: | 79 | case MAX8997_MOVIE_PIN_CONTROL_MODE: |
116 | mask = led->id ? | 80 | mask = MAX8997_LED1_MOVIE_PIN_MASK | |
81 | MAX8997_LED0_MOVIE_PIN_MASK; | ||
82 | val = led->id ? | ||
117 | MAX8997_LED1_MOVIE_PIN_MASK : MAX8997_LED0_MOVIE_PIN_MASK; | 83 | MAX8997_LED1_MOVIE_PIN_MASK : MAX8997_LED0_MOVIE_PIN_MASK; |
118 | led->cdev.max_brightness = MAX8997_LED_MOVIE_MAX_BRIGHTNESS; | 84 | led->cdev.max_brightness = MAX8997_LED_MOVIE_MAX_BRIGHTNESS; |
119 | break; | 85 | break; |
@@ -123,8 +89,8 @@ static void max8997_led_set_mode(struct max8997_led *led, | |||
123 | } | 89 | } |
124 | 90 | ||
125 | if (mask) { | 91 | if (mask) { |
126 | ret = max8997_update_reg(client, | 92 | ret = max8997_update_reg(client, MAX8997_REG_LEN_CNTL, val, |
127 | MAX8997_REG_LEN_CNTL, mask, mask); | 93 | mask); |
128 | if (ret) | 94 | if (ret) |
129 | dev_err(led->iodev->dev, | 95 | dev_err(led->iodev->dev, |
130 | "failed to update register(%d)\n", ret); | 96 | "failed to update register(%d)\n", ret); |
@@ -276,11 +242,9 @@ static int __devinit max8997_led_probe(struct platform_device *pdev) | |||
276 | return -ENODEV; | 242 | return -ENODEV; |
277 | } | 243 | } |
278 | 244 | ||
279 | led = kzalloc(sizeof(*led), GFP_KERNEL); | 245 | led = devm_kzalloc(&pdev->dev, sizeof(*led), GFP_KERNEL); |
280 | if (led == NULL) { | 246 | if (led == NULL) |
281 | ret = -ENOMEM; | 247 | return -ENOMEM; |
282 | goto err_mem; | ||
283 | } | ||
284 | 248 | ||
285 | led->id = pdev->id; | 249 | led->id = pdev->id; |
286 | snprintf(name, sizeof(name), "max8997-led%d", pdev->id); | 250 | snprintf(name, sizeof(name), "max8997-led%d", pdev->id); |
@@ -315,23 +279,17 @@ static int __devinit max8997_led_probe(struct platform_device *pdev) | |||
315 | 279 | ||
316 | ret = led_classdev_register(&pdev->dev, &led->cdev); | 280 | ret = led_classdev_register(&pdev->dev, &led->cdev); |
317 | if (ret < 0) | 281 | if (ret < 0) |
318 | goto err_led; | 282 | return ret; |
319 | 283 | ||
320 | ret = device_create_file(led->cdev.dev, &dev_attr_mode); | 284 | ret = device_create_file(led->cdev.dev, &dev_attr_mode); |
321 | if (ret != 0) { | 285 | if (ret != 0) { |
322 | dev_err(&pdev->dev, | 286 | dev_err(&pdev->dev, |
323 | "failed to create file: %d\n", ret); | 287 | "failed to create file: %d\n", ret); |
324 | goto err_file; | 288 | led_classdev_unregister(&led->cdev); |
289 | return ret; | ||
325 | } | 290 | } |
326 | 291 | ||
327 | return 0; | 292 | return 0; |
328 | |||
329 | err_file: | ||
330 | led_classdev_unregister(&led->cdev); | ||
331 | err_led: | ||
332 | kfree(led); | ||
333 | err_mem: | ||
334 | return ret; | ||
335 | } | 293 | } |
336 | 294 | ||
337 | static int __devexit max8997_led_remove(struct platform_device *pdev) | 295 | static int __devexit max8997_led_remove(struct platform_device *pdev) |
@@ -340,7 +298,6 @@ static int __devexit max8997_led_remove(struct platform_device *pdev) | |||
340 | 298 | ||
341 | device_remove_file(led->cdev.dev, &dev_attr_mode); | 299 | device_remove_file(led->cdev.dev, &dev_attr_mode); |
342 | led_classdev_unregister(&led->cdev); | 300 | led_classdev_unregister(&led->cdev); |
343 | kfree(led); | ||
344 | 301 | ||
345 | return 0; | 302 | return 0; |
346 | } | 303 | } |
@@ -354,17 +311,7 @@ static struct platform_driver max8997_led_driver = { | |||
354 | .remove = __devexit_p(max8997_led_remove), | 311 | .remove = __devexit_p(max8997_led_remove), |
355 | }; | 312 | }; |
356 | 313 | ||
357 | static int __init max8997_led_init(void) | 314 | module_platform_driver(max8997_led_driver); |
358 | { | ||
359 | return platform_driver_register(&max8997_led_driver); | ||
360 | } | ||
361 | module_init(max8997_led_init); | ||
362 | |||
363 | static void __exit max8997_led_exit(void) | ||
364 | { | ||
365 | platform_driver_unregister(&max8997_led_driver); | ||
366 | } | ||
367 | module_exit(max8997_led_exit); | ||
368 | 315 | ||
369 | MODULE_AUTHOR("Donggeun Kim <dg77.kim@samsung.com>"); | 316 | MODULE_AUTHOR("Donggeun Kim <dg77.kim@samsung.com>"); |
370 | MODULE_DESCRIPTION("MAX8997 LED driver"); | 317 | MODULE_DESCRIPTION("MAX8997 LED driver"); |
diff --git a/drivers/leds/leds-mc13783.c b/drivers/leds/leds-mc13783.c index 4cc6a2e3df34..2a5d43400677 100644 --- a/drivers/leds/leds-mc13783.c +++ b/drivers/leds/leds-mc13783.c | |||
@@ -280,7 +280,8 @@ static int __devinit mc13783_led_probe(struct platform_device *pdev) | |||
280 | return -EINVAL; | 280 | return -EINVAL; |
281 | } | 281 | } |
282 | 282 | ||
283 | led = kcalloc(pdata->num_leds, sizeof(*led), GFP_KERNEL); | 283 | led = devm_kzalloc(&pdev->dev, pdata->num_leds * sizeof(*led), |
284 | GFP_KERNEL); | ||
284 | if (led == NULL) { | 285 | if (led == NULL) { |
285 | dev_err(&pdev->dev, "failed to alloc memory\n"); | 286 | dev_err(&pdev->dev, "failed to alloc memory\n"); |
286 | return -ENOMEM; | 287 | return -ENOMEM; |
@@ -289,7 +290,7 @@ static int __devinit mc13783_led_probe(struct platform_device *pdev) | |||
289 | ret = mc13783_leds_prepare(pdev); | 290 | ret = mc13783_leds_prepare(pdev); |
290 | if (ret) { | 291 | if (ret) { |
291 | dev_err(&pdev->dev, "unable to init led driver\n"); | 292 | dev_err(&pdev->dev, "unable to init led driver\n"); |
292 | goto err_free; | 293 | return ret; |
293 | } | 294 | } |
294 | 295 | ||
295 | for (i = 0; i < pdata->num_leds; i++) { | 296 | for (i = 0; i < pdata->num_leds; i++) { |
@@ -344,8 +345,6 @@ err_register: | |||
344 | cancel_work_sync(&led[i].work); | 345 | cancel_work_sync(&led[i].work); |
345 | } | 346 | } |
346 | 347 | ||
347 | err_free: | ||
348 | kfree(led); | ||
349 | return ret; | 348 | return ret; |
350 | } | 349 | } |
351 | 350 | ||
@@ -372,7 +371,7 @@ static int __devexit mc13783_led_remove(struct platform_device *pdev) | |||
372 | 371 | ||
373 | mc13xxx_unlock(dev); | 372 | mc13xxx_unlock(dev); |
374 | 373 | ||
375 | kfree(led); | 374 | platform_set_drvdata(pdev, NULL); |
376 | return 0; | 375 | return 0; |
377 | } | 376 | } |
378 | 377 | ||
diff --git a/drivers/leds/leds-netxbig.c b/drivers/leds/leds-netxbig.c index 73973fdbd8be..e37618e363cf 100644 --- a/drivers/leds/leds-netxbig.c +++ b/drivers/leds/leds-netxbig.c | |||
@@ -362,14 +362,14 @@ static int __devinit netxbig_led_probe(struct platform_device *pdev) | |||
362 | if (!pdata) | 362 | if (!pdata) |
363 | return -EINVAL; | 363 | return -EINVAL; |
364 | 364 | ||
365 | leds_data = kzalloc(sizeof(struct netxbig_led_data) * pdata->num_leds, | 365 | leds_data = devm_kzalloc(&pdev->dev, |
366 | GFP_KERNEL); | 366 | sizeof(struct netxbig_led_data) * pdata->num_leds, GFP_KERNEL); |
367 | if (!leds_data) | 367 | if (!leds_data) |
368 | return -ENOMEM; | 368 | return -ENOMEM; |
369 | 369 | ||
370 | ret = gpio_ext_init(pdata->gpio_ext); | 370 | ret = gpio_ext_init(pdata->gpio_ext); |
371 | if (ret < 0) | 371 | if (ret < 0) |
372 | goto err_free_data; | 372 | return ret; |
373 | 373 | ||
374 | for (i = 0; i < pdata->num_leds; i++) { | 374 | for (i = 0; i < pdata->num_leds; i++) { |
375 | ret = create_netxbig_led(pdev, &leds_data[i], &pdata->leds[i]); | 375 | ret = create_netxbig_led(pdev, &leds_data[i], &pdata->leds[i]); |
@@ -386,9 +386,6 @@ err_free_leds: | |||
386 | delete_netxbig_led(&leds_data[i]); | 386 | delete_netxbig_led(&leds_data[i]); |
387 | 387 | ||
388 | gpio_ext_free(pdata->gpio_ext); | 388 | gpio_ext_free(pdata->gpio_ext); |
389 | err_free_data: | ||
390 | kfree(leds_data); | ||
391 | |||
392 | return ret; | 389 | return ret; |
393 | } | 390 | } |
394 | 391 | ||
@@ -404,7 +401,6 @@ static int __devexit netxbig_led_remove(struct platform_device *pdev) | |||
404 | delete_netxbig_led(&leds_data[i]); | 401 | delete_netxbig_led(&leds_data[i]); |
405 | 402 | ||
406 | gpio_ext_free(pdata->gpio_ext); | 403 | gpio_ext_free(pdata->gpio_ext); |
407 | kfree(leds_data); | ||
408 | 404 | ||
409 | return 0; | 405 | return 0; |
410 | } | 406 | } |
diff --git a/drivers/leds/leds-ns2.c b/drivers/leds/leds-ns2.c index 01cf89ec6944..10528dafb043 100644 --- a/drivers/leds/leds-ns2.c +++ b/drivers/leds/leds-ns2.c | |||
@@ -273,29 +273,23 @@ static int __devinit ns2_led_probe(struct platform_device *pdev) | |||
273 | if (!pdata) | 273 | if (!pdata) |
274 | return -EINVAL; | 274 | return -EINVAL; |
275 | 275 | ||
276 | leds_data = kzalloc(sizeof(struct ns2_led_data) * | 276 | leds_data = devm_kzalloc(&pdev->dev, sizeof(struct ns2_led_data) * |
277 | pdata->num_leds, GFP_KERNEL); | 277 | pdata->num_leds, GFP_KERNEL); |
278 | if (!leds_data) | 278 | if (!leds_data) |
279 | return -ENOMEM; | 279 | return -ENOMEM; |
280 | 280 | ||
281 | for (i = 0; i < pdata->num_leds; i++) { | 281 | for (i = 0; i < pdata->num_leds; i++) { |
282 | ret = create_ns2_led(pdev, &leds_data[i], &pdata->leds[i]); | 282 | ret = create_ns2_led(pdev, &leds_data[i], &pdata->leds[i]); |
283 | if (ret < 0) | 283 | if (ret < 0) { |
284 | goto err; | 284 | for (i = i - 1; i >= 0; i--) |
285 | 285 | delete_ns2_led(&leds_data[i]); | |
286 | return ret; | ||
287 | } | ||
286 | } | 288 | } |
287 | 289 | ||
288 | platform_set_drvdata(pdev, leds_data); | 290 | platform_set_drvdata(pdev, leds_data); |
289 | 291 | ||
290 | return 0; | 292 | return 0; |
291 | |||
292 | err: | ||
293 | for (i = i - 1; i >= 0; i--) | ||
294 | delete_ns2_led(&leds_data[i]); | ||
295 | |||
296 | kfree(leds_data); | ||
297 | |||
298 | return ret; | ||
299 | } | 293 | } |
300 | 294 | ||
301 | static int __devexit ns2_led_remove(struct platform_device *pdev) | 295 | static int __devexit ns2_led_remove(struct platform_device *pdev) |
@@ -309,7 +303,6 @@ static int __devexit ns2_led_remove(struct platform_device *pdev) | |||
309 | for (i = 0; i < pdata->num_leds; i++) | 303 | for (i = 0; i < pdata->num_leds; i++) |
310 | delete_ns2_led(&leds_data[i]); | 304 | delete_ns2_led(&leds_data[i]); |
311 | 305 | ||
312 | kfree(leds_data); | ||
313 | platform_set_drvdata(pdev, NULL); | 306 | platform_set_drvdata(pdev, NULL); |
314 | 307 | ||
315 | return 0; | 308 | return 0; |
diff --git a/drivers/leds/leds-pca9532.c b/drivers/leds/leds-pca9532.c index ceccab44b5b8..cee8a5b483ac 100644 --- a/drivers/leds/leds-pca9532.c +++ b/drivers/leds/leds-pca9532.c | |||
@@ -449,7 +449,6 @@ static int pca9532_probe(struct i2c_client *client, | |||
449 | { | 449 | { |
450 | struct pca9532_data *data = i2c_get_clientdata(client); | 450 | struct pca9532_data *data = i2c_get_clientdata(client); |
451 | struct pca9532_platform_data *pca9532_pdata = client->dev.platform_data; | 451 | struct pca9532_platform_data *pca9532_pdata = client->dev.platform_data; |
452 | int err; | ||
453 | 452 | ||
454 | if (!pca9532_pdata) | 453 | if (!pca9532_pdata) |
455 | return -EIO; | 454 | return -EIO; |
@@ -458,7 +457,7 @@ static int pca9532_probe(struct i2c_client *client, | |||
458 | I2C_FUNC_SMBUS_BYTE_DATA)) | 457 | I2C_FUNC_SMBUS_BYTE_DATA)) |
459 | return -EIO; | 458 | return -EIO; |
460 | 459 | ||
461 | data = kzalloc(sizeof(*data), GFP_KERNEL); | 460 | data = devm_kzalloc(&client->dev, sizeof(*data), GFP_KERNEL); |
462 | if (!data) | 461 | if (!data) |
463 | return -ENOMEM; | 462 | return -ENOMEM; |
464 | 463 | ||
@@ -469,11 +468,7 @@ static int pca9532_probe(struct i2c_client *client, | |||
469 | data->client = client; | 468 | data->client = client; |
470 | mutex_init(&data->update_lock); | 469 | mutex_init(&data->update_lock); |
471 | 470 | ||
472 | err = pca9532_configure(client, data, pca9532_pdata); | 471 | return pca9532_configure(client, data, pca9532_pdata); |
473 | if (err) | ||
474 | kfree(data); | ||
475 | |||
476 | return err; | ||
477 | } | 472 | } |
478 | 473 | ||
479 | static int pca9532_remove(struct i2c_client *client) | 474 | static int pca9532_remove(struct i2c_client *client) |
@@ -485,7 +480,6 @@ static int pca9532_remove(struct i2c_client *client) | |||
485 | if (err) | 480 | if (err) |
486 | return err; | 481 | return err; |
487 | 482 | ||
488 | kfree(data); | ||
489 | return 0; | 483 | return 0; |
490 | } | 484 | } |
491 | 485 | ||
diff --git a/drivers/leds/leds-pca955x.c b/drivers/leds/leds-pca955x.c index 5f462dbf0dbb..aef3cf0432fe 100644 --- a/drivers/leds/leds-pca955x.c +++ b/drivers/leds/leds-pca955x.c | |||
@@ -293,15 +293,14 @@ static int __devinit pca955x_probe(struct i2c_client *client, | |||
293 | } | 293 | } |
294 | } | 294 | } |
295 | 295 | ||
296 | pca955x = kzalloc(sizeof(*pca955x), GFP_KERNEL); | 296 | pca955x = devm_kzalloc(&client->dev, sizeof(*pca955x), GFP_KERNEL); |
297 | if (!pca955x) | 297 | if (!pca955x) |
298 | return -ENOMEM; | 298 | return -ENOMEM; |
299 | 299 | ||
300 | pca955x->leds = kzalloc(sizeof(*pca955x_led) * chip->bits, GFP_KERNEL); | 300 | pca955x->leds = devm_kzalloc(&client->dev, |
301 | if (!pca955x->leds) { | 301 | sizeof(*pca955x_led) * chip->bits, GFP_KERNEL); |
302 | err = -ENOMEM; | 302 | if (!pca955x->leds) |
303 | goto exit_nomem; | 303 | return -ENOMEM; |
304 | } | ||
305 | 304 | ||
306 | i2c_set_clientdata(client, pca955x); | 305 | i2c_set_clientdata(client, pca955x); |
307 | 306 | ||
@@ -361,10 +360,6 @@ exit: | |||
361 | cancel_work_sync(&pca955x->leds[i].work); | 360 | cancel_work_sync(&pca955x->leds[i].work); |
362 | } | 361 | } |
363 | 362 | ||
364 | kfree(pca955x->leds); | ||
365 | exit_nomem: | ||
366 | kfree(pca955x); | ||
367 | |||
368 | return err; | 363 | return err; |
369 | } | 364 | } |
370 | 365 | ||
@@ -378,9 +373,6 @@ static int __devexit pca955x_remove(struct i2c_client *client) | |||
378 | cancel_work_sync(&pca955x->leds[i].work); | 373 | cancel_work_sync(&pca955x->leds[i].work); |
379 | } | 374 | } |
380 | 375 | ||
381 | kfree(pca955x->leds); | ||
382 | kfree(pca955x); | ||
383 | |||
384 | return 0; | 376 | return 0; |
385 | } | 377 | } |
386 | 378 | ||
diff --git a/drivers/leds/leds-pca9633.c b/drivers/leds/leds-pca9633.c index d8926fd031aa..edcd706c5631 100644 --- a/drivers/leds/leds-pca9633.c +++ b/drivers/leds/leds-pca9633.c | |||
@@ -108,7 +108,7 @@ static int __devinit pca9633_probe(struct i2c_client *client, | |||
108 | } | 108 | } |
109 | } | 109 | } |
110 | 110 | ||
111 | pca9633 = kcalloc(4, sizeof(*pca9633), GFP_KERNEL); | 111 | pca9633 = devm_kzalloc(&client->dev, 4 * sizeof(*pca9633), GFP_KERNEL); |
112 | if (!pca9633) | 112 | if (!pca9633) |
113 | return -ENOMEM; | 113 | return -ENOMEM; |
114 | 114 | ||
@@ -156,8 +156,6 @@ exit: | |||
156 | cancel_work_sync(&pca9633[i].work); | 156 | cancel_work_sync(&pca9633[i].work); |
157 | } | 157 | } |
158 | 158 | ||
159 | kfree(pca9633); | ||
160 | |||
161 | return err; | 159 | return err; |
162 | } | 160 | } |
163 | 161 | ||
@@ -171,8 +169,6 @@ static int __devexit pca9633_remove(struct i2c_client *client) | |||
171 | cancel_work_sync(&pca9633[i].work); | 169 | cancel_work_sync(&pca9633[i].work); |
172 | } | 170 | } |
173 | 171 | ||
174 | kfree(pca9633); | ||
175 | |||
176 | return 0; | 172 | return 0; |
177 | } | 173 | } |
178 | 174 | ||
diff --git a/drivers/leds/leds-pwm.c b/drivers/leds/leds-pwm.c index 3ed92f34bd44..f2e44c719437 100644 --- a/drivers/leds/leds-pwm.c +++ b/drivers/leds/leds-pwm.c | |||
@@ -57,7 +57,8 @@ static int led_pwm_probe(struct platform_device *pdev) | |||
57 | if (!pdata) | 57 | if (!pdata) |
58 | return -EBUSY; | 58 | return -EBUSY; |
59 | 59 | ||
60 | leds_data = kzalloc(sizeof(struct led_pwm_data) * pdata->num_leds, | 60 | leds_data = devm_kzalloc(&pdev->dev, |
61 | sizeof(struct led_pwm_data) * pdata->num_leds, | ||
61 | GFP_KERNEL); | 62 | GFP_KERNEL); |
62 | if (!leds_data) | 63 | if (!leds_data) |
63 | return -ENOMEM; | 64 | return -ENOMEM; |
@@ -103,8 +104,6 @@ err: | |||
103 | } | 104 | } |
104 | } | 105 | } |
105 | 106 | ||
106 | kfree(leds_data); | ||
107 | |||
108 | return ret; | 107 | return ret; |
109 | } | 108 | } |
110 | 109 | ||
@@ -121,8 +120,6 @@ static int __devexit led_pwm_remove(struct platform_device *pdev) | |||
121 | pwm_free(leds_data[i].pwm); | 120 | pwm_free(leds_data[i].pwm); |
122 | } | 121 | } |
123 | 122 | ||
124 | kfree(leds_data); | ||
125 | |||
126 | return 0; | 123 | return 0; |
127 | } | 124 | } |
128 | 125 | ||
diff --git a/drivers/leds/leds-regulator.c b/drivers/leds/leds-regulator.c index df7e963bddd3..25d382d60fa9 100644 --- a/drivers/leds/leds-regulator.c +++ b/drivers/leds/leds-regulator.c | |||
@@ -158,7 +158,7 @@ static int __devinit regulator_led_probe(struct platform_device *pdev) | |||
158 | return PTR_ERR(vcc); | 158 | return PTR_ERR(vcc); |
159 | } | 159 | } |
160 | 160 | ||
161 | led = kzalloc(sizeof(*led), GFP_KERNEL); | 161 | led = devm_kzalloc(&pdev->dev, sizeof(*led), GFP_KERNEL); |
162 | if (led == NULL) { | 162 | if (led == NULL) { |
163 | ret = -ENOMEM; | 163 | ret = -ENOMEM; |
164 | goto err_vcc; | 164 | goto err_vcc; |
@@ -169,7 +169,7 @@ static int __devinit regulator_led_probe(struct platform_device *pdev) | |||
169 | dev_err(&pdev->dev, "Invalid default brightness %d\n", | 169 | dev_err(&pdev->dev, "Invalid default brightness %d\n", |
170 | pdata->brightness); | 170 | pdata->brightness); |
171 | ret = -EINVAL; | 171 | ret = -EINVAL; |
172 | goto err_led; | 172 | goto err_vcc; |
173 | } | 173 | } |
174 | led->value = pdata->brightness; | 174 | led->value = pdata->brightness; |
175 | 175 | ||
@@ -190,7 +190,7 @@ static int __devinit regulator_led_probe(struct platform_device *pdev) | |||
190 | ret = led_classdev_register(&pdev->dev, &led->cdev); | 190 | ret = led_classdev_register(&pdev->dev, &led->cdev); |
191 | if (ret < 0) { | 191 | if (ret < 0) { |
192 | cancel_work_sync(&led->work); | 192 | cancel_work_sync(&led->work); |
193 | goto err_led; | 193 | goto err_vcc; |
194 | } | 194 | } |
195 | 195 | ||
196 | /* to expose the default value to userspace */ | 196 | /* to expose the default value to userspace */ |
@@ -201,8 +201,6 @@ static int __devinit regulator_led_probe(struct platform_device *pdev) | |||
201 | 201 | ||
202 | return 0; | 202 | return 0; |
203 | 203 | ||
204 | err_led: | ||
205 | kfree(led); | ||
206 | err_vcc: | 204 | err_vcc: |
207 | regulator_put(vcc); | 205 | regulator_put(vcc); |
208 | return ret; | 206 | return ret; |
@@ -216,7 +214,6 @@ static int __devexit regulator_led_remove(struct platform_device *pdev) | |||
216 | cancel_work_sync(&led->work); | 214 | cancel_work_sync(&led->work); |
217 | regulator_led_disable(led); | 215 | regulator_led_disable(led); |
218 | regulator_put(led->vcc); | 216 | regulator_put(led->vcc); |
219 | kfree(led); | ||
220 | return 0; | 217 | return 0; |
221 | } | 218 | } |
222 | 219 | ||
diff --git a/drivers/leds/leds-renesas-tpu.c b/drivers/leds/leds-renesas-tpu.c index 32fe337d5c68..9ee12c28059a 100644 --- a/drivers/leds/leds-renesas-tpu.c +++ b/drivers/leds/leds-renesas-tpu.c | |||
@@ -243,31 +243,30 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
243 | struct led_renesas_tpu_config *cfg = pdev->dev.platform_data; | 243 | struct led_renesas_tpu_config *cfg = pdev->dev.platform_data; |
244 | struct r_tpu_priv *p; | 244 | struct r_tpu_priv *p; |
245 | struct resource *res; | 245 | struct resource *res; |
246 | int ret = -ENXIO; | 246 | int ret; |
247 | 247 | ||
248 | if (!cfg) { | 248 | if (!cfg) { |
249 | dev_err(&pdev->dev, "missing platform data\n"); | 249 | dev_err(&pdev->dev, "missing platform data\n"); |
250 | goto err0; | 250 | goto err0; |
251 | } | 251 | } |
252 | 252 | ||
253 | p = kzalloc(sizeof(*p), GFP_KERNEL); | 253 | p = devm_kzalloc(&pdev->dev, sizeof(*p), GFP_KERNEL); |
254 | if (p == NULL) { | 254 | if (p == NULL) { |
255 | dev_err(&pdev->dev, "failed to allocate driver data\n"); | 255 | dev_err(&pdev->dev, "failed to allocate driver data\n"); |
256 | ret = -ENOMEM; | 256 | return -ENOMEM; |
257 | goto err0; | ||
258 | } | 257 | } |
259 | 258 | ||
260 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); | 259 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); |
261 | if (!res) { | 260 | if (!res) { |
262 | dev_err(&pdev->dev, "failed to get I/O memory\n"); | 261 | dev_err(&pdev->dev, "failed to get I/O memory\n"); |
263 | goto err1; | 262 | return -ENXIO; |
264 | } | 263 | } |
265 | 264 | ||
266 | /* map memory, let mapbase point to our channel */ | 265 | /* map memory, let mapbase point to our channel */ |
267 | p->mapbase = ioremap_nocache(res->start, resource_size(res)); | 266 | p->mapbase = ioremap_nocache(res->start, resource_size(res)); |
268 | if (p->mapbase == NULL) { | 267 | if (p->mapbase == NULL) { |
269 | dev_err(&pdev->dev, "failed to remap I/O memory\n"); | 268 | dev_err(&pdev->dev, "failed to remap I/O memory\n"); |
270 | goto err1; | 269 | return -ENXIO; |
271 | } | 270 | } |
272 | 271 | ||
273 | /* get hold of clock */ | 272 | /* get hold of clock */ |
@@ -275,7 +274,7 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
275 | if (IS_ERR(p->clk)) { | 274 | if (IS_ERR(p->clk)) { |
276 | dev_err(&pdev->dev, "cannot get clock\n"); | 275 | dev_err(&pdev->dev, "cannot get clock\n"); |
277 | ret = PTR_ERR(p->clk); | 276 | ret = PTR_ERR(p->clk); |
278 | goto err2; | 277 | goto err0; |
279 | } | 278 | } |
280 | 279 | ||
281 | p->pdev = pdev; | 280 | p->pdev = pdev; |
@@ -294,7 +293,7 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
294 | p->ldev.flags |= LED_CORE_SUSPENDRESUME; | 293 | p->ldev.flags |= LED_CORE_SUSPENDRESUME; |
295 | ret = led_classdev_register(&pdev->dev, &p->ldev); | 294 | ret = led_classdev_register(&pdev->dev, &p->ldev); |
296 | if (ret < 0) | 295 | if (ret < 0) |
297 | goto err3; | 296 | goto err1; |
298 | 297 | ||
299 | /* max_brightness may be updated by the LED core code */ | 298 | /* max_brightness may be updated by the LED core code */ |
300 | p->min_rate = p->ldev.max_brightness * p->refresh_rate; | 299 | p->min_rate = p->ldev.max_brightness * p->refresh_rate; |
@@ -302,14 +301,11 @@ static int __devinit r_tpu_probe(struct platform_device *pdev) | |||
302 | pm_runtime_enable(&pdev->dev); | 301 | pm_runtime_enable(&pdev->dev); |
303 | return 0; | 302 | return 0; |
304 | 303 | ||
305 | err3: | 304 | err1: |
306 | r_tpu_set_pin(p, R_TPU_PIN_UNUSED, LED_OFF); | 305 | r_tpu_set_pin(p, R_TPU_PIN_UNUSED, LED_OFF); |
307 | clk_put(p->clk); | 306 | clk_put(p->clk); |
308 | err2: | ||
309 | iounmap(p->mapbase); | ||
310 | err1: | ||
311 | kfree(p); | ||
312 | err0: | 307 | err0: |
308 | iounmap(p->mapbase); | ||
313 | return ret; | 309 | return ret; |
314 | } | 310 | } |
315 | 311 | ||
@@ -327,7 +323,6 @@ static int __devexit r_tpu_remove(struct platform_device *pdev) | |||
327 | clk_put(p->clk); | 323 | clk_put(p->clk); |
328 | 324 | ||
329 | iounmap(p->mapbase); | 325 | iounmap(p->mapbase); |
330 | kfree(p); | ||
331 | return 0; | 326 | return 0; |
332 | } | 327 | } |
333 | 328 | ||
diff --git a/drivers/leds/leds-s3c24xx.c b/drivers/leds/leds-s3c24xx.c index bd0a5ed49c42..942f0ea18178 100644 --- a/drivers/leds/leds-s3c24xx.c +++ b/drivers/leds/leds-s3c24xx.c | |||
@@ -45,17 +45,19 @@ static void s3c24xx_led_set(struct led_classdev *led_cdev, | |||
45 | { | 45 | { |
46 | struct s3c24xx_gpio_led *led = to_gpio(led_cdev); | 46 | struct s3c24xx_gpio_led *led = to_gpio(led_cdev); |
47 | struct s3c24xx_led_platdata *pd = led->pdata; | 47 | struct s3c24xx_led_platdata *pd = led->pdata; |
48 | int state = (value ? 1 : 0) ^ (pd->flags & S3C24XX_LEDF_ACTLOW); | ||
48 | 49 | ||
49 | /* there will be a short delay between setting the output and | 50 | /* there will be a short delay between setting the output and |
50 | * going from output to input when using tristate. */ | 51 | * going from output to input when using tristate. */ |
51 | 52 | ||
52 | s3c2410_gpio_setpin(pd->gpio, (value ? 1 : 0) ^ | 53 | gpio_set_value(pd->gpio, state); |
53 | (pd->flags & S3C24XX_LEDF_ACTLOW)); | ||
54 | |||
55 | if (pd->flags & S3C24XX_LEDF_TRISTATE) | ||
56 | s3c2410_gpio_cfgpin(pd->gpio, | ||
57 | value ? S3C2410_GPIO_OUTPUT : S3C2410_GPIO_INPUT); | ||
58 | 54 | ||
55 | if (pd->flags & S3C24XX_LEDF_TRISTATE) { | ||
56 | if (value) | ||
57 | gpio_direction_output(pd->gpio, state); | ||
58 | else | ||
59 | gpio_direction_input(pd->gpio); | ||
60 | } | ||
59 | } | 61 | } |
60 | 62 | ||
61 | static int s3c24xx_led_remove(struct platform_device *dev) | 63 | static int s3c24xx_led_remove(struct platform_device *dev) |
@@ -63,7 +65,6 @@ static int s3c24xx_led_remove(struct platform_device *dev) | |||
63 | struct s3c24xx_gpio_led *led = pdev_to_gpio(dev); | 65 | struct s3c24xx_gpio_led *led = pdev_to_gpio(dev); |
64 | 66 | ||
65 | led_classdev_unregister(&led->cdev); | 67 | led_classdev_unregister(&led->cdev); |
66 | kfree(led); | ||
67 | 68 | ||
68 | return 0; | 69 | return 0; |
69 | } | 70 | } |
@@ -74,7 +75,8 @@ static int s3c24xx_led_probe(struct platform_device *dev) | |||
74 | struct s3c24xx_gpio_led *led; | 75 | struct s3c24xx_gpio_led *led; |
75 | int ret; | 76 | int ret; |
76 | 77 | ||
77 | led = kzalloc(sizeof(struct s3c24xx_gpio_led), GFP_KERNEL); | 78 | led = devm_kzalloc(&dev->dev, sizeof(struct s3c24xx_gpio_led), |
79 | GFP_KERNEL); | ||
78 | if (led == NULL) { | 80 | if (led == NULL) { |
79 | dev_err(&dev->dev, "No memory for device\n"); | 81 | dev_err(&dev->dev, "No memory for device\n"); |
80 | return -ENOMEM; | 82 | return -ENOMEM; |
@@ -89,27 +91,27 @@ static int s3c24xx_led_probe(struct platform_device *dev) | |||
89 | 91 | ||
90 | led->pdata = pdata; | 92 | led->pdata = pdata; |
91 | 93 | ||
94 | ret = devm_gpio_request(&dev->dev, pdata->gpio, "S3C24XX_LED"); | ||
95 | if (ret < 0) | ||
96 | return ret; | ||
97 | |||
92 | /* no point in having a pull-up if we are always driving */ | 98 | /* no point in having a pull-up if we are always driving */ |
93 | 99 | ||
94 | if (pdata->flags & S3C24XX_LEDF_TRISTATE) { | 100 | s3c_gpio_setpull(pdata->gpio, S3C_GPIO_PULL_NONE); |
95 | s3c2410_gpio_setpin(pdata->gpio, 0); | 101 | |
96 | s3c2410_gpio_cfgpin(pdata->gpio, S3C2410_GPIO_INPUT); | 102 | if (pdata->flags & S3C24XX_LEDF_TRISTATE) |
97 | } else { | 103 | gpio_direction_input(pdata->gpio); |
98 | s3c2410_gpio_pullup(pdata->gpio, 0); | 104 | else |
99 | s3c2410_gpio_setpin(pdata->gpio, 0); | 105 | gpio_direction_output(pdata->gpio, |
100 | s3c2410_gpio_cfgpin(pdata->gpio, S3C2410_GPIO_OUTPUT); | 106 | pdata->flags & S3C24XX_LEDF_ACTLOW ? 1 : 0); |
101 | } | ||
102 | 107 | ||
103 | /* register our new led device */ | 108 | /* register our new led device */ |
104 | 109 | ||
105 | ret = led_classdev_register(&dev->dev, &led->cdev); | 110 | ret = led_classdev_register(&dev->dev, &led->cdev); |
106 | if (ret < 0) { | 111 | if (ret < 0) |
107 | dev_err(&dev->dev, "led_classdev_register failed\n"); | 112 | dev_err(&dev->dev, "led_classdev_register failed\n"); |
108 | kfree(led); | ||
109 | return ret; | ||
110 | } | ||
111 | 113 | ||
112 | return 0; | 114 | return ret; |
113 | } | 115 | } |
114 | 116 | ||
115 | static struct platform_driver s3c24xx_led_driver = { | 117 | static struct platform_driver s3c24xx_led_driver = { |
diff --git a/drivers/leds/leds-sunfire.c b/drivers/leds/leds-sunfire.c index 1757396b20b3..134d9a4b34f1 100644 --- a/drivers/leds/leds-sunfire.c +++ b/drivers/leds/leds-sunfire.c | |||
@@ -132,15 +132,13 @@ static int __devinit sunfire_led_generic_probe(struct platform_device *pdev, | |||
132 | if (pdev->num_resources != 1) { | 132 | if (pdev->num_resources != 1) { |
133 | printk(KERN_ERR PFX "Wrong number of resources %d, should be 1\n", | 133 | printk(KERN_ERR PFX "Wrong number of resources %d, should be 1\n", |
134 | pdev->num_resources); | 134 | pdev->num_resources); |
135 | err = -EINVAL; | 135 | return -EINVAL; |
136 | goto out; | ||
137 | } | 136 | } |
138 | 137 | ||
139 | p = kzalloc(sizeof(*p), GFP_KERNEL); | 138 | p = devm_kzalloc(&pdev->dev, sizeof(*p), GFP_KERNEL); |
140 | if (!p) { | 139 | if (!p) { |
141 | printk(KERN_ERR PFX "Could not allocate struct sunfire_drvdata\n"); | 140 | printk(KERN_ERR PFX "Could not allocate struct sunfire_drvdata\n"); |
142 | err = -ENOMEM; | 141 | return -ENOMEM; |
143 | goto out; | ||
144 | } | 142 | } |
145 | 143 | ||
146 | for (i = 0; i < NUM_LEDS_PER_BOARD; i++) { | 144 | for (i = 0; i < NUM_LEDS_PER_BOARD; i++) { |
@@ -156,20 +154,15 @@ static int __devinit sunfire_led_generic_probe(struct platform_device *pdev, | |||
156 | if (err) { | 154 | if (err) { |
157 | printk(KERN_ERR PFX "Could not register %s LED\n", | 155 | printk(KERN_ERR PFX "Could not register %s LED\n", |
158 | lp->name); | 156 | lp->name); |
159 | goto out_unregister_led_cdevs; | 157 | for (i--; i >= 0; i--) |
158 | led_classdev_unregister(&p->leds[i].led_cdev); | ||
159 | return err; | ||
160 | } | 160 | } |
161 | } | 161 | } |
162 | 162 | ||
163 | dev_set_drvdata(&pdev->dev, p); | 163 | dev_set_drvdata(&pdev->dev, p); |
164 | 164 | ||
165 | return 0; | 165 | return 0; |
166 | |||
167 | out_unregister_led_cdevs: | ||
168 | for (i--; i >= 0; i--) | ||
169 | led_classdev_unregister(&p->leds[i].led_cdev); | ||
170 | kfree(p); | ||
171 | out: | ||
172 | return err; | ||
173 | } | 166 | } |
174 | 167 | ||
175 | static int __devexit sunfire_led_generic_remove(struct platform_device *pdev) | 168 | static int __devexit sunfire_led_generic_remove(struct platform_device *pdev) |
@@ -180,8 +173,6 @@ static int __devexit sunfire_led_generic_remove(struct platform_device *pdev) | |||
180 | for (i = 0; i < NUM_LEDS_PER_BOARD; i++) | 173 | for (i = 0; i < NUM_LEDS_PER_BOARD; i++) |
181 | led_classdev_unregister(&p->leds[i].led_cdev); | 174 | led_classdev_unregister(&p->leds[i].led_cdev); |
182 | 175 | ||
183 | kfree(p); | ||
184 | |||
185 | return 0; | 176 | return 0; |
186 | } | 177 | } |
187 | 178 | ||
diff --git a/drivers/leds/leds-tca6507.c b/drivers/leds/leds-tca6507.c index 6c1c14f31635..dabcf7ae8d0f 100644 --- a/drivers/leds/leds-tca6507.c +++ b/drivers/leds/leds-tca6507.c | |||
@@ -687,7 +687,7 @@ static int __devinit tca6507_probe(struct i2c_client *client, | |||
687 | NUM_LEDS); | 687 | NUM_LEDS); |
688 | return -ENODEV; | 688 | return -ENODEV; |
689 | } | 689 | } |
690 | tca = kzalloc(sizeof(*tca), GFP_KERNEL); | 690 | tca = devm_kzalloc(&client->dev, sizeof(*tca), GFP_KERNEL); |
691 | if (!tca) | 691 | if (!tca) |
692 | return -ENOMEM; | 692 | return -ENOMEM; |
693 | 693 | ||
@@ -727,7 +727,6 @@ exit: | |||
727 | if (tca->leds[i].led_cdev.name) | 727 | if (tca->leds[i].led_cdev.name) |
728 | led_classdev_unregister(&tca->leds[i].led_cdev); | 728 | led_classdev_unregister(&tca->leds[i].led_cdev); |
729 | } | 729 | } |
730 | kfree(tca); | ||
731 | return err; | 730 | return err; |
732 | } | 731 | } |
733 | 732 | ||
@@ -743,7 +742,6 @@ static int __devexit tca6507_remove(struct i2c_client *client) | |||
743 | } | 742 | } |
744 | tca6507_remove_gpio(tca); | 743 | tca6507_remove_gpio(tca); |
745 | cancel_work_sync(&tca->work); | 744 | cancel_work_sync(&tca->work); |
746 | kfree(tca); | ||
747 | 745 | ||
748 | return 0; | 746 | return 0; |
749 | } | 747 | } |
@@ -758,18 +756,7 @@ static struct i2c_driver tca6507_driver = { | |||
758 | .id_table = tca6507_id, | 756 | .id_table = tca6507_id, |
759 | }; | 757 | }; |
760 | 758 | ||
761 | static int __init tca6507_leds_init(void) | 759 | module_i2c_driver(tca6507_driver); |
762 | { | ||
763 | return i2c_add_driver(&tca6507_driver); | ||
764 | } | ||
765 | |||
766 | static void __exit tca6507_leds_exit(void) | ||
767 | { | ||
768 | i2c_del_driver(&tca6507_driver); | ||
769 | } | ||
770 | |||
771 | module_init(tca6507_leds_init); | ||
772 | module_exit(tca6507_leds_exit); | ||
773 | 760 | ||
774 | MODULE_AUTHOR("NeilBrown <neilb@suse.de>"); | 761 | MODULE_AUTHOR("NeilBrown <neilb@suse.de>"); |
775 | MODULE_DESCRIPTION("TCA6507 LED/GPO driver"); | 762 | MODULE_DESCRIPTION("TCA6507 LED/GPO driver"); |
diff --git a/drivers/leds/leds.h b/drivers/leds/leds.h index e77c7f8dcdd4..d02acd496126 100644 --- a/drivers/leds/leds.h +++ b/drivers/leds/leds.h | |||
@@ -17,7 +17,7 @@ | |||
17 | #include <linux/rwsem.h> | 17 | #include <linux/rwsem.h> |
18 | #include <linux/leds.h> | 18 | #include <linux/leds.h> |
19 | 19 | ||
20 | static inline void led_set_brightness(struct led_classdev *led_cdev, | 20 | static inline void __led_set_brightness(struct led_classdev *led_cdev, |
21 | enum led_brightness value) | 21 | enum led_brightness value) |
22 | { | 22 | { |
23 | if (value > led_cdev->max_brightness) | 23 | if (value > led_cdev->max_brightness) |
diff --git a/drivers/leds/ledtrig-backlight.c b/drivers/leds/ledtrig-backlight.c index e2726867c5d4..b941685f2227 100644 --- a/drivers/leds/ledtrig-backlight.c +++ b/drivers/leds/ledtrig-backlight.c | |||
@@ -46,9 +46,9 @@ static int fb_notifier_callback(struct notifier_block *p, | |||
46 | 46 | ||
47 | if ((n->old_status == UNBLANK) ^ n->invert) { | 47 | if ((n->old_status == UNBLANK) ^ n->invert) { |
48 | n->brightness = led->brightness; | 48 | n->brightness = led->brightness; |
49 | led_set_brightness(led, LED_OFF); | 49 | __led_set_brightness(led, LED_OFF); |
50 | } else { | 50 | } else { |
51 | led_set_brightness(led, n->brightness); | 51 | __led_set_brightness(led, n->brightness); |
52 | } | 52 | } |
53 | 53 | ||
54 | n->old_status = new_status; | 54 | n->old_status = new_status; |
@@ -87,9 +87,9 @@ static ssize_t bl_trig_invert_store(struct device *dev, | |||
87 | 87 | ||
88 | /* After inverting, we need to update the LED. */ | 88 | /* After inverting, we need to update the LED. */ |
89 | if ((n->old_status == BLANK) ^ n->invert) | 89 | if ((n->old_status == BLANK) ^ n->invert) |
90 | led_set_brightness(led, LED_OFF); | 90 | __led_set_brightness(led, LED_OFF); |
91 | else | 91 | else |
92 | led_set_brightness(led, n->brightness); | 92 | __led_set_brightness(led, n->brightness); |
93 | 93 | ||
94 | return num; | 94 | return num; |
95 | } | 95 | } |
diff --git a/drivers/leds/ledtrig-default-on.c b/drivers/leds/ledtrig-default-on.c index a4ef54b9d508..eac1f1b1adac 100644 --- a/drivers/leds/ledtrig-default-on.c +++ b/drivers/leds/ledtrig-default-on.c | |||
@@ -19,7 +19,7 @@ | |||
19 | 19 | ||
20 | static void defon_trig_activate(struct led_classdev *led_cdev) | 20 | static void defon_trig_activate(struct led_classdev *led_cdev) |
21 | { | 21 | { |
22 | led_set_brightness(led_cdev, led_cdev->max_brightness); | 22 | __led_set_brightness(led_cdev, led_cdev->max_brightness); |
23 | } | 23 | } |
24 | 24 | ||
25 | static struct led_trigger defon_led_trigger = { | 25 | static struct led_trigger defon_led_trigger = { |
diff --git a/drivers/leds/ledtrig-gpio.c b/drivers/leds/ledtrig-gpio.c index f057c101b896..ba215dc42f98 100644 --- a/drivers/leds/ledtrig-gpio.c +++ b/drivers/leds/ledtrig-gpio.c | |||
@@ -54,12 +54,12 @@ static void gpio_trig_work(struct work_struct *work) | |||
54 | 54 | ||
55 | if (tmp) { | 55 | if (tmp) { |
56 | if (gpio_data->desired_brightness) | 56 | if (gpio_data->desired_brightness) |
57 | led_set_brightness(gpio_data->led, | 57 | __led_set_brightness(gpio_data->led, |
58 | gpio_data->desired_brightness); | 58 | gpio_data->desired_brightness); |
59 | else | 59 | else |
60 | led_set_brightness(gpio_data->led, LED_FULL); | 60 | __led_set_brightness(gpio_data->led, LED_FULL); |
61 | } else { | 61 | } else { |
62 | led_set_brightness(gpio_data->led, LED_OFF); | 62 | __led_set_brightness(gpio_data->led, LED_OFF); |
63 | } | 63 | } |
64 | } | 64 | } |
65 | 65 | ||
diff --git a/drivers/leds/ledtrig-heartbeat.c b/drivers/leds/ledtrig-heartbeat.c index a019fbb70880..1edc7463ce83 100644 --- a/drivers/leds/ledtrig-heartbeat.c +++ b/drivers/leds/ledtrig-heartbeat.c | |||
@@ -74,7 +74,7 @@ static void led_heartbeat_function(unsigned long data) | |||
74 | break; | 74 | break; |
75 | } | 75 | } |
76 | 76 | ||
77 | led_set_brightness(led_cdev, brightness); | 77 | __led_set_brightness(led_cdev, brightness); |
78 | mod_timer(&heartbeat_data->timer, jiffies + delay); | 78 | mod_timer(&heartbeat_data->timer, jiffies + delay); |
79 | } | 79 | } |
80 | 80 | ||
diff --git a/drivers/leds/ledtrig-ide-disk.c b/drivers/leds/ledtrig-ide-disk.c index ec099fcbcb00..2cd7c0cf5924 100644 --- a/drivers/leds/ledtrig-ide-disk.c +++ b/drivers/leds/ledtrig-ide-disk.c | |||
@@ -12,39 +12,22 @@ | |||
12 | */ | 12 | */ |
13 | 13 | ||
14 | #include <linux/module.h> | 14 | #include <linux/module.h> |
15 | #include <linux/jiffies.h> | ||
16 | #include <linux/kernel.h> | 15 | #include <linux/kernel.h> |
17 | #include <linux/init.h> | 16 | #include <linux/init.h> |
18 | #include <linux/timer.h> | ||
19 | #include <linux/leds.h> | 17 | #include <linux/leds.h> |
20 | 18 | ||
21 | static void ledtrig_ide_timerfunc(unsigned long data); | 19 | #define BLINK_DELAY 30 |
22 | 20 | ||
23 | DEFINE_LED_TRIGGER(ledtrig_ide); | 21 | DEFINE_LED_TRIGGER(ledtrig_ide); |
24 | static DEFINE_TIMER(ledtrig_ide_timer, ledtrig_ide_timerfunc, 0, 0); | 22 | static unsigned long ide_blink_delay = BLINK_DELAY; |
25 | static int ide_activity; | ||
26 | static int ide_lastactivity; | ||
27 | 23 | ||
28 | void ledtrig_ide_activity(void) | 24 | void ledtrig_ide_activity(void) |
29 | { | 25 | { |
30 | ide_activity++; | 26 | led_trigger_blink_oneshot(ledtrig_ide, |
31 | if (!timer_pending(&ledtrig_ide_timer)) | 27 | &ide_blink_delay, &ide_blink_delay, 0); |
32 | mod_timer(&ledtrig_ide_timer, jiffies + msecs_to_jiffies(10)); | ||
33 | } | 28 | } |
34 | EXPORT_SYMBOL(ledtrig_ide_activity); | 29 | EXPORT_SYMBOL(ledtrig_ide_activity); |
35 | 30 | ||
36 | static void ledtrig_ide_timerfunc(unsigned long data) | ||
37 | { | ||
38 | if (ide_lastactivity != ide_activity) { | ||
39 | ide_lastactivity = ide_activity; | ||
40 | /* INT_MAX will set each LED to its maximum brightness */ | ||
41 | led_trigger_event(ledtrig_ide, INT_MAX); | ||
42 | mod_timer(&ledtrig_ide_timer, jiffies + msecs_to_jiffies(10)); | ||
43 | } else { | ||
44 | led_trigger_event(ledtrig_ide, LED_OFF); | ||
45 | } | ||
46 | } | ||
47 | |||
48 | static int __init ledtrig_ide_init(void) | 31 | static int __init ledtrig_ide_init(void) |
49 | { | 32 | { |
50 | led_trigger_register_simple("ide-disk", &ledtrig_ide); | 33 | led_trigger_register_simple("ide-disk", &ledtrig_ide); |
diff --git a/drivers/leds/ledtrig-oneshot.c b/drivers/leds/ledtrig-oneshot.c new file mode 100644 index 000000000000..2c029aa5c4f1 --- /dev/null +++ b/drivers/leds/ledtrig-oneshot.c | |||
@@ -0,0 +1,204 @@ | |||
1 | /* | ||
2 | * One-shot LED Trigger | ||
3 | * | ||
4 | * Copyright 2012, Fabio Baltieri <fabio.baltieri@gmail.com> | ||
5 | * | ||
6 | * Based on ledtrig-timer.c by Richard Purdie <rpurdie@openedhand.com> | ||
7 | * | ||
8 | * This program is free software; you can redistribute it and/or modify | ||
9 | * it under the terms of the GNU General Public License version 2 as | ||
10 | * published by the Free Software Foundation. | ||
11 | * | ||
12 | */ | ||
13 | |||
14 | #include <linux/module.h> | ||
15 | #include <linux/kernel.h> | ||
16 | #include <linux/init.h> | ||
17 | #include <linux/device.h> | ||
18 | #include <linux/ctype.h> | ||
19 | #include <linux/slab.h> | ||
20 | #include <linux/leds.h> | ||
21 | #include "leds.h" | ||
22 | |||
23 | #define DEFAULT_DELAY 100 | ||
24 | |||
25 | struct oneshot_trig_data { | ||
26 | unsigned int invert; | ||
27 | }; | ||
28 | |||
29 | static ssize_t led_shot(struct device *dev, | ||
30 | struct device_attribute *attr, const char *buf, size_t size) | ||
31 | { | ||
32 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
33 | struct oneshot_trig_data *oneshot_data = led_cdev->trigger_data; | ||
34 | |||
35 | led_blink_set_oneshot(led_cdev, | ||
36 | &led_cdev->blink_delay_on, &led_cdev->blink_delay_off, | ||
37 | oneshot_data->invert); | ||
38 | |||
39 | /* content is ignored */ | ||
40 | return size; | ||
41 | } | ||
42 | static ssize_t led_invert_show(struct device *dev, | ||
43 | struct device_attribute *attr, char *buf) | ||
44 | { | ||
45 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
46 | struct oneshot_trig_data *oneshot_data = led_cdev->trigger_data; | ||
47 | |||
48 | return sprintf(buf, "%u\n", oneshot_data->invert); | ||
49 | } | ||
50 | |||
51 | static ssize_t led_invert_store(struct device *dev, | ||
52 | struct device_attribute *attr, const char *buf, size_t size) | ||
53 | { | ||
54 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
55 | struct oneshot_trig_data *oneshot_data = led_cdev->trigger_data; | ||
56 | unsigned long state; | ||
57 | int ret; | ||
58 | |||
59 | ret = kstrtoul(buf, 0, &state); | ||
60 | if (ret) | ||
61 | return ret; | ||
62 | |||
63 | oneshot_data->invert = !!state; | ||
64 | |||
65 | if (oneshot_data->invert) | ||
66 | __led_set_brightness(led_cdev, LED_FULL); | ||
67 | else | ||
68 | __led_set_brightness(led_cdev, LED_OFF); | ||
69 | |||
70 | return size; | ||
71 | } | ||
72 | |||
73 | static ssize_t led_delay_on_show(struct device *dev, | ||
74 | struct device_attribute *attr, char *buf) | ||
75 | { | ||
76 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
77 | |||
78 | return sprintf(buf, "%lu\n", led_cdev->blink_delay_on); | ||
79 | } | ||
80 | |||
81 | static ssize_t led_delay_on_store(struct device *dev, | ||
82 | struct device_attribute *attr, const char *buf, size_t size) | ||
83 | { | ||
84 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
85 | unsigned long state; | ||
86 | int ret; | ||
87 | |||
88 | ret = kstrtoul(buf, 0, &state); | ||
89 | if (ret) | ||
90 | return ret; | ||
91 | |||
92 | led_cdev->blink_delay_on = state; | ||
93 | |||
94 | return size; | ||
95 | } | ||
96 | static ssize_t led_delay_off_show(struct device *dev, | ||
97 | struct device_attribute *attr, char *buf) | ||
98 | { | ||
99 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
100 | |||
101 | return sprintf(buf, "%lu\n", led_cdev->blink_delay_off); | ||
102 | } | ||
103 | |||
104 | static ssize_t led_delay_off_store(struct device *dev, | ||
105 | struct device_attribute *attr, const char *buf, size_t size) | ||
106 | { | ||
107 | struct led_classdev *led_cdev = dev_get_drvdata(dev); | ||
108 | unsigned long state; | ||
109 | int ret; | ||
110 | |||
111 | ret = kstrtoul(buf, 0, &state); | ||
112 | if (ret) | ||
113 | return ret; | ||
114 | |||
115 | led_cdev->blink_delay_off = state; | ||
116 | |||
117 | return size; | ||
118 | } | ||
119 | |||
120 | static DEVICE_ATTR(delay_on, 0644, led_delay_on_show, led_delay_on_store); | ||
121 | static DEVICE_ATTR(delay_off, 0644, led_delay_off_show, led_delay_off_store); | ||
122 | static DEVICE_ATTR(invert, 0644, led_invert_show, led_invert_store); | ||
123 | static DEVICE_ATTR(shot, 0200, NULL, led_shot); | ||
124 | |||
125 | static void oneshot_trig_activate(struct led_classdev *led_cdev) | ||
126 | { | ||
127 | struct oneshot_trig_data *oneshot_data; | ||
128 | int rc; | ||
129 | |||
130 | oneshot_data = kzalloc(sizeof(*oneshot_data), GFP_KERNEL); | ||
131 | if (!oneshot_data) | ||
132 | return; | ||
133 | |||
134 | led_cdev->trigger_data = oneshot_data; | ||
135 | |||
136 | rc = device_create_file(led_cdev->dev, &dev_attr_delay_on); | ||
137 | if (rc) | ||
138 | goto err_out_trig_data; | ||
139 | rc = device_create_file(led_cdev->dev, &dev_attr_delay_off); | ||
140 | if (rc) | ||
141 | goto err_out_delayon; | ||
142 | rc = device_create_file(led_cdev->dev, &dev_attr_invert); | ||
143 | if (rc) | ||
144 | goto err_out_delayoff; | ||
145 | rc = device_create_file(led_cdev->dev, &dev_attr_shot); | ||
146 | if (rc) | ||
147 | goto err_out_invert; | ||
148 | |||
149 | led_cdev->blink_delay_on = DEFAULT_DELAY; | ||
150 | led_cdev->blink_delay_off = DEFAULT_DELAY; | ||
151 | |||
152 | led_cdev->activated = true; | ||
153 | |||
154 | return; | ||
155 | |||
156 | err_out_invert: | ||
157 | device_remove_file(led_cdev->dev, &dev_attr_invert); | ||
158 | err_out_delayoff: | ||
159 | device_remove_file(led_cdev->dev, &dev_attr_delay_off); | ||
160 | err_out_delayon: | ||
161 | device_remove_file(led_cdev->dev, &dev_attr_delay_on); | ||
162 | err_out_trig_data: | ||
163 | kfree(led_cdev->trigger_data); | ||
164 | } | ||
165 | |||
166 | static void oneshot_trig_deactivate(struct led_classdev *led_cdev) | ||
167 | { | ||
168 | struct oneshot_trig_data *oneshot_data = led_cdev->trigger_data; | ||
169 | |||
170 | if (led_cdev->activated) { | ||
171 | device_remove_file(led_cdev->dev, &dev_attr_delay_on); | ||
172 | device_remove_file(led_cdev->dev, &dev_attr_delay_off); | ||
173 | device_remove_file(led_cdev->dev, &dev_attr_invert); | ||
174 | device_remove_file(led_cdev->dev, &dev_attr_shot); | ||
175 | kfree(oneshot_data); | ||
176 | led_cdev->activated = false; | ||
177 | } | ||
178 | |||
179 | /* Stop blinking */ | ||
180 | led_set_brightness(led_cdev, LED_OFF); | ||
181 | } | ||
182 | |||
183 | static struct led_trigger oneshot_led_trigger = { | ||
184 | .name = "oneshot", | ||
185 | .activate = oneshot_trig_activate, | ||
186 | .deactivate = oneshot_trig_deactivate, | ||
187 | }; | ||
188 | |||
189 | static int __init oneshot_trig_init(void) | ||
190 | { | ||
191 | return led_trigger_register(&oneshot_led_trigger); | ||
192 | } | ||
193 | |||
194 | static void __exit oneshot_trig_exit(void) | ||
195 | { | ||
196 | led_trigger_unregister(&oneshot_led_trigger); | ||
197 | } | ||
198 | |||
199 | module_init(oneshot_trig_init); | ||
200 | module_exit(oneshot_trig_exit); | ||
201 | |||
202 | MODULE_AUTHOR("Fabio Baltieri <fabio.baltieri@gmail.com>"); | ||
203 | MODULE_DESCRIPTION("One-shot LED trigger"); | ||
204 | MODULE_LICENSE("GPL"); | ||
diff --git a/drivers/leds/ledtrig-timer.c b/drivers/leds/ledtrig-timer.c index 9010f7abaf2c..f774d0592204 100644 --- a/drivers/leds/ledtrig-timer.c +++ b/drivers/leds/ledtrig-timer.c | |||
@@ -104,7 +104,7 @@ static void timer_trig_deactivate(struct led_classdev *led_cdev) | |||
104 | } | 104 | } |
105 | 105 | ||
106 | /* Stop blinking */ | 106 | /* Stop blinking */ |
107 | led_brightness_set(led_cdev, LED_OFF); | 107 | led_set_brightness(led_cdev, LED_OFF); |
108 | } | 108 | } |
109 | 109 | ||
110 | static struct led_trigger timer_led_trigger = { | 110 | static struct led_trigger timer_led_trigger = { |
diff --git a/drivers/leds/ledtrig-transient.c b/drivers/leds/ledtrig-transient.c index 83179f435e1e..398f1042c43e 100644 --- a/drivers/leds/ledtrig-transient.c +++ b/drivers/leds/ledtrig-transient.c | |||
@@ -41,7 +41,7 @@ static void transient_timer_function(unsigned long data) | |||
41 | struct transient_trig_data *transient_data = led_cdev->trigger_data; | 41 | struct transient_trig_data *transient_data = led_cdev->trigger_data; |
42 | 42 | ||
43 | transient_data->activate = 0; | 43 | transient_data->activate = 0; |
44 | led_set_brightness(led_cdev, transient_data->restore_state); | 44 | __led_set_brightness(led_cdev, transient_data->restore_state); |
45 | } | 45 | } |
46 | 46 | ||
47 | static ssize_t transient_activate_show(struct device *dev, | 47 | static ssize_t transient_activate_show(struct device *dev, |
@@ -72,7 +72,7 @@ static ssize_t transient_activate_store(struct device *dev, | |||
72 | if (state == 0 && transient_data->activate == 1) { | 72 | if (state == 0 && transient_data->activate == 1) { |
73 | del_timer(&transient_data->timer); | 73 | del_timer(&transient_data->timer); |
74 | transient_data->activate = state; | 74 | transient_data->activate = state; |
75 | led_set_brightness(led_cdev, transient_data->restore_state); | 75 | __led_set_brightness(led_cdev, transient_data->restore_state); |
76 | return size; | 76 | return size; |
77 | } | 77 | } |
78 | 78 | ||
@@ -80,7 +80,7 @@ static ssize_t transient_activate_store(struct device *dev, | |||
80 | if (state == 1 && transient_data->activate == 0 && | 80 | if (state == 1 && transient_data->activate == 0 && |
81 | transient_data->duration != 0) { | 81 | transient_data->duration != 0) { |
82 | transient_data->activate = state; | 82 | transient_data->activate = state; |
83 | led_set_brightness(led_cdev, transient_data->state); | 83 | __led_set_brightness(led_cdev, transient_data->state); |
84 | transient_data->restore_state = | 84 | transient_data->restore_state = |
85 | (transient_data->state == LED_FULL) ? LED_OFF : LED_FULL; | 85 | (transient_data->state == LED_FULL) ? LED_OFF : LED_FULL; |
86 | mod_timer(&transient_data->timer, | 86 | mod_timer(&transient_data->timer, |
@@ -203,7 +203,7 @@ static void transient_trig_deactivate(struct led_classdev *led_cdev) | |||
203 | 203 | ||
204 | if (led_cdev->activated) { | 204 | if (led_cdev->activated) { |
205 | del_timer_sync(&transient_data->timer); | 205 | del_timer_sync(&transient_data->timer); |
206 | led_set_brightness(led_cdev, transient_data->restore_state); | 206 | __led_set_brightness(led_cdev, transient_data->restore_state); |
207 | device_remove_file(led_cdev->dev, &dev_attr_activate); | 207 | device_remove_file(led_cdev->dev, &dev_attr_activate); |
208 | device_remove_file(led_cdev->dev, &dev_attr_duration); | 208 | device_remove_file(led_cdev->dev, &dev_attr_duration); |
209 | device_remove_file(led_cdev->dev, &dev_attr_state); | 209 | device_remove_file(led_cdev->dev, &dev_attr_state); |
diff --git a/include/linux/leds.h b/include/linux/leds.h index 39eee41d8c6f..3aade1d8f410 100644 --- a/include/linux/leds.h +++ b/include/linux/leds.h | |||
@@ -38,6 +38,9 @@ struct led_classdev { | |||
38 | #define LED_SUSPENDED (1 << 0) | 38 | #define LED_SUSPENDED (1 << 0) |
39 | /* Upper 16 bits reflect control information */ | 39 | /* Upper 16 bits reflect control information */ |
40 | #define LED_CORE_SUSPENDRESUME (1 << 16) | 40 | #define LED_CORE_SUSPENDRESUME (1 << 16) |
41 | #define LED_BLINK_ONESHOT (1 << 17) | ||
42 | #define LED_BLINK_ONESHOT_STOP (1 << 18) | ||
43 | #define LED_BLINK_INVERT (1 << 19) | ||
41 | 44 | ||
42 | /* Set LED brightness level */ | 45 | /* Set LED brightness level */ |
43 | /* Must not sleep, use a workqueue if needed */ | 46 | /* Must not sleep, use a workqueue if needed */ |
@@ -103,7 +106,25 @@ extern void led_blink_set(struct led_classdev *led_cdev, | |||
103 | unsigned long *delay_on, | 106 | unsigned long *delay_on, |
104 | unsigned long *delay_off); | 107 | unsigned long *delay_off); |
105 | /** | 108 | /** |
106 | * led_brightness_set - set LED brightness | 109 | * led_blink_set_oneshot - do a oneshot software blink |
110 | * @led_cdev: the LED to start blinking | ||
111 | * @delay_on: the time it should be on (in ms) | ||
112 | * @delay_off: the time it should ble off (in ms) | ||
113 | * @invert: blink off, then on, leaving the led on | ||
114 | * | ||
115 | * This function makes the LED blink one time for delay_on + | ||
116 | * delay_off time, ignoring the request if another one-shot | ||
117 | * blink is already in progress. | ||
118 | * | ||
119 | * If invert is set, led blinks for delay_off first, then for | ||
120 | * delay_on and leave the led on after the on-off cycle. | ||
121 | */ | ||
122 | extern void led_blink_set_oneshot(struct led_classdev *led_cdev, | ||
123 | unsigned long *delay_on, | ||
124 | unsigned long *delay_off, | ||
125 | int invert); | ||
126 | /** | ||
127 | * led_set_brightness - set LED brightness | ||
107 | * @led_cdev: the LED to set | 128 | * @led_cdev: the LED to set |
108 | * @brightness: the brightness to set it to | 129 | * @brightness: the brightness to set it to |
109 | * | 130 | * |
@@ -111,7 +132,7 @@ extern void led_blink_set(struct led_classdev *led_cdev, | |||
111 | * software blink timer that implements blinking when the | 132 | * software blink timer that implements blinking when the |
112 | * hardware doesn't. | 133 | * hardware doesn't. |
113 | */ | 134 | */ |
114 | extern void led_brightness_set(struct led_classdev *led_cdev, | 135 | extern void led_set_brightness(struct led_classdev *led_cdev, |
115 | enum led_brightness brightness); | 136 | enum led_brightness brightness); |
116 | 137 | ||
117 | /* | 138 | /* |
@@ -150,6 +171,10 @@ extern void led_trigger_event(struct led_trigger *trigger, | |||
150 | extern void led_trigger_blink(struct led_trigger *trigger, | 171 | extern void led_trigger_blink(struct led_trigger *trigger, |
151 | unsigned long *delay_on, | 172 | unsigned long *delay_on, |
152 | unsigned long *delay_off); | 173 | unsigned long *delay_off); |
174 | extern void led_trigger_blink_oneshot(struct led_trigger *trigger, | ||
175 | unsigned long *delay_on, | ||
176 | unsigned long *delay_off, | ||
177 | int invert); | ||
153 | 178 | ||
154 | #else | 179 | #else |
155 | 180 | ||
diff --git a/include/linux/platform_data/leds-lm3556.h b/include/linux/platform_data/leds-lm3556.h new file mode 100644 index 000000000000..4b4e7d6b0527 --- /dev/null +++ b/include/linux/platform_data/leds-lm3556.h | |||
@@ -0,0 +1,50 @@ | |||
1 | /* | ||
2 | * Simple driver for Texas Instruments LM3556 LED Flash driver chip (Rev0x03) | ||
3 | * Copyright (C) 2012 Texas Instruments | ||
4 | * | ||
5 | * This program is free software; you can redistribute it and/or modify | ||
6 | * it under the terms of the GNU General Public License version 2 as | ||
7 | * published by the Free Software Foundation. | ||
8 | * | ||
9 | */ | ||
10 | |||
11 | #ifndef __LINUX_LM3556_H | ||
12 | #define __LINUX_LM3556_H | ||
13 | |||
14 | #define LM3556_NAME "leds-lm3556" | ||
15 | |||
16 | enum lm3556_pin_polarity { | ||
17 | PIN_LOW_ACTIVE = 0, | ||
18 | PIN_HIGH_ACTIVE, | ||
19 | }; | ||
20 | |||
21 | enum lm3556_pin_enable { | ||
22 | PIN_DISABLED = 0, | ||
23 | PIN_ENABLED, | ||
24 | }; | ||
25 | |||
26 | enum lm3556_strobe_usuage { | ||
27 | STROBE_EDGE_DETECT = 0, | ||
28 | STROBE_LEVEL_DETECT, | ||
29 | }; | ||
30 | |||
31 | enum lm3556_indic_mode { | ||
32 | INDIC_MODE_INTERNAL = 0, | ||
33 | INDIC_MODE_EXTERNAL, | ||
34 | }; | ||
35 | |||
36 | struct lm3556_platform_data { | ||
37 | enum lm3556_pin_enable torch_pin_en; | ||
38 | enum lm3556_pin_polarity torch_pin_polarity; | ||
39 | |||
40 | enum lm3556_strobe_usuage strobe_usuage; | ||
41 | enum lm3556_pin_enable strobe_pin_en; | ||
42 | enum lm3556_pin_polarity strobe_pin_polarity; | ||
43 | |||
44 | enum lm3556_pin_enable tx_pin_en; | ||
45 | enum lm3556_pin_polarity tx_pin_polarity; | ||
46 | |||
47 | enum lm3556_indic_mode indicator_mode; | ||
48 | }; | ||
49 | |||
50 | #endif /* __LINUX_LM3556_H */ | ||
diff --git a/net/mac80211/led.c b/net/mac80211/led.c index 1bf7903496f8..bcffa6903129 100644 --- a/net/mac80211/led.c +++ b/net/mac80211/led.c | |||
@@ -276,7 +276,7 @@ static void ieee80211_stop_tpt_led_trig(struct ieee80211_local *local) | |||
276 | 276 | ||
277 | read_lock(&tpt_trig->trig.leddev_list_lock); | 277 | read_lock(&tpt_trig->trig.leddev_list_lock); |
278 | list_for_each_entry(led_cdev, &tpt_trig->trig.led_cdevs, trig_list) | 278 | list_for_each_entry(led_cdev, &tpt_trig->trig.led_cdevs, trig_list) |
279 | led_brightness_set(led_cdev, LED_OFF); | 279 | led_set_brightness(led_cdev, LED_OFF); |
280 | read_unlock(&tpt_trig->trig.leddev_list_lock); | 280 | read_unlock(&tpt_trig->trig.leddev_list_lock); |
281 | } | 281 | } |
282 | 282 | ||