- 16 Jul, 2017 40 commits
-
-
Simon Sandström authored
Fixes checkpatch.pl warnings "spaces preferred around that <operator>". Signed-off-by:
Simon Sandström <simon@nikanor.nu> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Janusz Lisiecki authored
This patch fixes the following Sparse warnings in ks_wlan_net.c: drivers/staging/ks7010/ks_wlan_net.c:1359:24: warning: cast to restricted __le16 link_ap_info_t structure field 'capability' has native order and is used everywhere in the code in such way (i.e get_ap_information, get_current_ap). Both sides of assignment are u16 (native order) so 'le16_to_cpu' is not needed and wrong. Signed-off-by:
Janusz Lisiecki <janusz.lisiecki@gmail.com> Signed-off-by:
-
Lukasz Faber authored
Fixes the following checkpatch errors: - open brace '{' following enum go on the same line - space required after that close brace '}' Signed-off-by:Lukasz Faber <faber@agh.edu.pl> Signed-off-by:
-
Arnd Bergmann authored
I ran into a stack frame size warning in code that hasn't changed much recently, no idea why I didn't spot this earlier despite build many thousand randconfigs. drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c: In function 'ieee80211_rx_mgt_rsl': drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:2563:1: error: the frame size of 1676 bytes is larger than 1024 bytes [-Werror=frame-larger-than=] We have a couple of other copies of the same function in the kernel, and they simply don't put a gigantic structure on the stack, so I'm changing this copy to do the same as the others, replacing the stack variable with a GFP_ATOMIC allocation. Signed-off-by:
Arnd Bergmann <arnd@arndb.de> Signed-off-by:
-
Colin Ian King authored
Don't populate array gamma_par_mask on the stack but instead make it static. Makes the object code smaller by 148 bytes: Before: text data bss dec hex filename 2993 1104 0 4097 1001 drivers/staging/fbtft/fb_st7789v.o After: text data bss dec hex filename 2757 1192 0 3949 f6d drivers/staging/fbtft/fb_st7789v.o Signed-off-by:
Colin Ian King <colin.king@canonical.com> Signed-off-by:
-
Rui Teng authored
Comparing two user space addresses to avoid sparse error: drivers/staging//lustre/lnet/selftest/conrpc.c:490:30: error: incompatible types in comparison expression (different address spaces) Signed-off-by:
Rui Teng <rui.teng@linux.vnet.ibm.com> Acked-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
Bincy K Philip authored
Trivial fix for Line over 80 characters Moved the comment to top of the definition Signed-off-by:
Bincy K Philip <bincy_k_philip@yahoo.co.in> Signed-off-by:
-
Gilad Ben-Yossef authored
Remove unnecessary whitespace before a quoted newline in strings. Signed-off-by:
Gilad Ben-Yossef <gilad@benyossef.com> Signed-off-by:
-
Gilad Ben-Yossef authored
Avoid unnecessary line continuation in log function call. Signed-off-by:
-
Gilad Ben-Yossef authored
Re-write predicate to avoid constant comparison. Signed-off-by:
-
Gilad Ben-Yossef authored
Rename aead_req_struct fields from CamelCase to snake_case. Signed-off-by:
-
Gilad Ben-Yossef authored
Turn local vars and function parameters names in CamelCase to snake_case. Signed-off-by:
-
Gilad Ben-Yossef authored
Fix needless casting to unsigned long long in printk for DMA addresses by using proper %pad format. Signed-off-by:
-
Gilad Ben-Yossef authored
struct ssi_aead_ctx has some nested structure defined, resulting in code accessing them to be very unreadable. Move out the nested structure definitions out of the struct and use the change to make the code accessing it more readable and better coding style compliant by shortening lines and properly matching alignment, Signed-off-by:
-
Gilad Ben-Yossef authored
Remove unnecessary parentheses from ccree driver code. Signed-off-by:
-
Amitoj Kaur Chawla authored
Use offset_in_page macro instead of (var & ~PAGE_MASK) The Coccinelle semantic patch used to make this change is as follows: // <smpl> @@ unsigned long p; @@ - p & ~PAGE_MASK + offset_in_page(p) // </smpl> Signed-off-by:
Amitoj Kaur Chawla <amitoj1606@gmail.com> Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Mateusz Nowotyński authored
Use BIT macro instead of left shifting in android/ion/ion.h Signed-off-by:
Mateusz Nowotyński <maxmati4@gmail.com> Signed-off-by:
-
Mateusz Nowotyński authored
Fixes code alignment issue in ion/ion.h Signed-off-by:
-
Mateusz Nowotyński authored
Removes unnecessary blank lines in android/ion/ion_system_heap.c Signed-off-by:
-
Mateusz Nowotyński authored
Converts sizeof(type) to sizeof(variable) in android/ion/ion_cma_heap.c Signed-off-by:
-
Mateusz Nowotyński authored
Removes unnecessary blank lines in android/ion/ion_cma_heap.c Signed-off-by:
-
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:
-
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:
-
