[PATCH] staging: vt6655: fix sparse warning "cast removes address space of expression"

2014-05-31 Thread James A Shackleford
Add missing __user macro to second parameter of ethtool_ioctl(). This removes the need for the offending (void *) cast of the user space pointer rq->ifr_data. Signed-off-by: James A Shackleford --- drivers/staging/vt6655/device_main.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-

[PATCH] staging: android: binder: Fix 10 overlong lines

2014-05-31 Thread Daniel Dressler
This patch was made following Greg Kroah-Hartman's newbie guide. It is not intended to make any logical alterations to codeflow. Of note to reviews this patch switch's a call to kzalloc to kcalloc on line 2837 Neil Brown gave good advice on how to align the over-long function calls. He suggested u

Re: [PATCH] Staging: android: binder: Fix over-80-char lines

2014-05-31 Thread Daniel Dressler
Thank you Neil I was not sure if space-based alignment was allowed so thank you for demonstrating the change. It does make the code much easier to read. Similar thank you for how to indent over-long function calls. I had mis-understood the coding standard to mean overflow needed to be on the far

[PATCH 07/12] staging: rtl8723au: Don't carry ht_cap_info and ht_param in struct wlan_bcn_info

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen These were just stored to check whether the beacon info had changed and print a debug message in this case. If we ever need this again, we can determine this change differently. Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 22

[PATCH 03/12] staging: rtl8723au: Use struct ieee80211_mcs_info to describe MCS information

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen The original RTL code defined the MCS information as a 16 byte array and blindly applied it's masking over the full array, ignoring the fact that only the first 10 bytes describe the rx_mask and the rest are different fields. Signed-off-by: Jes Sorensen --- drivers/staging/r

[PATCH 01/12] staging: rtl8723au: rtw_mlme_ext.c: Fix up some of the bad formatting

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 274 +++--- 1 file changed, 160 insertions(+), 114 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c ind

[PATCH 11/12] staging: rtl8723au: collect_bss_info23a(): Collect beacon_interval correct

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Only retrieve beacon_interval from frames where it is present. This avoids setting BeaconPeriod to garbage for other frame types. Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-)

[PATCH 08/12] staging: rtl8723au: rtw_check_bcn_info23a(): Use capability bit define

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_wlan_util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723au/core/rtw_wlan_util.c b/drivers/staging/rtl8723au/core/rtw_wlan_util.c index 8c007cd..67fb7ec 100644 --

[PATCH 02/12] staging: rtl8723au: rtw_mlme.c: Remove some tab and parenthesis abuse

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_mlme.c | 26 -- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c b/drivers/staging/rtl8723au/core/rtw_mlme.c index 7170258..bd8

[PATCH 05/12] staging: rtl8723au: Use correct struct for WLAN_EID_HT_CAPABILITY

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen WLAN_EID_HT_CAPABILITY carries a struct ieee80211_ht_operation - use the right struct instead of a self invented one. Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_mlme.c | 6 +++--- drivers/staging/rtl8723au/include/wifi.h | 8 2 files changed

[PATCH 10/12] staging: rtl8723au: Rewrite update_beacon23a_info() tto use cfg80211_find_ie()

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c| 5 ++-- drivers/staging/rtl8723au/core/rtw_wlan_util.c | 38 ++-- drivers/staging/rtl8723au/include/rtw_mlme_ext.h | 9 +++--- 3 files changed, 22 insertions(+), 30 dele

[PATCH 06/12] staging: rtl8723au: Eliminate struct HT_info_element

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Yet another duplicate version of struct ieee80211_ht_operation Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_ap.c | 14 +++-- drivers/staging/rtl8723au/core/rtw_ieee80211.c | 6 +++--- drivers/staging/rtl8723au/core/rtw_ioctl_set.c

[PATCH 04/12] staging: rtl8723au: Use struct ieee80211_ht_cap and associated #defines

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_ap.c| 9 ++-- drivers/staging/rtl8723au/core/rtw_ioctl_set.c | 11 ++-- drivers/staging/rtl8723au/core/rtw_mlme.c | 14 +++-- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 42 +

[PATCH 00/12] staging-next: rtl8723au: More fixes and duplicate defines

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Hi Greg, A smaller set of patches this time. It's mainly cleanups, removal of yet more locally duplicated structs, and some endian fixes. Note the endian fixes shouldn't really cause an urgeng 3.15 backport since I am unaware of any big endian hardware currently using this chi

[PATCH 12/12] staging: rtl8723au: collect_bss_info23a(): ieee80211_mgmt capab_info is little endian

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c index 4d6ec40..1e1f845 1

[PATCH 09/12] staging: rtl8723au: Remmove buddy-adapter linking

2014-05-31 Thread Jes . Sorensen
From: Jes Sorensen This sort of stuff should be handled by the bonding driver Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_pwrctrl.c | 16 drivers/staging/rtl8723au/core/rtw_wlan_util.c | 25 - drivers/staging/rtl8723au/include/d

[PATCH 01/01] linux-next: staging: lustre: fix integer as NULL pointer warnings

2014-05-31 Thread Alexandr Terekhov
Fix several sparse warnings "Using plain integer as NULL pointer" Signed-off-by: Alexandr Terekhov drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 4 ++-- drivers/staging/lustre/lustre/obdclass/obd_mount.c | 4 ++-- drivers/staging/lustre/lustre/obdecho/lproc_echo.c | 4 ++-- drivers/stagi

Re: [PATCH 2/2] gpio: gpiolib: set gpiochip_remove retval to void

2014-05-31 Thread Dan Carpenter
On Sat, May 31, 2014 at 09:35:39AM +0200, Lars-Peter Clausen wrote: > On 05/31/2014 01:29 AM, Greg KH wrote: > >On Fri, May 30, 2014 at 08:16:59PM +0200, Lars-Peter Clausen wrote: > >>On 05/30/2014 07:33 PM, David Daney wrote: > >>>On 05/30/2014 04:39 AM, Geert Uytterhoeven wrote: > On Fri, May

[PATCH] staging: gdm724x: Added blank lines after declarations in gdm_lte.c

2014-05-31 Thread Scott Weir
This patch fixes following checkpatch.pl warning: WARNING: Missing a blank line after declarations in gdm_lte.c Signed-off-by: Scott Weir --- drivers/staging/gdm724x/gdm_lte.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/gdm724x/gdm_lte.c b/drivers/staging/gdm724x/gdm_l

Re: [PATCH 2/2] gpio: gpiolib: set gpiochip_remove retval to void

2014-05-31 Thread Lars-Peter Clausen
On 05/31/2014 01:29 AM, Greg KH wrote: On Fri, May 30, 2014 at 08:16:59PM +0200, Lars-Peter Clausen wrote: On 05/30/2014 07:33 PM, David Daney wrote: On 05/30/2014 04:39 AM, Geert Uytterhoeven wrote: On Fri, May 30, 2014 at 1:30 PM, abdoulaye berthe wrote: --- a/drivers/gpio/gpiolib.c +++ b/