- 16 Jul, 2017 39 commits
-
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Signed-off-by:
Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com> Signed-off-by:
David Kershner <david.kershner@unisys.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Added kernel-doc like comments and cleaned up the inline comments in the structures. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Added kernel-doc like comments and cleaned up the inline comments in the structures. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Signed-off-by:
-
David Kershner authored
Several MACROS were not in use by the code, get rid of them. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Added kernel-doc like comments and cleaned up the inline comments in the structures. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Added kernel-doc like comments and cleaned up the inline comments in the structures. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of lines and fixed spacing in comments. Added kernel-doc like comments and cleaned up the inline comments in the structures. Signed-off-by:
-
Dmitriy Cherkasov authored
After removing commented out code, ksocknal_csum() becomes a useless wrapper for crc32_le(). Remove it, and instead call crc32_le() directly. Fixes the following checkpatch warning: WARNING: space prohibited before semicolon Signed-off-by:
Dmitriy Cherkasov <dmitriy@oss-tech.org> Signed-off-by:
-
Thomas Gardner authored
The below warning is resolved by removing the cpu_to_le32() call. This call was redundant; vdm_run_state_machine() ensures that SVDM responses have the correct endianness before sending. typec/tcpm.c:1019:49: warning: incorrect type in assignment (different base types) typec/tcpm.c:1019:49: expected unsigned int [unsigned] [usertype] <noident> typec/tcpm.c:1019:49: got restricted __le32 [usertype] <noident> Signed-off-by:
Thomas Gardner <tmg@fastmail.com> Reviewed-by:
Guenter Roeck <linux@roeck-us.net> Signed-off-by:
-
Shreeya Patel authored
Move constant to the right of a relational operator. This coding style is more common for the kernel code. Problem found by checkpatch. Signed-off-by:
Shreeya Patel <shreeya.patel23498@gmail.com> Signed-off-by:
-
Muhammad Falak R Wani authored
Replace the explicit "if (val != NULL)" with if (val). Signed-off-by:
Muhammad Falak R Wani <falakreyaz@gmail.com> Signed-off-by:
-
Arushi Singhal authored
Fix checkpatch issues: "CHECK: Alignment should match open parenthesis". Signed-off-by:
Arushi Singhal <arushisinghal19971997@gmail.com> Signed-off-by:
-
Mitchell Tasman authored
Adjust formatting of several comments to keep line length within the 80 column limit preferred by the Linux kernel coding style. Signed-off-by:
Mitchell Tasman <tasman@leaflabs.com> Acked-by:
Viresh Kumar <viresh.kumar@linaro.org> Signed-off-by:
-
Aleksey Rybalkin authored
According to checkpatch warning, block comments should align the * on each line. Also, preferred style for multi-line comments is starting the comment text after the second *. Signed-off-by:
Aleksey Rybalkin <aleksey@rybalkin.org> Acked-by:
Johan Hovold <johan@kernel.org> Reviewed-by:
-
Ilia Sergachev authored
txq_count receives a value from wilc_wlan_handle_txq() and therefore should be u32 found using sparse: drivers/staging/wilc1000/linux_wlan.c:306:58: warning: incorrect type in argument 2 (different signedness) Signed-off-by:
Ilia Sergachev <ilia.sergachev@unibas.ch> Signed-off-by:
-
Dmitriy Cherkasov authored
Add missing parameter name to fix the following checkpatch.pl warning: WARNING: function definition argument 'struct device *' should also have an identifier name Signed-off-by:
-
Colin Ian King authored
Trivial fix to typos in netdev_err error messages. I should have spotted this in a previous round of spelling mistake checks on this driver but this one slipped through. Signed-off-by:
Colin Ian King <colin.king@canonical.com> Signed-off-by:
-
Joe Perches authored
All uses of refresh_scan() have a constant 1 as the second argument. Remove it and neaten the function declaration. Miscellanea: o Remove overly deep indentation by using continue; o Remove unnecessary test of the address of a static array as it's always non-zero o Rename and use proper type for the first void * argument as it's always the struct wilc_priv * o Move variables into loop where used Signed-off-by:
Joe Perches <joe@perches.com> Signed-off-by:
-
Simon Sandström authored
Fixes checkpatch.pl alignment warnings. Signed-off-by:
Simon Sandström <simon@nikanor.nu> Signed-off-by:
-
Simon Sandström authored
Fixes checkpatch.pl alignment warnings. Signed-off-by:
-
Simon Sandström authored
Fixes checkpatch.pl alignment warnings. Signed-off-by:
-
Simon Sandström authored
Fixes checkpatch.pl alignment warnings. Signed-off-by:
-
Simon Sandström authored
Fixes checkpatch.pl alignment warnings. Signed-off-by:
-
Simon Sandström authored
Fixes checkpatch.pl alignment warnings. Signed-off-by:
-
Simon Sandström authored
Fixes checkpatch.pl alignment warnings. Signed-off-by:
-
Gilad Ben-Yossef authored
The ccree driver had its own FIPS support, complete with a test harness comparable to crypto testmgr and an implementation which disables crypto functionality on FIPS test error detection, either in Linux or from TEE. This patch removes the duplication, while reimplementing the handling of TEE reported FIPS errors according to the kernel policy of inducing a panic in such an event. Signed-off-by:
Gilad Ben-Yossef <gilad@benyossef.com> Signed-off-by:
-
Tyler Olivieri authored
Patch to fix following checkpatch error: ERROR: switch and case should be at the same indent Signed-off-by:
Tyler Olivieri <sleepingzucchini@gmail.com> Acked-by:
-
Tyler Olivieri authored
Patch to fix following checkpatch warning: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable Signed-off-by:
-
Tyler Olivieri authored
Patch to fix following checkpatch error: ERROR: do not use assignment in if condition Signed-off-by:
-
Tyler Olivieri authored
Patch to fix checkpatch errors: ERROR: that open brace { should be on the previous line ERROR: open brace '{' following function declarations go on the next line Signed-off-by: -
Tyler Olivieri authored
Patch to remove checkpatch warning: WARNING: Statements terminations use 1 semicolon Signed-off-by:
-
Gustavo A. R. Silva authored
Fix the following checkpatch warning: CHECK: Prefer kmalloc(sizeof(*buff_mgr_handle)...) over kmalloc(sizeof(struct buff_mgr_handle)...) Signed-off-by:
Gustavo A. R. Silva <garsilva@embeddedor.com> Acked-by:
-
Gustavo A. R. Silva authored
The assignment operator implicitly converts a void pointer to the type of the pointer it is assigned to. This issue was detected using Coccinelle and the following semantic patch: @@ expression * e; expression arg1, arg2; type T; @@ - e=(T*) + e= kmalloc(arg1, arg2); Signed-off-by:
-
Karthik Tummala authored
Fixed following checkpatch.pl warning: WARNING: Prefer using '"%s...", __func__' to using the function's name, in a string It is prefered to use '%s & __func__' instead of function name for logging. Signed-off-by:
Karthik Tummala <karthik@techveda.org> Acked-by:
-
- 15 Jul, 2017 1 commit
-
-
Linus Torvalds authored
-
