ar71xx: make leds-gpio usable with single-ended GPIOs
authorGabor Juhos <juhosg@freemail.hu>
Thu, 18 Jan 2018 12:50:31 +0000 (13:50 +0100)
committerJohn Crispin <john@phrozen.org>
Mon, 22 Jan 2018 09:53:32 +0000 (10:53 +0100)
Add patches for the leds-gpio driver to make it usable with
open-drain and open-source kind of GPIO lines.

This type of functionality is required by various MikroTik boards.

Signed-off-by: Gabor Juhos <juhosg@freemail.hu>
target/linux/ar71xx/patches-4.4/442-leds-gpio-allow-to-use-OPEN_-DRAIN-SOURCE-flags-with.patch [new file with mode: 0644]
target/linux/ar71xx/patches-4.9/442-leds-gpio-allow-to-use-OPEN_-DRAIN-SOURCE-flags-with.patch [new file with mode: 0644]

diff --git a/target/linux/ar71xx/patches-4.4/442-leds-gpio-allow-to-use-OPEN_-DRAIN-SOURCE-flags-with.patch b/target/linux/ar71xx/patches-4.4/442-leds-gpio-allow-to-use-OPEN_-DRAIN-SOURCE-flags-with.patch
new file mode 100644 (file)
index 0000000..c17c879
--- /dev/null
@@ -0,0 +1,45 @@
+From 183148e0789bee1cd5c46ba49afcb211f636f8a2 Mon Sep 17 00:00:00 2001
+From: Gabor Juhos <juhosg@freemail.hu>
+Date: Mon, 15 Jan 2018 15:01:14 +0100
+Subject: [PATCH] leds: gpio: allow to use OPEN_{DRAIN,SOURCE} flags with
+ legacy GPIOs
+
+LEDs which are connected to open-source or open-drain type of GPIO lines
+can be used only, if those are defined via devicetree.
+Add two new fields to 'struct gpio_led' in order to make it possible to
+specify this type of GPIO lines to the leds-gpio driver via platform data.
+Also update the create_gpio_led() function to set the GPIOF_OPEN_DRAIN and
+GPIOF_OPEN_SOURCE flags for the given GPIO line.
+
+Signed-off-by: Gabor Juhos <juhosg@freemail.hu>
+---
+ drivers/leds/leds-gpio.c | 6 ++++++
+ include/linux/leds.h     | 2 ++
+ 2 files changed, 8 insertions(+)
+
+--- a/drivers/leds/leds-gpio.c
++++ b/drivers/leds/leds-gpio.c
+@@ -110,6 +110,12 @@ static int create_gpio_led(const struct
+               if (template->active_low)
+                       flags |= GPIOF_ACTIVE_LOW;
++              if (template->open_drain)
++                      flags |= GPIOF_OPEN_DRAIN;
++
++              if (template->open_source)
++                      flags |= GPIOF_OPEN_SOURCE;
++
+               ret = devm_gpio_request_one(parent, template->gpio, flags,
+                                           template->name);
+               if (ret < 0)
+--- a/include/linux/leds.h
++++ b/include/linux/leds.h
+@@ -350,6 +350,8 @@ struct gpio_led {
+       unsigned        retain_state_suspended : 1;
+       unsigned        default_state : 2;
+       /* default_state should be one of LEDS_GPIO_DEFSTATE_(ON|OFF|KEEP) */
++      unsigned        open_drain : 1;
++      unsigned        open_source : 1;
+       struct gpio_desc *gpiod;
+ };
+ #define LEDS_GPIO_DEFSTATE_OFF                0
diff --git a/target/linux/ar71xx/patches-4.9/442-leds-gpio-allow-to-use-OPEN_-DRAIN-SOURCE-flags-with.patch b/target/linux/ar71xx/patches-4.9/442-leds-gpio-allow-to-use-OPEN_-DRAIN-SOURCE-flags-with.patch
new file mode 100644 (file)
index 0000000..26c8cb5
--- /dev/null
@@ -0,0 +1,45 @@
+From 183148e0789bee1cd5c46ba49afcb211f636f8a2 Mon Sep 17 00:00:00 2001
+From: Gabor Juhos <juhosg@freemail.hu>
+Date: Mon, 15 Jan 2018 15:01:14 +0100
+Subject: [PATCH] leds: gpio: allow to use OPEN_{DRAIN,SOURCE} flags with
+ legacy GPIOs
+
+LEDs which are connected to open-source or open-drain type of GPIO lines
+can be used only, if those are defined via devicetree.
+Add two new fields to 'struct gpio_led' in order to make it possible to
+specify this type of GPIO lines to the leds-gpio driver via platform data.
+Also update the create_gpio_led() function to set the GPIOF_OPEN_DRAIN and
+GPIOF_OPEN_SOURCE flags for the given GPIO line.
+
+Signed-off-by: Gabor Juhos <juhosg@freemail.hu>
+---
+ drivers/leds/leds-gpio.c | 6 ++++++
+ include/linux/leds.h     | 2 ++
+ 2 files changed, 8 insertions(+)
+
+--- a/drivers/leds/leds-gpio.c
++++ b/drivers/leds/leds-gpio.c
+@@ -100,6 +100,12 @@ static int create_gpio_led(const struct
+               if (template->active_low)
+                       flags |= GPIOF_ACTIVE_LOW;
++              if (template->open_drain)
++                      flags |= GPIOF_OPEN_DRAIN;
++
++              if (template->open_source)
++                      flags |= GPIOF_OPEN_SOURCE;
++
+               ret = devm_gpio_request_one(parent, template->gpio, flags,
+                                           template->name);
+               if (ret < 0)
+--- a/include/linux/leds.h
++++ b/include/linux/leds.h
+@@ -380,6 +380,8 @@ struct gpio_led {
+       unsigned        panic_indicator : 1;
+       unsigned        default_state : 2;
+       /* default_state should be one of LEDS_GPIO_DEFSTATE_(ON|OFF|KEEP) */
++      unsigned        open_drain : 1;
++      unsigned        open_source : 1;
+       struct gpio_desc *gpiod;
+ };
+ #define LEDS_GPIO_DEFSTATE_OFF                0