[PATCH] rtl8712: pwrctrl_priv: Replace semaphore lock with mutex

2016-08-21 Thread Binoy Jayan
here. Hence, using mutex_lock instead of the interruptible version. Removing the now unused _enter_pwrlock and _down_sema. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> Tested-by: Larry Finger <larry.fin...@lwfinger.net> --- drive

[PATCH] staging: wilc1000: txq_event: Fix coding error

2016-07-21 Thread Binoy Jayan
Fix incorrect usage of completion interface by replacing 'wait_for_completion' with 'complete'. This error was introduced accidentally while replacing semaphores with mutexes. Reported-by: Jiri Slaby <jsl...@suse.cz> Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- dri

[PATCH v4 2/3] staging: wilc1000: Replace kthread with workqueue for host interface

2016-06-22 Thread Binoy Jayan
-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/TODO | 5 + drivers/staging/wilc1000/host_interface.c | 282 -- 2 files changed, 75 insertions(+), 212 deletions(-) diff --git a/drivers/staging/wilc1000/TODO b/drivers/staging/wilc100

[PATCH v4 3/3] staging: wilc1000: Change interface wilc_mq_send to wilc_enqueue_cmd

2016-06-22 Thread Binoy Jayan
Replace the interface 'wilc_mq_send' with 'wilc_enqueue_cmd' and remove the now unused structures 'message' and 'message_queue'. Restructure switch statement in the work queue helper function host_if_work and remove unwanted indentation. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.

[PATCH v4 0/3] *** staging: wilc1000: Replace semaphores ***

2016-06-22 Thread Binoy Jayan
nction 'host_if_work' since there is a mutual dependency. New changes in v4 Remove unused identifier 'hif_msg_q' which causes the build error. Binoy Binoy Jayan (3): staging: wilc1000: message_queue: Move code to host interface staging: wilc1000: Replace kthread with workqueue for host int

[PATCH v4 1/3] staging: wilc1000: message_queue: Move code to host interface

2016-06-22 Thread Binoy Jayan
Move the contents of wilc_msgqueue.c and wilc_msgqueue.h into host_interface.c, remove 'wilc_msgqueue.c' and 'wilc_msgqueue.h'. This is done so as to restructure the implementation of the kthread 'hostIFthread' using a work queue. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- d

[PATCH v3 3/3] staging: wilc1000: Change interface wilc_mq_send to wilc_enqueue_cmd

2016-06-22 Thread Binoy Jayan
Replace the interface 'wilc_mq_send' with 'wilc_enqueue_cmd' and remove the now unused structures 'message' and 'message_queue'. Restructure switch statement in the work queue helper function host_if_work and remove unwanted indentation. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.

[PATCH v3 1/3] staging: wilc1000: message_queue: Move code to host interface

2016-06-22 Thread Binoy Jayan
Move the contents of wilc_msgqueue.c and wilc_msgqueue.h into host_interface.c, remove 'wilc_msgqueue.c' and 'wilc_msgqueue.h'. This is done so as to restructure the implementation of the kthread 'hostIFthread' using a work queue. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- d

[PATCH v3 2/3] staging: wilc1000: Replace kthread with workqueue for host interface

2016-06-22 Thread Binoy Jayan
-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/TODO | 5 + drivers/staging/wilc1000/host_interface.c | 282 -- 2 files changed, 75 insertions(+), 212 deletions(-) diff --git a/drivers/staging/wilc1000/TODO b/drivers/staging/wilc100

[PATCH v3 0/3] *** staging: wilc1000: Replace semaphores ***

2016-06-22 Thread Binoy Jayan
aition for 'host_if_work' since there is a mutual dependency. Binoy Binoy Jayan (3): staging: wilc1000: message_queue: Move code to host interface staging: wilc1000: Replace kthread with workqueue for host interface staging: wilc1000: Change interface wilc_mq_send to wilc_enque

[PATCH v2 2/2] staging: wilc1000: Replace kthread with workqueue for host interface

2016-06-20 Thread Binoy Jayan
-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/wilc1000/TODO | 5 + drivers/staging/wilc1000/host_interface.c | 542 +++--- 2 files changed, 198 insertions(+), 349 deletions(-) diff --git a/dri

[PATCH v2 0/2] *** staging: wilc1000: Replace semaphores ***

