leds: leds-pwm: add DT support for LEDs wired to supply (b0571e7e) · Commits · 戴 / test

The non-DT driver allowed an active low property to be specified, but DT is missing this in its description. Add the property to the DT binding document, making it optional. It defaults to active high, which retains compatibility with existing descriptions. This should only be used for causes where the LED is wired to supply, and the PWM does not sensibly support its own inversion. Signed-off-by:

Russell King <rmk+kernel@arm.linux.org.uk> Signed-off-by:

Bryan Wu <cooloney@gmail.com>

Documentation/devicetree/bindings/leds/leds-pwm.txt

+2 −0

Original line number	Diff line number	Diff line
		@@ -13,6 +13,8 @@ LED sub-node properties:
		For the pwms and pwm-names property please refer to:
		Documentation/devicetree/bindings/pwm/pwm.txt
		- max-brightness : Maximum brightness possible for the LED
		- active-low : (optional) For PWMs where the LED is wired to supply
		rather than ground.
		- label : (optional)
		see Documentation/devicetree/bindings/leds/common.txt
		- linux,default-trigger : (optional)

drivers/leds/leds-pwm.c

+1 −0

Original line number	Diff line number	Diff line
		@@ -154,6 +154,7 @@ static int led_pwm_create_of(struct device dev, struct led_pwm_priv priv)

		led.default_trigger = of_get_property(child,
		"linux,default-trigger", NULL);
		led.active_low = of_property_read_bool(child, "active-low");
		of_property_read_u32(child, "max-brightness",
		&led.max_brightness);

Admin message