linux-stable.git/drivers/leds, branch v5.10.7

leds: turris-omnia: check for LED_COLOR_ID_RGB instead LED_COLOR_ID_MULTI

2020-12-30T10:53:22+00:00

[ Upstream commit 98650b0874171cc443251f7b369d3b1544db9d4e ]

LED core does not allow LED_COLOR_ID_MULTI for now and instead for RGB
LEDs prefers LED_COLOR_ID_RGB.

Signed-off-by: Marek Behún 
Fixes: 77dce3a22e89 ("leds: disallow /sys/class/leds/*:multi:* for now")
Signed-off-by: Pavel Machek 
Signed-off-by: Sasha Levin

leds: lp50xx: Fix an error handling path in 'lp50xx_probe_dt()'

2020-12-30T10:53:22+00:00

[ Upstream commit 6d8d014c7dcf85a79da71ef586d06d03d2cae558 ]

In case of memory allocation failure, we must release some resources as
done in all other error handling paths of the function.

'goto child_out' instead of a direct return so that 'fwnode_handle_put()'
is called when we break out of a 'device_for_each_child_node' loop.

Fixes: 242b81170fb8 ("leds: lp50xx: Add the LP50XX family of the RGB LED driver")
Signed-off-by: Christophe JAILLET 
Acked-by: Dan Murphy 
Signed-off-by: Pavel Machek 
Signed-off-by: Sasha Levin

leds: netxbig: add missing put_device() call in netxbig_leds_get_of_pdata()

2020-12-30T10:53:21+00:00

[ Upstream commit 311066aa9ebcd6f1789c829da5039ca02f2dfe46 ]

if of_find_device_by_node() succeed, netxbig_leds_get_of_pdata() doesn't
have a corresponding put_device(). Thus add jump target to fix the
exception handling for this function implementation.

Fixes: 2976b1798909 ("leds: netxbig: add device tree binding")
Signed-off-by: Yu Kuai 
Signed-off-by: Pavel Machek 
Signed-off-by: Sasha Levin

leds: pwm: Remove platform_data support

2020-10-07T10:02:58+00:00

Since commit 141f15c66d94 ("leds: pwm: remove header") that platform
interface is not usable from outside and there seems to be no in tree
user anymore.  All in-tree users of the leds-pwm driver seem to use DT
currently.  Getting rid of the old platform interface allows the
leds-pwm driver to switch over from 'devm_led_classdev_register()' to
'devm_led_classdev_register_ext()'.

Signed-off-by: Alexander Dahl 
Cc: Denis Osterland-Heim 
Reviewed-by: Marek Behún 
Signed-off-by: Pavel Machek

leds: lm3697: Fix out-of-bound access

2020-10-05T18:36:09+00:00

If both LED banks aren't used in device tree, an out-of-bounds
condition in lm3697_init occurs because of the for loop assuming that
all the banks are used.  Fix it by adding a variable that contains the
number of used banks.

Signed-off-by: Gabriel David 
[removed extra rename, minor tweaks]
Signed-off-by: Pavel Machek 
Cc: stable@kernel.org

leds: ns2: do not guard OF match pointer with of_match_ptr

2020-09-30T17:22:58+00:00

Do not match OF match pointer with of_match_ptr, so that even if
CONFIG_OF is disabled, the driver can still be bound via another method.

Move definition of of_ns2_leds_match just before ns2_led_driver
definition, since it is not needed sooner.

Signed-off-by: Marek Behún 
Tested-by: Simon Guinot 
Signed-off-by: Pavel Machek

leds: ns2: convert to fwnode API

2020-09-30T17:22:58+00:00

Convert from OF api to fwnode API, so that it is possible to bind this
driver without device-tree.

The fwnode API does not expose a function to read a specific element of
an array. We therefore change the types of the ns2_led_modval structure
so that we can read the whole modval array with one fwnode call.

Signed-off-by: Marek Behún 
Tested-by: Simon Guinot 
Signed-off-by: Pavel Machek

leds: tlc591xx: fix leak of device node iterator

2020-09-30T17:20:46+00:00

In one of the error paths of the for_each_child_of_node loop in
tlc591xx_probe, add missing call to of_node_put.

Fixes: 1ab4531ad132 ("leds: tlc591xx: simplify driver by using the managed led API")

Signed-off-by: Tobias Jordan 
Reviewed-by: Marek Behún 
Reviewed-by: Tomi Valkeinen 
Signed-off-by: Pavel Machek

leds: pca963x: use struct led_init_data when registering

2020-09-30T17:15:42+00:00

By using struct led_init_data when registering we do not need to parse
`label` DT property. Moreover `label` is deprecated and if it is not
present but `color` and `function` are, LED core will compose a name
from these properties instead.

Previously if the `label` DT property was not present, the code composed
name for the LED in the form
  "pca963x:%d:%.2x:%u"
For backwards compatibility we therefore set init_data->default_label
to this value so that the LED will not get a different name if `label`
property is not present, nor are `color` and `function`.

Signed-off-by: Marek Behún 
Cc: Peter Meerwald 
Cc: Ricardo Ribalda 
Cc: Zahari Petkov 
Signed-off-by: Pavel Machek

leds: pca963x: register LEDs immediately after parsing, get rid of platdata

2020-09-30T17:15:41+00:00

Register LEDs immediately after parsing their properties.
This allows us to get rid of platdata, and since no one in tree uses
header linux/platform_data/leds-pca963x.h, remove it.

Signed-off-by: Marek Behún 
Cc: Peter Meerwald 
Cc: Ricardo Ribalda 
Cc: Zahari Petkov 
Signed-off-by: Pavel Machek