Re: [PATCH] ath9k_htc: check seq number instead of cmd id for timeout

2015-04-05 Thread Fred Chou
Hi all, May I have an update on the patch status please? Thanks and regards, Fred On 13/3/2015 4:32 PM, Fred Chou wrote: > From: Fred Chou > > As the driver may send multiple wmi commands with identical cmd id, > it is more robust to check seq number for timeout instead. >

Re: [PATCH] ath9k_htc: check seq number instead of cmd id for timeout

2015-03-15 Thread Fred Chou
On 15/03/2015 14:48, Oleksij Rempel wrote: > thank you for your patch. Looks good for me. Did you tested it? Yes. Working all right for two days. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

[PATCH] ath9k_htc: check seq number instead of cmd id for timeout

2015-03-13 Thread Fred Chou
From: Fred Chou As the driver may send multiple wmi commands with identical cmd id, it is more robust to check seq number for timeout instead. Signed-off-by: Fred Chou --- drivers/net/wireless/ath/ath9k/wmi.c | 12 ++-- drivers/net/wireless/ath/ath9k/wmi.h | 2 +- 2 files changed, 7

Re: [PATCH] ath9k_htc: avoid memcpy when downloading firmware

2015-03-03 Thread Fred Chou
On 3/3/2015 2:00 PM, Eric Dumazet wrote: > On Tue, 2015-03-03 at 12:24 +0800, Fred Chou wrote: >> From: Fred Chou >> >> The temporary buffer to hold firmware data is not really needed, >> and memcpy can be avoided by using data pointer instead. >> >> Signe

[PATCH] ath9k_htc: avoid memcpy when downloading firmware

2015-03-02 Thread Fred Chou
From: Fred Chou The temporary buffer to hold firmware data is not really needed, and memcpy can be avoided by using data pointer instead. Signed-off-by: Fred Chou --- drivers/net/wireless/ath/ath9k/hif_usb.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a

[PATCH] net: wireless: rt2x00: use helper to check capability/requirement

2014-12-26 Thread Fred Chou
From: Fred Chou Use rt2x00_has_cap_flag macro to check rt2x00dev->cap_flags. Signed-off-by: Fred Chou --- drivers/net/wireless/rt2x00/rt2x00config.c | 4 ++-- drivers/net/wireless/rt2x00/rt2x00dev.c | 18 +- drivers/net/wireless/rt2x00/rt2x00firmware.c |

[PATCH] fs: fat: use MSDOS_SB macro to get msdos_sb_info

2014-12-25 Thread Fred Chou
From: Fred Chou Use the MSDOS_SB macro to get msdos_sb_info, instead of coding it directly. Signed-off-by: Fred Chou --- fs/fat/inode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/fat/inode.c b/fs/fat/inode.c index 1956dae..0f52d5f 100644 --- a/fs/fat/inode.c +++ b

[PATCH] staging: vt6655: remove unused function in mac.c

2014-12-09 Thread Fred Chou
From: Fred Chou Removed the unused function MACvGetShortRetryLimit, which also fixed the following sparse warning: drivers/staging/vt6655/mac.c:162:6: warning: symbol 'MACvGetShortRetryLimit' was not declared. Should it be static? Signed-off-by: Fred Chou --- drivers/staging/vt

[PATCH] staging:lustre:lustre:ptlrpc: fix sparse warnings in lproc_ptlrpc.c

2014-12-08 Thread Fred Chou
From: Fred Chou Fixed the below warning in sparse: drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c:184:6: warning: symbol 'ptlrpc_lprocfs_register' was not declared. Should it be static? Signed-off-by: Fred Chou --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 7

Re: [PATCH v3] staging: rtl8192u: fix sparse warnings

2014-12-04 Thread Fred Chou
On 4/12/2014 4:30 PM, Sudip Mukherjee wrote: > On Thu, Dec 04, 2014 at 04:12:47PM +0800, Fred Chou wrote: >> >> >> On 4/12/2014 3:34 PM, Sudip Mukherjee wrote: >>> On Thu, Dec 04, 2014 at 10:50:43AM +0800, Fred Chou wrote: >>>> From: Fred Chou >&g

Re: [PATCH v3] staging: rtl8192u: fix sparse warnings

2014-12-04 Thread Fred Chou
On 4/12/2014 3:34 PM, Sudip Mukherjee wrote: > On Thu, Dec 04, 2014 at 10:50:43AM +0800, Fred Chou wrote: >> From: Fred Chou >> >> Fixed the following warnings in sparse: >> >> drivers/staging/rtl8192u/r8192U_core.c:670:6: warning: >> symbol 'dum

[PATCH v3] staging: rtl8192u: fix sparse warnings

2014-12-03 Thread Fred Chou
From: Fred Chou Fixed the following warnings in sparse: drivers/staging/rtl8192u/r8192U_core.c:670:6: warning: symbol 'dump_eprom' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1489:5: warning: symbol 'ComputeTxTime&#x

[PATCH v2] staging: rt8192u: fix sparse warnings in r8192U_core.c

2014-12-02 Thread Fred Chou
From: Fred Chou v2: alignment to match open parenthesis Fixed the following warnings in sparse: drivers/staging/rtl8192u/r8192U_core.c:670:6: warning: symbol 'dump_eprom' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1489:

[PATCH] staging: rt8192u: fix sparse warnings in r8192U_core.c

2014-12-02 Thread Fred Chou
From: Fred Chou Fixed the following warnings in sparse: drivers/staging/rtl8192u/r8192U_core.c:670:6: warning: symbol 'dump_eprom' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1489:5: warning: symbol 'ComputeTxTime&#x

[PATCH] staging: rt8192u: fix sparse warnings in r8192U_core.c

2014-12-02 Thread Fred Chou
; was not declared. Should it be static? Signed-off-by: Fred Chou --- drivers/staging/rtl8192u/r8192U_core.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r8192U_core.c index 7640386..6b647

[PATCH] vme: remove redundant else condition

2014-08-27 Thread Fred Chou
The else condition is redundant after a return. Remove these redundant else conditions. Signed-off-by: Fred Chou --- drivers/staging/vme/devices/vme_pio2_gpio.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/vme/devices/vme_pio2_gpio.c b