commit 5caca611a7cb71236e30b8c28d5f3f6a0986b39b
author Wang Hai <wanghai38@huawei.com> Thu Jul 30 15:39:39 2020 +0800
committer Greg Kroah-Hartman <gregkh@linuxfoundation.org> Fri Aug 21 11:02:03 2020 +0200
tree 145510edd7e29bfdd3e70fe8c8aab929f9c1b456
parent 9c3150f0db63c8cd7cc6ea65c036728ff74ec6b4

wl1251: fix always return 0 error

[ 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

diff --git a/drivers/net/wireless/ti/wl1251/event.c b/drivers/net/wireless/ti/wl1251/event.c
index d0593bc..daddeaa 100644
--- a/drivers/net/wireless/ti/wl1251/event.c
+++ b/drivers/net/wireless/ti/wl1251/event.c
@@ -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)