Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1324832

[PATCH v2 1/2] rtlwifi: Fix improve function 'rtl_addr_delay()' in core.c

Show all headers | View raw

Conditional codes in rtl_addr_delay() were improved in readability and
performance by using switch codes.

Signed-off-by: Byeoungwook Kim <quddnr145@gmail.com>
Reviewed-by: Julian Calaby <julian.calaby@gmail.com>
---
V2 split in separate patchs.
 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 4ae421e..05f432c 100644
--- a/drivers/net/wireless/realtek/rtlwifi/core.c
+++ b/drivers/net/wireless/realtek/rtlwifi/core.c
@@ -37,18 +37,26 @@
 
 void rtl_addr_delay(u32 addr)
 {
-	if (addr == 0xfe)
+	switch (addr) {
+	case 0xfe:
 		mdelay(50);
-	else if (addr == 0xfd)
+		break;
+	case 0xfd:
 		mdelay(5);
-	else if (addr == 0xfc)
+		break;
+	case 0xfc:
 		mdelay(1);
-	else if (addr == 0xfb)
+		break;
+	case 0xfb:
 		udelay(50);
-	else if (addr == 0xfa)
+		break;
+	case 0xfa:
 		udelay(5);
-	else if (addr == 0xf9)
+		break;
+	case 0xf9:
 		udelay(1);
+		break;
+	};
 }
 EXPORT_SYMBOL(rtl_addr_delay);
 
-- 
2.5.0

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 1/2] rtlwifi: Fix improve function 'rtl_addr_delay()' in  core.c Byeoungwook Kim <quddnr145@gmail.com> - 2016-02-03 03:20 +0100
  Re: [PATCH v2 1/2] rtlwifi: Fix improve function 'rtl_addr_delay()'  in core.c kbuild test robot <lkp@intel.com> - 2016-02-03 04:40 +0100
  [PATCH] rtlwifi: fix semicolon.cocci warnings kbuild test robot <lkp@intel.com> - 2016-02-03 04:40 +0100
    Re: [PATCH] rtlwifi: fix semicolon.cocci warnings Larry Finger <Larry.Finger@lwfinger.net> - 2016-02-03 07:10 +0100

csiph-web