Avoid a negative conditional and an extra level of indentation in the
bigger part of the loop body.

Signed-off-by: Jakub Sitnicki <jsitni...@gmail.com>
---
 drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c | 28 +++++++++++-------------
 1 file changed, 13 insertions(+), 15 deletions(-)

diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c 
b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c
index 79efdc2..cfb18db 100644
--- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c
+++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.c
@@ -2010,22 +2010,20 @@ static int rtl8xxxu_read_efuse(struct rtl8xxxu_priv 
*priv)
                }
                for (i = 0; i < EFUSE_MAX_WORD_UNIT; i++) {
                        /* Check word enable condition in the section */
-                       if (!(word_mask & BIT(i))) {
-                               ret = rtl8xxxu_read_efuse8(priv,
-                                                          efuse_addr++,
-                                                          &val8);
-                               if (ret)
-                                       goto exit;
-                               priv->efuse_wifi.raw[map_addr++] = val8;
-
-                               ret = rtl8xxxu_read_efuse8(priv,
-                                                          efuse_addr++,
-                                                          &val8);
-                               if (ret)
-                                       goto exit;
-                               priv->efuse_wifi.raw[map_addr++] = val8;
-                       } else
+                       if (word_mask & BIT(i)) {
                                map_addr += 2;
+                               continue;
+                       }
+
+                       ret = rtl8xxxu_read_efuse8(priv, efuse_addr++, &val8);
+                       if (ret)
+                               goto exit;
+                       priv->efuse_wifi.raw[map_addr++] = val8;
+
+                       ret = rtl8xxxu_read_efuse8(priv, efuse_addr++, &val8);
+                       if (ret)
+                               goto exit;
+                       priv->efuse_wifi.raw[map_addr++] = val8;
                }
        }
 
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to