wl1251: fix always return 0 error

Wang Hai · Kalle Valo · commit 20e6421344b5 · 2020-08-02T18:25:14.000+03:00
wl1251_event_ps_report() should not always return 0 because wl1251_ps_set_mode() may fail. Change it to return 'ret'. Fixes: f7ad1ee ("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
diff --git a/drivers/net/wireless/ti/wl1251/event.c b/drivers/net/wireless/ti/wl1251/event.c
@@ -70,7 +70,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)

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ static int wl1251_event_ps_report(struct wl1251 *wl,`
`70`	`70`	`break;`
`71`	`71`	`}`
`72`	`72`
`73`		`- return 0;`
	`73`	`+ return ret;`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`static void wl1251_event_mbox_dump(struct event_mailbox *mbox)`