- Feb 28, 2014
-
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in functions.c WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by:Monam Agarwal <monamagarwal123@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in functions.c WARNING: please no spaces at the start of a line in Signed-off-by:
-
Daeseok Youn authored
After spin_lock() is called, all of if-else conditions in this brace should reach the end of else and spin_unlock() must be called. So It doesn't need to call spin_unlock() without a return statement for handling an error. Also sparse says: drivers/staging/ced1401/usb1401.c:1080:28: warning: context imbalance in 'Handle1401Esc' - unexpected unlock Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Daeseok Youn <daeseok.youn@gmail.com> Signed-off-by:
-
Chase Southwood authored
The variable ui_Command is as of right now being cleared to a value of zero between everytime that it writes to a port and then takes a new value from a port. Seems like this zeroing is unnecessary, so we can just remove these lines. Signed-off-by:
Chase Southwood <chase.southwood@yahoo.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Kumar Amit Mehta authored
Return -ENOMEM in ni_E_init if ni_gpct_device_construct returns NULL Signed-off-by:
Kumar Amit Mehta <gmate.amit@gmail.com> Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in comedidev.h WARNING: Fix unnecessary space after function pointer Signed-off-by:
-
- Feb 25, 2014
-
-
Mark Hounschell authored
The last patch series exposed some dead code causing compile warnings. This patch removes that dead code and fixes the warnings Signed-off-by:
Mark Hounschell <markh@compro.net> Signed-off-by:
-
Chase Southwood authored
There are a couple of cases where a comment being on the same line as a statement is causing the line to be over 80 characters long. This is an easy fix, move these comments to the previous line. Signed-off-by:
-
Chase Southwood authored
There were some conditional blocks that had an unnecessary level of indentation in them. We can remove this to improve code clarity. Signed-off-by:
-
Chase Southwood authored
This patch further cleans up the comments in hwdrv_apci035.c, converting them to kernel style and removing some commented conditional statements that are unused. Signed-off-by:
-
Monam Agarwal authored
Signed-off-by:
-
Malcolm Priestley authored
Replace with enum assign as u8 type. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
Remove commented out, white space and camel case. Camel case changes pContext -> context pDevice -> priv ContextType -> context_type ulBufLen -> buf_len Signed-off-by:
-
Malcolm Priestley authored
change pContext->bBoolInUse to false on connection errors. Signed-off-by:
-
Malcolm Priestley authored
Declare in order of pointer use. Signed-off-by:
-
Malcolm Priestley authored
Remove calls with break s_bCommandComplete(pDevice); spin_unlock_irq(&pDevice->lock); return; Add single call to s_bCommandComplete; Signed-off-by:
-
Malcolm Priestley authored
device_set_multi is an atomic call, in order to reduce atomic area of driver move code to be called from vRunCommand. Later the atomic area of vRunCommand can be reduced. Change existing code in device_set_multi to new function vnt_configure_filter minus its locks. Change device_set_multi to call bScheduleCommand device_set_multi is nolonger called from device open. Signed-off-by:
-
Malcolm Priestley authored
Replace with struct vnt_interrupt_buffer. Using only the live member of old structure pDataBuf -> data_buf bInUse -> in_use uDataLen is unused and dropped. Signed-off-by:
-
Malcolm Priestley authored
Remove comments, white space and camel case. Camel case changes pDevice -> priv ntStatus -> status Signed-off-by:
-
Malcolm Priestley authored
set intBuf.bInUse to false on return error. Signed-off-by:
-
Malcolm Priestley authored
Change to usb_fill_int_urb which has int_interval. Signed-off-by:
-
Jon Mason authored
Remove unused #defines, structure, and inlined functions for the rtl8192e driver. Compile tested only. Signed-off-by:
Jon Mason <jdmason@kudzu.us> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by:
-
Mark Hounschell authored
This patch adds in-kernel firmware loading support and removes support for the original userland firmware loading process. Signed-off-by:
-
Mark Hounschell authored
Renames driver file dgap_driver.c and dgap_driver.h to dgap.c and dgap.h because we are now single source and include file and better fits kernel naming conventions. Signed-off-by:
-
Mark Hounschell authored
This patch makes all merged and original functions static to dgap.c. Doing so has revealed more dead code via gcc warnings. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
Removes unneeded files dgap_trace.c and dgap_trace.h Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
There is a lot of cleanup work to do on these digi drivers and merging as much as is possible will make it easier. I also notice that many merged drivers are single source and header. Signed-off-by:
-
Mark Hounschell authored
This patch removes userland source code files downld.c and dgap_downld.h Signed-off-by:
-
