- 03 Jan, 2017 40 commits
-
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: Prefer kernel type 'u16' over 'uint16_t' Signed-off-by:
Saber Rezvani <irsaber@gmail.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Saber Rezvani authored
Fix the checkpatch.pl issue: CHECK: Prefer kernel type 'u8' over 'uint8_t' Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Ian Abbott authored
Fix checkpatch.pl warnings of the form "function definition argument 'foo' should also have an identifier name". Signed-off-by:
-
Piotr Gregor authored
Add names of parameters to function prototypes in comedi PCI. Checkpatch reports now no errors. Signed-off-by:
Piotr Gregor <piotrgregor@rsyncme.org> Reviewed-by:
-
Matthew Smith authored
Fixes "WARNING: Statements should start on a tabstop" from checkpatch.pl Signed-off-by:
Matthew Smith <matthew11235@outlook.com> Signed-off-by:
-
Jaewon Kim authored
Initial Commit 349c9e13 ("gpu: ion: add CMA heap") returns -1 in allocation failure. The returned value is passed up to userspace through ioctl. So user can misunderstand error reason as -EPERM(1) rather than -ENOMEM(12). This patch simply changed this to return -ENOMEM. Signed-off-by:
Jaewon Kim <jaewon31.kim@samsung.com> Acked-by:
Laura Abbott <labbott@redhat.com> Signed-off-by:
-
David Kershner authored
Add the proper error handling to my_device_destroy so it can be send back up the stack for further error reporting. Reported-by:
David Kershner <david.kershner@unisys.com> Reviewed-by:
David Binder <david.binder@unisys.com> Signed-off-by:
-
David Kershner authored
The function my_device_changestate was not returning errors up the stack. Reported-by:
-
David Kershner authored
The function save_crash_message returns an error, don't ignore it, respond appropriately and send it up. Reported-by:
-
David Kershner authored
Add proper error handling to the function my_device_create and propagate the error message up the stack. Reported-by:
-
Bryan Thompson authored
Remove the MAX_BUF define and associated comments. Signed-off-by:
Bryan Thompson <bryan.thompson@unisys.com> Reviewed-by:
Tim Sell <Timothy.Sell@unisys.com> Signed-off-by:
-
David Binder authored
Removes references to visorchipset in the /proc filesystem, and replaces them with references to the /sys filesystem. Also removes reference to the visorchipset driver, and replaces it with a reference to plain visorchipset (since it is no longer a standalone driver). Signed-off-by:
-
Erik Arfvidson authored
This patch removes unused pound defines in controlvmchannel.h. Signed-off-by:
Erik Arfvidson <erik.arfvidson@unisys.com> Signed-off-by:
-
David Binder authored
The 'retry' variable created in handle_command() is statically allocated, and its address is never set to NULL. Therefore conditionals to verify the validity of the retry variable's address are unnecessary. Reported-by:
-
David Binder authored
Control flow is now directed using a switch statement, triggered by the enum crash_obj_type function parameter, instead of a set of conditional statements. Reported-by:
-
Sameer Wadgaonkar authored
This patch shortens a few error message defines by removing ERROR from the define. Reported-by:
Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com> Signed-off-by:
-
Geliang Tang authored
Use rb_entry_safe() instead of container_of() to simplify the code. Signed-off-by:
Geliang Tang <geliangtang@gmail.com> Signed-off-by:
-
Tabrez khan authored
Remove unnecessary braces {} for single while statement. This warning is found using checkpatch.pl. Signed-off-by:Tabrez khan <khan.tabrez21@gmail.com> Signed-off-by:
-
Ben Evans authored
Change the style of lustre_get_wire_obdo and lustre_set_wire_obdo to conform to linux kernel standard. Signed-off-by:
Ben Evans <bevans@cray.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401 Reviewed-on: http://review.whamcloud.com/16917 Reviewed-on: http://review.whamcloud.com/19266Reviewed-by:
James Simmons <uja.ornl@yahoo.com> Reviewed-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
-
Ben Evans authored
Remove all obdo related functions from lustre_idl.h Create lustre_odbo.h. Include where appropriate. Make the functions lustre_get_wire_obdo and lustre_set_wire_obdo to not be inlined functions. Signed-off-by:
-
Quentin Lambert authored
A recent clean-up declared och_flags as a int rather than fmode_t. This lead to the following sparse warning: drivers/staging/lustre/lustre/llite/file.c:106:30: warning: restricted fmode_t degrades to integer This patch fixes this issue. Fixes: 0a120099 ("staging: lustre: cleanup lustre_lib.h") Signed-off-by:
Quentin Lambert <lambert.quentin@gmail.com> Acked-by:
-
James Simmons authored
It's not used anywhere outside of brw_test.c file. Highlighted by sparse. Signed-off-by:
-
Jinshan Xiong authored
Cleanup everything if an OSC is being evicted. Group lock is not well supported yet. Signed-off-by:
Jinshan Xiong <jinshan.xiong@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6271 Reviewed-on: http://review.whamcloud.com/14989Reviewed-by:
Bobi Jam <bobijam@hotmail.com> Reviewed-by:
-
Oleg Drokin authored
This avoids having an extern definition in a C file which is bad, and also silences sparse complaint as well. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Reviewed-by:
-
Oleg Drokin authored
Move it to obd.h, so that it's included from both the users and the actual definition, making sure they never get out of sync. This also silences a sparse warning. Signed-off-by:
-
Oleg Drokin authored
Make the declaration in a header, not as an extern in a C file, that is frowned upon. This also makes sparse a little bit more happy. Signed-off-by:
-
Oleg Drokin authored
This avoids frowned upon extern in the C file, and also shuts down a sparse warning of drivers/staging/lustre/lustre/osc/osc_dev.c:55:22: warning: symbol 'osc_caches' was not declared. Should it be static? Signed-off-by:
-
Oleg Drokin authored
It's not used anywhere outside of this file. Highlighted by sparse. Signed-off-by:
-
Oleg Drokin authored
It's not used anywhere out of this file. Highlighted by sparse. Signed-off-by:
-
