- 16 Jul, 2017 40 commits
-
-
Kamal Heib authored
Fix the following sparse warning: symbol '__ion_add_cma_heaps' was not declared. Should it be static? Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by:
Kamal Heib <kamalheib1@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Ioana Radulescu authored
All possible values of the switch statement are explicitly handled, so there's no need to have a default branch. Signed-off-by:
Ioana Radulescu <ruxandra.radulescu@nxp.com> Acked-by:
Bogdan Purcareata <bogdan.purcareata@nxp.com> Signed-off-by:
-
Erik Arfvidson authored
This patch fixes visorbus_main.c improper function brackets. for the following functions: partition_handle_show partition_guid_show partition_name_show channel_addr_show channel_bytes_show channel_id_show Signed-off-by:
Erik Arfvidson <erik.arfvidson@gmail.com> Signed-off-by:
David Kershner <david.kershner@unisys.com> Signed-off-by:
-
Zachary Dremann authored
As reported by checkpatch.pl, replace hardcoded uses of the current function's name in format strings with uses of __func__. Signed-off-by:
Zachary Dremann <dremann@gmail.com> Signed-off-by:
-
Zachary Dremann authored
Several structure initializations all had their members indented with 2 tabs instead of one. Remove the extra tabs. Signed-off-by:
-
Steve Wilson authored
Removed extra tabs in visorbus_get_device_by_id function. Signed-off-by:
Steve Wilson <steven.d.wilson@unisys.com> Signed-off-by:
-
David Binder authored
Removes unnecessary blank lines to create a more uniform coding style. Signed-off-by:
David Binder <david.binder@unisys.com> Signed-off-by:
-
David Binder authored
Adjusts presence of blank lines to create a more uniform coding style. Signed-off-by:
-
David Binder authored
Adjusts spacing and removes unnecessary blank lines to create a more uniform coding style. Signed-off-by:
-
David Binder authored
Adjusts spacing and removes unnecessary blank lines to create a more uniform coding style. Signed-off-by:
-
David Binder authored
Refactors existing static function comments to increase code readability. Signed-off-by:
-
David Binder authored
Renames MYDRVNAME to better fit with the surrounding driver namespace. Signed-off-by:
-
David Binder authored
Removes unused MYDRVNAME define. Signed-off-by:
-
David Binder authored
Fixes minor spelling mistake: stired -> stored. Signed-off-by:
-
Sameer Wadgaonkar authored
Removed comments from the right side of the lines. Signed-off-by:
Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com> 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:
-
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:
-
