Re: [PATCH v2 4/6] dt: bindings: lp8860: Add trigger binding to the lp8860

From: Rob Herring
Date: Thu Dec 07 2017 - 17:46:41 EST


On Tue, Dec 05, 2017 at 02:43:25PM -0600, Dan Murphy wrote:
> Add a default trigger optional node to the child node.
> This will allow the driver to set the trigger for a backlight.
>
> Signed-off-by: Dan Murphy <dmurphy@xxxxxx>
> ---
>
> v2 - Moved binding changes to first patch in the series.
>
> Documentation/devicetree/bindings/leds/leds-lp8860.txt | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/leds/leds-lp8860.txt b/Documentation/devicetree/bindings/leds/leds-lp8860.txt
> index 22b594d95162..3452c9c10499 100644
> --- a/Documentation/devicetree/bindings/leds/leds-lp8860.txt
> +++ b/Documentation/devicetree/bindings/leds/leds-lp8860.txt
> @@ -21,6 +21,8 @@ Required child properties:
>
> Optional child properties:
> - label : see Documentation/devicetree/bindings/leds/common.txt
> + - linux,default-trigger : (optional)

You already said this is optional.

> + see Documentation/devicetree/bindings/leds/common.txt
>
> Example:
>
> @@ -34,6 +36,7 @@ Example:
> display_cluster: display_cluster@0 {
> reg=<0>;
> label = "display_cluster";
> + linux,default-trigger = "backlight";
> };
> }
>
> --
> 2.15.0.124.g7668cbc60
>