summaryrefslogtreecommitdiffstats
path: root/drivers/staging/vt6656
Commit message (Collapse)AuthorAgeFilesLines
* staging: vt6656: iwctl_giwaplist() re warn: possible memory leak of 'qual'Malcolm Priestley2012-11-151-1/+5
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl_giwaplist/device_ioctl : use off stack buffers.Malcolm Priestley2012-11-132-51/+68
| | | | | | | | | | | Calls ioctl SIOCGIWAPLIST use off stack buffers. clears up warning messages. main_usb.c:2015:1: warning: the frame size of 1888 bytes is larger than 1024 bytes [-Wframe-larger-than=] iwctl.c:683:1: warning: the frame size of 1280 bytes is larger than 1024 bytes [-Wframe-larger-than=] Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove WPA_SUPPLICANT_DRIVER_WEXT_SUPPORT macroMalcolm Priestley2012-11-139-61/+1
| | | | | | | | | Already removed in parts of driver. Removed in remainder. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64bit fixes: vCommandTimerWait change calculation of timer.Malcolm Priestley2012-11-132-11/+13
| | | | | | | | | | | | | | | The timer appears to run too fast/race on 64 bit systems. Using msecs_to_jiffies seems to cause a deadlock on 64 bit. A calculation of (MSecond * HZ) / 1000 appears to run satisfactory. Change BSSIDInfoCount to u32. After this patch the driver can be successfully connect on little endian 64/32 bit systems. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes correct sizes of ↵Malcolm Priestley2012-11-131-21/+21
| | | | | | | NDIS_802_11_ASSOCIATION_INFORMATION Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes TKIPvMixKey remove unsigned longMalcolm Priestley2012-11-131-21/+19
| | | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes pdwIV is now u32Malcolm Priestley2012-11-131-1/+1
| | | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64bit fixes: key.c/h change unsigned long to u32Malcolm Priestley2012-11-132-7/+7
| | | | | | | Fixes long issues. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes: fix long warning messages.Malcolm Priestley2012-11-134-24/+51
| | | | | | | | | Fixes long warning messages from patch [PATCH 08/14] staging: vt6656: 64 bit fixes : correct all type sizes Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes : correct all type sizesMalcolm Priestley2012-11-131-5/+5
| | | | | | | After this patch all BYTE/WORD/DWORD types can be replaced with the appropriate u sizes. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes: int.h replaced DWORD with u32Malcolm Priestley2012-11-131-2/+2
| | | | | | | Fixes size of long issues. Signed-off-by: Malcolm Priestley<tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes: rxtx.h Replace DWORD with u32.Malcolm Priestley2012-11-131-4/+4
| | | | | | | | Size of long issues. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes: use u32 for QWORD definition.Malcolm Priestley2012-11-131-2/+4
| | | | | | | Size of long issues replace with u32. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit fixes: RXbBulkInProcessData FramesizeMalcolm Priestley2012-11-131-10/+10
| | | | | | | | | Size of long issues. Framesize correction replace DWORD with u32. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64bit fixes: desh.h fix size. replace DWORD with u32.Malcolm Priestley2012-11-131-14/+14
| | | | | | | Fixes size of long issues replace DWORD with u32. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64bit fixes: dpc.c incorrect addressing of void structure.Malcolm Priestley2012-11-131-3/+3
| | | | | | | Fixes the deadlock on 64 bit. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: 64 bit- Correctly address void structure.Malcolm Priestley2012-11-131-6/+4
| | | | | | | Fixes 64 bit deadlock on successful association. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: fix viawget_wpa_param sizeMalcolm Priestley2012-11-131-6/+4
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: nsDMA_tx_packet: free tx context bBoolInUseMalcolm Priestley2012-11-131-0/+1
| | | | | | This reduces the number of free TX urbs when key is not found. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: dead code byCntMeasure/TKIPCounterMeasuresInvokedMalcolm Priestley2012-11-133-7/+0
| | | | | | | remove TKIPCounterMeasuresInvoked and byCntMeasure Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: dead code remove upc.hMalcolm Priestley2012-11-132-163/+0
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: free skbuff and relocate on bReAllocSkb == FALSE.Malcolm Priestley2012-11-131-2/+7
| | | | | | | | | | | | | bReAllocSkb == FALSE is when data is not fed to user land. Free and relocate skbuff. Where data is not sent return = FALSE from RXbBulkInProcessData. This appears to reduce false errors when a reused skbuff has old data. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: no need to bIndicateReceive when no bytes to read.Malcolm Priestley2012-11-131-1/+2
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: [BUG] PIPEnsSendBulkOut free bBoolInUseMalcolm Priestley2012-10-221-0/+1
| | | | | | | | Eventually, when there is enough errors we run out of free TX urbs and connection stalls. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655-6: shift wrap in hostap_set_encryption()Dan Carpenter2012-10-221-3/+3
| | | | | | | | abySeq is an unsigned char so shifting more than 31 bits will lead to a shift wrapping bug. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655-6: shift wrap buf in s_vFillTxKey()Dan Carpenter2012-10-221-1/+1
| | | | | | | | byKeyIndex is an unsigned char between 0 and 0xf. If it is any value higher than 1, then we will hit an integer wrap issue here. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Keep firmware loading local and release firware.Malcolm Priestley2012-10-223-14/+10
| | | | | | | | | | Firmware is retained unreleased for the entire duration of the driver. When done release firmware and if the need be request firmware again. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: [BUG] out of bound array reference in RFbSetPower.Malcolm Priestley2012-10-221-0/+3
| | | | | | | | | | | | Calling RFbSetPower with uCH zero value will cause out of bound array reference. This causes 64 bit kernels to oops on boot. Note: Driver does not function on 64 bit kernels and should be blacklisted on them. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Merge 3.6-rc6 into staging-nextGreg Kroah-Hartman2012-09-162-20/+20
|\ | | | | | | | | | | | | This pulls in the staging tree fixes in 3.6-rc6 into our branch to resolve the merge issues. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: vt6656: [BUG] - Failed connection, incorrect endian.Malcolm Priestley2012-09-042-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hi, This patch fixes a bug with driver failing to negotiate a connection. The bug was traced to commit 203e4615ee9d9fa8d3506b9d0ef30095e4d5bc90 staging: vt6656: removed custom definitions of Ethernet packet types In that patch, definitions in include/linux/if_ether.h replaced ones in tether.h which had both big and little endian definitions. include/linux/if_ether.h only refers to big endian values, cpu_to_be16 should be used for the correct endian architectures. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Cc: stable <stable@vger.kernel.org> # 2.6.37+ Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | Staging: vt6656: using is_broadcast_ether_addr() to simplify the codeWei Yongjun2012-09-041-6/+2
| | | | | | | | | | | | | | Using is_broadcast_ether_addr() to simplify the code. Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging "vt6656" Typo rename Embeded to Embedded.Justin P. Mattock2012-09-042-27/+27
| | | | | | | | | | Signed-off-by: Justin P. Mattock <justinmattock@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging "vt6656" Typo rename Caculate to Calculate.Justin P. Mattock2012-09-044-36/+36
| | | | | | | | | | Signed-off-by: Justin P. Mattock <justinmattock@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging "vt6656" Fix typos in comments, and in a printk message.Justin P. Mattock2012-09-0425-75/+75
| | | | | | | | | | Signed-off-by: Justin P. Mattock <justinmattock@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | Merge 3.6-rc3 into staging-nextGreg Kroah-Hartman2012-08-271-1/+1
|\ \ | |/ | | | | | | | | This picks up fixes we want in this branch to allow us to properly test. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * USB: vt6656: remove __devinit* from the struct usb_device_id tableGreg Kroah-Hartman2012-08-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This structure needs to always stick around, even if CONFIG_HOTPLUG is disabled, otherwise we can oops when trying to probe a device that was added after the structure is thrown away. Thanks to Fengguang Wu and Bjørn Mork for tracking this issue down. Reported-by: Fengguang Wu <fengguang.wu@intel.com> Reported-by: Bjørn Mork <bjorn@mork.no> Cc: stable <stable@vger.kernel.org> CC: Forest Bond <forest@alittletooquiet.net> CC: Marcos Paulo de Souza <marcos.souza.org@gmail.com> CC: "David S. Miller" <davem@davemloft.net> CC: Jesper Juhl <jj@chaosbits.net> CC: Jiri Pirko <jpirko@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6656: Fix typo in vt6656Masanari Iida2012-08-134-8/+8
| | | | | | | | | | | | | | Correct spelling typo in staging/vt6656 Signed-off-by: Masanari Iida <standby24x7@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6656: print small buffers with %*phAndy Shevchenko2012-08-131-5/+2
| | | | | | | | | | | | Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Cc: Forest Bond <forest@alittletooquiet.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6656: don't leak 'param' in vt6656_hostap_ioctl() when returning ↵Jesper Juhl2012-08-131-6/+6
|/ | | | | | | | | | | -EOPNOTSUPP Don't return -EOPNOTSUPP directly in switch case's since it'll leak the memory allocated to 'param' when that variable goes out of scope without having been assigned to anything. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/vt6656: cleanup coding style in vResetCommandTimerDevendra Naga2012-07-161-14/+14
| | | | | | | | | the following fixes... removed spaces at start of a line and used tabs Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/vt6656: remove unnecessary return in BSSvSecondTxDataDevendra Naga2012-07-161-1/+0
| | | | | | | | this return is at the function end, and function is returning nothing.. i.e a void. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/vt6656: coding style fix in BSSvSecondTxDataDevendra Naga2012-07-161-28/+28
| | | | | | | | | | | | | | this function seemed bit more coding style fix... The following fixes: remove spaces at start of line and use tabs use space between if and ( give a space in a multiplication operation use space after = and another variable/constant Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove unsed macrosMarcos Paulo de Souza2012-07-093-5/+0
| | | | | | | | These macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove all commented macrosMarcos Paulo de Souza2012-07-098-20/+0
| | | | | | | | These macros were reported by forgotten-macros tool (https://github.com/marcosps/forgotten_macros). Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: hostap.c: Remove commented codeMarcos Paulo de Souza2012-07-091-43/+0
| | | | | Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: hostap.c: Remove unused macros and some useless commentsMarcos Paulo de Souza2012-07-091-20/+0
| | | | | | | | The unused macros were reporteds by forgotten-macros tool(https://github.com/marcosps/forgotten_macros). Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: desc.h: Remove commented macros and useless commentsMarcos Paulo de Souza2012-07-091-19/+0
| | | | | | | | The commented macros are reported by forgotten-macros tool(https://github.com/marcosps/forgotten_macros). Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: remove a few remaining redundant newlinesJesper Juhl2012-07-091-5/+1
| | | | | | | I missed a few redundant newlines the first time. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: Fix up a few remaining brace issuesJesper Juhl2012-07-091-12/+11
| | | | | | | | | | My previous cleanup patches missed a few cases of redundant/missing/replaced curly braces. This should fix up the last ones. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Acked-by : Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: fix up spacing around operatorsJesper Juhl2012-07-091-7/+6
| | | | | | | | | A few cases were missed in my previous cleanup, this takes care of the last cases of missing space (or too much space (as in a newline)) around operators ('=', '==', ',', '<'). Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
OpenPOWER on IntegriCloud