firefly-linux-kernel-4.4.55.git
10 years agostaging: vt6656: Remove PIPEnsControlOutAsyn/CONTROLnsRequestOutAsyn
Malcolm Priestley [Thu, 15 May 2014 21:49:24 +0000 (22:49 +0100)]
staging: vt6656: Remove PIPEnsControlOutAsyn/CONTROLnsRequestOutAsyn

This now the same as PIPEnsControlOut.

Replace with PIPEnsControlOut/CONTROLnsRequestOut.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: PIPEnsControlIn use usb_control_msg
Malcolm Priestley [Thu, 15 May 2014 21:49:23 +0000 (22:49 +0100)]
staging: vt6656: PIPEnsControlIn use usb_control_msg

Now that this function is nolonger atomic useusb_control_msg.

Return STATUS_FAILURE if ntStatus less than wLength

Remove the old struct urb and struct usb_ctrlrequest structures.

Remove fMP_CONTROL_READS flag.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: usbpipe.c PIPEnsControlOut use usb_control_msg
Malcolm Priestley [Thu, 15 May 2014 21:49:22 +0000 (22:49 +0100)]
staging: vt6656: usbpipe.c PIPEnsControlOut use usb_control_msg

Now that this fucntion is nolonger atomic change to usb_control_msg

Return STATUS_FAILURE if ntStatus less than wLength.

Remove fMP_CONTROL_WRITES flag

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes device_xmit
Malcolm Priestley [Thu, 15 May 2014 21:49:21 +0000 (22:49 +0100)]
staging: vt6656: lock changes device_xmit

Replace locks with the spin_lock_irqsave and spin_unlock_irqrestore

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: s_nsBulkInUsbIoCompleteRead.
Malcolm Priestley [Thu, 15 May 2014 21:49:20 +0000 (22:49 +0100)]
staging: vt6656: lock changes: s_nsBulkInUsbIoCompleteRead.

Use spin_lock_irqsave/spin_unlock_irqrestore

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: usbpipe.c add mutex lock.
Malcolm Priestley [Thu, 15 May 2014 21:49:19 +0000 (22:49 +0100)]
staging: vt6656: lock changes: usbpipe.c add mutex lock.

PIPEnsControlOut and PIPEnsControlIn are nolonger
atomic but they do need a usb lock.

Add new mutex lock for these functions and PIPEnsControlOutAsyn.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: vRunCommand nsDMA_tx_packet
Malcolm Priestley [Thu, 15 May 2014 21:49:18 +0000 (22:49 +0100)]
staging: vt6656: lock changes: vRunCommand nsDMA_tx_packet

vRunCommand is nolonger atomic add lock for nsDMA_tx_packet.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: bMgrPrepareBeaconToSend add lock.
Malcolm Priestley [Thu, 15 May 2014 21:49:17 +0000 (22:49 +0100)]
staging: vt6656: lock changes: bMgrPrepareBeaconToSend add lock.

bMgrPrepareBeaconToSend is nolonger atomic

csBeacon_xmit must be locked add spin lock.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: vDMA0_tx_80211
Malcolm Priestley [Thu, 15 May 2014 21:49:16 +0000 (22:49 +0100)]
staging: vt6656: lock changes: vDMA0_tx_80211

vDMA0_tx_80211 is atomically called.

RFbSetPower can not be called atomically.

If wCurrentRate is different to pDevice->wCurrentRate call
bScheduleCommand WLAN_CMD_SETPOWER.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: csMgmt_xmit.
Malcolm Priestley [Thu, 15 May 2014 21:49:15 +0000 (22:49 +0100)]
staging: vt6656: lock changes: csMgmt_xmit.

csMgmt_xmit is nolonger atomically called but part of must be.

Reorder the function so that RFbSetPower is called first.

Add new lock spinlock for the remainder of the function.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: RXvMngWorkItem.
Malcolm Priestley [Thu, 15 May 2014 21:49:14 +0000 (22:49 +0100)]
staging: vt6656: lock changes: RXvMngWorkItem.

Narrow atomic locks in RXvMngWorkItem.

We must lock the DequeueRCB and RXvFreeRCB so that
they are in sync.

vMgrRxManagePacket can nolonger be atomically called.

