summaryrefslogtreecommitdiffstats
path: root/drivers/staging/vt6656/iwctl.c
Commit message (Collapse)AuthorAgeFilesLines
* staging: vt6656: replaced custom BYTE definition with u8Andres More2013-03-111-12/+12
| | | | | | | | | | Checkpatch findings were not resolved, only direct replacement. sed -i 's/\bBYTE\b/u8/g' drivers/staging/vt6656/*.[ch] sed -i 's/\bPBYTE\b/u8 */g' drivers/staging/vt6656/*.[ch] Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: replaced custom TRUE definition with trueAndres More2013-02-151-24/+24
| | | | | | | | | Checkpatch findings were not resolved, just direct replacement. sed -i 's/\bTRUE\b/true/g' drivers/staging/vt6656/*.[ch] Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: replaced custom FALSE definition with falseAndres More2013-02-151-18/+18
| | | | | | | | | Checkpatch findings were not resolved, just direct replacement. sed -i 's/\bFALSE\b/false/g' drivers/staging/vt6656/*.[ch] Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl/key/rf use new structures.Malcolm Priestley2013-01-071-50/+50
| | | | | | | | This patch cleans up function declarations, definitions and local variables were appropriate replacing types defined in "ttype.h" with linux/types.h Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: wpa_set_keys remove fcpfkernelMalcolm Priestley2012-11-271-1/+1
| | | | | | | | | Only TRUE is ever called, remove FALSE code. Since copy_from_user has been removed, the unlock..lock spin locks have been removed. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: extend wireless definitions to v22Malcolm Priestley2012-11-271-8/+10
| | | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: remove redundant IWEVCUSTOM events.Malcolm Priestley2012-11-271-7/+0Star
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: dead code remove ioctl.c/hMalcolm Priestley2012-11-271-5/+0Star
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: move move all commands to iw_handler.Malcolm Priestley2012-11-271-122/+259
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl_giwaplist() re warn: possible memory leak of 'qual'Malcolm Priestley2012-11-161-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-141-32/+41
| | | | | | | | | | | 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-131-17/+1Star
| | | | | | | | | 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: fix viawget_wpa_param sizeMalcolm Priestley2012-11-131-6/+4Star
| | | | | Signed-off-by: Malcolm Priestley <tvboxspy@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-041-1/+1
| | | | | Signed-off-by: Justin P. Mattock <justinmattock@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: remove a few remaining redundant newlinesJesper Juhl2012-07-091-5/+1Star
| | | | | | | 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/+11Star
| | | | | | | | | | 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/+6Star
| | | | | | | | | 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>
* staging: vt6656: iwctl: Clean up braces on 'if' statementsJesper Juhl2012-06-251-84/+61Star
| | | | | | | | Clean up braces on 'if' statements to (mostly) match coding style. A few other bits, like removing a few blank lines and such may have snug in. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: Cleanup spacing around operators (mostly)Jesper Juhl2012-06-251-89/+79Star
| | | | | | | | | This cleans up spacing around operators according to CodingStyle. (A few deletions of empty lines that were missed by a previous patch are also included. A few bits and pieces broken on multiple lines were put one one line as well). Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: one statement per line (if)Jesper Juhl2012-06-251-1/+2
| | | | | | | Statement following 'if' should be on its own line. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: comment cleanupJesper Juhl2012-06-251-132/+133
| | | | | | | | | | | All comments now have one space between the code they follow and the comment start chars and one space after the comment start chars and the comment text. Also cleaned up some spacing within comments - mostly removed space before ':' and added space after ',' etc. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: space after if/for/while/switchJesper Juhl2012-06-251-72/+72
| | | | | | | | Our coding style dictates a space after if/for/while/switch and the opening parenthesis. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: remove spaces between casts and variablesJesper Juhl2012-06-251-74/+74
| | | | | | | | We do not usually write a space between a cast and the variable being converted. this patch removes such spaces where they occour in iwctl.c Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: remove unneeded scope block in iwctl_siwessid()Jesper Juhl2012-06-251-37/+34Star
| | | | | | | | Reduce indentation by removing completely redundant scope block in the iwctl_siwessid() function. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: remove redundant cast (to PSDevice)Jesper Juhl2012-06-251-29/+29
| | | | | | | | | netdev_priv() returns a void*, so there is no reason to explicitly cast to (PSDevice) when assigning to a variable of type PSDevice. The cast is done implicitly. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: ensure one space between defined name and valueJesper Juhl2012-06-251-2/+2
| | | | | Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: Fix up variable declarations (whitespace and ↵Jesper Juhl2012-06-251-90/+109
| | | | | | | | | | | | 'one-per-line') Ensure that we have only one variable declaration per line and for each of those lines we have one space between type and variable name and if there's assignment, then we have one space on each side of the equals sign (and no more than 80 characters per line). Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: remove redundant ';'Jesper Juhl2012-06-251-1/+1
| | | | | | | else clauses don't need to be terminated with ';'. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: Fix indentationJesper Juhl2012-06-251-620/+620
| | | | | | | | | Fix indentation for the entire files to use tabs rather than spaces and also make sure everything is indented to the proper depth according to context. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: fix up function declarations/prototypesJesper Juhl2012-06-251-132/+65Star
| | | | | | | | | | | | | | | | | The way function prototypes/declarations are written in iwctl.[ch] is an inconsistent mess. This patch makes the whole thing consistent by putting the first function arguments (op to a column width of at most 80) on the same line as the function name and the remaining ones on the following line indented by two tabs. Besides getting rid of the current tabs vs spaces mess it also shortens the files quite a bit and puts them more in line with most other kernel files. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl: Remove redundant blank linesJesper Juhl2012-06-251-76/+0Star
| | | | | | | | | There are a number of excessive blank lines in iwctl.c - this patch gets rid of most of those that I personally considered the most obvious ones. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Correct a few assignments to be compares in ↵Jesper Juhl2012-06-231-3/+3
| | | | | | | | | | | | | | | | | | | iwctl_siwencodeext() The result of "foo = bar" is true, so in statements such as ... if((pDevice->bwextstep0 = TRUE)&&(param->u.wpa_key.key_index ==1)) ... an assignment is most likely not what was intended - a comparison was. As in: ... if ((pDevice->bwextstep0 == TRUE) && (param->u.wpa_key.key_index == 1)) ... There are a 3 such mistakes in the iwctl_siwencodeext() function. This patch fixes them all. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl.c: Rewrite siwrts funcitonMarcos Paulo de Souza2012-02-091-17/+9Star
| | | | | | | | | | | | | This function has the following issues: Parameter info and extra are not used Wrong error handling(the function not return -EINVAL when it happens) This patch simplifies this funtion, remove the not used parameters and fix the error handilng. Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl.c: Remove return statement of iwctl_giwrateMarcos Paulo de Souza2012-02-091-5/+1Star
| | | | | | | | This function will always return 0, and this data is not used by who calls this function. Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl.c: Remove return statement of iwctl_giwessidMarcos Paulo de Souza2012-02-091-4/+1Star
| | | | | | | | This function will always return 0, and this data is not used by who calls this function. Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove return statement of iwctl_giwrangeMarcos Paulo de Souza2012-02-091-4/+1Star
| | | | | | | | The function iwctl_giwrange will always return 0, and this data is not used by who calls this function. Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: Remove return statement of iwctl_giwmodeMarcos Paulo de Souza2012-02-091-3/+1Star
| | | | | | | | This function will always return 0, and this data is not used by who calls this function. Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl.c: Remove useless functionMarcos Paulo de Souza2012-02-091-11/+1Star
| | | | | | | The funciton iwctl_commit does nothing, and can be removed. Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6656: iwctl.c: Remove commented codeMarcos Paulo de Souza2012-02-091-172/+0Star
| | | | | Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging:vt6656: iwctl.c: Removed unneeded functionMarcos Paulo de Souza2011-11-301-12/+0Star
| | | | | | | | | | Removed the function iwctl_giwnwid, that just return a error code. Changes v1 to v2: Removed same functions of vt6655 and vt6656. Signed-off-by: Marcos Paulo de Souza <marcos.mage@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* drivers/staging: Remove unnecessary semicolonsJoe Perches2010-11-161-1/+1
| | | | | Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: Use available error codesJulia Lawall2010-09-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An error code is stored in a variable, but 0 is returned instead. Use the variable instead of 0. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r@ local idexpression x; constant C; @@ if (...) { ... x = -C ... when != x ( return <+...x...+>; | return NULL; | return; | * return ...; ) } // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: vt6656: removed not useful commentsAndres More2010-08-311-13/+2Star
| | | | | | | Removed comments about who changed/added lines, they do not seem useful. Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: vt6656: added spaces around '||'Andres More2010-08-311-1/+2
| | | | | | | Cleared checkpatch ERROR: spaces required around that '||' Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: vt6656: removed Calcu_LinkQual definitionAndres More2010-07-221-8/+0Star
| | | | | | | Removed an always defined macro, perhaps used to patch the driver Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: vt6656: removed update_BssList definitionAndres More2010-07-221-2/+0Star
| | | | | | | Removed an always defined macro, perhaps used to patch the driver Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: vt6656: removed Safe_Close definitionAndres More2010-07-221-27/+3Star
| | | | | | | Removed an always defined macro, perhaps used to patch the driver Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: vt6656: removed '#if 1'/'#if 0' definitionsAndres More2010-07-221-18/+0Star
| | | | | | | Removed code guarded by always false definitions. Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: vt6656: replace IS_ETH_ADDRESS_EQUAL with compare_ether_addrAndres More2010-06-241-35/+4Star
| | | | | | | | | | Code cleanup, removed custom macro to compare Ethernet addresses. Some checkpatch warnings on indentation were not resolved. The return value of compare_ether_addr was reversed, something to consider. Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: vt6656: replaced IS_BROADCAST_ADDRESS with is_broadcast_ether_addrAndres More2010-06-241-1/+1
| | | | | | | | Replaced and removed the custom definition. Minor checkpatch warnings on long lines and indentation were not cleared. Signed-off-by: Andres More <more.andres@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
generated by cgit v1.2.3-55-g7522 (git 2.39.0) at 2024-07-22 09:38:25 +0200