On Tue, Feb 04, 2014 at 09:57:42PM +0800, Liu Ying wrote:
> The D25 LED controlled by gpio on the i.MX51 babbage
> board is a diagnostic LED according to the board design.
> This patch adds the relevant device tree nodes to the
> i.MX51 babbage device tree file to support this LED.
> 
> Signed-off-by: Liu Ying <[email protected]>
> ---
>  arch/arm/boot/dts/imx51-babbage.dts |   17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx51-babbage.dts 
> b/arch/arm/boot/dts/imx51-babbage.dts
> index be1407c..8d6a74b 100644
> --- a/arch/arm/boot/dts/imx51-babbage.dts
> +++ b/arch/arm/boot/dts/imx51-babbage.dts
> @@ -81,6 +81,17 @@
>               };
>       };
>  
> +     leds {
> +             compatible = "gpio-leds";
> +             pinctrl-names = "default";
> +             pinctrl-0 = <&led_pin_gpio2_6>;
> +
> +             led-diagnostic {
> +                     label = "diagnostic";
> +                     gpios = <&gpio2 6 0>;

Just out of curiosity, how will you use/trigger the led?

> +             };
> +     };
> +
>       sound {
>               compatible = "fsl,imx51-babbage-sgtl5000",
>                            "fsl,imx-audio-sgtl5000";
> @@ -280,6 +291,12 @@
>                               MX51_PAD_CSPI1_RDY__GPIO4_26 0x80000000
>                       >;
>               };
> +
> +             led_pin_gpio2_6: led_gpio2_6 {

This might be copied from some existing file, but I would hope the name
can be more generic, something like the following

        pinctrl_gpio_leds: gpioledsgrp {

, so that when we have more gpio controlled leds to add, we can just add
more pins into the same group without concerning the name.

Shawn

> +                     fsl,pins = <
> +                             MX51_PAD_EIM_D22__GPIO2_6 0x80000000
> +                     >;
> +             };
>       };
>  };
>  
> -- 
> 1.7.9.5
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to