- 25 Sep, 2013 40 commits
-
-
Wei Yongjun authored
usb_free_urb(NULL) is safe and this check is not required. Signed-off-by:
Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Peter Senna Tschudin authored
Convert 0 to false and 1 to true when assigning values to bool variables. Inspired by commit 3db1cd5c. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @@ bool b; @@ ( -b = 0 +b = false | -b = 1 +b = true ) Signed-off-by:
Peter Senna Tschudin <peter.senna@gmail.com> Signed-off-by:
-
Hanjun Guo authored
In acpi_bus_register_driver(), there is an if (acpi_disabled) check, so the if(acpi_disabled) before it is reduplicate, remove it. Signed-off-by:
Hanjun Guo <hanjun.guo@linaro.org> Signed-off-by:
-
Sachin Kamat authored
Driver core sets driver data to NULL upon failure or remove. Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by:
-
Sachin Kamat authored
Driver core sets driver data to NULL upon failure or remove. Signed-off-by:
-
Sachin Kamat authored
Driver core sets driver data to NULL upon failure or remove. Signed-off-by:
-
Sachin Kamat authored
Driver core sets driver data to NULL upon failure or remove. Signed-off-by:
-
Sachin Kamat authored
Driver core sets driver data to NULL upon failure or remove. Signed-off-by:
-
Iker Pedrosa authored
White space deleted before semicolons. Signed-off-by:
Iker Pedrosa <ikerpedrosam@gmail.com> Signed-off-by:
-
Iker Pedrosa authored
Erased all the errors given by checkpatch stating trailing whitespace. Signed-off-by:
-
Iker Pedrosa authored
Second of the patches that fixes the lines over 80 characters in phy_calibration.c Signed-off-by:
-
Iker Pedrosa authored
First of the patches that fixes the lines over 80 characters in phy_calibration.c Signed-off-by:
-
Iker Pedrosa authored
Deleted declaration of external functions that weren't used on this driver. Signed-off-by:
-
Iker Pedrosa authored
Prototype of two functions added to the header to avoid the use of extern. Signed-off-by:
-
Iker Pedrosa authored
The following function declarations have been removed because they aren't implemented. Signed-off-by:
-
Dominik Paulus authored
getaddrinfo() leaves the order of the returned addrinfo structs unspecified. On systems with bindv6only disabled (this is the default), PF_INET6 sockets bind to IPv4, too. Thus, IPv6 support in usbipd was broken when getaddrinfo returned first IPv4 and then IPv6 addrinfos, as the IPv6 bind failed with EADDRINUSE. This patch uses seperate sockets for IPv4 and IPv6 and sets IPV6_V6ONLY on all IPv6 sockets. Two command line arguments, -4 and -6 were added to manually select the socket family. Signed-off-by:
Tobias Polzer <tobias.polzer@fau.de> Signed-off-by:
Dominik Paulus <dominik.paulus@fau.de> Signed-off-by:
-
Tobias Polzer authored
One missing include was added. Signed-off-by:
-
Dominik Paulus authored
With --with-tcp-wrappers=no specified, the build system reset LIBS to the empty string and thus fails to link against libsysfs. Signed-off-by:
-
Elad Wexler authored
Coding style issues Signed-off-by:
Elad Wexler <eladwexler77@yahoo.com> Signed-off-by:
-
navin patidar authored
vhci_hcd is a virtual usb host controller, so no need to check for dma. Signed-off-by:
navin patidar <navinp@cdac.in> Signed-off-by:
-
navin patidar authored
ret == 0 means success, anything else is failure. Signed-off-by:
-
navin patidar authored
ret == 0 means success, anything else is failure. Signed-off-by:
-
Xie XiuQi authored
Remove unnecessary work pending test before calling schedule_work(). It has been tested in queue_work_on() already. No functional changed. Signed-off-by:
Xie XiuQi <xiexiuqi@huawei.com> Cc: Jens Frederich <jfrederich@gmail.com> Cc: Daniel Drake <dsd@laptop.org> Cc: Jon Nettleton <jon.nettleton@gmail.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Reviewed-by:
Tejun Heo <tj@kernel.org> Signed-off-by:
Andrew Morton <akpm@linux-foundation.org> Signed-off-by:
-
Peter Senna Tschudin authored
The variable ownbit_flag is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:
-
Peter Senna Tschudin authored
The variables bMultiBand and bKeyTableFull are only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:
-
Peter Senna Tschudin authored
The variable Trigger is only assigned the values true and false. Change its type to bool. The simplified semantic patch that find this problem is as follows (http://coccinelle.lip6.fr/): @exists@ type T; identifier b; @@ - T + bool b = ...; ... when any b = \(true\|false\) Signed-off-by:
Pavel Machek <pavel@ucw.cz> Signed-off-by:
-
Shalin Mehta authored
The function implementations of the PHSModule didn't match with the declaration. The functions are static in the declaration but in the implemntation they are non-static Signed-off-by:
Shalin Mehta <shalinmehta85@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
with struct vnt_tx_buffer and attach vnt_tx_fifo_head to pTX_Buffer structure. The the calling pbyTxBufferAddr points to the same address as vnt_tx_buffer->fifo_head and is still in use. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
In code it is referred to as fifo to renamed to typedef struct tagSTxBufHead -> struct vnt_tx_fifo_head This is only needed by rxtc.c so moved to rxtx.h Signed-off-by:
-
Masanari Iida authored
Correct spelling typo in comment Signed-off-by:
Masanari Iida <standby24x7@gmail.com> Signed-off-by:
-
Masanari Iida authored
Correct spelling typo in staging/vt6655 and staging/vt6656 Signed-off-by:
-
Masanari Iida authored
Correct spelling typo in staging/octeon Signed-off-by:
-
Avinash kumar authored
removed checpatch warning : WARNING: extern prototypes should be avoided in .h files Signed-off-by:
Avinash Kumar <avi.kp.137@gmail.com> Signed-off-by:
-
Jon Bernard authored
This is a patch to lvfs_linux.c that fixes incorrect switch case statement indenting found by checkpatch.pl. Signed-off-by:
Jon Bernard <jbernard@tuxion.com> Signed-off-by:
-
Jon Bernard authored
This is a patch to lvfs_linux.c that removes the use of variable assignment within an if condition found by checkpatch.pl. Signed-off-by:
-
Masanari Iida authored
Correct spelling typo in lustre/llite Correct double words "to to" Signed-off-by:
-
Masanari Iida authored
Correct spelling typos in comment and debug message, within luster/libcfs Signed-off-by:
-
Lidza Louina authored
This patch removes these smatch warnings: redundant null check on dgap_TmpWriteBuf calling kfree() redundant null check on brd->SerialDriver->ttys calling kfree() redundant null check on brd->PrintDriver->ttys calling kfree() The code checked to see if these variables are null before freeing. This check isn't needed. Signed-off-by:
Lidza Louina <lidza.louina@gmail.com> Signed-off-by:
-
Lidza Louina authored
This patch removes this smatch warning: info: ignoring unreachable code. There were instances where there was extra code after the default action in switch statements. These default actions ended with a break so the code wasn't being run at anytime. This patch removes that extra code. Signed-off-by:
-
Lidza Louina authored
This patch removes these smatch warnings from dgap_driver.c: redundant null check on dgap_config_buf calling kfree() redundant null check on brd->flipbuf calling kfree() redundant null check on brd->flipflagbuf calling kfree() Signed-off-by:
-
