[PATCH 4/5] staging: wilc1000: remove unused struct semaphore SemHandleUpdateStats

2016-03-27 Thread Chaehyun Lim
struct semaphore SemHandleUpdateStats is defined but never used in this driver, so just remove it. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 1 - drivers/staging/wilc1000/wilc_wfi_netdevice.h | 1 - 2 files changed, 2

[PATCH 5/5] staging: wilc1000: use mutex instead of struct semaphore hSemScanReq

2016-03-27 Thread Chaehyun Lim
This patch replaces struct semaphore hSemScanReq with struct mutex scan_req_lock. It is better to use mutex than semaphore. Signed-off-by: Chaehyun Lim --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 10 +- drivers/staging/wilc1000/wilc_wfi_netdevice.h

[PATCH 3/5] staging: wilc1000: change data type of wid argument in wilc_wlan_cfg_get_val

2016-03-27 Thread Chaehyun Lim
This patch changes data type of wid argument in wilc_wlan_cfg_get_val from u32 to u16. It is better to change data type of wid because wid has one of enum WID_T that is data type of u16. And, there is no need to use u16 type casting when calling wilc_wlan_cfg_get_wid_value function.

Re: [PATCH] Staging: rtl8723au: Remove function rtw_enqueue_{recvbuf23a/recvbuf23a_to_head}

2016-03-27 Thread Julian Calaby
Hi All, On Sat, Mar 26, 2016 at 5:24 PM, Bhumika Goyal wrote: > The functions rtw_enqueue_recvbuf23a and rtw_enqueue_recvbuf23a_to_head > are never used anywhere in the kernel. So, remove their definition and > prototype. > Grepped to find occurences. > > Signed-off-by:

Re: [PATCH] staging: dgnc: replace dgnc_offset_table with bit shift.

2016-03-27 Thread DaeSeok Youn
2016-03-25 22:15 GMT+09:00 walter harms : > > > Am 25.03.2016 12:33, schrieb Daeseok Youn: >> the dgnc_offset_table has a same value with (1 << port). >> So I tried to replace dgnc_offset_table array with 1 << port. >> And also there are redundant assignments(tmp and current_port)

[PATCH 8/9] staging: lustre: libcfs: add CFS_FAULT_CHECK()

2016-03-27 Thread James Simmons
Add the macro CFS_FAULT_CHECK() which behaves like CFS_FAIL_CHECK() except that any site may be matched by setting CFS_FAULT (0x0200) in cfs_fail_loc. Add cfs_fail_err for use as a return value with CFS_FAULT_CHECK(). Signed-off-by: John L. Hammond Intel-bug-id:

[PATCH 9/9] staging: lustre: libcfs: remove cfs_workitem_t typedefs

2016-03-27 Thread James Simmons
Convert cfs_workitem_t to proper structure. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/17202 Reviewed-by: John L. Hammond Reviewed-by: Dmitry Eremin

[PATCH 5/9] staging: lustre: libcfs: move all cpt handling to libcfs_cpu.h

2016-03-27 Thread James Simmons
Move the CPT handling declartions out of libcfs_private.h to libcfs_cpu.h where it belongs. Signed-off-by: frank zago Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/15913

[PATCH 7/9] staging: lustre: libcfs: replace direct HZ access with kernel APIs

2016-03-27 Thread James Simmons
From: Jian Yu On some customers' systems, the kernel was compiled with HZ defined to 100, instead of 1000. This improves performance for HPC applications. However, to use these systems with Lustre, customers have to re-build Lustre for the kernel because Lustre directly uses

[PATCH 1/9] staging: lustre: libcfs: remove function declarations in libcfs.h

2016-03-27 Thread James Simmons
A few function declarations are in libcfs.h that are not needed. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/13874 Reviewed-by: Dmitry Eremin Reviewed-by: John L.

[PATCH 2/9] staging: lustre: libcfs: remove cfs_signal_pending wrapper

2016-03-27 Thread James Simmons
Use signal_pending() directly instead of a one line function wrapper. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/13874 Reviewed-by: Dmitry Eremin Reviewed-by: John L.

[PATCH 3/9] staging: lustre: libcfs: remove atomic cpt allocations

2016-03-27 Thread James Simmons
libcfs contains functions to perform atomic memory operations. These functions have never been used so remove them. Signed-off-by: frank zago Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on:

[PATCH 4/9] staging: lustre: libcfs: remove cfs_percpt_[current|index]

2016-03-27 Thread James Simmons
The functions cfs_percpt_current() and cfs_percpt_index() are not used anywhere. Remove it. Signed-off-by: frank zago Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/15913

[PATCH 6/9] staging: lustre: libcfs: add lock-class for cfs_percpt_lock

2016-03-27 Thread James Simmons
From: Liang Zhen initialise lock-class for each sublock of cfs_percpt_lock to eliminate false alarm ""possible recursive locking detected" Signed-off-by: Liang Zhen Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6432 Reviewed-on:

[PATCH] Staging: media: bcm2048: defined region_configs[] array as const array

2016-03-27 Thread Claudiu Beznea
This patch defines region_configs[] array as const array since it is not changed anywhere in code. Signed-off-by: Claudiu Beznea --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] Staging: comedi: Fix 'unsigned' warning style

2016-03-27 Thread Aniket Sharma
This patch fixes the checkpatch.pl warning: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' + unsigned runflags; WARNING: Prefer 'unsigned int' to bare use of 'unsigned' +struct comedi_device *comedi_dev_get_from_minor(unsigned minor); Signed-off-by: Aniket Sharma

[PATCH] Staging: comedi: Fix 'unsigned' warning style

2016-03-27 Thread Aniket Sharma
This patch fixes the checkpatch.pl warning: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' + unsigned runflags; WARNING: Prefer 'unsigned int' to bare use of 'unsigned' +struct comedi_device *comedi_dev_get_from_minor(unsigned minor); Signed-off-by: Aniket Sharma

[PATCH] staging/lustre/libcfs: Copy correct amount in libcfs_ioctl_getdata

2016-03-27 Thread green
From: Oleg Drokin Commit b8ff756bc351 ("staging: lustre: libcfs: merge code from libcfs_ioctl into libcfs_ioctl_getdata") introduced a problem copying just a single pointer worth of data from userspace instead of whole libcfs_ioctl_hdr structure. Adjust the copying amount.

[PATCH] Staging: vt6655: defined some arrays as const arrays since they are not changed anywhere in code

2016-03-27 Thread Claudiu Beznea
This patch defines byVT3253B0_AGC4_RFMD2959[], byVT3253B0_AIROHA2230[], byVT3253B0_UW2451[] and byVT3253B0_AGC[] arrays as const arrays since they are not change anywhere in code. After this change some lines broke the 80 characters per line rule. To fix this, some changed lines were splited on

[PATCH] rtllib: change function name of Dot11d_Reset

2016-03-27 Thread Ming Li
hi, i rename Dot11d_Reset function. Regards, Ming Li Signed-off-by: Ming Li --- drivers/staging/rtl8192e/dot11d.c | 4 ++-- drivers/staging/rtl8192e/dot11d.h | 2 +- drivers/staging/rtl8192e/rtllib_softmac.c | 2 +- 3 files changed, 4 insertions(+), 4