Some triggers create sysfs files when they are enabled. Send a uevent
"change" notification whenever the trigger is changed to allow userspace
processes such as udev to modify permissions on the new files.

A change notification will also be sent during registration of led class
devices or led triggers if the default trigger of an led class device
is found.

(bryan.wu@canonical.com: rename trigger to trig to fix the build error)
Signed-off-by: NColin Cross <ccross@android.com>
Acked-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Acked-by: NHenrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

(a) use LP5523_ENABLE rather than magic number 0x40
(b) use min_t() in lp5523_mux_parse()
(c) skip while loop and just return if invalid command
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

The return value of this function is not handled any place, so
make it as void type.

And three if-statements are replaced with switch-statements.
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Turning off the brightness of each channel is required
when removing the driver.

So use flush_work() rather than cancel_work_sync() to execute
remaining brightness works.
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

The name of each led channel is configurable.
If the name is NULL, just use the channel id for making the channel name
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Use of_get_child_count() instead of custom implementation.
Signed-off-by: NTobias Klauser <klto@zhaw.ch>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Use the wrapper functions, so we can directly pass a struct
platfrom_device.
Signed-off-by: NTobias Klauser <tklauser@distanz.ch>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Instead of having to define the match table to NULL if CONFIG_OF isn't
set, use the of_match_ptr() macro which will do this for us.
Signed-off-by: NTobias Klauser <tklauser@distanz.ch>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Delay execution of led_set_brightness() if need to stop soft-blink
timer.

This allows led_set_brightness to be called in hard-irq context even if
soft-blink was activated on that LED.
Signed-off-by: NFabio Baltieri <fabio.baltieri@gmail.com>
Cc: Pawel Moll <pawel.moll@arm.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Signed-off-by: NAxel Lin <axel.lin@gmail.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

There are more variants of Clevo M5x0V laptops and Positivo Mobile is just one of them.
Another one is UMAX VisionBook 2400WXC. Use only generic Clevo DMI strings to match all
variants.
Signed-off-by: NOndrej Zary <linux@rainbow-software.org>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Enable autoloading of leds-clevo-mail by using MODULE_DEVICE_TABLE.
Signed-off-by: NOndrej Zary <linux@rainbow-software.org>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

bfe4c041 "leds: convert Renesas TPU LED driver to devm_kzalloc() and
cleanup error exit path" introduced a possible case in which r_tpu_probe
calls iounmap on a wild pointer. This changes the one case that was
missed in the same way as the other error paths.

Without this patch, building kota2_defconfig results in:

drivers/leds/leds-renesas-tpu.c: In function 'r_tpu_probe':
drivers/leds/leds-renesas-tpu.c:246:6: warning: 'ret' may be used uninitialized in this function [-Wuninitialized]
drivers/leds/leds-renesas-tpu.c:308:17: warning: 'p' may be used uninitialized in this function [-Wuninitialized]
Signed-off-by: NArnd Bergmann <arnd@arndb.de>
Cc: Magnus Damm <damm@opensource.se>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

This reverts commit a0193cbe.

The problem with the original commit was that it caused a warning with
the MMC trigger calling del_timer_sync from hard-irq context.
Signed-off-by: NFabio Baltieri <fabio.baltieri@gmail.com>
Reported-by: NPawel Moll <pawel.moll@arm.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

We need to do left shift (cfg->num + LP8788_ISINK_SCALE_OFFSET) bits for
updating scale configuration.
Signed-off-by: NAxel Lin <axel.lin@gmail.com>
Acked-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Tested-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

at default case of switch we return with out unlocking the mutex.
Signed-off-by: NDevendra Naga <develkernel412222@gmail.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Use the error codes from the lower levels instead of -ENODEV.
Signed-off-by: NDan Carpenter <dan.carpenter@oracle.com>
Acked-by: NJan-Simon Möller <jansimon.moeller@gmx.de>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

This can return without freeing the bl_work struct.  Also there was no
check for allocation failure.
Signed-off-by: NDan Carpenter <dan.carpenter@oracle.com>
Acked-by: NJan-Simon Möller <jansimon.moeller@gmx.de>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

TI LP8788 PMU has the current sink as the keyboard led driver.
The brightness is controlled by the i2c commands.
Configurable parameters can be defined in the platform side.

Patch v2.
(a) use workqueue on changing the brightness

(b) use mutex_lock/unlock when the brightness is set
    and the led block of lp8788 device is enabled

(c) remove err_dev on _probe()
    : just return as returned value if any errors

(d) replace module_init/exit() with module_platform_driver()

(e) add led configuration structure and loading them by default
    if platform data is null
Signed-off-by: NMilo(Woogyom) Kim <milo.kim@ti.com>
Acked-by: NMark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Add driver for BlinkM device to drivers/leds/.
Add entry to MAINTAINERS file.
Add documentation in Documentation/leds/.

A BlinkM is a RGB LED controlled through I2C.

This driver implements an interface to the LED framework
and another sysfs group to access the internal options
of the BlinkM.

rev6: Use module_i2c_driver().
rev5: Removed own workqueue in favor of events wq.
rev4: Fixed comments by Bryan Wu.
rev3: Fixed issues found by Jonathan Neuschäfer.

(bryan.wu@canonical.com: remove 2 trailing whitespace)
Signed-off-by: NJan-Simon Möller <jansimon.moeller@gmx.de>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

It is not necessary to call max8997_update_reg() twice just for updating
MAX8997_REG_LEN_CNTL register. With proper val and mask arguments to
max8997_update_reg() call, this can be done in one max8997_update_reg() call.
And then we can remove max8997_led_clear_mode() function.
Signed-off-by: NAxel Lin <axel.lin@gmail.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

why?
because if the led_classdev_register fails we wont do gpio free
because using devm_ functions, there is no need for error paths
Signed-off-by: NDevendra Naga <devendra.aaru@gmail.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Simon Guinot <sguinot@lacie.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Guennadi Liakhovetski <lg@denx.de>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Shreshtha Kumar SAHU <shreshthakumar.sahu@stericsson.com>
Acked-by: NLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Neil Brown <neilb@suse.de>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Philippe Retornaz <philippe.retornaz@epfl.ch>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Michael Hennerich <hennerich@blackfin.uclinux.org>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Nate Case <ncase@xes-inc.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Riku Voipio <riku.voipio@iki.fi>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Daniel Mack <daniel@caiaq.de>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Magnus Damm <damm@opensource.se>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Mathias Nyman <mathias.nyman@nokia.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Peter Meerwald <p.meerwald@bct-electronic.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Kim Kyuwon <q1.kim@samsung.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Eric Miao <eric.miao@canonical.com>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Cc: Antonio Ospite <ospite@studenti.unina.it>
Signed-off-by: NBryan Wu <bryan.wu@canonical.com>

Signed-off-by: NBryan Wu <bryan.wu@canonical.com>