staging: rtl8192e: Remove unused if clauses in rtl92e_set_wireless_mode
authorPhilipp Hortmann <philipp.g.hortmann@gmail.com>
Mon, 27 Mar 2023 21:02:51 +0000 (23:02 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 28 Mar 2023 11:34:38 +0000 (13:34 +0200)
Equation (support_mode & WIRELESS_MODE_N_24G) will always be true. Remove
resulting dead code.

Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/359ed203ab08afe5c2f449f40d2958868b4e55b4.1679949171.git.philipp.g.hortmann@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8192e/rtl8192e/rtl_core.c

index c039783..689a5e1 100644 (file)
@@ -598,25 +598,8 @@ void rtl92e_set_wireless_mode(struct net_device *dev, u8 wireless_mode)
        struct r8192_priv *priv = rtllib_priv(dev);
        u8 support_mode = (WIRELESS_MODE_N_24G | WIRELESS_MODE_G | WIRELESS_MODE_B);
 
-       if ((wireless_mode == WIRELESS_MODE_AUTO) ||
-           ((wireless_mode & support_mode) == 0)) {
-               if (support_mode & WIRELESS_MODE_N_24G) {
-                       wireless_mode = WIRELESS_MODE_N_24G;
-               } else if (support_mode & WIRELESS_MODE_N_5G) {
-                       wireless_mode = WIRELESS_MODE_N_5G;
-               } else if ((support_mode & WIRELESS_MODE_A)) {
-                       wireless_mode = WIRELESS_MODE_A;
-               } else if ((support_mode & WIRELESS_MODE_G)) {
-                       wireless_mode = WIRELESS_MODE_G;
-               } else if ((support_mode & WIRELESS_MODE_B)) {
-                       wireless_mode = WIRELESS_MODE_B;
-               } else {
-                       netdev_info(dev,
-                                   "%s(): Unsupported mode requested. Fallback to 802.11b\n",
-                                   __func__);
-                       wireless_mode = WIRELESS_MODE_B;
-               }
-       }
+       if ((wireless_mode == WIRELESS_MODE_AUTO) || ((wireless_mode & support_mode) == 0))
+               wireless_mode = WIRELESS_MODE_N_24G;
 
        if ((wireless_mode & (WIRELESS_MODE_B | WIRELESS_MODE_G)) ==
            (WIRELESS_MODE_G | WIRELESS_MODE_B))