Re: [PATCH] rtlwifi: Change long delays to sleeps

2016-02-20 Thread Larry Finger
On 02/19/2016 11:48 PM, ByeoungWook Kim wrote: 2016-02-16 7:12 GMT+09:00 Larry Finger : Routine rtl_addr_delay() uses delay statements in code that can sleep. To improve system responsiveness, the various delay statements are changed. In addition, routines

Re: [PATCH] rtlwifi: Change long delays to sleeps

2016-02-20 Thread ByeoungWook Kim
2016-02-20 23:06 GMT+09:00 Larry Finger : > On 02/19/2016 11:48 PM, ByeoungWook Kim wrote: >> >> 2016-02-16 7:12 GMT+09:00 Larry Finger : >>> >>> Routine rtl_addr_delay() uses delay statements in code that can >>> sleep. To improve system

[PATCH] rtlwifi: pass struct rtl_stats by reference as it is more efficient

2016-02-20 Thread Colin King
From: Colin Ian King passing rtl_stats by value is inefficient; the structure is over 300 bytes in size and generally just one field (packet_report_type) is being accessed, so the pass by value is a relatively large overhead. This change just affects just the

at76c50x-usb: usb_put_dev() if at76_load_internal_fw() succeed

2016-02-20 Thread Alexey Khoroshilov
I do not see any reason to call usb_put_dev() if at76_load_internal_fw() succeed. But it is here from the very beginning. Any suggestions? -- Thank you, Alexey -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majord...@vger.kernel.org More

[PATCH] at76c50x-usb: avoid double usb_put_dev() after downloading internal firmware in at76_probe()

2016-02-20 Thread Alexey Khoroshilov
There is no need in usb_put_dev() if at76_load_internal_fw() succeed. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- drivers/net/wireless/atmel/at76c50x-usb.c | 2 -- 1 file changed, 2 deletions(-) diff --git