leds: Let the binding document example for leds-gpio follow the gpio bindings
authorOlliver Schinagl <oliver@schinagl.nl>
Wed, 21 Jan 2015 21:33:47 +0000 (13:33 -0800)
committerBryan Wu <cooloney@gmail.com>
Mon, 2 Mar 2015 19:58:31 +0000 (11:58 -0800)
In the gpio bindings documents it is requested to use the marco's in
include/dt-bindings/gpio/gpio.h whenever possible. The gpios in the led
drivers don't seem to form an exception, so update the example in the
document bindings.

Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
Documentation/devicetree/bindings/leds/leds-gpio.txt

index f77148f..fea1ebf 100644 (file)
@@ -26,16 +26,18 @@ LED sub-node properties:
 
 Examples:
 
+#include <dt-bindings/gpio/gpio.h>
+
 leds {
        compatible = "gpio-leds";
        hdd {
                label = "IDE Activity";
-               gpios = <&mcu_pio 0 1>; /* Active low */
+               gpios = <&mcu_pio 0 GPIO_ACTIVE_LOW>;
                linux,default-trigger = "ide-disk";
        };
 
        fault {
-               gpios = <&mcu_pio 1 0>;
+               gpios = <&mcu_pio 1 GPIO_ACTIVE_HIGH>;
                /* Keep LED on if BIOS detected hardware fault */
                default-state = "keep";
        };
@@ -44,11 +46,11 @@ leds {
 run-control {
        compatible = "gpio-leds";
        red {
-               gpios = <&mpc8572 6 0>;
+               gpios = <&mpc8572 6 GPIO_ACTIVE_HIGH>;
                default-state = "off";
        };
        green {
-               gpios = <&mpc8572 7 0>;
+               gpios = <&mpc8572 7 GPIO_ACTIVE_HIGH>;
                default-state = "on";
        };
 };
@@ -57,7 +59,7 @@ leds {
        compatible = "gpio-leds";
 
        charger-led {
-               gpios = <&gpio1 2 0>;
+               gpios = <&gpio1 2 GPIO_ACTIVE_HIGH>;
                linux,default-trigger = "max8903-charger-charging";
                retain-state-suspended;
        };