[GIT PULL] Staging driver fixes for 4.0-rc5

The following changes since commit 9eccca0843205f87c00404b663188b88eb248051: Linux 4.0-rc3 (2015-03-08 16:09:09 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/ tags/staging-4.0-rc5 for you to fetch changes up to 1f51d5801859e

[PATCH 10/18] staging: octeon-usb: don't poll interrupts early

Don't poll interrupts before we have registered the HCD. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index 74e2f56..1a53574 10064

[PATCH 02/18] staging: octeon-usb: eliminate 64-bit register access wrappers

Eliminate 64-bit register access wrappers. Since we don't implement register-level debugging outputs, these are useless. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 108 ++-- 1 file changed, 33 insertions(+), 75 deletions(-) diff --git

[PATCH 00/18] staging: octeon-usb: cleanups and reliability improvements

Hi, Here's some cleanups and error handling improvements for octeon-usb. Tested on two different OCTEON setups, EdgeRouter Lite & D-Link DSR-1000N mainly with USB mass storage. A. Aaro Koskinen (18): staging: octeon-usb: remove internal function parameter sanity checks staging: octeon-usb:

[PATCH 01/18] staging: octeon-usb: remove internal function parameter sanity checks

Remove some function parameter sanity checks from internal functions where we can rely them being called with sane parameters. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 41 - 1 file changed, 41 deletions(-) diff --git a/drivers/st

[PATCH 08/18] staging: octeon-usb: never retry after xacterr

Never retry after xacterr. The DMA engine cannot handle that properly and may result in wrong transfer count and hang. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 38 + 1 file changed, 5 insertions(+), 33 deletions(-) diff --git a/d

[PATCH 07/18] staging: octeon-usb: program DMA engine based on transfer direction

Program DMA engine based on transfer direction. We are only transferring to one direction at a time. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c

[PATCH 13/18] staging: octeon-usb: move fifo setup to controller reset from port reset

Move FIFO setup to controller reset from port reset. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index 1ad69c5..

[PATCH 03/18] staging: octeon-usb: remove useless prefix from internal routines

Original CVMX code used "__" to distinguis internal USB functions from public ones. But in Linux all cvmx_usb routines are internal to this driver, so such separation is useless. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 424 +++- 1 fi

[PATCH 04/18] staging: octeon-usb: remove useless prefix from pipe flags

Remove useless prefix from pipe flags and adjust comments. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 33 - 1 file changed, 16 insertions(+), 17 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/o

[PATCH 11/18] staging: octeon-usb: use USB stack to turn on power

Use USB stack to turn on the power bit. It will also do the necessary delays. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/stagi

[PATCH 06/18] staging: octeon-usb: use __BITFIELD_FIELD

Use __BITFIELD_FIELD to define bitfields. OCTEON can at least in theory run also in little-endian mode, so the bitfield definitions should not assume big-endian byte order. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.h | 513 +--- 1 file cha

[PATCH 05/18] staging: octeon-usb: delete redundant field from octeon_temp_buffer

Since we are not moving/realigning the original pointer returned by kmalloc, we don't need to store the value in a separate field. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/octe

[PATCH 14/18] staging: octeon-usb: rewrite port reset

Don't disable port before resetting. Also perform reset unconditionally when requested. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 28 +++- 1 file changed, 3 insertions(+), 25 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd

[PATCH 12/18] staging: octeon-usb: move fifo setup into a separate routine

Move FIFO setup into a separate routine. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 107 1 file changed, 53 insertions(+), 54 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-h

[PATCH 15/18] staging: octeon-usb: move cvmx_usb_shutdown()

Move cvmx_usb_shutdown() to allow its use from some other functions. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 66 - 1 file changed, 32 insertions(+), 34 deletions(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers

[PATCH 17/18] staging: octeon-usb: try to recover from failed hardware reset

On some hardware the USB fails to initialize to sane state after cold boot. We can detect this based on some unexpected interrupt bits, and recover by re-initializing. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 30 +++--- 1 file changed, 27

[PATCH 18/18] stating: octeon-usb: cvmx_usb_initialize(): eliminate extra indentation

Eliminate extra indentation by putting all the code into the same block. Also delete a bogus comment. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 152 ++-- 1 file changed, 68 insertions(+), 84 deletions(-) diff --git a/drivers/staging/o

[PATCH 16/18] staging: octeon-usb: fail and warn if DMA counters are wrong

Fail and WARN if DMA counters are wrong. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c index c761a39..54c23c4 100644 ---

[PATCH 09/18] staging: octeon-usb: refactor usbn block init

Move one-time state data structure initialization out of cvmx_usb_initialize(). This enables to re-initialize USBN HW block without messing up data structures. Signed-off-by: Aaro Koskinen --- drivers/staging/octeon-usb/octeon-hcd.c | 57 +++-- 1 file changed, 25 ins

[PATCH 1/3] staging: rtl8723au: Fix trailing whitespace

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/odm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723au/hal/odm.c b/drivers/staging/rtl8723au/hal/odm.c index 428b2b3..ec543cf 100644 --- a/drivers/staging/rtl8723au/hal

[PATCH 2/3] staging: rtl8723au: update_txdesc(): bagg_pkt is always false

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/hal/rtl8723au_xmit.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c b/drivers/staging/rtl8723au/hal/rtl8723au_xmit.c index 1759487..6bc1297 100

[PATCH 3/3] staging: rtl8723au: Do not byteswap timeout twice

From: Jes Sorensen Signed-off-by: Jes Sorensen --- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c index d01ca1d..c98920e 10064

[PATCH 0/3] staging: rtl8723au: 3 fixes

From: Jes Sorensen Hi Greg, 3 fixes for the rtl8723au staging driver - one fixes a whitespace mess I created. One is an old one I forgot to post a while ago, and the least one fixes an issue where a timeout value would get byteswapped twice which is somewhat serious. Cheers, Jes Jes Sorensen (

[PATCH 1/2] octeon-usb:Fix coding style issue with space between function name and opening bracket

Fixes the checkpath.pl error where the opening bracket has a unneeded space between it and the function name for a if statement in the marco,VMX_WAIT_FOR_FIELD32. Signed-off-by: Nicholas Krause --- drivers/staging/octeon-usb/octeon-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH 2/2] octeon:Make of_device_id structure const in octeon-hcd.c

This fixes the checkpatch.pl warning about recommending the of_device_id structure,octeon_usb_match to be defined const in octeon-hcd.c Signed-off-by: Nicholas Krause --- drivers/staging/octeon-usb/octeon-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/o