There is no need for an overall lock.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: vRunCommand remove locks.
Malcolm Priestley [Thu, 15 May 2014 21:49:13 +0000 (22:49 +0100)]
staging: vt6656: lock changes: vRunCommand remove locks.

Removing atomic locked paths to usbpipe.c PIPEnsControlOut and PIPEnsControlIn

This function will nolonger need any locking.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: lock changes: Remove spin locks.
Malcolm Priestley [Thu, 15 May 2014 21:49:12 +0000 (22:49 +0100)]
staging: vt6656: lock changes: Remove spin locks.

Remove atomic paths to usbpipe.c PIPEnsControlOut and PIPEnsControlIn

Remove from
PIPEnsControlOut
PIPEnsControlIn
BSSvSecondCallBack
vt6656_hostap_ioctl
iwctl_siwmode
iwctl_siwencode
iwctl_siwpower
iwctl_siwencodeext
KeyvInitTable
device_init_registers
device_open
device_close

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: struct vnt_usb_send_context remove camel case.
Malcolm Priestley [Thu, 15 May 2014 21:49:11 +0000 (22:49 +0100)]
staging: vt6656: struct vnt_usb_send_context remove camel case.

Camel case changes
pDevice -> priv
pPacket -> skb
pUrb -> urb
uBufLen -> buf_len;
bBoolInUse -> in_use
Data -> data

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: nsDMA_tx_packet Move pContext later.
Malcolm Priestley [Thu, 15 May 2014 21:49:10 +0000 (22:49 +0100)]
staging: vt6656: nsDMA_tx_packet Move pContext later.

call s_vGetFreeContext later in function skiping large
chunk of code.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging/wlan-ng: code refactoring
Denis Pithon [Mon, 12 May 2014 15:22:46 +0000 (17:22 +0200)]
staging/wlan-ng: code refactoring

Extract new static function from p80211netdev_rx_bh() to fix coding
style issue (too many leading tabs).

Signed-off-by: Denis Pithon <denis.pithon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging/wlan-ng: remove function declaration
Denis Pithon [Mon, 12 May 2014 15:22:45 +0000 (17:22 +0200)]
staging/wlan-ng: remove function declaration

Removed useless function prototype: static function p80211netdev_rx_bh()
is defined before being used.

Signed-off-by: Denis Pithon <denis.pithon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging/wlan-ng: update function header comment
Denis Pithon [Mon, 12 May 2014 15:22:44 +0000 (17:22 +0200)]
staging/wlan-ng: update function header comment

Header comment of p80211netdev_rx_bh() does not match function
prototype. Fixed.

Signed-off-by: Denis Pithon <denis.pithon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: wlan-ng: fix bad symbol declaration
Neil Armstrong [Fri, 11 Apr 2014 13:00:49 +0000 (15:00 +0200)]
staging: wlan-ng: fix bad symbol declaration

With sparse, the following error appears :
  CHECK   drivers/staging/wlan-ng/p80211netdev.c
drivers/staging/wlan-ng/cfg80211.c:710:6: warning: symbol 'prism2_connect_result' was not declared. Should it be static?
drivers/staging/wlan-ng/cfg80211.c:719:6: warning: symbol 'prism2_disconnected' was not declared. Should it be static?
drivers/staging/wlan-ng/cfg80211.c:725:6: warning: symbol 'prism2_roamed' was not declared. Should it be static?

Move functions declaration to coherent internal header file.

Signed-off-by: Neil 'Superna' Armstrong <superna9999@gmail.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Removed unnecessary spaces after cast from hfa384x_usb.c
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:35:15 +0000 (23:35 +0200)]
driver: staging: wlan-ng: Removed unnecessary spaces after cast from hfa384x_usb.c

Fixed: No space is necessary after a cast

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Removed #if 0 lines from hfa384x_usb.c
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:35:06 +0000 (23:35 +0200)]
driver: staging: wlan-ng: Removed #if 0 lines from hfa384x_usb.c

Removed lines that were ignored by #if 0

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Removed multiple assignments
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:34:56 +0000 (23:34 +0200)]
driver: staging: wlan-ng: Removed multiple assignments

Removed multiple assignments from hfa384x_usb.c

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Fixed white spaces issues
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:34:48 +0000 (23:34 +0200)]
driver: staging: wlan-ng: Fixed white spaces issues

