[PATCH v2] ARM: ep93xx: Convert to use descriptors for GPIO LEDs

[PATCH v2] ARM: ep93xx: Convert to use descriptors for GPIO LEDs

[Linus Walleij]

This converts the EP93xx to use GPIO descriptors for the
LEDs.

Cc: Nikita Shubin <nikita.shubin@maquefel.me>
Cc: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Cc: Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Lukasz Majewski <lukma@denx.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
ChangeLog v1->v2:
- Fix syntax error (doubled left parens)
---
 arch/arm/mach-ep93xx/core.c | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-ep93xx/core.c b/arch/arm/mach-ep93xx/core.c
index 2d58e273c96d..318c09ee4723 100644
--- a/arch/arm/mach-ep93xx/core.c
+++ b/arch/arm/mach-ep93xx/core.c
@@ -425,10 +425,8 @@ void __init ep93xx_register_spi(struct ep93xx_spi_info *info,
 static const struct gpio_led ep93xx_led_pins[] __initconst = {
 	{
 		.name	= "platform:grled",
-		.gpio	= EP93XX_GPIO_LINE_GRLED,
 	}, {
 		.name	= "platform:rdled",
-		.gpio	= EP93XX_GPIO_LINE_RDLED,
 	},
 };
 
@@ -437,6 +435,16 @@ static const struct gpio_led_platform_data ep93xx_led_data __initconst = {
 	.leds		= ep93xx_led_pins,
 };
 
+static struct gpiod_lookup_table ep93xx_leds_gpio_table = {
+	.dev_id = "leds-gpio",
+	.table = {
+		/* Use local offsets on gpiochip/port "E" */
+		GPIO_LOOKUP_IDX("E", 0, NULL, 0, GPIO_ACTIVE_HIGH),
+		GPIO_LOOKUP_IDX("E", 1,	NULL, 1, GPIO_ACTIVE_HIGH),
+		{ }
+	},
+};
+
 /*************************************************************************
  * EP93xx pwm peripheral handling
  *************************************************************************/
@@ -989,6 +997,7 @@ struct device __init *ep93xx_init_devices(void)
 	platform_device_register(&ep93xx_ohci_device);
 	platform_device_register(&ep93xx_wdt_device);
 
+	gpiod_add_lookup_table(&ep93xx_leds_gpio_table);
 	gpio_led_register_device(-1, &ep93xx_led_data);
 
 	return parent;
-- 
2.37.2

Re: [PATCH v2] ARM: ep93xx: Convert to use descriptors for GPIO LEDs

[Alexander Sverdlin]

Hello Linus!

On Sat, 2022-09-03 at 00:04 +0200, Linus Walleij wrote:
> This converts the EP93xx to use GPIO descriptors for the
> LEDs.
> 
> Cc: Nikita Shubin <nikita.shubin@maquefel.me>
> Cc: Alexander Sverdlin <alexander.sverdlin@gmail.com>
> Cc: Hartley Sweeten <hsweeten@visionengravers.com>
> Cc: Lukasz Majewski <lukma@denx.de>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

I've tested it on EDB9302-similar custom board, both LEDs
work nicely.

Tested-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>

> ---
> ChangeLog v1->v2:
> - Fix syntax error (doubled left parens)
> ---
>  arch/arm/mach-ep93xx/core.c | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-ep93xx/core.c b/arch/arm/mach-ep93xx/core.c
> index 2d58e273c96d..318c09ee4723 100644
> --- a/arch/arm/mach-ep93xx/core.c
> +++ b/arch/arm/mach-ep93xx/core.c
> @@ -425,10 +425,8 @@ void __init ep93xx_register_spi(struct ep93xx_spi_info *info,
>  static const struct gpio_led ep93xx_led_pins[] __initconst = {
>         {
>                 .name   = "platform:grled",
> -               .gpio   = EP93XX_GPIO_LINE_GRLED,
>         }, {
>                 .name   = "platform:rdled",
> -               .gpio   = EP93XX_GPIO_LINE_RDLED,
>         },
>  };
>  
> @@ -437,6 +435,16 @@ static const struct gpio_led_platform_data ep93xx_led_data __initconst = {
>         .leds           = ep93xx_led_pins,
>  };
>  
> +static struct gpiod_lookup_table ep93xx_leds_gpio_table = {
> +       .dev_id = "leds-gpio",
> +       .table = {
> +               /* Use local offsets on gpiochip/port "E" */
> +               GPIO_LOOKUP_IDX("E", 0, NULL, 0, GPIO_ACTIVE_HIGH),
> +               GPIO_LOOKUP_IDX("E", 1, NULL, 1, GPIO_ACTIVE_HIGH),
> +               { }
> +       },
> +};
> +
>  /*************************************************************************
>   * EP93xx pwm peripheral handling
>   *************************************************************************/
> @@ -989,6 +997,7 @@ struct device __init *ep93xx_init_devices(void)
>         platform_device_register(&ep93xx_ohci_device);
>         platform_device_register(&ep93xx_wdt_device);
>  
> +       gpiod_add_lookup_table(&ep93xx_leds_gpio_table);
>         gpio_led_register_device(-1, &ep93xx_led_data);
>  
>         return parent;

-- 
Alexander Sverdlin.