Re: [PATCH v3] rtlwifi: Fix improve function 'rtl_addr_delay()' in core.c
On Wed, Feb 03, 2016 at 02:21:46PM +0900, Byeoungwook Kim wrote: > Conditional codes in rtl_addr_delay() were improved in readability and > performance by using switch codes. > > Reviewed-by: Julian Calaby> Signed-off-by: Byeoungwook Kim > Signed-off-by: Fengguang Wu How you are using Signed-off-by: of Fengguang Wu? did i missed seeing any mail from Fengguang in your previous versions? regards sudip
Re: [PATCH v3] rtlwifi: Fix improve function 'rtl_addr_delay()' in core.c
Hi Sudip, This patch is exsisted depencency like next. http://marc.info/?l=linux-wireless=145447963305712=2 I wrote review and the followup patch. but i seem to that don't write a Cc document. Sorry for your confused. Regards, Byeoungwook. 2016-02-03 17:56 GMT+09:00 Sudip Mukherjee: > On Wed, Feb 03, 2016 at 02:21:46PM +0900, Byeoungwook Kim wrote: >> Conditional codes in rtl_addr_delay() were improved in readability and >> performance by using switch codes. >> >> Reviewed-by: Julian Calaby >> Signed-off-by: Byeoungwook Kim >> Signed-off-by: Fengguang Wu > > How you are using Signed-off-by: of Fengguang Wu? > > did i missed seeing any mail from Fengguang in your previous versions? > > regards > sudip
[PATCH v3] rtlwifi: Fix improve function 'rtl_addr_delay()' in core.c
Conditional codes in rtl_addr_delay() were improved in readability and performance by using switch codes. Reviewed-by: Julian CalabySigned-off-by: Byeoungwook Kim Signed-off-by: Fengguang Wu --- V3 remove unneeded semicolon. 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..63cda78 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