In hfa384x_usb.c:
Moved Logical continuations to the correct lines
Removed unnecessary blank lines

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Fixed Alignment to match open parenthesis
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:34:40 +0000 (23:34 +0200)]
driver: staging: wlan-ng: Fixed Alignment to match open parenthesis

Fixed Alignment to match open parenthesis in hfa384x_usb.c

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Removed Unnecessary space after function pointer name
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:34:12 +0000 (23:34 +0200)]
driver: staging: wlan-ng: Removed Unnecessary space after function pointer name

Removed Unnecessary space after function pointer name

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Setting character pointers as const
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:34:01 +0000 (23:34 +0200)]
driver: staging: wlan-ng: Setting character pointers as const

changed declaration of ctlx_str to:
static const char * const ctlx_str[]

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: switched to pr_warn
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:33:53 +0000 (23:33 +0200)]
driver: staging: wlan-ng: switched to pr_warn

changed printk(KERN_WARNING .. to pr_warn

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Removed unnecessary typedefs from hfa384x_usb.c
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:33:45 +0000 (23:33 +0200)]
driver: staging: wlan-ng: Removed unnecessary typedefs from hfa384x_usb.c

Removed unnecessary typedefs from hfa384x_usb.c

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodriver: staging: wlan-ng: Fixed Breaking long lines and strings style rule
Sherif Shehab Aldin [Sat, 5 Apr 2014 21:33:16 +0000 (23:33 +0200)]
driver: staging: wlan-ng: Fixed Breaking long lines and strings style rule

Fixed coding style rule "Breaking long lines and strings" for hfa384x_usb.c

Signed-off-by: Sherif Shehab Aldin <shehabaldin.sherif@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rts5208: fix static checker warnings
Micky Ching [Wed, 2 Apr 2014 09:58:52 +0000 (17:58 +0800)]
staging: rts5208: fix static checker warnings

The patch fa590c222fba: "staging: rts5208: add support for rts5208
and rts5288" from Nov 12, 2013, leads to the following static checker
warning:
drivers/staging/rts5208/rtsx_chip.c:107 rtsx_enable_bus_int()
warn: add curly braces?
This warning is produced because incorrect code indent.

Signed-off-by: Micky Ching <micky_ching@realsil.com.cn>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: gs_fpgaboot: Fixed code style issues
Rocco Folino [Wed, 2 Apr 2014 22:47:46 +0000 (00:47 +0200)]
staging: gs_fpgaboot: Fixed code style issues

Fixed missing a blank line after declarations warning

Signed-off-by: Rocco Folino <lordzen87@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging/line6: blank line after declaration cleanup
Jerry Snitselaar [Thu, 24 Apr 2014 07:31:48 +0000 (00:31 -0700)]
staging/line6: blank line after declaration cleanup

Fix coding style warnings reported by checkpath.

Signed-off-by: Jerry Snitselaar <dev@snitselaar.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodrivers: staging: ft1000: Fix lines over 80 characters
Artem Fetishev [Sun, 11 May 2014 18:14:30 +0000 (21:14 +0300)]
drivers: staging: ft1000: Fix lines over 80 characters

..plus some function arg indentation which I haven't noticed at first.

Signed-off-by: Artem Fetishev <wwctrsrx@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodrivers: staging: ft1000: Remove typedefs
Artem Fetishev [Sun, 11 May 2014 18:14:29 +0000 (21:14 +0300)]
drivers: staging: ft1000: Remove typedefs

Remove typedefs according to the "Chapter 5: Typedefs" from
Documentation/CodingStyle.

Signed-off-by: Artem Fetishev <wwctrsrx@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodrivers: staging: ft1000: Use pr_info instead of printk
Artem Fetishev [Sun, 11 May 2014 18:14:28 +0000 (21:14 +0300)]
drivers: staging: ft1000: Use pr_info instead of printk

Improve coding style by fixing this checkstyle warning:

