ath9k: unset the default LED pin if used by platform buttons
authorMathias Kresin <dev@kresin.me>
Fri, 3 Nov 2017 06:25:21 +0000 (07:25 +0100)
committerMathias Kresin <dev@kresin.me>
Sat, 4 Nov 2017 08:50:13 +0000 (09:50 +0100)
Unset the default LED gpio pin if the same gpio pin is used by a button
defined via platform button. It prevents the change of the GPIO value
on wireless up/down or wireless traffic.

Fixes: FS#1129
Signed-off-by: Mathias Kresin <dev@kresin.me>
package/kernel/mac80211/patches/549-ath9k_enable_gpio_buttons.patch

index f3fead0a1aef41e05957fda39128b13eaebb9652..e86e3e38bd69eeef66147d805894885e5dde64d1 100644 (file)
@@ -29,7 +29,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
  
  #ifdef CPTCFG_MAC80211_LEDS
  
-@@ -129,6 +131,64 @@ static void ath9k_unregister_gpio_chip(s
+@@ -129,6 +131,67 @@ static void ath9k_unregister_gpio_chip(s
        sc->gpiochip = NULL;
  }
  
@@ -59,6 +59,9 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
 +              return;
 +
 +      for (i = 0; i < pdata->num_btns; i++) {
++              if (pdata->btns[i].gpio == sc->sc_ah->led_pin)
++                              sc->sc_ah->led_pin = -1;
++
 +              ath9k_hw_gpio_request_in(sc->sc_ah, pdata->btns[i].gpio,
 +                                       "ath9k-gpio");
 +              bt[i].gpio = sc->gpiochip->gchip.base + pdata->btns[i].gpio;
@@ -94,7 +97,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
  #else /* CONFIG_GPIOLIB */
  
  static inline void ath9k_register_gpio_chip(struct ath_softc *sc)
-@@ -139,6 +199,14 @@ static inline void ath9k_unregister_gpio
+@@ -139,6 +202,14 @@ static inline void ath9k_unregister_gpio
  {
  }
  
@@ -109,7 +112,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
  #endif /* CONFIG_GPIOLIB */
  
  /********************************/
-@@ -262,6 +330,7 @@ void ath_deinit_leds(struct ath_softc *s
+@@ -262,6 +333,7 @@ void ath_deinit_leds(struct ath_softc *s
  {
        struct ath_led *led;
  
@@ -117,7 +120,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
        while (!list_empty(&sc->leds)) {
                led = list_first_entry(&sc->leds, struct ath_led, list);
  #ifdef CONFIG_GPIOLIB
-@@ -301,6 +370,7 @@ void ath_init_leds(struct ath_softc *sc)
+@@ -301,6 +373,7 @@ void ath_init_leds(struct ath_softc *sc)
        }
  
        ath_fill_led_pin(sc);