* [PATCH] wl12xx: scan: using pm_runtime_resume_and_get instead of pm_runtime_get_sync
@ 2022-04-20 9:02 cgel.zte
2022-04-23 12:46 ` wl12xx: scan: use pm_runtime_resume_and_get() instead of pm_runtime_get_sync() Kalle Valo
0 siblings, 1 reply; 2+ messages in thread
From: cgel.zte @ 2022-04-20 9:02 UTC (permalink / raw)
To: kvalo
Cc: davem, kuba, linux-wireless, netdev, linux-kernel, Minghao Chi,
Zeal Robot
From: Minghao Chi <chi.minghao@zte.com.cn>
Using pm_runtime_resume_and_get() to replace pm_runtime_get_sync and
pm_runtime_put_noidle. This change is just to simplify the code, no
actual functional changes.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
---
drivers/net/wireless/ti/wlcore/scan.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/wireless/ti/wlcore/scan.c b/drivers/net/wireless/ti/wlcore/scan.c
index 29fa51c37e88..b414305acc32 100644
--- a/drivers/net/wireless/ti/wlcore/scan.c
+++ b/drivers/net/wireless/ti/wlcore/scan.c
@@ -53,11 +53,9 @@ void wl1271_scan_complete_work(struct work_struct *work)
wl->scan.req = NULL;
wl->scan_wlvif = NULL;
- ret = pm_runtime_get_sync(wl->dev);
- if (ret < 0) {
- pm_runtime_put_noidle(wl->dev);
+ ret = pm_runtime_resume_and_get(wl->dev);
+ if (ret < 0)
goto out;
- }
if (test_bit(WLVIF_FLAG_STA_ASSOCIATED, &wlvif->flags)) {
/* restore hardware connection monitoring template */
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: wl12xx: scan: use pm_runtime_resume_and_get() instead of pm_runtime_get_sync()
2022-04-20 9:02 [PATCH] wl12xx: scan: using pm_runtime_resume_and_get instead of pm_runtime_get_sync cgel.zte
@ 2022-04-23 12:46 ` Kalle Valo
0 siblings, 0 replies; 2+ messages in thread
From: Kalle Valo @ 2022-04-23 12:46 UTC (permalink / raw)
To: cgel.zte
Cc: davem, kuba, linux-wireless, netdev, linux-kernel, Minghao Chi,
Zeal Robot
cgel.zte@gmail.com wrote:
> From: Minghao Chi <chi.minghao@zte.com.cn>
>
> Using pm_runtime_resume_and_get() to replace pm_runtime_get_sync and
> pm_runtime_put_noidle. This change is just to simplify the code, no
> actual functional changes.
>
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Patch applied to wireless-next.git, thanks.
c94e36908467 wl12xx: scan: use pm_runtime_resume_and_get() instead of pm_runtime_get_sync()
--
https://patchwork.kernel.org/project/linux-wireless/patch/20220420090247.2588680-1-chi.minghao@zte.com.cn/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-04-23 12:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-20 9:02 [PATCH] wl12xx: scan: using pm_runtime_resume_and_get instead of pm_runtime_get_sync cgel.zte
2022-04-23 12:46 ` wl12xx: scan: use pm_runtime_resume_and_get() instead of pm_runtime_get_sync() Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).