WARNING: Prefer [subsystem eg: netdev]_info([subsystem]dev, ... then dev_info(dev, ... then pr_info(...  to printk(KERN_INFO ...

Signed-off-by: Artem Fetishev <wwctrsrx@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodrivers: staging: ft1000: Fix IOCTL_FT1000_* marcos
Artem Fetishev [Sun, 11 May 2014 18:14:27 +0000 (21:14 +0300)]
drivers: staging: ft1000: Fix IOCTL_FT1000_* marcos

Fixes include:
 - Add closing brackets;
 - Remove sizeof (_IOR and _IOW macro can accept types);
 - Replace _IOW which size of 0 (the last arg) with _IO.
 - Actually these macros are not used anywhere and might be removed.

Signed-off-by: Artem Fetishev <wwctrsrx@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agodrivers: staging: ft1000: Fix indentation
Artem Fetishev [Sun, 11 May 2014 18:14:26 +0000 (21:14 +0300)]
drivers: staging: ft1000: Fix indentation

Fixes indentation in header files.

Signed-off-by: Artem Fetishev <wwctrsrx@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: ft1000: remove explicit static initialisation
Angus Gibson [Mon, 7 Apr 2014 02:29:30 +0000 (12:29 +1000)]
staging: ft1000: remove explicit static initialisation

Fix checkpatch.pl error where a static variable was explicitly
initialised to false

Signed-off-by: Angus Gibson <darthshrine@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agoStaging:ft1000_debug.c:Renaming __FUNCTION__
Paul McQuade [Thu, 3 Apr 2014 16:21:22 +0000 (17:21 +0100)]
Staging:ft1000_debug.c:Renaming __FUNCTION__

__Function__ gets renamed with __func__

Signed-off-by: Paul McQuade <paulmcquad@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agoStaging: ft1000-usb: use netdev_warn
Anders Darander [Tue, 1 Apr 2014 06:29:40 +0000 (08:29 +0200)]
Staging: ft1000-usb: use netdev_warn

Fix coding style issue by replacing printk with netdev_warn.

Signed-off-by: Anders Darander <anders.darander@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agoStaging: ft1000-usb: wrap macros in do while
Anders Darander [Tue, 1 Apr 2014 06:29:39 +0000 (08:29 +0200)]
Staging: ft1000-usb: wrap macros in do while

Fix coding style issue by wrapping the macro in a do {} while (0).

Signed-off-by: Anders Darander <anders.darander@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: dgnc: UART_IIR_XOFF code is buggy
Dan Carpenter [Wed, 14 May 2014 16:13:07 +0000 (19:13 +0300)]
staging: dgnc: UART_IIR_XOFF code is buggy

The UART_IIR_XOFF was supposed to be a no-op but, because there was a
missing semi-colon, the if statement is not "Empty".  I have just deleted
this code because it was supposed to be a no-op anyway.  UART_IIR_XOFF
is a standard define and not something specific to this driver.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: dgnc: use dev_err() instead of printk()
Martin Kepplinger [Tue, 29 Apr 2014 09:43:08 +0000 (11:43 +0200)]
staging: dgnc: use dev_err() instead of printk()

Use dev_err() insted of printk() in order to provice userspace with
more useful information and use the common kernel coding style.

Signed-off-by: Martin Kepplinger <martink@posteo.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: dgnc: redundant NULL checks before kfree()
Vitaly Osipov [Sat, 26 Apr 2014 14:59:17 +0000 (00:59 +1000)]
staging: dgnc: redundant NULL checks before kfree()

Removed redundant NULL checks before calling kfree()

Signed-off-by: Vitaly Osipov <vitaly.osipov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agoStaging: dgnc: dgnc_driver: fixed incorrect braces
Son P. Nguyen [Sun, 20 Apr 2014 18:05:39 +0000 (18:05 +0000)]
Staging: dgnc: dgnc_driver: fixed incorrect braces

Fixed incorrect braces (coding style)

Signed-off-by: Son P. Nguyen <fastmutex@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: dgnc: cleanup dgnc_finalize_board_init()
Antoine Sirinelli [Thu, 27 Mar 2014 21:31:55 +0000 (22:31 +0100)]
staging: dgnc: cleanup dgnc_finalize_board_init()

Trivial cleanups:
- Transform a printk() to a dev_err() call
- Fix 2 lines over 80 characters

Signed-off-by: Antoine Sirinelli <antoine@monte-stello.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: dgap: Add Mark Hounschell to maintainers list for dgap
Mark Hounschell [Fri, 25 Apr 2014 18:32:15 +0000 (14:32 -0400)]
staging: dgap: Add Mark Hounschell to maintainers list for dgap

Add Mark Hounschell to the MAINTAINERs list for the dgap driver

Signed-off-by: Mark Hounschell <markh@compro.net>
Suggested-by: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: fix potential leak in vt6656_hostap_ioctl()
Christian Engelmayer [Wed, 7 May 2014 19:43:37 +0000 (21:43 +0200)]
staging: vt6656: fix potential leak in vt6656_hostap_ioctl()

Commit fb841d67 (staging: vt6656: don't leak 'param' in vt6656_hostap_ioctl()
when returning -EOPNOTSUPP) cleaned up direct returns in the ioctl switch
statement that leaked already allocated memory. Fix the same issue for
VIAWGET_HOSTAPD_SCAN_REQ and VIAWGET_HOSTAPD_MLME that are not supported by
this driver. Detected by Coverity - CID 144381.

Signed-off-by: Christian Engelmayer <cengelma@gmx.at>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: desh.h Remove unused macros and structures.
Malcolm Priestley [Wed, 30 Apr 2014 20:31:13 +0000 (21:31 +0100)]
staging: vt6656: desh.h Remove unused macros and structures.

All are dead and of no use.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: rxtx: Remove unused uDMAIdx from driver.
Malcolm Priestley [Wed, 30 Apr 2014 20:31:12 +0000 (21:31 +0100)]
staging: vt6656: rxtx: Remove unused uDMAIdx from driver.

Remove from functions and callers.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: make spin_lock_irq() human readable
Martin Kepplinger [Tue, 6 May 2014 05:58:39 +0000 (07:58 +0200)]
staging: vt6656: make spin_lock_irq() human readable

Don't require FIRMWAREbDownload() to, first off, unlock a held lock.
Thus do all locking in main_usb.c and hold it for a insignificantly
shorter period of time. This makes the affected area significantly more
readable though.

Signed-off-by: Martin Kepplinger <martink@posteo.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6655: fix checkpatch bracing issues
Guido Martínez [Sat, 19 Apr 2014 19:45:00 +0000 (16:45 -0300)]
staging: vt6655: fix checkpatch bracing issues

This patchs fixes tons of warnings such as:

  WARNING: braces {} are not necessary for single statement blocks
  #354: FILE: drivers/staging/vt6655/wmgr.c:354:
  +       for (ii = 0; ii < WLAN_BSSID_LEN; ii++) {
  +               pMgmt->abyDesireBSSID[ii] = 0xFF;
  +       }

Please note: this patch only fixes bracing issues (and there is still a
lot to do); so if you run checkpatch it _will_ throw a lot of errors.
Use --test-only=braces

Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6655: remove dead code
Guido Martínez [Sat, 19 Apr 2014 19:44:59 +0000 (16:44 -0300)]
staging: vt6655: remove dead code

Remove dead code in many places on this driver.

Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: Use proper target pointer in memcpy()
Tobias Klauser [Fri, 25 Apr 2014 09:53:58 +0000 (11:53 +0200)]
staging: vt6656: Use proper target pointer in memcpy()

The coverity scanner marked these two memcpy()'s as causing a buffer
overflow in CIDs 142743 and 142744. This is due the h_dest member of
struct ethhdr being used as a target (size ETH_ALEN) in memcpy, but the
copy is of size ETH_HLEN. However, the intention here seems to be to
copy the entire ethernet header. Make that clear by specifying the
proper destination buffer.

Also remove the unnecessary casts of the source argument.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: vt6656: Remove unnecesssary NULL check
Tobias Klauser [Fri, 25 Apr 2014 13:39:12 +0000 (15:39 +0200)]
staging: vt6656: Remove unnecesssary NULL check

pFifoHead points to tx_buffer->fifo_head which can never be NULL.
We also don't need to check for tx_buffer being NULL instead, since it
always points to ->Data of struct vnt_usb_send_context - the pointer to
which was checked before calling s_vGenerateTxParameter().

Silences a dereference before NULL check warning reported by the coverity
scanner in CID 1127221.

Cc: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agoStaging: vt6655: tether: fixed a brace coding style issue
Nicolas Del Piano [Fri, 25 Apr 2014 17:36:56 +0000 (14:36 -0300)]
Staging: vt6655: tether: fixed a brace coding style issue

Fixed a coding style issue.

Signed-off-by: Nicolas Del Piano <ndel314@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: clean up the pipe driver 64bit ifdefs
Alan [Mon, 12 May 2014 15:57:22 +0000 (16:57 +0100)]
goldfish: clean up the pipe driver 64bit ifdefs

Use the 64bit helper method to scrub most of the ifdefs from the driver. The
pipe reading has a funny case we can't scrub completely.

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: clean up the checkpatch warnings
Alan [Mon, 12 May 2014 15:57:14 +0000 (16:57 +0100)]
goldfish: clean up the checkpatch warnings

Mostly spacing changes, also making the operations structure const

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: clean up tty 64bit support
Alan [Mon, 12 May 2014 15:57:05 +0000 (16:57 +0100)]
goldfish: clean up tty 64bit support

Use the helper we now have available.

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: clean up staging ifdefs
Alan [Mon, 12 May 2014 15:56:56 +0000 (16:56 +0100)]
goldfish: clean up staging ifdefs

Use the write helpers and add an audio helper

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: Allow 64bit builds
Alan [Mon, 12 May 2014 15:55:35 +0000 (16:55 +0100)]
goldfish: Allow 64bit builds

We can now enable the 64bit option for the Goldfish 64bit emulator.

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: enable 64-bit tty driver
Jun Tian [Mon, 12 May 2014 15:55:26 +0000 (16:55 +0100)]
goldfish: enable 64-bit tty driver

Signed-off-by: Jun Tian <jun.j.tian@intel.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish_nand.c: remove length restrictions
Nick Kralevich [Mon, 12 May 2014 15:55:16 +0000 (16:55 +0100)]
goldfish_nand.c: remove length restrictions

Remove limitations on the size of a read / write request.
AFAIK, there's no consequence to allowing the upper layers
to specify different read / write sizes.

This is needed to support running ext4 on goldfish.

Bug: https://code.google.com/p/android/issues/detail?id=38561

Signed-off-by: Nick Kralevich <nnk@google.com>
Signed-off-by: Jun Tian <jun.j.tian@intel.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: add support for 64bit to the virtual bus
Octavian Purdila [Mon, 12 May 2014 15:55:05 +0000 (16:55 +0100)]
goldfish: add support for 64bit to the virtual bus

This patchs adds a new register to pass the upper 32bits for the
device name address when running in 64bit mode.

Signed-off-by: Octavian Purdila <octavian.purdila@intel.com>
Signed-off-by: Jun Tian <jun.j.tian@intel.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: fix kernel panic when using multiple adb connection
Jun Tian [Mon, 12 May 2014 15:54:57 +0000 (16:54 +0100)]
goldfish: fix kernel panic when using multiple adb connection

When using multiple adb on 64 bit kernel to transfer data,
the goldfish pipe interrupt will crash the kernel.

Signed-off-by: Jun Tian <jun.j.tian@intel.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agogoldfish: 64-bit pipe driver for goldfish platform
Jun Tian [Mon, 12 May 2014 15:54:46 +0000 (16:54 +0100)]
goldfish: 64-bit pipe driver for goldfish platform

Support 64-bit channel and address for the goldfish pipe driver.

Signed-off-by: Jun Tian <jun.j.tian@intel.com>
Signed-off-by: Octavian Purdila <octavian.purdila@intel.com>
Signed-off-by: Brian Wood <brian.j.wood@intel.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove unused pointer in rtw_wdev_free()
Christian Engelmayer [Wed, 7 May 2014 19:21:58 +0000 (21:21 +0200)]
staging: rtl8723au: Remove unused pointer in rtw_wdev_free()

Pointer 'pwdev_priv' in function rtw_wdev_free() is unused - thus remove it.

Signed-off-by: Christian Engelmayer <cengelma@gmx.at>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: fix some confusing indenting
Dan Carpenter [Fri, 9 May 2014 13:04:31 +0000 (15:04 +0200)]
staging: rtl8723au: fix some confusing indenting

The break and the "unknown++" are at the same indent level so my static
checker complains.  The if statement should be indented more.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove unused struct odm_ra_info
Jes Sorensen [Fri, 9 May 2014 13:04:30 +0000 (15:04 +0200)]
staging: rtl8723au: Remove unused struct odm_ra_info

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Since struct odm_sta_info isn't used, remove it completely
Jes Sorensen [Fri, 9 May 2014 13:04:29 +0000 (15:04 +0200)]
staging: rtl8723au: Since struct odm_sta_info isn't used, remove it completely

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove unused variable IOTPeer
Jes Sorensen [Fri, 9 May 2014 13:04:28 +0000 (15:04 +0200)]
staging: rtl8723au: Remove unused variable IOTPeer

For bizarre reason we have unused variables IOTPeer both in struct
sta_info and struct odm_sta_info.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: struct odm_mac_info isn't used
Jes Sorensen [Fri, 9 May 2014 13:04:27 +0000 (15:04 +0200)]
staging: rtl8723au: struct odm_mac_info isn't used

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: struct phy_info and struct odm_phy_info are identical
Jes Sorensen [Fri, 9 May 2014 13:04:26 +0000 (15:04 +0200)]
staging: rtl8723au: struct phy_info and struct odm_phy_info are identical

We don't need two copies of the same struct, it just leads to
pointless typecasts.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove empty *_free_xmit_priv() and friends
Jes Sorensen [Fri, 9 May 2014 13:04:25 +0000 (15:04 +0200)]
staging: rtl8723au: Remove empty *_free_xmit_priv() and friends

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Eliminate no-op function rtl8723au_init_recvbuf()
Jes Sorensen [Fri, 9 May 2014 13:04:24 +0000 (15:04 +0200)]
staging: rtl8723au: Eliminate no-op function rtl8723au_init_recvbuf()

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove another unused queue from struct recv_priv
Jes Sorensen [Fri, 9 May 2014 13:04:23 +0000 (15:04 +0200)]
staging: rtl8723au: Remove another unused queue from struct recv_priv

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove some unused lists from struct sta_priv
Jes Sorensen [Fri, 9 May 2014 13:04:22 +0000 (15:04 +0200)]
staging: rtl8723au: Remove some unused lists from struct sta_priv

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: rtw_pwrctrl.c: Reduce spaghettification
Jes Sorensen [Fri, 9 May 2014 13:04:21 +0000 (15:04 +0200)]
staging: rtl8723au: rtw_pwrctrl.c: Reduce spaghettification

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: pattrib->priority can't be > 15, no need for CHKBIT()
Jes Sorensen [Fri, 9 May 2014 13:04:20 +0000 (15:04 +0200)]
staging: rtl8723au: pattrib->priority can't be > 15, no need for CHKBIT()

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: qos_asm23a(): Don't pass through an invalid priority
Jes Sorensen [Fri, 9 May 2014 13:04:19 +0000 (15:04 +0200)]
staging: rtl8723au: qos_asm23a(): Don't pass through an invalid priority

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: set_qos() keep userpriority in a u8
Jes Sorensen [Fri, 9 May 2014 13:04:18 +0000 (15:04 +0200)]
staging: rtl8723au: set_qos() keep userpriority in a u8

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Another case of missing 'tid' bounds checking.
Jes Sorensen [Fri, 9 May 2014 13:04:17 +0000 (15:04 +0200)]
staging: rtl8723au: Another case of missing 'tid' bounds checking.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: 'keyid' is always 0-3, no need for CHKBIT()
Jes Sorensen [Fri, 9 May 2014 13:04:16 +0000 (15:04 +0200)]
staging: rtl8723au: 'keyid' is always 0-3, no need for CHKBIT()

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: No need for CHKBIT since 16 + 12 is still < 32
Jes Sorensen [Fri, 9 May 2014 13:04:15 +0000 (15:04 +0200)]
staging: rtl8723au: No need for CHKBIT since 16 + 12 is still < 32

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: tid is 4 bits, so we need a 16 bit value to hold a bitmap
Jes Sorensen [Fri, 9 May 2014 13:04:14 +0000 (15:04 +0200)]
staging: rtl8723au: tid is 4 bits, so we need a 16 bit value to hold a bitmap

Remove unused add_enable_bitmap and candidate_tid_bitmap from struct
mlme_ext_info.

In addition we don't need to use CHKBIT when we know we are dealing
with a value in the range 0-15.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove local #define BIT()
Jes Sorensen [Fri, 9 May 2014 13:04:13 +0000 (15:04 +0200)]
staging: rtl8723au: Remove local #define BIT()

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Call c2h_handler_8723a() directly instead of via HAL interface
Jes Sorensen [Fri, 9 May 2014 13:04:12 +0000 (15:04 +0200)]
staging: rtl8723au: Call c2h_handler_8723a() directly instead of via HAL interface

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: The driver only has one c2h_id_filter
Jes Sorensen [Fri, 9 May 2014 13:04:11 +0000 (15:04 +0200)]
staging: rtl8723au: The driver only has one c2h_id_filter

Since we only have one function for this, there is no point in keeping
the HAL silliness.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: c2h_evt_hdl() is always called with 3rd argument = NULL
Jes Sorensen [Fri, 9 May 2014 13:04:10 +0000 (15:04 +0200)]
staging: rtl8723au: c2h_evt_hdl() is always called with 3rd argument = NULL

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Declare c2h_evt_hdl static
Jes Sorensen [Fri, 9 May 2014 13:04:09 +0000 (15:04 +0200)]
staging: rtl8723au: Declare c2h_evt_hdl static

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Move rtw_do_join23a() prototype to rtw_ioctl_set.h
Jes Sorensen [Fri, 9 May 2014 13:04:08 +0000 (15:04 +0200)]
staging: rtl8723au: Move rtw_do_join23a() prototype to rtw_ioctl_set.h

rtw_do_join23a() is declared in rtw_ioctl_set.c so move the prototype
to it's correct place, and remove duplicate protoype from rtw_mlme.c

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Move {WPA,RSN}_TKIP_CIPHER23A to rtw_wlan_util.c and mark them...
Jes Sorensen [Fri, 9 May 2014 13:04:07 +0000 (15:04 +0200)]
staging: rtl8723au: Move {WPA,RSN}_TKIP_CIPHER23A to rtw_wlan_util.c and mark them static

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Move REALTEK96B_IE23A to rtw_mlme_ext.c and mark it static
Jes Sorensen [Fri, 9 May 2014 13:04:06 +0000 (15:04 +0200)]
staging: rtl8723au: Move REALTEK96B_IE23A to rtw_mlme_ext.c and mark it static

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove unused ADPT_{FMT,AGR} macros
Jes Sorensen [Fri, 9 May 2014 13:04:05 +0000 (15:04 +0200)]
staging: rtl8723au: Remove unused ADPT_{FMT,AGR} macros

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Get rid of FUNC_NDEV_{FMT,ARG}
Jes Sorensen [Fri, 9 May 2014 13:04:04 +0000 (15:04 +0200)]
staging: rtl8723au: Get rid of FUNC_NDEV_{FMT,ARG}

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove FUNC_ADPT_{FMT,ARG}
Jes Sorensen [Fri, 9 May 2014 13:04:03 +0000 (15:04 +0200)]
staging: rtl8723au: Remove FUNC_ADPT_{FMT,ARG}

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Get rid of NDEV_FMT/NDEV_ARG macros
Jes Sorensen [Fri, 9 May 2014 13:04:02 +0000 (15:04 +0200)]
staging: rtl8723au: Get rid of NDEV_FMT/NDEV_ARG macros

Silly use-once macros that simply obfuscates the code.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Remove some unused prototypes
Jes Sorensen [Fri, 9 May 2014 13:04:01 +0000 (15:04 +0200)]
staging: rtl8723au: Remove some unused prototypes

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Eliminate RTW_STATUS_CODE23a()
Jes Sorensen [Fri, 9 May 2014 13:04:00 +0000 (15:04 +0200)]
staging: rtl8723au: Eliminate RTW_STATUS_CODE23a()

Inline the parsing of status codes. Long term we should pass back the
proper status codes and get rid of the ugly _FAIL/_SUCCESS mess. This
is one small step in that direction.

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
10 years agostaging: rtl8723au: Eliminate _rtw_queue_empty23a()
Jes Sorensen [Fri, 9 May 2014 13:03:59 +0000 (15:03 +0200)]
staging: rtl8723au: Eliminate _rtw_queue_empty23a()

This was just a silly wrapper around list_empty()

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>