Dan Carpenter dan.carpen...@oracle.com writes:
The return from myid() isn't aligned correctly for ether_addr_copy().
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
Sorry, this makes no sense, just fix it properly!
drivers/staging/rtl8723au/include/rtw_eeprom.h:
struct eeprom_priv {
Dan Carpenter dan.carpen...@oracle.com writes:
On Wed, Oct 08, 2014 at 03:59:33PM +0200, Jes Sorensen wrote:
Dan Carpenter dan.carpen...@oracle.com writes:
The return from myid() isn't aligned correctly for ether_addr_copy().
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
Sorry
Joe Perches j...@perches.com writes:
On Wed, 2014-10-08 at 15:46 +0300, Dan Carpenter wrote:
On Wed, Oct 08, 2014 at 05:26:11AM -0700, Joe Perches wrote:
On Wed, 2014-10-08 at 13:40 +0300, Dan Carpenter wrote:
The return from myid() isn't aligned correctly for ether_addr_copy().
Hey
Joe Perches j...@perches.com writes:
On Wed, 2014-10-08 at 16:33 +0200, Jes Sorensen wrote:
Joe Perches j...@perches.com writes:
On Wed, 2014-10-08 at 15:46 +0300, Dan Carpenter wrote:
On Wed, Oct 08, 2014 at 05:26:11AM -0700, Joe Perches wrote:
On Wed, 2014-10-08 at 13:40 +0300, Dan
From: Jes Sorensen jes.soren...@redhat.com
This fixes a potential endian bug
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/hal_com.c | 2 +-
drivers/staging/rtl8723au/include/rtw_xmit.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/usb_ops_linux.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/staging/rtl8723au/os_dep/usb_ops_linux.c
b/drivers/staging/rtl8723au/os_dep
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/rtw_xmit.h | 1 -
drivers/staging/rtl8723au/os_dep/os_intfs.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/rtw_xmit.h
b
From: Jes Sorensen jes.soren...@redhat.com
Hi,
Please find attached a couple of fixes and cleanups for the rtl8723au
driver. This includes a fix for the mac_addr alignment reported by Dan
Carpenter and two patches from Greg Donald.
Cheers,
Jes
Greg Donald (2):
drivers: staging: rtl8723au
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_sreset.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_sreset.c
b/drivers/staging/rtl8723au/core
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c| 2 +-
drivers/staging/rtl8723au/include/rtw_xmit.h | 1 -
drivers/staging/rtl8723au/os_dep/os_intfs.c | 1 -
3 files changed, 1 insertion(+), 3
From: Jes Sorensen jes.soren...@redhat.com
We don't do anything with these, so get rid of them
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c| 6 --
drivers/staging/rtl8723au/include/rtw_xmit.h | 5 -
drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Be consistent in the use of enum VCS_TYPE
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_wlan_util.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8723au/core
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 3 ---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 2 +-
drivers/staging/rtl8723au/include/wlan_bssdef.h | 1 -
3 files changed, 1 insertion
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/wlan_bssdef.h | 10 --
1 file changed, 10 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/wlan_bssdef.h
b/drivers/staging/rtl8723au/include
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/rtw_xmit.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/rtw_xmit.h
b/drivers/staging/rtl8723au
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c| 2 --
drivers/staging/rtl8723au/include/rtw_xmit.h | 1 -
2 files changed, 3 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c
b
From: Jes Sorensen jes.soren...@redhat.com
Make sure struct eeprom_priv-mac_addr is 2 byte aligned to work with
ether_addr_copy()
Reported-by: Dan Carpenter dan.carpen...@oracle.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/rtw_eeprom.h | 2 +-
1
From: Greg Donald gdon...@gmail.com
Fix checkpatch.pl 'foo * bar' should be 'foo *bar' errors
Signed-off-by: Greg Donald gdon...@gmail.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_ap.c| 2 +-
drivers/staging/rtl8723au/core/rtw_led.c
From: Greg Donald gdon...@gmail.com
Fix checkpatch.pl open brace '{' following struct go on the same line errors
Signed-off-by: Greg Donald gdon...@gmail.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_security.c | 3 +--
drivers/staging
Greg KH gre...@linuxfoundation.org writes:
On Fri, Oct 10, 2014 at 09:41:23PM +0200, jes.soren...@redhat.com wrote:
From: Jes Sorensen jes.soren...@redhat.com
Hi,
Please find attached a couple of fixes and cleanups for the rtl8723au
driver. This includes a fix for the mac_addr alignment
Greg KH gre...@linuxfoundation.org writes:
On Sat, Oct 11, 2014 at 05:32:49AM -0700, Greg KH wrote:
On Fri, Oct 10, 2014 at 09:41:34PM +0200, jes.soren...@redhat.com wrote:
From: Jes Sorensen jes.soren...@redhat.com
We don't do anything with these, so get rid of them
Signed-off
Joe Perches j...@perches.com writes:
On Sun, 2014-10-26 at 16:18 +, Paul McQuade wrote:
ERROR: spaces required around that ':' (ctx:VxE)
Signed-off-by: Paul McQuade paulmcq...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 16
1 file changed, 8
Paul McQuade paulmcq...@gmail.com writes:
create marco for max_rate values
Signed-off-by: Paul McQuade paulmcq...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 58
--
1 file changed, 26 insertions(+), 32 deletions(-)
diff --git
Joe Perches j...@perches.com writes:
On Mon, 2014-10-27 at 06:45 +0100, Jes Sorensen wrote:
Joe Perches j...@perches.com writes:
On Sun, 2014-10-26 at 16:18 +, Paul McQuade wrote:
ERROR: spaces required around that ':' (ctx:VxE)
[]
diff --git a/drivers/staging/rtl8723au/core
Joe Perches j...@perches.com writes:
On Mon, 2014-10-27 at 07:16 +0100, Jes Sorensen wrote:
Joe Perches j...@perches.com writes:
On Mon, 2014-10-27 at 06:45 +0100, Jes Sorensen wrote:
Joe Perches j...@perches.com writes:
On Sun, 2014-10-26 at 16:18 +, Paul McQuade wrote:
ERROR
Sanjeev Sharma sanjeev_sha...@mentor.com writes:
This is a patch to the rtw_cmd.c file that fixes
Error reported by checkpatch.
Signed-off-by: Sanjeev Sharma sanjeev_sha...@mentor.com
---
drivers/staging/rtl8723au/core/rtw_cmd.c | 83
+++-
1 file changed, 40
Joe Perches j...@perches.com writes:
On Mon, 2014-10-27 at 09:43 +0100, Jes Sorensen wrote:
Sanjeev Sharma sanjeev_sha...@mentor.com writes:
This is a patch to the rtw_cmd.c file that fixes
Error reported by checkpatch.
[]
diff --git a/drivers/staging/rtl8723au/core/rtw_cmd.c
b/drivers
Sharma, Sanjeev sanjeev_sha...@mentor.com writes:
-Original Message-
From: Jes Sorensen [mailto:jes.soren...@redhat.com]
Sent: Monday, October 27, 2014 2:13 PM
To: Sharma, Sanjeev
Cc: larry.fin...@lwfinger.net; gre...@linuxfoundation.org;
linux-wirel...@vger.kernel.org; de
Sharma, Sanjeev sanjeev_sha...@mentor.com writes:
-Original Message-
From: Joe Perches [mailto:j...@perches.com]
Sent: Monday, October 27, 2014 8:23 PM
To: Jes Sorensen
Cc: Sharma, Sanjeev; larry.fin...@lwfinger.net;
gre...@linuxfoundation.org; linux-wirel...@vger.kernel.org;
de
Sharma, Sanjeev sanjeev_sha...@mentor.com writes:
-Original Message-
From: Greg KH [mailto:gre...@linuxfoundation.org]
Sent: Thursday, November 06, 2014 9:13 PM
To: Sharma, Sanjeev
Cc: larry.fin...@lwfinger.net; jes.soren...@redhat.com;
de...@driverdev.osuosl.org;
().
Signed-off-by: Chris Ruffin cmruf...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Looks fine to me
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c
b/drivers
Arend van Spriel ar...@broadcom.com writes:
On 10-11-14 21:21, Jes Sorensen wrote:
Chris Ruffin cmruf...@gmail.com writes:
Using a u16 pointer typecast for a result from htons() results in
the following warning from sparse:
drivers/staging/rtl8723au/core/rtw_xmit.c:1279:36: warning
From: Jes Sorensen jes.soren...@redhat.com
Hi,
Here's a set of patches removing a pile of unused code from the
rtl8723au drivers. It also includes an alignment issue pointed out by
Arend van Spriel.
Note this set goes on top of Chris Ruffin's patch:
staging: rtl8723au: change typecast to match
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 4
drivers/staging/rtl8723au/hal/usb_halinit.c | 4
drivers/staging/rtl8723au/include/rtl8723a_hal.h | 2 --
3 files changed, 10
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c| 17 -
drivers/staging/rtl8723au/include/rtw_xmit.h | 2 --
2 files changed, 19 deletions(-)
diff --git a/drivers/staging/rtl8723au
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_recv.c | 40 ---
1 file changed, 4 insertions(+), 36 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_recv.c
b/drivers
From: Jes Sorensen jes.soren...@redhat.com
No point of masking out high bits since we never set the value to
anything exceeding bits 0-3.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c | 2 +-
drivers/staging/rtl8723au/hal
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c| 40 ++--
drivers/staging/rtl8723au/include/rtw_xmit.h | 1 -
2 files changed, 20 insertions(+), 21 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Reported-by: Arend van Spriel ar...@broadcom.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au/core
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c | 26 +++---
1 file changed, 7 insertions(+), 19 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c
b/drivers
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 248 --
drivers/staging/rtl8723au/include/rtl8723a_xmit.h | 1 -
2 files changed, 249 deletions(-)
diff --git a/drivers
From: Jes Sorensen jes.soren...@redhat.com
The rtl8723au relies on hw led support, so no point carrying a large
unused sw led infrastructure around.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/Makefile |2 -
drivers/staging/rtl8723au/core
Dan Carpenter dan.carpen...@oracle.com writes:
On Mon, Nov 10, 2014 at 06:11:39PM -0500, jes.soren...@redhat.com wrote:
diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c
b/drivers/staging/rtl8723au/core/rtw_xmit.c
index 18a9f34..f8b1243 100644
---
---
1 file changed, 78 deletions(-)
Looks good to me
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
index 9a75eb6..3e61a45 100644
--- a/drivers/staging
Kinka Huang kinkabr...@gmail.com writes:
Signed-off-by: Kinka Huang kinkabr...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Eeeek, nice catch!
Acked-by: Jes Sorensen jes.soren...@redhat.com
diff --git a/drivers/staging
Athira Lekshmi andnlnb...@gmail.com writes:
Fixed the checkpatch error:
ERROR: spaces required around that ''
Signed-off-by: Athira Lekshmi andnlnb...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_cmd.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_recv.c| 104 ---
drivers/staging/rtl8723au/include/rtw_recv.h | 2 -
2 files changed, 46 insertions(+), 60 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 16 +++-
1 file changed, 7 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/usb_halinit.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/usb_halinit.c
b/drivers/staging/rtl8723au/hal
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme.c| 6 ++
drivers/staging/rtl8723au/core/rtw_recv.c| 10 --
drivers/staging/rtl8723au/include/rtw_recv.h | 1 -
3 files changed, 6
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 37 ++---
1 file changed, 18 insertions(+), 19 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/usb_halinit.c
b
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/usb_halinit.c
b/drivers/staging/rtl8723au/hal
From: Jes Sorensen jes.soren...@redhat.com
Hi,
Pleased find attached a couple of fixes and cleanups for the rtl8723au
staging driver. One serious bug fixe in 13/28, but follow-on changes
showed the code was never actually called.
Should be good for staging-next, don't think any of it is more
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 3 +--
drivers/staging/rtl8723au/hal/usb_ops_linux.c | 19 ---
drivers/staging/rtl8723au/include/rtl8723a_recv.h | 1
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/Makefile| 1 -
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 15 +++
drivers/staging/rtl8723au/hal/rtl8723a_xmit.c | 31
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/usb_halinit.c
b/drivers/staging/rtl8723au/hal/usb_halinit.c
index
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b/drivers
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 27 ---
drivers/staging/rtl8723au/include/usb_ops.h | 2 +-
drivers/staging/rtl8723au/os_dep/usb_intf.c | 3 ++-
3 files
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 18 --
1 file changed, 4 insertions(+), 14 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_rf6052.c | 229 +---
1 file changed, 123 insertions(+), 106 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_rf6052.c
b
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 11 ---
drivers/staging/rtl8723au/include/rtl8723a_hal.h | 4
2 files changed, 15 deletions(-)
diff --git a/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/odm_HWConfig.c | 13 ++---
1 file changed, 2 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/odm_HWConfig.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 15 ---
drivers/staging/rtl8723au/include/rtl8723a_hal.h | 2 --
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Get rid of the alternate code paths since these weren't used and have
never been tested according to the in-code comments.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 322
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 2 --
drivers/staging/rtl8723au/include/rtl8723a_hal.h | 3 ---
2 files changed, 5 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_xmit.c| 5 -
drivers/staging/rtl8723au/include/rtw_xmit.h | 1 -
2 files changed, 6 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c | 62 +++
1 file changed, 30 insertions(+), 32 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c
b
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 4 ++--
drivers/staging/rtl8723au/hal/usb_ops_linux.c | 2 +-
drivers/staging/rtl8723au/include/rtl8723a_recv.h | 2 --
drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Use a separate variable to hold the value of REG_SPS0_CTRL to avoid it
getting clobbered and writing out garbage to it, for the case of
eRFPowerState == rf_sleep/rf_off bRegSSPwrLvl == 1.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers
From: Jes Sorensen jes.soren...@redhat.com
Hi,
This removes a pointless NULL pointer check in the USB endpoint
parsing code, allowing for some cleaning up. I have made the same
change to the rtl8723au driver, which I will push out with the next
set of fixes.
Cheers,
Jes
Jes Sorensen (1
From: Jes Sorensen jes.soren...@redhat.com
struct usb_host_interface points to an array of
struct usb_host_endpoints - it makes no sense to do a NULL pointer
check for each pointer.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8188eu/os_dep/usb_intf.c | 33
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c| 12
drivers/staging/rtl8723au/include/Hal8723APhyCfg.h | 1 -
2 files changed, 13 deletions(-)
diff --git a/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/usb_halinit.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/usb_halinit.c
b/drivers/staging/rtl8723au/hal
From: Jes Sorensen jes.soren...@redhat.com
Use the correct register names when setting parameters to reduce 80M
spur. In addition apply updated 80M spur reduction parameters.
Original patch supplied by Andrew Chang andrew.ch...@realtek.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/Hal8723PwrSeq.h | 8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/Hal8723PwrSeq.h
b/drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c
b/drivers/staging/rtl8723au
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/Hal8723APhyCfg.h | 10 --
1 file changed, 10 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/Hal8723APhyCfg.h
b/drivers/staging/rtl8723au
From: Jes Sorensen jes.soren...@redhat.com
struct usb_host_interface points to an array of
struct usb_host_endpoints - it makes no sense to do a NULL pointer
check for each pointer.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/os_dep/usb_intf.c | 72
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/HalPwrSeqCmd.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8723au/hal/HalPwrSeqCmd.c
b/drivers/staging/rtl8723au
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/Hal8723PwrSeq.h | 40 +++
1 file changed, 11 insertions(+), 29 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/Hal8723PwrSeq.h
b
From: Jes Sorensen jes.soren...@redhat.com
Hi,
This set removes a bunch of unused code, in addition it includes a fix
for using the right register definitions to when setting parameters to
reduce 80M spur.
This set should apply cleanly to staging-next as of today.
Cheers,
Jes
Jes Sorensen (12
From: Jes Sorensen jes.soren...@redhat.com
This is a no-op change, but it improves consistency.
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/Hal8723PwrSeq.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8723au
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/hal/rtl8723a_phycfg.c| 19 ---
drivers/staging/rtl8723au/include/Hal8723APhyCfg.h | 5 -
2 files changed, 8 insertions(+), 16 deletions(-)
diff
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/include/Hal8723APhyCfg.h | 39 --
1 file changed, 39 deletions(-)
diff --git a/drivers/staging/rtl8723au/include/Hal8723APhyCfg.h
b/drivers/staging
Krzysztof Konopko k...@konagma.com writes:
On 12/12/14 00:53, Larry Finger wrote:
In RTL8188EU, both BA_starting_seqctrl and TXOP_limit are unsigned short.
That's not quite the case. `TXOP_limit` is __le16 in RTL8188EU [1].
It's __le16 even in your GitHub repo [2]. And that made me
Krzysztof Konopko k...@konagma.com writes:
Some struct fields in wifi.h are meant to be __le16 bu were declared as
unsigned short. This was reported by sparse:
rtw_wlan_util.c:538:24: warning: cast to restricted __le16
rtw_wlan_util.c:1544:29: warning: cast to restricted __le16
From: Jes Sorensen jes.soren...@redhat.com
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 jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme.c | 6 +++---
drivers/staging
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
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
From: Jes Sorensen jes.soren...@redhat.com
Yet another duplicate version of struct ieee80211_ht_operation
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_ap.c | 14 +++--
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 6
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
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
From: Jes Sorensen jes.soren...@redhat.com
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
From: Jes Sorensen jes.soren...@redhat.com
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
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
From: Jes Sorensen jes.soren...@redhat.com
This sort of stuff should be handled by the bonding driver
Signed-off-by: Jes Sorensen jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_pwrctrl.c | 16
drivers/staging/rtl8723au/core/rtw_wlan_util.c | 25
From: Jes Sorensen jes.soren...@redhat.com
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 jes.soren...@redhat.com
---
drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 11 ---
1
1 - 100 of 671 matches
Mail list logo
