From: Byeoungwook Kim <[email protected]>

Doing it this way improves readability.

Signed-off-by: Byeoungwook Kim <[email protected]>
[Rewrote commit message]
Signed-off-by: Julian Calaby <[email protected]>

---

... or does it.

Honesltly, this is only in this list as Byeoungwook asked about it and I
can see _some_ sort of reason why it exists.

Larry, if you don't like this, please NACK it and we'll never speak of it
again.

Thanks,

Julian Calaby
---
 drivers/net/wireless/realtek/rtlwifi/core.c | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/drivers/net/wireless/realtek/rtlwifi/core.c 
b/drivers/net/wireless/realtek/rtlwifi/core.c
index 0f48048..1cf367f 100644
--- a/drivers/net/wireless/realtek/rtlwifi/core.c
+++ b/drivers/net/wireless/realtek/rtlwifi/core.c
@@ -53,18 +53,26 @@ EXPORT_SYMBOL(channel5g_80m);
 
 void rtl_addr_delay(u32 addr)
 {
-       if (addr == 0xfe)
+       switch (addr) {
+       case 0xfe:
                msleep(50);
-       else if (addr == 0xfd)
+               break;
+       case 0xfd:
                msleep(5);
-       else if (addr == 0xfc)
+               break;
+       case 0xfc:
                msleep(1);
-       else if (addr == 0xfb)
+               break;
+       case 0xfb:
                usleep_range(50, 100);
-       else if (addr == 0xfa)
+               break;
+       case 0xfa:
                usleep_range(5, 10);
-       else if (addr == 0xf9)
+               break;
+       case 0xf9:
                usleep_range(1, 2);
+               break;
+       }
 }
 EXPORT_SYMBOL(rtl_addr_delay);
 
-- 
2.7.0

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

Reply via email to