- 20 Mar, 2019 17 commits
-
-
George Hilliard authored
When waiting on completions, use the _io variant so the caller is charged as using I/O. This should have no effect on the module's functionality, only improve CPU accounting. Signed-off-by:
George Hilliard <thirtythreeforty@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
George Hilliard authored
The driver previously grabbed the SD pins for itself, ignoring the pin controller. Remove this, and allow the pinctrl subsystem to set up the pins using the device tree mappings. This allows this driver to work on related devices that have a different pin controller mapping, such as the MT7688. The hardcoded bit index was incorrect on that device. The driver now needs a pin controller reference in its device tree node: sdhci: /* ... */ { compatible = "ralink,mt7620-sdhci"; pinctrl-names = "default"; pinctrl-0 = <&sdhci_pins>; // ... }; This change could break SD controller functionality on existing devices whose device trees do not specify a pin controller and state for the SD node. Signed-off-by: -
George Hilliard authored
These comments don't contain useful code or alternate implementation ideas. Remove them. Signed-off-by:
-
George Hilliard authored
The compiler complains: drivers/staging/mt7621-mmc/dbg.c: In function ‘msdc_debug_proc_write’: drivers/staging/mt7621-mmc/dbg.c:237:12: warning: unused variable ‘size’ [-Wunused-variable] int mode, size; ^~~~ drivers/staging/mt7621-mmc/dbg.c:237:6: warning: unused variable ‘mode’ [-Wunused-variable] int mode, size; ^~~~ Remove these declarations. Signed-off-by: -
Nishka Dasgupta authored
Remove return variable and return return value directly. Issue found by Coccinelle using ret.cocci. Signed-off-by:
Nishka Dasgupta <nishka.dasgupta@yahoo.com> Acked-by:
Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
Wentao Cai authored
Replace '--help--' in Kconfig and indent the subsequent text to silence checkpatch.pl warning: WARNING: prefer 'help' over '---help---' for new help texts Signed-off-by:
Wentao Cai <etsai042@gmail.com> Signed-off-by:
-
Wentao Cai authored
Replace '--help--' in Kconfig and indent the subsequent text to silence checkpatch.pl warning: WARNING: prefer 'help' over '---help---' for new help texts Signed-off-by:
-
Wentao Cai authored
Replace '--help--' in Kconfig and indent the subsequent text to silence checkpatch.pl warning: WARNING: prefer 'help' over '---help---' for new help texts Signed-off-by:
-
Vatsala Narang authored
Edit CamelCase function name Issue found by checkpatch.pl Signed-off-by:
Vatsala Narang <vatsalanarang@gmail.com> Signed-off-by:
-
Sanjana Sanikommu authored
Fix the spacce around else. Issue found by checkpatch.pl ERROR: spaces required before that open brace '{' ERROR: spaces required before that close brace '}' Signed-off-by:Sanjana Sanikommu <sanjana99reddy99@gmail.com> Signed-off-by:
-
Sanjana Sanikommu authored
Replace NULL comparisons in the file. Issue found by checkpatch.pl Signed-off-by:
-
Sanjana Sanikommu authored
Remove unnecessary blank lines. Issue found by checkpatch.pl Signed-off-by:
-
Sanjana Sanikommu authored
Add spaces around the operators. Issue found by checkpatch.pl CHECK: spaces preferred around that '+' CHECK: spaces preferred around that '*' Signed-off-by:
-
Sanjana Sanikommu authored
Ajdust alignment to match open paranthesis. Issue found by checkpatch.pl CHECK: Alignment should match open paranthesis. Signed-off-by:
-
Nishka Dasgupta authored
Remove return variable and return the return value directly. Issue suggested by ret.cocci. Signed-off-by:
-
Nishka Dasgupta authored
Remove parentheses around variable. Signed-off-by:
-
Julian Merida authored
Fix all checkpatch issues: "CHECK: Alignment should match open parenthesis" Signed-off-by:
Julian Merida <julianmr97@gmail.com> Reviewed-by:
Gao Xiang <gaoxiang25@huawei.com> Signed-off-by:
-
- 19 Mar, 2019 10 commits
-
-
Bhagyashri Dighole authored
Convert a ternary statement into a if statement which is detected while resolving "WARNING: line over 80 characters". Use BIT() macro instead manually left shifting. Signed-off-by:
Bhagyashri Dighole <digholebhagyashri@gmail.com> Signed-off-by:
-
Emanuel Bennici authored
Improve Code readability by adding Tabs and Spaces after #define Signed-off-by:
Emanuel Bennici <benniciemanuel78@gmail.com> Signed-off-by:
-
Sanjana Sanikommu authored
Fix the check as per Linux kernel style for use of spaces. Issue found by checkpatch.pl CHECK:spaces preffered around that '<<' Signed-off-by:
-
Madhumitha Prabakaran authored
Include missing NULL check for kmalloc in function rtw_init_evt_priv. Signed-off-by:
Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by:
-
Madhumitha Prabakaran authored
Replace NULL comparison with ! in function rtw_init_cmd_priv, to maintain Linux kernel coding style. Signed-off-by:
-
Madhumitha Prabakaran authored
Change values for standard error codes in functions rtw_init_cmd_priv and rtw_init_evt_priv, as _SUCCESS should be 0 and _FAIL should be -ENOMEM. Also, change the values in corresponding call sites of the functions. Signed-off-by:
-
Madhumitha Prabakaran authored
Remove unnecessary local variable 'res' in function and replace the value directly in the return of the function. Signed-off-by:
-
Sergey Senozhatsky authored
cpu_to_le16() is capable enough to detect __builtin_constant_p() and to use an appropriate compile time ___constant_swahbXX() function. So we can use cpu_to_le16() instead of __constant_cpu_to_le16(). Signed-off-by:
Sergey Senozhatsky <sergey.senozhatsky@gmail.com> Signed-off-by:
-
Colin Ian King authored
The range check on auth_type is redundant as there is a prior check on the auth_type values and the only way the block is entered is if auth_type is one of TYPE_PMK1, TYPE_GMK1 and TYPE_GMK2. Hence the auth_type check can be removed. Detected by static analysis with cppcheck. Signed-off-by:
Colin Ian King <colin.king@canonical.com> Acked-by:
Jeremy Sowden <jeremy@azazel.net> Signed-off-by:
-
Michael Straube authored
All defined P2P* in wifi.h are unused in the driver code, so remove them. Signed-off-by:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
-
- 18 Mar, 2019 13 commits
-
-
Sanjana Sanikommu authored
Adjust indentation for the condition statement. Issue found by checkpatch.pl WARNING: suspect code indent for condition statements(8,24) Signed-off-by:
-
Sanjana Sanikommu authored
Replace NULL comparison with '!' in the file rtw_mlme.c Issue found by checkpatch.pl semantic patch results for rtw_mlme.c CHECK:Comparision to NULL could be written "!candidate" Signed-off-by:
-
Sanjana Sanikommu authored
Concatenate a multi-line string constant into a single line.Issue found by checkpatch.pl. WARNING:quoted string split across line. Signed-off-by:
-
Branden Bonaby authored
Align function to prevent it from going over the 80 character a line limit for readability. WARNING: line over 80 characters Signed-off-by:
Branden Bonaby <brandonbonaby94@gmail.com> Signed-off-by:
-
Jules Irenge authored
Replace printk with pr_info to solve checkpatch.pl warning: " WARNING: printk() should include KERN_<LEVEL> facility level" Signed-off-by:
Jules Irenge <jbi.octave@gmail.com> Signed-off-by:
-
Sanjana Sanikommu authored
Move the constant to the right side of comparision. Issue found by checkpatch.pl semantic patch results for rtw_ap.c WARNING:Comparision should place the constant on the right side of the test. Signed-off-by:
-
Bhanusree Pola authored
Avoid useless parentheses to the right hand side of an assignment. Issue found using coccinelle. The semantic patch that fixes the problem is as follows // <smpl> @r1 disable paren@ expression value,e; @@ ( - value = (e) + value = e ) @r2 depends on r1@ expression value,e; constant c; @@ ( - value = (e == c) + value = (e == c) | - value = (e <= c) + value = (e <= c) | - value = (e >= c) + value = (e >= c) | - value = (e != c) + value = (e != c) ) // </smpl> Signed-off-by:
Bhanusree Pola <bhanusreemahesh@gmail.com> Signed-off-by:
-
Emanuel Bennici authored
The Functions `void rtw_surveydone_event_callback`, `void rtw_indicate_connect`, `void rtw_stadel_event_callback` and `void _rtw_join_timeout_handler` are using the Macro CONFIG_INTEL_WIDI which doesn't exists anymore. Signed-off-by:
-
Emanuel Bennici authored
The Functions `static int rtw_widi_set()` and `static int rtw_widi_set_probe_request()` part of dead code due to CONFIG_INTEL_WIDI. The fix removes the deadcode. Signed-off-by:
-
Emanuel Bennici authored
Remove Dead Code in `static int rtw_wps_start`, `struct iw_handler rtw_private_handler[]` and `struct iw_priv_args rtw_private_args[]` since they use the Macro CONFIG_INTEL_WIDI witch doesn't exists any more. Signed-off-by:
-
Emanuel Bennici authored
The Functions `u8 rtw_init_drv_sw` and `u8 rtw_free_drv_sw` uses the Macro CONFIG_INTEL_WIDI witch doesn't exists anymore. This Patch removes those redundant Code Lines. Signed-off-by:
-
Emanuel Bennici authored
`uint loadparam` uses the Macro CONFIG_INTEL_WIDI witch doesn't exists anymore. Signed-off-by:
-
Emanuel Bennici authored
The `case INTEl_WIDI_WK_CID`-Statement in Function `u8 rtw_drvextra_cmd_hdl` wouldn't be used because of CONFIG_INTEL_WIDI. Signed-off-by:
-
