mwifiex: Remove unused 'pm_flag' variable
authorKirtika Ruchandani <kirtika.ruchandani@gmail.com>
Thu, 24 Nov 2016 01:26:06 +0000 (17:26 -0800)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 29 Nov 2016 15:27:49 +0000 (17:27 +0200)
mwifiex_sdio_resume() intializes pm_flag, just like
mwifiex_sdio_suspend(), but does not use it. Compiling with W=1 gives
the following warning, fix it.
mwifiex/sdio.c: In function ‘mwifiex_sdio_resume’:
mwifiex/sdio.c:234:16: warning: variable ‘pm_flag’ set but not used [-Wunused-but-set-variable]

sdio_get_host_pm_caps() is just an acessor, so the call to it is safe
to remove. The unused variable seems to be present since
5e6e3a92b9a4 which introduced mwifiex_sdio_resume().

Fixes: 5e6e3a92b9a4 ("wireless: mwifiex: initial commit for Marvell mwifiex driver")
Cc: Bing Zhao <bzhao@marvell.com>
Cc: Amitkumar Karwar <akarwar@marvell.com>
Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/marvell/mwifiex/sdio.c

index 1235f04..8b33a57 100644 (file)
@@ -186,9 +186,7 @@ static int mwifiex_sdio_resume(struct device *dev)
        struct sdio_func *func = dev_to_sdio_func(dev);
        struct sdio_mmc_card *card;
        struct mwifiex_adapter *adapter;
-       mmc_pm_flag_t pm_flag = 0;
 
-       pm_flag = sdio_get_host_pm_caps(func);
        card = sdio_get_drvdata(func);
        if (!card || !card->adapter) {
                dev_err(dev, "resume: invalid card or adapter\n");