2016-06-20 Thread Binoy Jayan
hanges to 'wilc_mq_send' No easy way found to split the patch to change the interface 'wilc_mq_send' and to 'wilc_enqueue_cmd' as the parameters 'mq' 'send_buf' and 'send_buf_size' itself are part of the message queue implementation. Binoy Jayan (2): staging: wilc1000: message_queue: Mov

[PATCH v2 1/2] staging: wilc1000: message_queue: Move code to host interface

2016-06-20 Thread Binoy Jayan
Move the contents of wilc_msgqueue.c and wilc_msgqueue.h into host_interface.c, remove 'wilc_msgqueue.c' and 'wilc_msgqueue.h'. This is done so as to restructure the implementation of the kthread 'hostIFthread' using a work queue. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Re

[PATCH] staging: unisys: visorbus: Replace semaphore with mutex

2016-06-19 Thread Binoy Jayan
The semaphore 'visordriver_callback_lock' is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/unisys/include/visorbus.h | 3 ++- drivers/staging/unisys/visorbus/visorbus_

[PATCH v2] staging: gdm724x: Replace semaphore netlink with mutex

2016-06-14 Thread Binoy Jayan
Replace semaphore netlink_mutex with mutex. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/gdm724x/netlink_k.c | 11 +-- 1 file changed, 1 insertion(+), 10 deleti

[PATCH v3 3/5] staging: wilc1000: Replace semaphore cfg_event with completion

2016-06-14 Thread Binoy Jayan
The semaphore 'cfg_event' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/wilc1000/linux_wlan.c | 2 +- drivers/staging/wilc1000/wilc_wfi_netd

[PATCH v3 4/5] staging: wilc1000: Replace semaphore sync_event with completion

2016-06-14 Thread Binoy Jayan
The semaphore 'sync_event' is used as completion, so convert it to a struct completion type. Also, return -ETIME if the return value of wait_for_completion_timeout is 0. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/stag

[PATCH v3 5/5] staging: wilc1000: Remove semaphore close_exit_sync

2016-06-14 Thread Binoy Jayan
. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/wilc1000/linux_wlan.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index 39fe

[PATCH v3 2/5] staging: wilc1000: Replace semaphore txq_add_to_head_cs with mutex

2016-06-14 Thread Binoy Jayan
The semaphore 'txq_add_to_head_cs' is a simple mutex, so it should be written as one. Semaphores are going away in the future. Also, removing the timeout scenario as the error handling code does not propagate the timeout properly. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Re

[PATCH v3 1/5] staging: wilc1000: Replace semaphore txq_event with completion

2016-06-14 Thread Binoy Jayan
The semaphore 'txq_event' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/wilc1000/linux_wlan.c | 8 drivers/staging/wilc1000/wilc_wfi

[PATCH v3 0/5] *** staging: wilc1000: Replace semaphores with mutexes or completions ***

2016-06-14 Thread Binoy Jayan
. Whitespace removed in patch 3 2. Removed semaphore 'close_exit_sync' 3. To rework on patch 6 and send in a seperate patch series Binoy Jayan (5): staging: wilc1000: Replace semaphore txq_event with completion staging: wilc1000: Replace semaphore txq_add_to_head_cs with mutex staging

[PATCH v2 1/5] staging: wilc1000: Replace semaphore txq_event with completion

2016-06-14 Thread Binoy Jayan
The semaphore 'txq_event' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/linux_wlan.c | 8 drivers/staging/wilc1000/wilc_wfi_netdevice.h | 3 ++- drivers/staging/wi

Re: [PATCH 0/7] *** staging: wilc1000: Replace semaphores with mutexes or completions ***

2016-06-13 Thread Binoy Jayan
On 13 June 2016 at 19:59, Arnd Bergmann <a...@arndb.de> wrote: > On Monday, June 13, 2016 4:07:32 PM CEST Binoy Jayan wrote: >> Hi, >> >> These are a set of patches which removes semaphores from: >> >> drivers/staging/wilc1000 >> >> These are

[PATCH] staging: gdm724x: Replace semaphore netlink with mutex

2016-06-13 Thread Binoy Jayan
Replace semaphore netlink_mutex with mutex. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/gdm724x/netlink_k.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/staging/gdm724x/netli

[PATCH 7/7] staging: wilc1000: Remove unused inclusion of semaphore header

2016-06-13 Thread Binoy Jayan
Remove unused inclusions of semaphore header and remove the same from the todo list. Also remove the now unused wrapper 'wilc_lock_timeout'. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/TODO

