- 11 Sep, 2014 40 commits
-
-
Domagoj Trsan authored
Signed-off-by:
Domagoj Trsan <domagoj.trsan@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Domagoj Trsan authored
Signed-off-by:
-
Sudip Mukherjee authored
fixed sparse warning : context imbalance in 'resume_device' unexpected unlock this patch will generate warning from checkpatch for lines over 80 character , but since those are user-visible strings so it was not modified. Signed-off-by:Sudip Mukherjee <sudip@vectorindia.org> Acked-by:
Benjamin Romer <benjamin.romer@unisys.com> Tested-by:
-
Nitin Kuppelur authored
This is a patch to the octeon-hcd.c file that fixes removes dummy label i.e. label followed by return of void function Signed-off-by:
Nitin Kuppelur <nitinkuppelur@gmail.com> Signed-off-by:
-
Sanjeev Sharma authored
This is a patch to the r8190_rtl8256.c file that fixes commenting style Error Signed-off-by:
Sanjeev Sharma <Sanjeev_Sharma@mentor.com> Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl 'code indent should use tabs where possible' errors Signed-off-by:
Greg Donald <gdonald@gmail.com> Signed-off-by:
-
John L. Hammond authored
Move the definition of struct lvfs_run_ctxt to the one file that needed it (lustre/include/obd.h). Remove the then unneeded headers lustre/include/lvfs.h and lustre/include/linux/lvfs.h. Signed-off-by:
John L. Hammond <john.hammond@intel.com> Signed-off-by:
-
John L. Hammond authored
The upcall cache is unused so remove include/linux/libcfs/lucache.h and lustre/libcfs/upcall_cache.c. Signed-off-by:
-
John L. Hammond authored
Remove the unused "lvfs" functions obd_lvfs_fid2dentry(), ll_lookup_one_len(), l_dput(), lustre_rename(), push_ctxt(), and pop_ctxt(). Remove the unused members of struct lvfs_run_ctxt. Signed-off-by:
-
John L. Hammond authored
The definitions in lustre/include/linux/lvfs_linux.h are not used so remove that header. Signed-off-by:
-
James Simmons authored
Remove include/linux/libcfs/linux/portals_compat25.h. . remove some unused/unnecessary macros such as smp_num_cpus / SIGNAL_MASK_ASSERT etc. . replace some macros with direct kernel API calls such as RECALC_SIGPENDING/CLEAR_SIGPENDING/CURRENT_SECONDS, cfs_wait_event_interruptible/_exclusive etc. Signed-off-by:
Liu Xuezhao <xuezhao.liu@emc.com> Signed-off-by:
Peng Tao <tao.peng@emc.com> Signed-off-by:
James Simmons <uja.ornl@gmail.com> Signed-off-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
Keith Mannthey <keith.mannthey@intel.com> Signed-off-by:
-
John L. Hammond authored
Expand the two uses of ll_inode_blksize() and remove the then unnecessary header lustre/include/linux/lustre_common.h. Signed-off-by:
Bob Glossman <bob.glossman@intel.com> Reviewed-by:
Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
-
James Simmons authored
The lvfs layer is obsolete and unused by Lustre clients. So remove it and integrate the last of the procfs code into the obdclass layer. Signed-off-by:
Mikhail Pershin <mike.pershin@intel.com> Signed-off-by:
Jian Yu <jian.yu@intel.com> Reviewed-by:
-
Hugues Morisset authored
Signed-off-by:
Hugues Morisset <morisset.hugues@gmail.com> Signed-off-by:
-
Hugues Morisset authored
Signed-off-by:
-
Hugues Morisset authored
Signed-off-by:
-
Julia Lawall authored
The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Oleg Drokin authored
This function is no longer used anywhere. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
-
Mikhail Pershin authored
OUT service is used to server both MDS-MDS updates and MDS-OST therefore services on MDT and OST are set to use the same request and reply portals to be fully unified and able to serve any type of requests. Signed-off-by:
wangdi <di.wang@intel.com> Reviewed-by:
Fan Yong <fan.yong@intel.com> Reviewed-by:
-
Greg Donald authored
Fix checkpatch.pl 'code indent should use tabs where possible' errors Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl "space prohibited after that open parenthesis '('" errors Signed-off-by: -
Ian Abbott authored
The `comedi_board(dev)` inline function calls just return `dev->board_ptr`. Expand the inline function calls. Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Ian Abbott authored
The `comedi_board(dev)` inline function calls just return `dev->board_ptr`. Expand the inline function calls. Signed-off-by:
-
Chase Southwood authored
Returns at the end of void functions are useless. Remove this one. Signed-off-by:
Chase Southwood <chase.southwood@gmail.com> Reviewed-by:
-
Chase Southwood authored
The statement nested in an else after a return may be brought out one indent level, the else is useless. Signed-off-by:
-
Chase Southwood authored
Signed-off-by:
-
Chase Southwood authored
This patch fixes improper comment indentation, removes dead code and obsolete comments, and conforms remaining comments to a consistent kernel commenting style. Signed-off-by:
-
H Hartley Sweeten authored
This driver currently allows a cmd->stop_src == TRIG_COUNT with a cmd->stop_arg of 0 for both the analog input and output async commands. The (*do_cmd) for both subdevices sets up and starts the command without handling the "empty acquisition". This results in the interrupt functions trying to transfer 0 data samples. Validate that the cmd->stop_arg is >= 1 in the (*do_cmdtest) to prevent the "empty acquisition". Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
-
H Hartley Sweeten authored
This driver currently allows a cmd->stop_src == TRIG_COUNT with a cmd->stop_arg of 0 for both the analog input and output async commands. The (*do_cmd) for both subdevices sets up and starts the command without handling the "empty acquisition". This results in the interrupt functions trying to transfer 0 data samples. Validate that the cmd->stop_arg is >= 1 in the (*do_cmdtest) to prevent the "empty acquisition". Signed-off-by:
-
H Hartley Sweeten authored
The only limit for the stop_arg with a stop_src == TRIG_COUNT is that there is at least 1 scan in order to prevent an empty acquisition. The max test is bogus. Fix the validation. Signed-off-by:
-
H Hartley Sweeten authored
When the cmd->stop_src == TRIG_COUNT, the 'ai_sample_count' is the cmd->stop_arg. For a stop_src == TRIG_NONE the 'ai_sample_count' is currently set to 1 but the driver never actually uses the value. Simplify the 'stop_count' initialization by just setting it to the cmd->stop_arg. Signed-off-by:
-
H Hartley Sweeten authored
The end-of-acquisition only applies when the cmd->stop_src == TRIG_COUNT. Refactor the code in s626_handle_eos_interrupt() that detects the end-of- acquisition to clarify this. Signed-off-by:
-
H Hartley Sweeten authored
This member of the private data is set based on the cmd->stop_src. Just use the cmd->stop_src directly and remove the member. Signed-off-by:
-
H Hartley Sweeten authored
This driver currently allows a cmd->stop_src == TRIG_COUNT with a cmd->stop_arg of 0. The (*do_cmd) then sets up and starts the command without handling the "empty acquisition". This results in the interrupt function trying to transfer 0 data samples. Validate that the cmd->stop_arg is >= 1 in the (*do_cmdtest) to prevent the "empty acquisition". Signed-off-by:
-
H Hartley Sweeten authored
This function always returns '0' so the comedi_event() is never done by the callers. Change the return type to void and remove the comedi_event() dead code. Signed-off-by:
-
H Hartley Sweeten authored
When the cmd->stop_src == TRIG_COUNT, the 'stop_count' is the cmd->stop_arg. For any other stop_src the 'stop_count' is 0, which is also the cmd->stop_arg. Simplify the 'stop_count' initialization. Signed-off-by:
-
H Hartley Sweeten authored
This member of the private data is set based on the cmd->stop_src. Just use the cmd->stop_src directly and remove the member. Signed-off-by:
-
H Hartley Sweeten authored
This driver currently allows a cmd->stop_src == TRIG_COUNT with a cmd->stop_arg of 0. This causes the (*do_cmd) to immediately generate a COMEDI_CB_EOA event without acquiring any data. This "empty acquisition" async command is not really useful. Validate that the cmd->stop_arg is >= 1 in the (*do_cmdtest) and remove the "empty acquisition" code. Signed-off-by:
-
H Hartley Sweeten authored
This function always returns '0' so the comedi_event() is never done by the callers. Change the return type to void and remove the comedi_event() dead code. Signed-off-by:
-
H Hartley Sweeten authored
When the cmd->stop_src == TRIG_COUNT, the 'stop_count' is the cmd->stop_arg. For any other stop_src the 'stop_count' is 0, which is also the cmd->stop_arg. Simplify the 'stop_count' initialization. Signed-off-by:
-
