wl1251: fix always return 0 error
authorWang Hai <wanghai38@huawei.com>
Thu, 30 Jul 2020 07:39:39 +0000 (15:39 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 21 Aug 2020 09:02:03 +0000 (11:02 +0200)
[ Upstream commit 20e6421344b5bc2f97b8e2db47b6994368417904 ]

wl1251_event_ps_report() should not always return 0 because
wl1251_ps_set_mode() may fail. Change it to return 'ret'.

Fixes: f7ad1eed4d4b ("wl1251: retry power save entry")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wang Hai <wanghai38@huawei.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200730073939.33704-1-wanghai38@huawei.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/wireless/ti/wl1251/event.c

index d0593bc1f1a929449ef0dff7956d2c8681be2e35..daddeaa66bf4a5d0ea72b48dbf65c2312c6ca704 100644 (file)
@@ -84,7 +84,7 @@ static int wl1251_event_ps_report(struct wl1251 *wl,
                break;
        }
 
-       return 0;
+       return ret;
 }
 
 static void wl1251_event_mbox_dump(struct event_mailbox *mbox)