[PATCH v3] ARM: mx5/mx53_evk.c: Add LED support

[PATCH v3] ARM: mx5/mx53_evk.c: Add LED support

[Fabio Estevam]

On the mx53evk board there is an LED connected to GPIO7_7.

Add support for it.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/mach-mx5/board-mx53_evk.c |   19 +++++++++++++++++++
 1 files changed, 19 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-mx5/board-mx53_evk.c b/arch/arm/mach-mx5/board-mx53_evk.c
index 2af3f43..3e4e40b 100644
--- a/arch/arm/mach-mx5/board-mx53_evk.c
+++ b/arch/arm/mach-mx5/board-mx53_evk.c
@@ -37,6 +37,7 @@
 #define MX53_EVK_FEC_PHY_RST	IMX_GPIO_NR(7, 6)
 #define EVK_ECSPI1_CS0		IMX_GPIO_NR(2, 30)
 #define EVK_ECSPI1_CS1		IMX_GPIO_NR(3, 19)
+#define MX53EVK_LED		IMX_GPIO_NR(7, 7)
 
 #include "crm_regs.h"
 #include "devices-imx53.h"
@@ -60,6 +61,8 @@ static iomux_v3_cfg_t mx53_evk_pads[] = {
 	/* ecspi chip select lines */
 	MX53_PAD_EIM_EB2__GPIO2_30,
 	MX53_PAD_EIM_D19__GPIO3_19,
+	/* LED */
+	MX53_PAD_PATA_DA_1__GPIO7_7,
 };
 
 static const struct imxuart_platform_data mx53_evk_uart_pdata __initconst = {
@@ -117,6 +120,20 @@ static const struct spi_imx_master mx53_evk_spi_data __initconst = {
 	.num_chipselect = ARRAY_SIZE(mx53_evk_spi_cs),
 };
 
+static struct gpio_led mx53_evk_leds[] = {
+	{
+		.name			= "led-green",
+		.default_trigger	= "heartbeat",
+		.active_low		= 0,
+		.gpio			= MX53EVK_LED,
+	},
+};
+
+static struct gpio_led_platform_data mx53_evk_leds_data = {
+	.leds = mx53_evk_leds,
+	.num_leds = ARRAY_SIZE(mx53_evk_leds),
+};
+
 static void __init mx53_evk_board_init(void)
 {
 	mxc_iomux_v3_setup_multiple_pads(mx53_evk_pads,
@@ -135,6 +152,8 @@ static void __init mx53_evk_board_init(void)
 		ARRAY_SIZE(mx53_evk_spi_board_info));
 	imx53_add_ecspi(0, &mx53_evk_spi_data);
 	imx53_add_imx2_wdt(0, NULL);
+	platform_device_register_resndata(NULL, "leds-gpio", -1,
+		NULL, 0, &mx53_evk_leds_data, sizeof(mx53_evk_leds_data));
 }
 
 static void __init mx53_evk_timer_init(void)
-- 
1.6.0.4

[PATCH v3] ARM: mx5/mx53_evk.c: Add LED support

[Sascha Hauer]

On Mon, Apr 04, 2011 at 02:49:51PM -0300, Fabio Estevam wrote:
> On the mx53evk board there is an LED connected to GPIO7_7.

Can we delay this until the generic helper function is merged?

Sascha

> 
> Add support for it.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/mach-mx5/board-mx53_evk.c |   19 +++++++++++++++++++
>  1 files changed, 19 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/mach-mx5/board-mx53_evk.c b/arch/arm/mach-mx5/board-mx53_evk.c
> index 2af3f43..3e4e40b 100644
> --- a/arch/arm/mach-mx5/board-mx53_evk.c
> +++ b/arch/arm/mach-mx5/board-mx53_evk.c
> @@ -37,6 +37,7 @@
>  #define MX53_EVK_FEC_PHY_RST	IMX_GPIO_NR(7, 6)
>  #define EVK_ECSPI1_CS0		IMX_GPIO_NR(2, 30)
>  #define EVK_ECSPI1_CS1		IMX_GPIO_NR(3, 19)
> +#define MX53EVK_LED		IMX_GPIO_NR(7, 7)
>  
>  #include "crm_regs.h"
>  #include "devices-imx53.h"
> @@ -60,6 +61,8 @@ static iomux_v3_cfg_t mx53_evk_pads[] = {
>  	/* ecspi chip select lines */
>  	MX53_PAD_EIM_EB2__GPIO2_30,
>  	MX53_PAD_EIM_D19__GPIO3_19,
> +	/* LED */
> +	MX53_PAD_PATA_DA_1__GPIO7_7,
>  };
>  
>  static const struct imxuart_platform_data mx53_evk_uart_pdata __initconst = {
> @@ -117,6 +120,20 @@ static const struct spi_imx_master mx53_evk_spi_data __initconst = {
>  	.num_chipselect = ARRAY_SIZE(mx53_evk_spi_cs),
>  };
>  
> +static struct gpio_led mx53_evk_leds[] = {
> +	{
> +		.name			= "led-green",
> +		.default_trigger	= "heartbeat",
> +		.active_low		= 0,
> +		.gpio			= MX53EVK_LED,
> +	},
> +};
> +
> +static struct gpio_led_platform_data mx53_evk_leds_data = {
> +	.leds = mx53_evk_leds,
> +	.num_leds = ARRAY_SIZE(mx53_evk_leds),
> +};
> +
>  static void __init mx53_evk_board_init(void)
>  {
>  	mxc_iomux_v3_setup_multiple_pads(mx53_evk_pads,
> @@ -135,6 +152,8 @@ static void __init mx53_evk_board_init(void)
>  		ARRAY_SIZE(mx53_evk_spi_board_info));
>  	imx53_add_ecspi(0, &mx53_evk_spi_data);
>  	imx53_add_imx2_wdt(0, NULL);
> +	platform_device_register_resndata(NULL, "leds-gpio", -1,
> +		NULL, 0, &mx53_evk_leds_data, sizeof(mx53_evk_leds_data));
>  }
>  
>  static void __init mx53_evk_timer_init(void)
> -- 
> 1.6.0.4
> 
> 
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

[PATCH v3] ARM: mx5/mx53_evk.c: Add LED support

[Fabio Estevam]

On 4/6/2011 3:39 AM, Sascha Hauer wrote:
> On Mon, Apr 04, 2011 at 02:49:51PM -0300, Fabio Estevam wrote:
>> On the mx53evk board there is an LED connected to GPIO7_7.
> 
> Can we delay this until the generic helper function is merged?

Sure. I will post v4 after Uwe?s gpio-leds generic helper function reaches mainline.

Regards,

Fabio Estevam