summaryrefslogtreecommitdiff
path: root/drivers/staging/vt6656
AgeCommit message (Collapse)Author
2014-03-19staging: vt6656: s_uGetRTSCTSRsvTime fix return.Malcolm Priestley
Always returns to __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-03-19staging: vt6656: vnt_rxtx_rsvtime_le16 fix return.Malcolm Priestley
Always returns to __le16 type. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-03-19staging: vt6656: s_uGetDataDuration use __le16.Malcolm Priestley
s_uGetDataDuration always returns to __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-03-19staging: vt6656: Correct return type s_uGetRTSCTSDuration.Malcolm Priestley
s_uGetRTSCTSDuration always returns in __le16 type Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-03-19staging: vt6656: rxtx endian correct return durationMalcolm Priestley
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>
2014-03-19staging: vt6656: vnt_time_stamp_off fix return to __le16Malcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-03-19staging: vt6656: struct vnt_tx_short_buf_head correct base typesMalcolm Priestley
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>
2014-03-19staging: vt6656: rxtx Replace wTimeStampOff* variablesMalcolm Priestley
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>
2014-03-19staging: vt6656: rxtx Replace wCTSTxRrvTime_baMalcolm Priestley
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>
2014-03-19staging: vt6656: rxtx: Replace wCTSDuration*Malcolm Priestley
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>
2014-03-19staging: vt6656: rxtx: Replace wTxRrvTime*Malcolm Priestley
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>
2014-03-19staging: vt6656: rxtx: Replace wRTSTxRrvTime*Malcolm Priestley
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>
2014-03-19staging: vt6656: rxtx: Replace wRTSDuration* variablesMalcolm Priestley
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>
2014-03-19staging: vt6656: rxtx Replace wDuration* variables.Malcolm Priestley
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>
2014-03-18staging: vt6656: Fix sparse warningsHimangi Saraogi
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>
2014-03-17staging: vt6656: [BUG] Fix Warning BOGUS urb xfer, pipe 3 != type 1Malcolm Priestley
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>
2014-03-17staging: vt6656: [BUG] set usb_fill_int_urb interval to 1Malcolm Priestley
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>
2014-03-16staging: vt6656: power.c Fix line over 80 characters.Aybuke Ozdemir
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>
2014-03-16staging: vt6656: Use __releases/__acquires annotationsIulia Manda
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>
2014-03-13/staging/vt6656: Delete extra paranthesis from macro definitionAndreea-Cristina Bernat
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>
2014-03-10staging:vt6656: Removed assignment in if statement.Chi Pham
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>
2014-03-08Staging:vt6656: Replace some kzalloc calls with kcalloc in iwctl.cIulia Manda
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>
2014-02-28staging: vt6656: Remove unused tasklets tasklet_struct CmdWorkItem and ↵Malcolm Priestley
EventWorkItem are not used. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-28staging: vt6656: dead code removetypedef struct tagSQuietControlMalcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-28staging: vt6656: clean up device_free_int_bufs.Malcolm Priestley
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>
2014-02-28staging: vt6656: clean up device_free_tx_bufsMalcolm Priestley
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>
2014-02-28staging: vt6656: clean up device_free_rx_bufs.Malcolm Priestley
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>
2014-02-28staging: vt6656: Clean up device_alloc_bufsMalcolm Priestley
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>
2014-02-28staging: vt6656: Replace dev_alloc_skb with netdev_alloc_skb.Malcolm Priestley
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>
2014-02-28staging: vt6656: Clean up RXvFreeRCB.Malcolm Priestley
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>
2014-02-28staging: vt6656: Clean up RXvWorkItemMalcolm Priestley
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>
2014-02-27staging: vt6656: s_nsBulkOutIoCompleteWrite Remove variable status.Malcolm Priestley
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>
2014-02-27staging: vt6656: s_nsBulkOutIoCompleteWrite remove buf_len.Malcolm Priestley
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>
2014-02-27staging: vt6656: Remove unused variables packets*Malcolm Priestley
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>
2014-02-27staging: vt6656: Remove unused variable bEventAvailable.Malcolm Priestley
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>
2014-02-27staging: vt6656: struct vnt_usb_send_context remove sEthHeader.Malcolm Priestley
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>
2014-02-27staging: vt6656: Remove all variables ulInt*Malcolm Priestley
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>
2014-02-27staging: vt6656: Remove all variables ulBulk*Malcolm Priestley
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>
2014-02-27staging: vt6656: struct vnt_usb_send_context struct size optimization.Malcolm Priestley
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>
2014-02-27staging: vt6656: Remove pointer cast to urb->context.Malcolm Priestley
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>
2014-02-27staging: vt6656: Cleanup PIPEnsBulkInUsbReadMalcolm Priestley
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>
2014-02-27staging: vt6656: clean up s_nsBulkInUsbIoCompleteReadMalcolm Priestley
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>
2014-02-27staging: vt6656: s_nsBulkInUsbIoCompleteRead Replace error handling.Malcolm Priestley
Use switch on urb->status Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-27staging: vt6656: s_nsBulkInUsbIoCompleteRead Remove bytesRead/bIndicateReceive.Malcolm Priestley
Replace with urb->actual_length. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-27staging: vt6656: s_nsBulkInUsbIoCompleteRead get rid of status.Malcolm Priestley
Just use urb->status directly. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-27staging: vt6656: clean up PIPEnsSendBulkOut.Malcolm Priestley
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>
2014-02-27staging: vt6656: usb_fill_bulk_urb get rid of void* cast.Malcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-27staging: vt6656: PIPEnsSendBulkOut reverse if statementMalcolm Priestley
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>
2014-02-24staging: vt6656: Remove typedef enum _CONTEXT_TYPEMalcolm Priestley
Replace with enum assign as u8 type. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-24staging: vt6656: clean up s_nsBulkOutIoCompleteWrite.Malcolm Priestley
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>