diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-06-16 11:01:10 +0200 |
|---|---|---|
| committer | Kalle Valo <kvalo@codeaurora.org> | 2016-06-18 16:36:00 +0300 |
| commit | 2095b1426c9c8088f7be28d70c6d7eb432640baa (patch) | |
| tree | 203f55763ac882ced64df56f5cafc966f10d7fca /drivers/net/wireless/marvell/mwifiex/main.c | |
| parent | 29477269a27daf8859217ced08f853be04e1bac7 (diff) | |
mwifiex: fix link error against sdio
Calling sdio_claim_host() from the interface independent part of
the mwifiex driver is not only a layering violation, but also causes
a link error if MMC support is disabled, or if CONFIG_MMC=m
and CONFIG_MWIFIEX=y:
drivers/net/built-in.o: In function `mwifiex_fw_dpc':
:(.text+0xff138): undefined reference to `sdio_claim_host'
:(.text+0xff158): undefined reference to `sdio_release_host'
The right way to do this is to have the sdio specific code in the
sdio driver front-end, and we already have a callback pointer that
we can use for this after exporting the generic fw download
function from the core driver.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 65c71efe1c59 ("mwifiex: fix racing condition when downloading firmware")
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to 'drivers/net/wireless/marvell/mwifiex/main.c')
| -rw-r--r-- | drivers/net/wireless/marvell/mwifiex/main.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/net/wireless/marvell/mwifiex/main.c b/drivers/net/wireless/marvell/mwifiex/main.c index 2b65334235ca..0e280f879b58 100644 --- a/drivers/net/wireless/marvell/mwifiex/main.c +++ b/drivers/net/wireless/marvell/mwifiex/main.c @@ -21,7 +21,6 @@ #include "wmm.h" #include "cfg80211.h" #include "11n.h" -#include "sdio.h" #define VERSION "1.0" @@ -515,7 +514,6 @@ static void mwifiex_fw_dpc(const struct firmware *firmware, void *context) struct semaphore *sem = adapter->card_sem; bool init_failed = false; struct wireless_dev *wdev; - struct sdio_mmc_card *card = adapter->card; if (!firmware) { mwifiex_dbg(adapter, ERROR, @@ -531,11 +529,7 @@ static void mwifiex_fw_dpc(const struct firmware *firmware, void *context) if (adapter->if_ops.dnld_fw) { ret = adapter->if_ops.dnld_fw(adapter, &fw); } else { - if (adapter->iface_type == MWIFIEX_SDIO) - sdio_claim_host(card->func); ret = mwifiex_dnld_fw(adapter, &fw); - if (adapter->iface_type == MWIFIEX_SDIO) - sdio_release_host(card->func); } if (ret == -1) |