- 16 Jul, 2017 40 commits
-
-
Gilad Ben-Yossef authored
Turn local vars and function parameters names in CamelCase to snake_case. Signed-off-by:
Gilad Ben-Yossef <gilad@benyossef.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
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:
-
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:
-
