- 06 Mar, 2018 40 commits
-
-
Ajay Singh authored
Fix 'line over 80 character' issues found by checkpatch.pl script by use of temporary variable and avoided leading tab. Signed-off-by:
Ajay Singh <ajay.kathat@microchip.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Ajay Singh authored
Fix 'Avoid camelCase' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 character' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 characters' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 characters' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'Avoid camelCase' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 characters' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 char' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 character' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 char' issues found by checkpatch.pl script in handle_cfg_param(). Rename variables and used temporary variables to fix the line over 80 characters issue. Signed-off-by:
-
Ajay Singh authored
Fix few smatch warning related to 'warn: always true condition'. handle_cfg_param() warn: always true condition '(cfg_param_attr->auth_timeout < 65536) => (0-u16max < 65536)' handle_cfg_param() warn: always true condition '(cfg_param_attr->rts_threshold < 65536) => (0-u16max < 65536)' handle_cfg_param() warn: always true condition '(cfg_param_attr->beacon_interval < 65536) => (0-u16max < 65536)' handle_cfg_param() warn: always true condition '(cfg_param_attr->site_survey_scan_time < 65536) => (0-u16max < 65536)' handle_cfg_param() warn: always true condition '(cfg_param_attr->active_scan_time < 65536) => (0-u16max < 65536)' handle_cfg_param() warn: always true condition '(cfg_param_attr->passive_scan_time < 65536) => (0-u16max < 65536)' Signed-off-by:
-
Ajay Singh authored
Fix "Avoid camelCase" issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Refactor wilc_wlan_handle_rxq() to fix line over 80 character issue found by checkpatch.pl script. Added a new function to split 'wilc_wlan_handle_rxq' function code. Signed-off-by:
-
Ajay Singh authored
Fix 'line over 80 characters' issue found by checkpatch.pl script. Signed-off-by:
-
Ajay Singh authored
Fix 'Alignment should match open parenthesis' issue found by checkpatch.pl script. Reduce the leading tab, to make space for open parenthesis match. Signed-off-by:
-
Quytelda Kahja authored
Indent the parameters for a function call that extends past 80 characters. Signed-off-by:
Quytelda Kahja <quytelda@tamalin.org> Signed-off-by:
-
Quytelda Kahja authored
Use a blank line after components_show() function declaration. Signed-off-by:
-
Quytelda Kahja authored
Replace literal bytestring with CIPHER_ID_WPA_WEP40 constant. Signed-off-by:
-
Dan Carpenter authored
These pointers can't be NULL so I have removed the checks. The checking was sort of problematic as well because it didn't make sense. In _nbu2ss_read_request_data() the && should have been ||. In nbu2ss_gad_get_frame() we know that "pgadget" is non-NULL and "udc" is an offset from "pgadget" so it can't be NULL. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Eric Anholt authored
It's not used in-tree, or in the downstream tree. Signed-off-by:
Eric Anholt <eric@anholt.net> Signed-off-by:
-
Santha Meena Ramamoorthy authored
Replace NULL comparison of the variable with the variable name or !variable to conform to the Linux kernel coding style. Issue found using checkpatch. Signed-off-by:
Santha Meena Ramamoorthy <santhameena13@gmail.com> Signed-off-by:
-
Santha Meena Ramamoorthy authored
Place constants on the right side of the test during comparisons to conform to the Linux kernel coding style. Issue found using checkpatch. Signed-off-by:
Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
Arushi Singhal authored
Use eth_zero_addr to assign zero address to the given address array instead of memset when the second argument in memset is address of zero. Coccinelle was used to do the replacement and add the header file linux/etherdevice.h if not already present. The Coccinelle semantic patch that makes this change is as follows: @header@ @@ #include <linux/etherdevice.h> @r1@ expression e; @@ -memset(e,0x00,ETH_ALEN); +eth_zero_addr(e); @includeheader depends on r1 && !header@ @@ + #include <linux/etherdevice.h> #include <...> Signed-off-by:
Arushi Singhal <arushisinghal19971997@gmail.com> Signed-off-by:
-
Arushi Singhal authored
Remove unnecessary semicolon using semicolon.cocci Coccinelle script. Signed-off-by:
-
Dafna Hirschfeld authored
Add spaces around arithmetic and bitwise operators to improve readability of the code. Issues found with checkpatch.pl Signed-off-by:
Dafna Hirschfeld <dafna3@gmail.com> Acked-by:
-
Dafna Hirschfeld authored
Remove unnecessary parentheses between 'address-of' operators and a struct members. Issues found with checkpatch.pl Signed-off-by:
-
Dafna Hirschfeld authored
Fix multiple blank lines and blank lines after braces. Issues found with checkpatch.pl Signed-off-by:
-
Arushi Singhal authored
Remove unnecessary parentheses around variables to conform to the Linux kernel coding style. Issue found using checkpatch. Signed-off-by:
-
Arushi Singhal authored
Add spaces around '|' to conform to the Linux kernel coding style. Issue found using checkpatch. CHECK: spaces preferred around that '|'. Signed-off-by:
-
Arushi Singhal authored
Add spaces around arithmetic operator '+', to conform to the Linux kernel coding style. Issue found using checkpatch. Signed-off-by:
-
Arushi Singhal authored
Remove extra blank line(s) to conform to the Linux kernel coding style. Issue found using checkpatch. Signed-off-by:
-
Arushi Singhal authored
Delete/Add tabs and spaces to align the code to fix the checkpatch issue: "CHECK: Alignment should match open parenthesis". Signed-off-by:
-
Arushi Singhal authored
Replace a mix of tabs and spaces indentation by tabs only. Fixed checkpatch warning "Statements should start on a tabstop". Signed-off-by:
-
Nishka Dasgupta authored
Change 'unsigned *' to 'unsigned int *'. Issue found with checkpatch. Signed-off-by:
Nishka Dasgupta <nishka.dasgupta_ug18@ashoka.edu.in> Acked-by:
-
Nishka Dasgupta authored
Format multiline comment by moving '*/' to a new line. Issue found with checkpatch. Signed-off-by:
-
Nishka Dasgupta authored
Add blank line after declaration. Issue found with checkpatch. Signed-off-by:
-
Arushi Singhal authored
Remove non-essential semicolon after 'else' and 'switch' statements. Issue found using semicolon.cocci Coccinelle script. Signed-off-by:
-
Santha Meena Ramamoorthy authored
Remove unneeded semicolon after 'while' and 'switch' statements. Issue found using semicolon.cocci Coccinelle script. Signed-off-by:
-
Santha Meena Ramamoorthy authored
Remove condition where if and else branch are identical. Issue found using cond_no_effect.cocci Coccinelle script. Signed-off-by:
-
Rinkle Jain authored
Replace line ending with '(' with function parameters to resolve style issue found by checkpath. Signed-off-by:Rinkle Jain <rinklejain96@gmail.com> Acked-by:
-
