From: Jérôme Pouiller Date: Tue, 17 Dec 2019 16:15:30 +0000 (+0000) Subject: staging: wfx: drop useless wfx_scan_complete() X-Git-Tag: v5.10.7~3370^2~145 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1a53df55346e1381406a6f4d161aa5acfa99a01c;p=platform%2Fkernel%2Flinux-rpi.git staging: wfx: drop useless wfx_scan_complete() Since wfx_scan_complete() is now only called from wfx_scan_complete_cb(), it make sense to merge the both functions. Signed-off-by: Jérôme Pouiller Link: https://lore.kernel.org/r/20191217161318.31402-47-Jerome.Pouiller@silabs.com Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/wfx/scan.c b/drivers/staging/wfx/scan.c index 397fe51..c043f2f 100644 --- a/drivers/staging/wfx/scan.c +++ b/drivers/staging/wfx/scan.c @@ -228,12 +228,6 @@ fail: schedule_work(&wvif->scan.work); } -static void wfx_scan_complete(struct wfx_vif *wvif) -{ - up(&wvif->scan.lock); - wfx_scan_work(&wvif->scan.work); -} - void wfx_scan_complete_cb(struct wfx_vif *wvif, const struct hif_ind_scan_cmpl *arg) { @@ -257,6 +251,7 @@ void wfx_scan_timeout(struct work_struct *work) wvif->scan.curr = wvif->scan.end; hif_stop_scan(wvif); } - wfx_scan_complete(wvif); + up(&wvif->scan.lock); + wfx_scan_work(&wvif->scan.work); } }