summaryrefslogtreecommitdiffstats
path: root/drivers/staging/bcm/PHSModule.c
AgeCommit message (Collapse)AuthorFilesLines
2014-07-15Staging: bcm: PHSModule.c: Shortened linesMatthias Beyer1-54/+89
This patch shortenes all lines where possible without code refactoring, as well as comment lines. It does not change the wording of comments. Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced if-else return code with simple return ↵Matthias Beyer1-9/+1
assertion Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced member accessing with variable in ↵Matthias Beyer1-8/+11
CreateSFToClassifierRuleMapping() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Simplified nested if statements by linking them ↵Matthias Beyer1-5/+4
with logical AND in GetPhsRuleEntry() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Simplified nested if statements by linking them ↵Matthias Beyer1-5/+4
with logical AND in GetServiceFlowEntry() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Simplified nested if statements by linking them ↵Matthias Beyer1-5/+3
with logical AND in GetServiceFlowEntry() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced member accessing with variable in ↵Matthias Beyer1-4/+5
GetServiceFlowEntry() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Added const keyword to ValidatePHSRUleComplete() ↵Matthias Beyer1-2/+2
argument Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Rewrote ValidatePHSRuleComplete()Matthias Beyer1-18/+4
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced indentation level with goto jump on bad ↵Matthias Beyer1-29/+32
condition Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced member accessing with variable in ↵Matthias Beyer1-12/+20
free_phs_service() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Added missing braces around else blockMatthias Beyer1-1/+2
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Whitespace cleanup in PhsCompress()Matthias Beyer1-3/+6
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced member accessing with variablesMatthias Beyer1-16/+24
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Reduced indentation by using jump label in ↵Matthias Beyer1-32/+34
PhsDeleteSFRules() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Shortened linesMatthias Beyer1-11/+15
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Reduced indentation level by using jump labelMatthias Beyer1-31/+34
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced nested if statements with logical AND ↵Matthias Beyer1-10/+10
concatenation of the conditions Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Replaced member accessing with variable in ↵Matthias Beyer1-7/+9
PhsDeletePHSRule() Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Outsourced debug printing for phs classifier entryMatthias Beyer1-18/+29
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Outsourced debug printing in own functionMatthias Beyer1-39/+56
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-15Staging: bcm: PHSModule.c: Whitespace & Indentation cleanupMatthias Beyer1-205/+384
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-09staging: bcm: add missing blank lines after declarationsPawel Lebioda1-0/+1
Fix "Missing blank line after declaration" warnings reported by checkpatch.pl. Signed-off-by: Pawel Lebioda <pawel.lebioda89@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-06-19staging: bcm: Remove unnecessary parenthesesMasanari Iida1-1/+1
This patch remove unnecessary parentheses in PHSModule.c Signed-off-by: Masanari Iida <standby24x7@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-05-26drivers/staging: Remove useless return variablesPeter Senna Tschudin1-6/+3
This patch remove variables that are initialized with a constant, are never updated, and are only used as parameter of return. Return the constant instead of using a variable. Verified by compilation only. The coccinelle script that find and fixes this issue is: // <smpl> @@ type T; constant C; identifier ret; @@ - T ret = C; ... when != ret when strict return - ret + C ; // </smpl> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-01-11staging: bcm : Fix typo in staging/bcmMasanari Iida1-2/+2
This patch fixed spelling typo in comment and printks withing staging/bcm. Signed-off-by: Masanari Iida <standby24x7@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-10-28staging: bcm: Replace FALSE with falseLisa Nguyen1-11/+11
Replace user-defined type FALSE with C defined false keyword. Signed-off-by: Lisa Nguyen <lisa@xenapiadmin.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-10-28staging: bcm: Replace BOOLEAN with boolLisa Nguyen1-9/+9
Remove user-defined BOOLEAN data type with C bool data type. Signed-off-by: Lisa Nguyen <lisa@xenapiadmin.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-09-25Staging: bcm: PHSModule.c: Matching the function definition with function ↵Shalin Mehta1-6/+6
declaration The function implementations of the PHSModule didn't match with the declaration. The functions are static in the declaration but in the implemntation they are non-static Signed-off-by: Shalin Mehta <shalinmehta85@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-03-11Staging: bcm: Fix spelling error in PHSModule.cKevin McKinney1-1/+1
This patch fixes a spelling error in PHSModule.c Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-03-11Staging: bcm: Properly format comments in PHSModule.cKevin McKinney1-271/+272
This patch properly formats comments, and removes them as needed in PHSModule.c. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-03-11Staging: bcm: Properly format braces in PHSModule.cKevin McKinney1-293/+140
This patch formats braces in PHSModule.c as reported by checkpatch.pl. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-03-11Staging: bcm: Fix all white space issues in PHSModule.cKevin McKinney1-596/+588
This patch fixes all white space issues in PHSModule.c as reported by checkpatch.pl. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-01-07Staging: bcm: Remove typedef for _S_PHS_RULE and call directly.Kevin McKinney1-28/+28
This patch removes typedef for _S_PHS_RULE, and changes the name of the struct to bcm_phs_rule. In addition, any calls to struct "S_PHS_RULE" are changed to call directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-01-07Staging: bcm: Remove typedef for _E_CLASSIFIER_ENTRY_CONTEXT and call directly.Kevin McKinney1-6/+6
This patch removes typedef for _E_CLASSIFIER_ENTRY_CONTEXT, and changes the name of the enum to bcm_phs_classifier_context. In addition, any calls to enum "_E_CLASSIFIER_ENTRY_CONTEXT" are changed to call directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-01-07Staging: bcm: Remove typedef for _S_CLASSIFIER_ENTRY and call directly.Kevin McKinney1-16/+16
This patch removes typedef for _S_CLASSIFIER_ENTRY, and changes the name of the struct to bcm_phs_classifier_entry. In addition, any calls to struct "_S_CLASSIFIER_ENTRY" are changed to call directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-01-07Staging: bcm: Remove typedef for _S_CLASSIFIER_TABLE and call directly.Kevin McKinney1-16/+16
This patch removes typedef for _S_CLASSIFIER_TABLE, and changes the name of the struct to bcm_phs_classifier_table. In addition, any calls to struct "S_CLASSIFIER_TABLE" are changed to call directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-01-07Staging: bcm: Remove typedef for _S_SERVICEFLOW_ENTRY and call directly.Kevin McKinney1-12/+12
This patch removes typedef for _S_SERVICEFLOW_ENTRY, and changes the name of the struct to bcm_phs_entry. In addition, any calls to struct "_S_SERVICEFLOW_ENTRY" are changed to call directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-01-07Staging: bcm: Remove typedef for _S_SERVICEFLOW_TABLE and call directly.Kevin McKinney1-7/+7
This patch removes typedef for _S_SERVICEFLOW_TABLE, and changes the name of the struct to bcm_phs_table. In addition, any calls to struct "S_SERVICEFLOW_TABLE" are changed to call directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2013-01-07Staging: bcm: Remove typedef for _PHS_DEVICE_EXTENSION and call directly.Kevin McKinney1-10/+10
This patch removes typedef for _PHS_DEVICE_EXTENSION, and changes the name of the struct to bcm_phs_extension. In addition, any calls to struct "PHS_DEVICE_EXTENSION, or *PPHS_DEVICE_EXTENSION;" are changed to call directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2012-08-17staging: bcm: Fix typo in drivers/bcmMasanari Iida1-2/+2
Correct spelling typo in drivers/bcm Signed-off-by: Masanari Iida <standby24x7@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2012-06-13staging bcm: spelling of suppress in codePeter Meerwald1-10/+10
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2012-06-13staging bcm: spelling of suppress in commentsPeter Meerwald1-5/+5
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2012-06-04Staging: bcm: Remove typedef for _MINI_ADAPTER and call directly.Kevin McKinney1-19/+19
This patch removes typedef for _MINI_ADAPTER, changes the name of the struct from _MINI_ADAPTER to bcm_mini_adapter. In addition, any calls to the following typedefs "MINI_ADAPTER, *PMINI_ADAPTER" are changed to call the struct directly. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2011-06-28staging: Fix recieve/receive typosJoe Perches1-12/+12
Just spelling fixes. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-11-01beceem: make local functions staticStephen Hemminger1-13/+59
Use namespace tool from kernel scripts to identify dead code and functions that should be static. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
2010-11-01beceem: remove unused code to dump headerStephen Hemminger1-31/+0
Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
2010-11-01beceem: remove OS wrapper libraryStephen Hemminger1-88/+43
Use native kernel functions for kmalloc/kfree directly Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
2010-10-05staging/bcm: add sparse annotationsArnd Bergmann1-1/+1
This marks up the code where sparse complains in most cases. Most of the changes are in the ioctl handling code, which gets __user annotations, finding one unchecked user access. The rest is mostly about marking functions static when they are only used in one file. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2010-09-08staging: Beeceem USB Wimax driverStephen Hemminger1-0/+1641
The Sprint 4G network uses a Wimax dongle with Beecem chipset. The driver is typical of out of tree drivers, but maybe useful for people, and the hardware is readily available. Here is a staging ready version (i.e warts and all) 0. Started with Rel_5.2.7.3P1_USB from Sprint4GDeveloperPack-1.1 1. Consolidated files in staging 2. Remove Dos cr/lf 3. Remove unnecessary ioctl from usbbcm_fops Applied patches that were in the developer pack, surprising there were ones for 2.6.35 already. This is compile tested only, see TODO for what still needs to be done. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>