[PATCH 4/7] staging: wilc1000: Replace semaphore sync_event with completion

2016-06-13 Thread Binoy Jayan
The semaphore 'sync_event' is used as completion, so convert it to a struct completion type. Also, return -ETIME if the return value of wait_for_completion_timeout is 0. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/linux_wlan.c | 10 +- d

[PATCH 5/7] staging: wilc1000: Replace semaphore close_exit_sync with completion

2016-06-13 Thread Binoy Jayan
The semaphore 'close_exit_sync' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/linux_wlan.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/staging/wi

[PATCH 1/7] staging: wilc1000: Replace semaphore txq_event with completion

2016-06-13 Thread Binoy Jayan
The semaphore 'txq_event' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/linux_wlan.c | 8 drivers/staging/wilc1000/wilc_wfi_netdevice.h | 3 ++- drivers/staging/wi

[PATCH 3/7] staging: wilc1000: Replace semaphore cfg_event with completion

2016-06-13 Thread Binoy Jayan
The semaphore 'cfg_event' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/linux_wlan.c | 2 +- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 +- drivers/staging/wilc1000/wilc_

[PATCH 6/7] staging: wilc1000: message_queue: Replace semaphore sem with completion

2016-06-13 Thread Binoy Jayan
The semaphore 'sem' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/wilc1000/wilc_msgqueue.c | 13 +++-- drivers/staging/wilc1000/wilc_msgqueue.h | 4 ++-- 2 files changed, 9 insertions

[PATCH 2/7] staging: wilc1000: Replace semaphore txq_add_to_head_cs with mutex

2016-06-13 Thread Binoy Jayan
The semaphore 'txq_add_to_head_cs' is a simple mutex, so it should be written as one. Semaphores are going away in the future. Also, removing the timeout scenario as the error handling code does not propagate the timeout properly. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- d

[PATCH 0/7] *** staging: wilc1000: Replace semaphores with mutexes or completions ***

