staging: wfx: drop useless wfx_scan_complete()
authorJérôme Pouiller <jerome.pouiller@silabs.com>
Tue, 17 Dec 2019 16:15:30 +0000 (16:15 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 18 Dec 2019 14:56:27 +0000 (15:56 +0100)
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 <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20191217161318.31402-47-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wfx/scan.c

index 397fe51..c043f2f 100644 (file)
@@ -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);
        }
 }