diff options
| author | Felix Fietkau <nbd@nbd.name> | 2018-09-20 10:06:49 +0200 |
|---|---|---|
| committer | Felix Fietkau <nbd@nbd.name> | 2018-09-20 10:09:10 +0200 |
| commit | f506de2cdaf92a5716298cf9d63cc686a1097a29 (patch) | |
| tree | ef709afc2cd0a898c01e896a86c8011c4a6d980a /package/kernel/mac80211/patches/548-ath9k_enable_gpio_chip.patch | |
| parent | 7347ea7453aed1f5e6bf222761e3e5bd6b5d13e7 (diff) | |
| download | upstream-f506de2cdaf92a5716298cf9d63cc686a1097a29.tar.gz upstream-f506de2cdaf92a5716298cf9d63cc686a1097a29.tar.bz2 upstream-f506de2cdaf92a5716298cf9d63cc686a1097a29.zip | |
ath9k: fix unloading the module
Registering a GPIO chip with the ath9k device as parent prevents unload,
because the gpiochip core increases the module use count.
Unfortunately, the only way to avoid this at the moment seems to be to
register the GPIO chip without a parent device
Signed-off-by: Felix Fietkau <nbd@nbd.name>
Diffstat (limited to 'package/kernel/mac80211/patches/548-ath9k_enable_gpio_chip.patch')
| -rw-r--r-- | package/kernel/mac80211/patches/548-ath9k_enable_gpio_chip.patch | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/package/kernel/mac80211/patches/548-ath9k_enable_gpio_chip.patch b/package/kernel/mac80211/patches/548-ath9k_enable_gpio_chip.patch index 31dac29ad7..0b76d330c6 100644 --- a/package/kernel/mac80211/patches/548-ath9k_enable_gpio_chip.patch +++ b/package/kernel/mac80211/patches/548-ath9k_enable_gpio_chip.patch @@ -45,7 +45,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name> #ifdef CPTCFG_ATH9K_DEBUGFS --- a/drivers/net/wireless/ath/ath9k/gpio.c +++ b/drivers/net/wireless/ath/ath9k/gpio.c -@@ -16,13 +16,135 @@ +@@ -16,13 +16,130 @@ #include "ath9k.h" #include <linux/ath9k_platform.h> @@ -126,11 +126,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name> + gc->sc = sc; + snprintf(gc->label, sizeof(gc->label), "ath9k-%s", + wiphy_name(sc->hw->wiphy)); -+#if LINUX_VERSION_CODE >= KERNEL_VERSION(4,5,0) -+ gc->gchip.parent = sc->dev; -+#else -+ gc->gchip.dev = sc->dev; -+#endif ++ + gc->gchip.label = gc->label; + gc->gchip.base = -1; /* determine base automatically */ + gc->gchip.ngpio = ah->caps.num_gpio_pins; @@ -145,7 +141,6 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name> + return; + } + -+ gc->gchip.owner = NULL; + sc->gpiochip = gc; +} + @@ -183,7 +178,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name> static void ath_fill_led_pin(struct ath_softc *sc) { struct ath_hw *ah = sc->sc_ah; -@@ -80,6 +202,12 @@ static int ath_add_led(struct ath_softc +@@ -80,6 +197,12 @@ static int ath_add_led(struct ath_softc else ath9k_hw_set_gpio(sc->sc_ah, gpio->gpio, gpio->active_low); @@ -196,7 +191,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name> return 0; } -@@ -136,17 +264,24 @@ void ath_deinit_leds(struct ath_softc *s +@@ -136,17 +259,24 @@ void ath_deinit_leds(struct ath_softc *s while (!list_empty(&sc->leds)) { led = list_first_entry(&sc->leds, struct ath_led, list); @@ -221,7 +216,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name> char led_name[32]; const char *trigger; int i; -@@ -156,6 +291,15 @@ void ath_init_leds(struct ath_softc *sc) +@@ -156,6 +286,15 @@ void ath_init_leds(struct ath_softc *sc) if (AR_SREV_9100(sc->sc_ah)) return; @@ -237,7 +232,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name> ath_fill_led_pin(sc); if (pdata && pdata->leds && pdata->num_leds) -@@ -180,6 +324,7 @@ void ath_init_leds(struct ath_softc *sc) +@@ -180,6 +319,7 @@ void ath_init_leds(struct ath_softc *sc) ath_create_gpio_led(sc, sc->sc_ah->led_pin, led_name, trigger, !sc->sc_ah->config.led_active_high); } |