2016-06-13 Thread Binoy Jayan
Hi, These are a set of patches which removes semaphores from: drivers/staging/wilc1000 These are part of a bigger effort to eliminate all semaphores from the linux kernel. They build correctly (individually and as a whole). NB: The changes are untested Thanks, Binoy Binoy Jayan (7

[PATCH 0/2] *** staging: lustre: Replace semaphore lock with mutex ***

2016-06-08 Thread Binoy Jayan
Hi, These are a set of patches which removes semaphores from: drivers/staging/lustre (lnet) These are part of a bigger effort to eliminate all semaphores from the linux kernel. They build correctly (individually and as a whole). Thanks, Binoy Binoy Jayan (2): staging: lustre

[PATCH 2/2] staging: lustre: lnet: Replace semaphore ln_rc_signal with completion

2016-06-08 Thread Binoy Jayan
The semaphore ln_rc_signal is used as completion, so convert it to struct completion. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/lustre/include/linux/lnet/lib-types.h | 3 ++- drivers/staging/lustre/lnet/lnet/ro

[PATCH 1/2] staging: lustre: lloop_device: Replace semaphore lo_sem with completion

2016-06-08 Thread Binoy Jayan
The semaphore 'lo_sem' in lloop_device is used as completion, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/lustre/lustre/llite/lloop.c | 14 +++--- 1 file changed, 7 insertions

[PATCH 0/2] *** r8723au: Replace semaphore lock with mutex ***

2016-06-08 Thread Binoy Jayan
: R8723AU "Realtek RTL8723AU Wireless LAN NIC driver" Thanks, Binoy Binoy Jayan (2): staging: r8723au: pwrctrl_priv: Replace semaphore lock with mutex staging: r8723au: Remove unused semaphores drivers/staging/rtl8723au/core/rtw_pwrctrl.c | 11 ++- drivers/staging/rtl8

[PATCH 2/2] staging: r8723au: Remove unused semaphores

2016-06-08 Thread Binoy Jayan
The semaphores xmit_sema, terminate_xmitthread_sema, tx_retevt and io_req have no users, hence remove all references to them. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8723au/core/rtw_xmit.c | 4 d

[PATCH 1/2] staging: r8723au: pwrctrl_priv: Replace semaphore lock with mutex

2016-06-08 Thread Binoy Jayan
The semaphore 'lock' in pwrctrl_priv is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8723au/core/rtw_pwrctrl.c| 11 ++- drivers/staging/rtl8723au/i

[PATCH v3 0/4] *** staging: r8188eu: Replace semaphores with mutexes or completions ***

2016-06-05 Thread Binoy Jayan
hardware: "Realtek RTL8188EU Wireless LAN NIC driver" Incorporated changes w.r.t. review comments - Changed changelog in patch 3/4 - Changed subject line to have driver name r8188eu - Removed misplaced 'i' in subject of patch 1/4 Thanks, Binoy Binoy Jayan (4): staging: r8188eu: Replace

[PATCH v3 4/4] staging: r8188eu: Remove unused semaphores

2016-06-05 Thread Binoy Jayan
The semaphores xmit_sema, terminate_xmitthread_sema and tx_retevt have no users, hence remove all references to them. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> Acked-by: Larry Finger <larry.fin...@lwfinger.net> --- drivers

[PATCH v3 1/4] staging: r8188eu: Replace semaphore cmd_queue_sema with completion

2016-06-05 Thread Binoy Jayan
The semaphore 'cmd_queue_sema' is used as completion, so convert it to struct completion. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> Acked-by: Larry Finger <larry.fin...@lwfinger.net> --- drivers/staging/rtl8188eu/cor

[PATCH v3 2/4] staging: r8188eu: Replace semaphore terminate_cmdthread_sema with completion

2016-06-05 Thread Binoy Jayan
The semaphore 'terminate_cmdthread_sema' is used as completion, so convert it to struct completion. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> Acked-by: Larry Finger <larry.fin...@lwfinger.net> --- drivers/staging/rtl818

[PATCH v3 3/4] staging: r8188eu: pwrctrl_priv: Replace semaphore 'lock' with mutex

2016-06-05 Thread Binoy Jayan
, using mutex_lock instead of the interruptible version. Also, remove the now unused wrappers _init_pwrlock, _enter_pwrlock, _exit_pwrlock and _rtw_down_sema. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> Acked-by: Larry Finge

[PATCH v2 2/4] rtl8188eu: Replace semaphore terminate_cmdthread_sema with completion

2016-06-05 Thread Binoy Jayan
The semaphore 'terminate_cmdthread_sema' is used as completion, so convert it to struct completion. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/rtl8188eu/core/rtw_cmd.c| 6 +++--- drivers/staging/rtl81

[PATCH v2 3/4] rtl8188eu: pwrctrl_priv: Replace semaphore 'lock' with mutex

2016-06-05 Thread Binoy Jayan
, using mutex_lock instead of the interruptible version. Also, remove the now unused wrappers _init_pwrlock, _enter_pwrlock, _exit_pwrlock and _rtw_down_sema. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/rt

[PATCH v2 0/4] *** rtl8188eu: Replace semaphores with mutexes or completions ***

2016-06-05 Thread Binoy Jayan
hardware: "Realtek RTL8188EU Wireless LAN NIC driver" Incorporated changes w.r.t. review comments: - Changed changelog in patch 3/4 Thanks, Binoy Binoy Jayan (4): irtl8188eu: Replace semaphore cmd_queue_sema with completion rtl8188eu: Replace semaphore terminate_cmdthread_sema with

[PATCH v2 4/4] rtl8188eu: Remove unused semaphores

2016-06-05 Thread Binoy Jayan
The semaphores xmit_sema, terminate_xmitthread_sema and tx_retevt have no users, hence remove all references to them. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/rtl8188eu/core/rtw_xmit.c | 4 dri

[PATCH v2 1/4] irtl8188eu: Replace semaphore cmd_queue_sema with completion

2016-06-05 Thread Binoy Jayan
The semaphore 'cmd_queue_sema' is used as completion, so convert it to struct completion. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> Reviewed-by: Arnd Bergmann <a...@arndb.de> --- drivers/staging/rtl8188eu/core/rtw_cmd.c| 6 +++--- drivers/staging/rtl8188eu/include/r

[PATCH 3/4] rtl8188eu: pwrctrl_priv: Replace semaphore 'lock' with mutex

2016-06-03 Thread Binoy Jayan
The semaphore 'lock' in pwrctrl_priv is a simple mutex, so it should be written as one. Semaphores are going away in the future. Also, remove the now unused wrappers _init_pwrlock, _enter_pwrlock, _exit_pwrlock and _rtw_down_sema. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This

[PATCH 2/4] rtl8188eu: Replace semaphore terminate_cmdthread_sema with completion

2016-06-03 Thread Binoy Jayan
The semaphore 'terminate_cmdthread_sema' is used as completion, so convert it to struct completion. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: irtl8188eu: Replace semaphore cmd_queue_sema with completion drivers/staging/rtl8188e

[PATCH 1/4] irtl8188eu: Replace semaphore cmd_queue_sema with completion

2016-06-03 Thread Binoy Jayan
The semaphore 'cmd_queue_sema' is used as completion, so convert it to struct completion. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8188eu/core/rtw_cmd.c| 6 +++--- drivers/staging/rtl8188eu/include/rtw_cmd.h | 2 +- drivers/staging/rtl8188eu/

[PATCH v2 4/4] rtl8192u: Remove unused semaphore rf_sem

2016-06-02 Thread Binoy Jayan
The semaphore 'rf_sem' in rtl8192u has no users, hence removing it. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192u: Replace semaphore scan_sem with mutex drivers/staging/rtl8192u/r8192U.h | 1 - drivers/staging/rt

[PATCH v2 2/4] rtl8192u: ieee80211_device: Replace semaphore wx_sem with mutex

2016-06-02 Thread Binoy Jayan
The semaphore 'wx_sem' in ieee80211_device is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192u: r8192_priv: Replace semaphore wx_sem with

[PATCH v2 3/4] rtl8192u: Replace semaphore scan_sem with mutex

2016-06-02 Thread Binoy Jayan
The semaphore 'scan_sem' in rtl8192u is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192u: ieee80211_device: Replace semaphore wx_sem with

[PATCH v2 1/4] rtl8192u: r8192_priv: Replace semaphore wx_sem with mutex

2016-06-02 Thread Binoy Jayan
The semaphore 'wx_sem' in r8192_priv is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8192u/r8192U.h | 2 +- drivers/staging/rtl8192u/r8192U_core.

[PATCH v2 0/4] *** rtl8192u: Replace semaphores with mutexes ***

2016-06-02 Thread Binoy Jayan
replacing it with a mutex. Thanks, Binoy Binoy Jayan (4): rtl8192u: r8192_priv: Replace semaphore wx_sem with mutex rtl8192u: ieee80211_device: Replace semaphore wx_sem with mutex rtl8192u: Replace semaphore scan_sem with mutex rtl8192u: Remove unused semaphore rf_sem drivers/stagin

[PATCH 2/4] rtl8192u: ieee80211_device: Replace semaphore wx_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'wx_sem' in ieee80211_device is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192u: r8192_priv: Replace semaphore wx_sem with

[PATCH 3/4] rtl8192u: Replace semaphore scan_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'scan_sem' in rtl8192u is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192u: ieee80211_device: Replace semaphore wx_sem with

[PATCH 1/4] rtl8192u: r8192_priv: Replace semaphore wx_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'wx_sem' in r8192_priv is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8192u/r8192U.h | 2 +- drivers/staging/rtl8192u/r8192U_core.

[PATCH 4/4] rtl8192u: Replace semaphore rf_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'rf_sem' in rtl8192u is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192u: Replace semaphore scan_sem with mutex drivers/s

[PATCH 0/4] *** rtl8192u: Replace semaphores with mutexes ***

2016-06-01 Thread Binoy Jayan
Hi, These are a set of patches which removes semaphores from: drivers/staging/rtl8192u They build correctly (individually and as a whole). NB: I have not tested this as I do not have the following hardware: "RealTek RTL8192U Wireless LAN NIC driver" Thanks, Binoy Binoy Jayan (4):

[PATCH v2 2/4] rtl8712: Replace semaphore terminate_cmdthread_sema with completion

2016-06-01 Thread Binoy Jayan
The semaphore 'terminate_cmdthread_sema' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8712: Replace semaphore cmd_queue_sema with completion drivers/staging/r

[PATCH v2 4/4] rtl8712: pwrctrl_priv: Replace semaphore lock with mutex

2016-06-01 Thread Binoy Jayan
here. Hence, using mutex_lock instead of the interruptible version. Removing the now unused _enter_pwrlock and _down_sema. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8712: intf_priv: Replace semaphore lock with completion drivers/s

[PATCH v2 0/4] *** rtl8712: Replace semaphores with mutex / completions ***

2016-06-01 Thread Binoy Jayan
ents w.r.t. PATCH v1: - Removed wrapper functions _wait_completion, _down_sema and _enter_pwrlock - Updated changelog to explain use of mutex_lock instead of mutex_lock_interruptible in [PATCH v2 4/4] Binoy Binoy Jayan (4): rtl8712: Replace semaphore cmd_queue_sema with completion rtl871

[PATCH v2 1/4] rtl8712: Replace semaphore cmd_queue_sema with completion

2016-06-01 Thread Binoy Jayan
The semaphore 'cmd_queue_sema' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8712/os_intfs.c| 2 +- drivers/staging/rtl8712/rtl8712_cmd.c | 2 +- drivers/staging/rtl8712/rtl871x

[PATCH v2 3/4] rtl8712: intf_priv: Replace semaphore lock with completion

2016-06-01 Thread Binoy Jayan
The semaphore 'lock' in 'intf_priv' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8712: Replace semaphore terminate_cmdthread_sema with completion drivers/staging/r

[PATCH 4/5] rtl8192e: Replace semaphore scan_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'scan_sem' in the rtl8192e is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192e: Replace semaphore rf_sem with mutex drivers/s

[PATCH 0/5] *** rtl8192e: Replace semaphore with mutex ***

2016-06-01 Thread Binoy Jayan
the following hardware: "RealTek RTL8192E Wireless LAN NIC driver" Thanks, Binoy Binoy Jayan (5): rtl8192e: rtllib_device: Replace semaphore wx_sem with mutex rtl8192e: r8192_priv: Replace semaphore wx_sem with mutex rtl8192e: Replace semaphore rf_sem with mutex rtl8192e: Replace

[PATCH 5/5] rtl8192e: Replace semaphore ips_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'ips_sem' in the rtl8192e is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192e: Replace semaphore scan_sem with mutex drivers/s

[PATCH 1/5] rtl8192e: rtllib_device: Replace semaphore wx_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'wx_sem' in the rtllib_device is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 4 +-- drivers/staging/rtl8192e/rtllib.h

[PATCH 3/5] rtl8192e: Replace semaphore rf_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'rf_sem' in the rtl8192e is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192e: r8192_priv: Replace semaphore wx_sem with

[PATCH 2/5] rtl8192e: r8192_priv: Replace semaphore wx_sem with mutex

2016-06-01 Thread Binoy Jayan
The semaphore 'wx_sem' in the r8192_priv is a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8192e: rtllib_device: Replace semaphore wx_sem with

[PATCH 4/4] rtl8712: pwrctrl_priv: Replace semaphore lock with mutex

2016-05-31 Thread Binoy Jayan
The semaphore 'lock' in 'pwrctrl_priv' is used as a simple mutex, so it should be written as one. Semaphores are going away in the future. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8712: intf_priv: Replace semaphor

[PATCH 1/4] rtl8712: Replace semaphore cmd_queue_sema with completion

2016-05-31 Thread Binoy Jayan
The semaphore 'cmd_queue_sema' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- drivers/staging/rtl8712/os_intfs.c| 2 +- drivers/staging/rtl8712/osdep_service.h | 7 +++ drivers/staging/rtl8712/rtl8712

[PATCH 3/4] rtl8712: intf_priv: Replace semaphore lock with completion

2016-05-31 Thread Binoy Jayan
The semaphore 'lock' in 'intf_priv' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8712: Replace semaphore terminate_cmdthread_sema with completion drivers/staging/r

[PATCH 2/4] rtl8712: Replace semaphore terminate_cmdthread_sema with completion

2016-05-31 Thread Binoy Jayan
The semaphore 'terminate_cmdthread_sema' is used as completion, so convert it to a struct completion type. Signed-off-by: Binoy Jayan <binoy.ja...@linaro.org> --- This patch depends on the following patch: rtl8712: Replace semaphore cmd_queue_sema with completion drivers/staging/r

[PATCH 0/4] *** rtl8712: Replace semaphores with mutex / completions ***

2016-05-31 Thread Binoy Jayan
Hi, These are a set of patches which removes semaphores from: drivers/staging/rtl8712 They build correctly (individually and as a whole). NB: I have not tested this as I do not have the following hardware: "RealTek RTL8712U (RTL8192SU) Wireless LAN NIC driver" Thanks, Binoy Bino