diff options
| author | Greg Kroah-Hartman | 2015-09-04 04:56:21 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2015-09-13 03:24:41 +0200 |
| commit | 642768eec5e5d8cd1e239828e3b5e23ae45b131a (patch) | |
| tree | 32a2b5c6d993cfdc481417aefd38e7e4fd46de92 /drivers/staging/wilc1000/wilc_wfi_netdevice.h | |
| parent | staging: wilc1000: remove mutex_init/destroy() wrappers (diff) | |
| download | kernel-qcow2-linux-642768eec5e5d8cd1e239828e3b5e23ae45b131a.tar.gz kernel-qcow2-linux-642768eec5e5d8cd1e239828e3b5e23ae45b131a.tar.xz kernel-qcow2-linux-642768eec5e5d8cd1e239828e3b5e23ae45b131a.zip | |
staging: wilc1000: remove wrappers around sema_init()
Just call the function directly. Also remove a pointless "deinit"
function that was empty to match the init function.
This also fixes a bug where txq_add_to_head_cs was a mutex structure
being used as a semaphore. See the fun things that happen when you use
void pointers instead of "real" types? Amazing that this worked at all,
someone got _very_ lucky. Whoever "Amr" is, they really didn't fix
BugID_4720 correctly :(
Cc: Johnny Kim <johnny.kim@atmel.com>
Cc: Rachel Kim <rachel.kim@atmel.com>
Cc: Dean Lee <dean.lee@atmel.com>
Cc: Chris Park <chris.park@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/wilc1000/wilc_wfi_netdevice.h')
| -rw-r--r-- | drivers/staging/wilc1000/wilc_wfi_netdevice.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.h b/drivers/staging/wilc1000/wilc_wfi_netdevice.h index 77f320d125e8..d1de0321749a 100644 --- a/drivers/staging/wilc1000/wilc_wfi_netdevice.h +++ b/drivers/staging/wilc1000/wilc_wfi_netdevice.h @@ -193,7 +193,7 @@ typedef struct { struct mutex txq_cs; /*Added by Amr - BugID_4720*/ - struct mutex txq_add_to_head_cs; + struct semaphore txq_add_to_head_cs; spinlock_t txq_spinlock; struct mutex rxq_cs; |