From mboxrd@z Thu Jan 1 00:00:00 1970 From: dmurphy@ti.com (Dan Murphy) Date: Wed, 5 Jun 2013 12:04:15 -0500 Subject: [PATCH 3/4] ARM: dts: omap5-uevm: Add LED support for uEVM blue LED In-Reply-To: <1370414770-1485-4-git-send-email-r.sricharan@ti.com> References: <1370414770-1485-1-git-send-email-r.sricharan@ti.com> <1370414770-1485-4-git-send-email-r.sricharan@ti.com> Message-ID: <51AF6F8F.4000702@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Sricharan Thanks for sending this up in the series. On 06/05/2013 01:46 AM, Sricharan R wrote: > From: Dan Murphy > > Add support for blue LED 1 off of GPIO 153. > Make the LED a heartbeat LED > Configure the MUX for GPIO output. > > Cc: Dan Murphy > Signed-off-by: Dan Murphy > [Sricharan R : Replaced constants with preprocessor macros] > Signed-off-by: Sricharan R > --- > arch/arm/boot/dts/omap5-uevm.dts | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts > index cf862df..a2aed5b 100644 > --- a/arch/arm/boot/dts/omap5-uevm.dts > +++ b/arch/arm/boot/dts/omap5-uevm.dts > @@ -66,6 +66,16 @@ > device = <&hsusb2_phy>; > clock-frequency = <19200000>; /* 19.2 MHz */ > }; > + > + leds { > + compatible = "gpio-leds"; > + led at 1 { > + label = "omap5:blue:usr1"; > + gpios = <&gpio5 25 GPIO_ACTIVE_HIGH>; /* gpio5_153 D1 LED */ > + linux,default-trigger = "heartbeat"; > + default-state = "off"; > + }; > + }; > }; > > &omap5_pmx_core { > @@ -77,6 +87,7 @@ > &mcbsp1_pins > &mcbsp2_pins > &usbhost_pins > + &led_gpio_pins > >; > > twl6040_pins: pinmux_twl6040_pins { > @@ -175,6 +186,12 @@ > 0x6e (PIN_OUTPUT | MUX_MODE6) /* gpio3_79 OUTPUT | MODE 6 ETH_NRESET */ > >; > }; > + > + led_gpio_pins: pinmux_led_gpio_pins { > + pinctrl-single,pins = < > + 0x196 (PIN_OUTPUT | MUX_MODE6) /* uart3_cts_rctx.gpio5_153 OUTPUT | MODE6 */ Same comment from Florian. We can drop the OUTPUT | MODE6 in the comment. > + >; > + }; > }; > > &omap5_pmx_wkup { -- ------------------ Dan Murphy