- 10 Jan, 2015 40 commits
-
-
Benjamin Romer authored
Just remove the one extra blank line. Signed-off-by:
Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by:
Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Benjamin Romer authored
Fix CamelCase names: pSignal => signal Remove unused parameters issueInterruptIfEmpty and interruptHandle, and update callers of this function. Signed-off-by:
-
Benjamin Romer authored
Correct alignment for a couple of functions and remove the space between a typecast and its target. Signed-off-by:
-
Benjamin Romer authored
Fix CamelCase name: Work_wakeup_polling_device_channels => work_wakeup_polling_device_channels Update references to use the new name. Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase name of this function: Initialize_incoming_thread => initialize_incoming_thread Update all references to use the new name. Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase function name: Process_Incoming => process_incoming Update all references to use the new name. Signed-off-by:
-
Benjamin Romer authored
Fix the function definition so that it is a single line. Fix CamelCase parameter names: busNo => bus_no devNo => dev_no Get rid of the goto and just break out of the for loop, since that does the exact same thing. Signed-off-by:
-
Benjamin Romer authored
Fix the function declaration so the type is on the same line as the name, and fix the CamelCase local variable name: TotalBytes => total_bytes Signed-off-by:
-
Benjamin Romer authored
Fix the function declaration so it is a single line, and fix CamelCase function parameter names: busNo => bus_no devNo => dev_no Signed-off-by:
-
Benjamin Romer authored
Fix the function declaration to be a single line, and rename the CamelCase parameter: busNo => bus_no Signed-off-by:
-
Benjamin Romer authored
Fix the function declaration so it is a single line. Rename CamelCase local variable names: busNo => bus_no devNo => dev_no Fix use of uuid_le_cmp() to check for 0 instead of using !uuid_le_cmp(). Fix spelling error DESTORY to correctly read DESTROY. Signed-off-by:
-
Benjamin Romer authored
Fix the function declaration so it is a single line. Fix CamelCase local variable names: busNo => bus_no devNo => dev_no Fix use of uuid_le_cmp() to check for 0 instead of using !uuid_le_cmp(). Signed-off-by:
-
Benjamin Romer authored
Fix the function declaration so it is on a single line. Fix CamelCase local variables: busNo => bus_no devNo => dev_no Fix use of uuid_le_cmp() to check for 0 instead of using !uuid_le_cmp(). Signed-off-by:
-
Benjamin Romer authored
OK, lets clean up this messy function. The entire thing is reflowed so spacing and indentation is correct. Tests that created extra indentation were swapped for equivalent negative tests that did not create extra indentation. Use of cmp_uuid_le() were fixed to test specifically for the zero case, and not to lazily use !cmp_uuid_le(). CamelCase local variable names were fixed: busNo => bus_no devNo => dev_no minSize => min_size pReqHandler => req_handler Away => cleanup Finally, the struct guest_msgs cmd declaration was moved to the beginning of the function, and cringing and wincing at the function was significantly reduced. Signed-off-by:
-
Benjamin Romer authored
Fix the CamelCase local variable: busNo => bus_no Signed-off-by:
-
Benjamin Romer authored
Fix CamelCase local variable names: busNo => bus_no deviceCount => dev_count Signed-off-by:
-
Benjamin Romer authored
Clean up the function definition so it's a single line. Remove the unnecessary goto statements and just return directly. Remove the unneeded local variable for the return result. Fix CamelCase parameters and local variable names: channelAddr => ch_addr channelBytes => ch_bytes pChan => ch Signed-off-by:
-
Benjamin Romer authored
Fix CamelCase names: ProcReadBufferValid => debug_buf_valid ProcReadBuffer => debug_buf BusListHead => bus_list BusListLock => bus_list_lock BusListCount => bus_list_count MaxBusCount => max_bus_count PhysicalDataChan => phys_data_chan PlatformNumber => platform_no Incoming_ThreadInfo => incoming_ti Incoming_Thread_Started => incoming_started List_Polling_Device_Channels => poll_dev_chan Lock_Polling_Device_Channels => poll_dev_lock Wakeup_Polling_Device_Channels => poll_dev_wake_q Go_Polling_Device_Channels => poll_dev_start Signed-off-by:
-
Benjamin Romer authored
Split the doubled assignment in uislib_mod_init() into two separate assignments. Signed-off-by:
-
Benjamin Romer authored
Get rid of the extra parenthesis in uislib_enable_channel_interrupts(). Signed-off-by:
-
Benjamin Romer authored
Fix the brackets in the else clause in Process_Incoming(). Signed-off-by:
-
Benjamin Romer authored
The if statement in info_debugfs_read() needs another set of brackets for the else clause. Signed-off-by:
Ken Depro <kenneth.depro@unisys.com> Signed-off-by:
-
Benjamin Romer authored
Fix the line spacing errors in uislib.c. Signed-off-by:
-
Benjamin Romer authored
Use the variable name rather than the type, and add a set of missing brackets to the if statement in create_device(). Signed-off-by:
-
Benjamin Romer authored
Fix the missing braces and logical continuation problems in create_bus(). Signed-off-by:
-
Benjamin Romer authored
Fix almost all of the parenthesis alignment problems in uislib.c. The 2 remaining issues are in create_device(), which is heavily indented and needs to be refactored entirely anyway. Signed-off-by:
-
Benjamin Romer authored
Remove all extraneous spaces from typecasts in uislib.c. Signed-off-by:
-
Benjamin Romer authored
Add a comment to the charqueue's spinlock to explain that it is a lock for the structure. Signed-off-by:
-
Benjamin Romer authored
Delete the following unused types, and unused function prototypes: VISORCHIPSET_SWITCH_INFO VISORCHIPSET_EXTERNALPORT_INFO VISORCHIPSET_INTERNALPORT_INFO visorchipset_get_switch_info() visorchipset_get_externalport_info() Signed-off-by:
-
Benjamin Romer authored
The functions in channels/* aren't used in a lot of places. In fact, the functions in channel.c can be moved to uislib/uisqueue.c, and the rest of the files in channels can be eliminated. This patch deletes the channels directory and files, removes it from all Kconfigs that referenced them, removes the reference in the Makefile, and moves the functions inside of channels.c to uislib/uisqueue.c. Signed-off-by:
-
Benjamin Romer authored
Nobody is using this file so remove it and the reference to it in visorchipset_main.c. Signed-off-by:
-
Benjamin Romer authored
Get rid of the extra blank lines in globals.h. Signed-off-by:
-
Benjamin Romer authored
Just get rid of the extra blank lines in this file. Signed-off-by:
-
H Hartley Sweeten authored
Update the MODULE_DESCRIPTION to something more useful than the generic "Comedi low-level driver". Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
H Hartley Sweeten authored
This board supports interrupts on change of state of the digital inputs. Add the necessary subdevice support and interrupt handler to allow async commands to detect the change of state. Signed-off-by:
-
H Hartley Sweeten authored
This board supports interrupts on change of state of the digital inputs. Introduce a helper function to enable/disable the interrupt. Use the new helper function to ensure that interrupts are initially disabled during the driver (*attach). Signed-off-by:
-
H Hartley Sweeten authored
The relay registers are readable. Read them during the attach to get the initial state of the digital outputs. Signed-off-by:
-
H Hartley Sweeten authored
Tidy up the multi-line comments to follow the kernel CodingStyle. Signed-off-by:
-
H Hartley Sweeten authored
For aesthetics, add some white space to the subdevice initialization. Signed-off-by:
-
H Hartley Sweeten authored
Subdevice 1 is a digial input not a digital I/O subdevice. Fix the type and, for aesthetics, rename the (*insn_bits) function used to read the inputs. Signed-off-by:
-
