summaryrefslogtreecommitdiffstats
path: root/drivers/staging/vt6656
Commit message (Collapse)AuthorAgeFilesLines
* staging: vt6656: s_uGetRTSCTSRsvTime fix return.Malcolm Priestley2014-03-191-3/+3
| | | | | | | Always returns to __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: vnt_rxtx_rsvtime_le16 fix return.Malcolm Priestley2014-03-191-1/+1
| | | | | | | Always returns to __le16 type. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_uGetDataDuration use __le16.Malcolm Priestley2014-03-191-2/+2
| | | | | | | s_uGetDataDuration always returns to __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Correct return type s_uGetRTSCTSDuration.Malcolm Priestley2014-03-191-2/+2
| | | | | | | s_uGetRTSCTSDuration always returns in __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx endian correct return durationMalcolm Priestley2014-03-191-4/+4
| | | | | | | All duration base types are __le16 correct back to cpu Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: vnt_time_stamp_off fix return to __le16Malcolm Priestley2014-03-191-1/+1
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: struct vnt_tx_short_buf_head correct base typesMalcolm Priestley2014-03-191-2/+2
| | | | | | | Both variables duration and time_stamp_off are __le16 types Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx Replace wTimeStampOff* variablesMalcolm Priestley2014-03-192-13/+13
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wTimeStampOff* -> time_stamp_off* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx Replace wCTSTxRrvTime_baMalcolm Priestley2014-03-192-2/+2
| | | | | | | Replace with cts_rrv_time_ba with __le16 base. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wCTSDuration*Malcolm Priestley2014-03-192-4/+4
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wCTSDuration* -> cts_duration* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wTxRrvTime*Malcolm Priestley2014-03-192-13/+13
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wTxRrvTime* -> rrv_time* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wRTSTxRrvTime*Malcolm Priestley2014-03-192-9/+9
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wRTSTxRrvTime* -> rts_rrv_time* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx: Replace wRTSDuration* variablesMalcolm Priestley2014-03-192-12/+12
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wRTSDuration* -> rts_duration* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: rxtx Replace wDuration* variables.Malcolm Priestley2014-03-192-56/+56
| | | | | | | | | | Fix base type to __le16 and remove camel case. Camel case changes wDuration* -> duration* Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Fix sparse warningsHimangi Saraogi2014-03-181-3/+3
| | | | | | | | | | This patch fixes the following sparse warnings: drivers/staging/vt6656/aes_ccmp.c:272:32: warning: restricted __le16 degrades to integer drivers/staging/vt6656/aes_ccmp.c:274:32: warning: restricted __le16 degrades to integer drivers/staging/vt6656/aes_ccmp.c:282:32: warning: restricted __le16 degrades to integer Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: [BUG] Fix Warning BOGUS urb xfer, pipe 3 != type 1Malcolm Priestley2014-03-171-1/+1
| | | | | | | | | | | Stable kernels will need patches staging: vt6656: s_nsInterruptUsbIoCompleteRead remove usb_fill_bulk_urb staging: vt6656: PIPEnsInterruptRead use usb_fill_int_urb and a backported version of this patch. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: [BUG] set usb_fill_int_urb interval to 1Malcolm Priestley2014-03-171-1/+1
| | | | | | | | | | | | | As result of patch staging: vt6656: [BUG] Fix Warning BOGUS urb xfer, pipe 3 != type 1 has resulted in sluggish TX performance due to interupt inverval being too long. Set int_interval to 1 as set in bInterval of endpoint 1. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: power.c Fix line over 80 characters.Aybuke Ozdemir2014-03-171-16/+28
| | | | | | | | Fix checkpatch.pl issue with line over 80 characters in power.c Signed-off-by: Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
* staging: vt6656: Use __releases/__acquires annotationsIulia Manda2014-03-171-0/+4
| | | | | | | | | | This patch fixes the following sparse warnings: drivers/staging/vt6656/usbpipe.c:147:24: warning: context imbalance in 'PIPEnsControlOut' - unexpected unlock drivers/staging/vt6656/usbpipe.c:209:24: warning: context imbalance in 'PIPEnsControlIn' - unexpected unlock Signed-off-by: Iulia Manda <iulia.manda21@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
* /staging/vt6656: Delete extra paranthesis from macro definitionAndreea-Cristina Bernat2014-03-141-1/+1
| | | | | | | This patch solves the error of checkpatch.pl by deleting the extra paranthesis. Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
* staging:vt6656: Removed assignment in if statement.Chi Pham2014-03-101-1/+2
| | | | | | | | | | | | | | | | | The following coccinelle script found the match: @@ expression E0, E1, E2; statement S0; @@ - if ((E1 = E2) == E0) + E1 = E2; + if (E1 == E0) S0 Signed-off-by: Chi Pham <fempsci@gmail.com> Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
* Staging:vt6656: Replace some kzalloc calls with kcalloc in iwctl.cIulia Manda2014-03-091-2/+2
| | | | | | | | | | This replacement is justified by the fact that kcalloc could prevent integer overflow resulted from the multiplications used in kzalloc. The semantic patch that showed this suggestion is kzalloc.cocci Signed-off-by: Iulia Manda <iulia.manda21@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove unused tasklets tasklet_struct CmdWorkItem and ↵Malcolm Priestley2014-03-012-6/+0Star
| | | | | | | EventWorkItem are not used. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: dead code removetypedef struct tagSQuietControlMalcolm Priestley2014-03-011-10/+0Star
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: clean up device_free_int_bufs.Malcolm Priestley2014-03-011-2/+3
| | | | | | | | | | Clean up white and camel case. Camel case change pDevice -> priv Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: clean up device_free_tx_bufsMalcolm Priestley2014-03-011-13/+14
| | | | | | | | | | | Clean up white space and camel case. Camel case changes pDevice -> priv pTxContext -> tx_context Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: clean up device_free_rx_bufs.Malcolm Priestley2014-03-011-15/+17
| | | | | | | | | | | cleanup white space and camel case. Camel case changes pDevice -> priv pRCB -> rcb Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Clean up device_alloc_bufsMalcolm Priestley2014-03-011-62/+76
| | | | | | | | | | | Clean up white space, comments and camel case. pDevice -> priv pTxContext -> tx_context pRCB -> rcb Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Replace dev_alloc_skb with netdev_alloc_skb.Malcolm Priestley2014-03-012-14/+8Star
| | | | | | | Remove code that points to netdev (pDevice/priv->dev) Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Clean up RXvFreeRCB.Malcolm Priestley2014-03-011-26/+27
| | | | | | | | | | | | Clean up white space, comments and camel case. Camel case changes pRCB -> rcb bReAllocSkb -> re_alloc_skb pDevice -> priv Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Clean up RXvWorkItemMalcolm Priestley2014-03-011-16/+20
| | | | | | | | | | | | Remove white space and camel case. Camel case changes pDevice -> priv ntStatus -> status pRCB -> rcb Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_nsBulkOutIoCompleteWrite Remove variable status.Malcolm Priestley2014-02-281-19/+5Star
| | | | | | | | | Remove status and move debug messages to switch statement. Remove duplicate second debug message. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_nsBulkOutIoCompleteWrite remove buf_len.Malcolm Priestley2014-02-281-3/+2Star
| | | | | | | Use context->uBufLen directly which is unsigned int. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove unused variables packets*Malcolm Priestley2014-02-282-10/+1Star
| | | | | | | | | These variables are unused and don't reach user. Remove them all. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove unused variable bEventAvailable.Malcolm Priestley2014-02-283-3/+0Star
| | | | | | | Flips from true to false but does nothing. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: struct vnt_usb_send_context remove sEthHeader.Malcolm Priestley2014-02-282-3/+2Star
| | | | | | | | | | | | | In nsDMA_tx_packet and bRelayPacketSend s_vSaveTxPktInfo points to sEthHeader.h_dest which is the wrong viable. Correct and point to pDevice->sTxEthHeader. This is only relevant in the non fuctioning hostapd access point mode, shortly to be removed from driver. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove all variables ulInt*Malcolm Priestley2014-02-282-9/+0Star
| | | | | | | None of these variables reach user or used by debug messages. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove all variables ulBulk*Malcolm Priestley2014-02-282-20/+0Star
| | | | | | | | | None of these variables reach user or in any debug messages. Remove them all. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: struct vnt_usb_send_context struct size optimization.Malcolm Priestley2014-02-271-2/+1Star
| | | | | | | | Reorder in size and remove dead member Next. Reported-by: Joe Perches <joe@perches.com> Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove pointer cast to urb->context.Malcolm Priestley2014-02-271-4/+3Star
| | | | | | | | Remove unnecessary cast to void * urb->context Reported-by: Joe Perches <joe@perches.com> Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Cleanup PIPEnsBulkInUsbReadMalcolm Priestley2014-02-271-27/+25Star
| | | | | | | | | | | | | Remove camel case and white space. Camel case changes pDevice -> priv pRCB -> rcb ntStatus -> status pUrb -> urb Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: clean up s_nsBulkInUsbIoCompleteReadMalcolm Priestley2014-02-271-22/+27
| | | | | | | | | | | | Clean up white space and camel case. Camel case changes pDevice -> priv pRCB -> rcb bReAllocSkb -> re_alloc_skb Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_nsBulkInUsbIoCompleteRead Replace error handling.Malcolm Priestley2014-02-271-15/+16
| | | | | | | Use switch on urb->status Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_nsBulkInUsbIoCompleteRead Remove bytesRead/bIndicateReceive.Malcolm Priestley2014-02-271-9/+3Star
| | | | | | | Replace with urb->actual_length. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: s_nsBulkInUsbIoCompleteRead get rid of status.Malcolm Priestley2014-02-271-4/+3Star
| | | | | | | Just use urb->status directly. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: clean up PIPEnsSendBulkOut.Malcolm Priestley2014-02-271-35/+26Star
| | | | | | | | | | | | Remove commented out code, white space and camel case. camel case changes pDevice -> priv pContext -> context pUrb -> urb Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: usb_fill_bulk_urb get rid of void* cast.Malcolm Priestley2014-02-271-8/+7Star
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: PIPEnsSendBulkOut reverse if statementMalcolm Priestley2014-02-271-6/+4Star
| | | | | | | | Get rid of unnessary else statement by !(MP_IS_READY(pDevice) && pDevice->Flags & fMP_POST_WRITES) Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove typedef enum _CONTEXT_TYPEMalcolm Priestley2014-02-253-11/+11
| | | | | | | | | Replace with enum assign as u8 type. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: clean up s_nsBulkOutIoCompleteWrite.Malcolm Priestley2014-02-251-45/+43Star
| | | | | | | | | | | | | Remove commented out, white space and camel case. Camel case changes pContext -> context pDevice -> priv ContextType -> context_type ulBufLen -> buf_len Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>