staging: r8188eu: drop another removal/stop check
authorMartin Kaiser <martin@kaiser.cx>
Mon, 7 Nov 2022 20:28:23 +0000 (21:28 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 8 Nov 2022 15:29:55 +0000 (16:29 +0100)
There's no need to check bDriverStopped and bSurpriseRemoved in
issue_probereq_ex.

The code path looks like

issue_probereq_ex
   _issue_probereq
      dump_mgntframe
      or
      dump_mgntframe_and_wait_ack

All paths from dump_mgntframe check the two variables.
dump_mgntframe_and_wait_ack contains a check as well.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20221107202824.61431-3-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/core/rtw_mlme_ext.c

index 17803ac..bfd6afd 100644 (file)
@@ -4496,9 +4496,6 @@ int issue_probereq_ex(struct adapter *padapter, struct ndis_802_11_ssid *pssid,
 
                i++;
 
-               if (padapter->bDriverStopped || padapter->bSurpriseRemoved)
-                       break;
-
                if (i < try_cnt && wait_ms > 0 && ret == _FAIL)
                        msleep(wait_ms);