[PATCH 010/108] staging: comedi: addi_apci_3120: introduce apci3120_timer_set_mode()

2014-11-04 Thread H Hartley Sweeten
Introduce a helper function to set the operation mode of a timer. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 57 +++--- drivers/staging/comedi/drivers/addi_apci_3120.c| 6 +++ 2 files

[PATCH 024/108] staging: comedi: addi_apci_3120: remove unnecessary reset of the scan sequence

2014-11-04 Thread H Hartley Sweeten
The scan sequence does not need to be reset when the driver is attached. Remove the code in apci3120_reset() that does this. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 7 --- 1 file changed, 7

[PATCH 025/108] staging: comedi: addi_apci_3120: tidy up scan chanlist programming

2014-11-04 Thread H Hartley Sweeten
Define the chanlist register and its bits and tidy up the programming of the scan chanlist. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 38 +++--- drivers/staging/comedi/drivers/addi_apci_3120

[PATCH 046/108] staging: comedi: addi_apci_3120: remove interrupt support from ai (*insn_read)

2014-11-04 Thread H Hartley Sweeten
w the comedi API and this operation mode is not part of the API. Remove the interrupt support from the (*insn_read) as well as the support code in the (*insn_config) and interrupt handler. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi

[PATCH 071/108] staging: comedi: addi_apci_3120: set scan length/start after programming chanlist

2014-11-04 Thread H Hartley Sweeten
The APCI-3120 documentation says that the PR/PA bits should be set after the chanlist sequence is programmed. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi_apci_3120.c | 8 1 file changed, 4 insertions(+), 4 deletions

[PATCH 057/108] staging: comedi: addi_apci_3120: move apci3120_di_insn_bits() to driver source

2014-11-04 Thread H Hartley Sweeten
Move this function from the included hwdrv_apci31210.c source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 13 - drivers/staging/comedi/drivers

[PATCH 012/108] staging: comedi: addi_apci_3120: rename private data 'us_OutputRegister'

2014-11-04 Thread H Hartley Sweeten
Rename this CamelCase member of the private data and tidy up the mask/set of its bits. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 110 - drivers/staging/comedi/drivers/addi_apci_3120.c

[PATCH 038/108] staging: comedi: addi_apci_3120: remove APCI3120_DISABLE_EOS_INT

2014-11-04 Thread H Hartley Sweeten
For aesthetics, remove this define and just use ~APCI3120_ENABLE_EOS_INT. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH 043/108] staging: comedi: addi_apci_3120: rename APCI3120_WRITE_MODE_SELECT

2014-11-04 Thread H Hartley Sweeten
For aesthetics, rename this define used for the mode register offset. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 42 +- drivers/staging/comedi/drivers/addi_apci_3120.c| 1 + 2 files

[PATCH 023/108] staging: comedi: addi_apci_3120: introduce apci3120_clr_timer2_interrupt()

2014-11-04 Thread H Hartley Sweeten
A dummy read of APCI3120_CTR0_REG clears the timer 2 interrupt. Introduce a helper function to clarify this. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 30 -- drivers/staging/comedi

[PATCH 018/108] staging: comedi: addi_apci_3120: introduce apci3120_timer_enable()

2014-11-04 Thread H Hartley Sweeten
Introduce a helper function to enable/disable a timer. The disable of timers 0 and 1 in apci3120_interrupt() is probably not needed. For now use the helper function to make sure the devpriv->ctrl bits are cleared correctly to disable the timers. Signed-off-by: H Hartley Sweeten Cc: Ian Abb

[PATCH 054/108] staging: comedi: addi_apci_3120: define status register bits

2014-11-04 Thread H Hartley Sweeten
Define the bits in the status register and use them to clarify the code. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 58 ++ drivers/staging/comedi/drivers/addi_apci_3120.c| 9 2

[PATCH 056/108] staging: comedi: addi_apci_3120: move apci3120_do_insn_bits() to driver source

2014-11-04 Thread H Hartley Sweeten
Move this function from the included hwdrv_apci31210.c source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 18 -- drivers/staging/comedi/drivers

[PATCH 050/108] staging: comedi: addi_apci_3120: absorb apci3120_interrupt_handle_eos()

2014-11-04 Thread H Hartley Sweeten
Absorb this simple function into apci3120_interrupt(). Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 28 +++--- 1 file changed, 8 insertions(+), 20 deletions(-) diff --git a/drivers/staging

[PATCH 042/108] staging: comedi: addi_apci_3120: define the timer 2 clock select bits

2014-11-04 Thread H Hartley Sweeten
For aesthetics, redefine the bits in the mode register used to select the clock for timer 2. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 11 --- drivers/staging/comedi/drivers/addi_apci_3120.c

[PATCH 015/108] staging: comedi: addi_apci_3120: tidy up APCI3120_ENABLE_TIMER[012]

2014-11-04 Thread H Hartley Sweeten
For aesthetics, replace these defines with a macro that returns the correct bit needed to set the gate bit to enable a timer. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 25 +- drivers

[PATCH 011/108] staging: comedi: addi_apci_3120: move timer helpers to main driver source

2014-11-04 Thread H Hartley Sweeten
Move the timer read/write and set mode helpers from the included source file into the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 53 -- drivers/staging/comedi

[PATCH 028/108] staging: comedi: addi_apci_3120: move ai range table to driver source

2014-11-04 Thread H Hartley Sweeten
Move the analog input comedi_lrange table from the included hwdrv_apci3120.c source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 14 -- drivers/staging

[PATCH 041/108] staging: comedi: addi_apci_3120: define the timer 2 operation bits

2014-11-04 Thread H Hartley Sweeten
For aesthetics, redefine the bits in the mode register used to set the operation mode of timer 2. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 23 -- drivers/staging/comedi/drivers

[PATCH 001/108] staging: comedi: addi_apci_3120: introduce apci3120_ns_to_timer()

2014-11-04 Thread H Hartley Sweeten
time. Set this member during the board attach. Introduce a helper function to calculate the divisor needed to generate a nanosecond time with a given timer. Use the new helper function in the driver to clarify the code. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman ---

[PATCH 069/108] staging: comedi: addi_apci_3120: tidy up timer 2 programming in apci3120_ai_cmd()

2014-11-04 Thread H Hartley Sweeten
Tidy up the programming of timer 2. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../staging/comedi/drivers/addi-data/hwdrv_apci3120.c| 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/drivers/staging/comedi/drivers/addi-data

[PATCH 016/108] staging: comedi: addi_apci_3120: rename APCI3120_ENABLE_EXT_TRIGGER

2014-11-04 Thread H Hartley Sweeten
For aesthetics, rename this define. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 7 ++- drivers/staging/comedi/drivers/addi_apci_3120.c | 1 + 2 files changed, 3 insertions(+), 5 deletions

[PATCH 005/108] staging: comedi: addi_apci_3120: tidy up CTR0 register defines

2014-11-04 Thread H Hartley Sweeten
TR0 register define in the digital output (*insn_bits) function and remove the current register define. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 38 +++--- drivers/staging/comedi/driv

[PATCH 059/108] staging: comedi: addi_apci_3120: move apci3120_ai_insn_read() to driver source

2014-11-04 Thread H Hartley Sweeten
Move this function, and its helper function, from the included hwdrv_apci31210.c source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 60 -- drivers

[PATCH 051/108] staging: comedi: addi_apci_3120: remove private data 'ui_AiNbrofChannels'

2014-11-04 Thread H Hartley Sweeten
This member of the private data is always the cmd->chanlist_len. Use that instead and remove the member. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 8 +++- drivers/staging/comedi/driv

[PATCH 058/108] staging: comedi: addi_apci_3120: move apci3120_ao_insn_write() to driver source

2014-11-04 Thread H Hartley Sweeten
Move this function, and its helper function, from the included hwdrv_apci31210.c source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 38 -- drivers

[PATCH 014/108] staging: comedi: addi_apci_3120: remove APCI3120_DISABLE_TIMER[012]

2014-11-04 Thread H Hartley Sweeten
For aesthetics, remove these defines and just use ~APCI3120_ENABLE_TIMER[012]. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git

[PATCH 040/108] staging: comedi: addi_apci_3120: define the "enable" bits in the mode register

2014-11-04 Thread H Hartley Sweeten
For aesthetics, redefine the bits in the mode register that enable interrupts and scanning. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 38 +- drivers/staging/comedi/drivers/addi_apci_3120

[PATCH 061/108] staging: comedi: addi_apci_3120: move apci3120_set_chanlist() to driver source

2014-11-04 Thread H Hartley Sweeten
Move this function from the included hwdrv_apci31210.c source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 28 -- drivers/staging/comedi/drivers

[PATCH 020/108] staging: comedi: addi_apci_3120: rename APCI3120_WR_ADDRESS

2014-11-04 Thread H Hartley Sweeten
For aesthetics, rename this define. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 11 +-- drivers/staging/comedi/drivers/addi_apci_3120.c | 1 + 2 files changed, 6 insertions(+), 6

[PATCH 048/108] staging: comedi: addi_apci_3120: remove private data 'ui_AiReadData'

2014-11-04 Thread H Hartley Sweeten
This member of the private data was used to return analog input samples that were acquired for the (*insn_read) using interrupts. The interrupt support code for the (*insn_read) has been removed. Remove this unused member from the private data. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc

[PATCH 080/108] staging: comedi: addi_apci_3120: move APCI3120_FIFO_ADVANCE_ON_BYTE_2

2014-11-04 Thread H Hartley Sweeten
Move this define to the main driver source and convert it to a bit-shift. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 6 -- drivers/staging/comedi/drivers/addi_apci_3120.c | 1 + 2 files

[PATCH 053/108] staging: comedi: addi_apci_3120: rename APCI3120_RD_STATUS

2014-11-04 Thread H Hartley Sweeten
For aesthetics, rename this define used for the status register offset. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 17 +++-- drivers/staging/comedi/drivers/addi_apci_3120.c | 3

[PATCH 003/108] staging: comedi: addi_apci_3120: introduce apci3120_timer_write()

2014-11-04 Thread H Hartley Sweeten
Introduce a helper function to select a timer and write a value to it. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 138 + drivers/staging/comedi/drivers/addi_apci_3120.c| 1 + 2 files

[PATCH 044/108] staging: comedi: addi_apci_3120: remove scanning from ai (*insn_read)

2014-11-04 Thread H Hartley Sweeten
w the comedi API and this operation mode is not part of the API. Remove the scanning support from the (*insn_read) as well as the support code in the (*insn_config) and interrupt handler. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi

[PATCH 070/108] staging: comedi: addi_apci_3120: reset fifo after programming chanlist

2014-11-04 Thread H Hartley Sweeten
The APCI-3120 documentation says that the FIFO should be reset after the chanlist sequence is programmed. Reset the FIFO after programming the chanlist and remove the extra FIFO resets in the driver. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging

[PATCH 060/108] staging: comedi: addi_apci_3120: remove check in apci3120_setup_chan_list()

2014-11-04 Thread H Hartley Sweeten
The n_chan check is not needed. This value will always be >= 1. Remove the unnecessary check. For aesthetics, rename the function and change it's return type to void. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../staging/comedi/drivers/a

[PATCH 055/108] staging: comedi: addi_apci_3120: remove private data 'ai_running'

2014-11-04 Thread H Hartley Sweeten
This member of the private data was used to determine if the interrupt routine was handling data for the (*insn_read) or an async command. Now that the (*insn_read) does not use interrupts this member is not needed. Remove the member and refactor the code. Signed-off-by: H Hartley Sweeten Cc

[PATCH 077/108] staging: comedi: addi_apci_3120: introduce apci3120_addon_write()

2014-11-04 Thread H Hartley Sweeten
Introduce a helper function to handle the common code that writes a 32-bit value to the 16-bit add-on register. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 71 +- 1 file changed, 29

[PATCH 031/108] staging: comedi: addi_apci_3120: rename private data 'b_ModeSelectRegister'

2014-11-04 Thread H Hartley Sweeten
Rename this CamelCase member of the private data and tidy up the mask/set of its bits. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 198 +++-- drivers/staging/comedi/drivers/addi_apci_3120.c

[PATCH 022/108] staging: comedi: addi_apci_3120: move apci3120_exttrig_enable() to driver source

2014-11-04 Thread H Hartley Sweeten
Move this helper function from the included source file into the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 11 --- drivers/staging/comedi/drivers/addi_apci_3120.c

[PATCH 026/108] staging: comedi: addi_apci_3120: remove 'check' param from apci3120_setup_chan_list()

2014-11-04 Thread H Hartley Sweeten
This parameter is always passed as '0'. Remove it and refactor the code. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 23 +- 1 file changed, 5 insertions(+), 18 deletions(-) di

[PATCH 006/108] staging: comedi: addi_apci_3120: fix counter and external interrupt disable

2014-11-04 Thread H Hartley Sweeten
The COMEDI_CB_EOA event will cause the core to call the (*cancel) operation which will disable the timers. Remove the unused define. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 21 ++---

[PATCH 068/108] staging: comedi: addi_apci_3120: tidy up timer programming in apci3120_ai_cmd()

2014-11-04 Thread H Hartley Sweeten
Tidy up the programming of timers 0 and 1 so that only one local variable is required. Also, remove the unnecessary clear of devpriv->timer_mode. This value will get set correctly by each apci3120_timer_set_mode() call. It's not necessary to clear it first. Signed-off-by: H Hartley Swe

[PATCH 065/108] staging: comedi: addi_apci_3120: move timer 2 enable in apci3120_cyclic_ai()

2014-11-04 Thread H Hartley Sweeten
Move the enable of timer 2 to avoid needing the extra if() check. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 062/108] staging: comedi: addi_apci_3120: factor DMA setup out of apci3120_cyclic_ai()

2014-11-04 Thread H Hartley Sweeten
For aesthetics, factor the DMA setup code out of apci3120_cyclic_ai(). Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 295 +++-- 1 file changed, 148 insertions(+), 147 deletions(-) diff --git a

[PATCH 013/108] staging: comedi: addi_apci_3120: tidy up devpriv->ctrl use

2014-11-04 Thread H Hartley Sweeten
write the register. Update the comments in apci3120_reset() and apci3120_cancel(). Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 34 -- drivers/staging/comedi/drivers/addi_apci_3120.c| 2 ++

[PATCH 073/108] staging: comedi: addi_apci_3120: tidy up devpriv->mode in apci3120_ai_cmd()

2014-11-04 Thread H Hartley Sweeten
There is no reason for the separate updates of the mode register in this function. Refactor the code so that the mode register is only updated at the end of the function after all the necessary bits have been set. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman

[PATCH 072/108] staging: comedi: addi_apci_3120: enable chanlist scanning if needed

2014-11-04 Thread H Hartley Sweeten
The 'SCAN_ENA' bit in the mode register needs to be set if the chanlist has more than 1 channel. Set the bit in apci3120_set_chanlist() if needed. The callers write the mode register after setting any additional bits, Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kro

[PATCH 029/108] staging: comedi: addi_apci_3120: remove APCI3120_DISABLE_ALL_INTERRUPT_WITHOUT_TIMER

2014-11-04 Thread H Hartley Sweeten
At the start of apci3120_ai_insn_read() the devpriv->b_ModeSelectRegister is set to '0'. Remove the unnecessary masking to clear the interrupt enable bits. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_ap

[PATCH 074/108] staging: comedi: addi_apci_3120: remove private data 'b_InterruptMode'

2014-11-04 Thread H Hartley Sweeten
This member of the private data is not really needed. The interrupt mode can be determined by checking the devpriv->mode value to see if the interrupt is enabled. Remove the unnecessary member. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../staging/com

[PATCH 067/108] staging: comedi: addi_apci_3120: absorb apci3120_cyclic_ai()

2014-11-04 Thread H Hartley Sweeten
This functon is called by apci3120_ai_cmd() with one additional parameter, 'mode', The 'mode' is based on the cmd->scan_begin_src. For aesthetics, absorb the function into apci3120_ai_cmd() and use the cmd->scan_begin_src directly to determine the 'mode'.

[PATCH 049/108] staging: comedi: addi_apci_3120: fix apci3120_ai_insn_read()

2014-11-04 Thread H Hartley Sweeten
but it returns insn->n. Fix this function to work like the core expects. Use comedi_timeout() to prevent a possible deadlock in the loop that waits for the end-of-conversion. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci

[PATCH 045/108] staging: comedi: addi_apci_3120: remove private data 'ui_EocEosConversionTime'

2014-11-04 Thread H Hartley Sweeten
m the private data and use a fixed 10us (1ns) conversion time in the (*insn_read). Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 10 +- drivers/staging/comedi/drivers/addi_apci_3120.c | 1

[PATCH 064/108] staging: comedi: addi_apci_3120: flip 'us_UseDma' test in apci3120_cyclic_ai()

2014-11-04 Thread H Hartley Sweeten
For aesthetics, flip this test and do the DMA setup first. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/staging/comedi

[PATCH 036/108] staging: comedi: addi_apci_3120: remove APCI3120_DISABLE_TIMER_INT

2014-11-04 Thread H Hartley Sweeten
For aesthetics, remove this define and just use ~APCI3120_ENABLE_TIMER_INT. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers

[PATCH 076/108] staging: comedi: addi_apci_3120: introduce apci3120_init_dma()

2014-11-04 Thread H Hartley Sweeten
Introduce a helper function to handle the common code that writes the DMA start address and number of acquisitions to the AMCC Add-on registers. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 75

[PATCH 032/108] staging: comedi: addi_apci_3120: remove unnecessary devpriv->mode masking

2014-11-04 Thread H Hartley Sweeten
The devpriv->mode is set to '0' at the start of apci3120_ai_insn_read() and apci3120_cyclic_ai(). There is no need to clear any of the bits or do any of the additional writes of those cleared bits. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../c

[PATCH 066/108] staging: comedi: addi_apci_3120: move start_src check into apci3120_cyclic_ai()

2014-11-04 Thread H Hartley Sweeten
For aesthetics, move the check of the cmd->start_src. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/staging/com

[PATCH 088/108] staging: comedi: addi_apci_3120: define the AI FIFO register

2014-11-04 Thread H Hartley Sweeten
Define the offset for the analog input FIFO register. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 4 +--- drivers/staging/comedi/drivers/addi_apci_3120.c | 3 ++- 2 files changed, 3 insertions

[PATCH 079/108] staging: comedi: addi_apci_3120: define the Add-On registers

2014-11-04 Thread H Hartley Sweeten
Define the add-On registers and bits and remove the "magic" numbers in the driver. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 40 +- drivers/staging/comedi/drivers/addi_apci_3120

[PATCH 087/108] staging: comedi: addi_apci_3120: don't use timer 2 to count scans

2014-11-04 Thread H Hartley Sweeten
the core using the private data membmer 'scans_done'. Use that to detect the End-Of-Scan for both DMA and none DMA modes. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 51 ++ 1 fil

[PATCH 090/108] staging: comedi: addi_apci_3120: fix timer (*insn_read)

2014-11-04 Thread H Hartley Sweeten
The timer subdevice in this driver does not follow the comedi API. Fix the (*insn_read) to work correctly. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 35 -- drivers/staging/comedi/drivers

[PATCH 089/108] staging: comedi: addi_apci_3120: define the AI software trigger register

2014-11-04 Thread H Hartley Sweeten
Define the offset for the analog input software trigger register. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 3 --- drivers/staging/comedi/drivers/addi_apci_3120.c | 1 + 2 files changed, 1

[PATCH 063/108] staging: comedi: addi_apci_3120: remove APCI3120_{ENABLE, DISABLE}

2014-11-04 Thread H Hartley Sweeten
These true/false defines don't add any significant clarity to the code. Remove them. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 22 +- 1 file changed, 9 insertions(+), 13 dele

[PATCH 084/108] staging: comedi: addi_apci_3120: rename private data 'us_UseDma'

2014-11-04 Thread H Hartley Sweeten
Rename this CamelCase member of the private data and convert it to a bit-field. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 2 +- drivers/staging/comedi/drivers/addi_apci_3120.c | 4 ++-- 2

[PATCH 091/108] staging: comedi: addi_apci_3120: fix timer (*insn_config)

2014-11-04 Thread H Hartley Sweeten
interrupt support code for the timer. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 197 + drivers/staging/comedi/drivers/addi_apci_3120.c| 85 - 2 files changed, 83 insertions(+), 199

[PATCH 095/108] staging: comedi: addi_apci_3120: remove unnecessary include

2014-11-04 Thread H Hartley Sweeten
The header is not needed by this driver. Remove the include. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi_apci_3120.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/addi_apci_3120.c b

[PATCH 092/108] staging: comedi: addi_apci_3120: fix cmd->convert_arg vaildation

2014-11-04 Thread H Hartley Sweeten
Regardless of the cmd->scan_begin_src, the minimum cmd->convert_arg value is 1 (10us). Fix the validation in apci3120_ai_cmdtest(). Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 9 ++---

[PATCH 093/108] staging: comedi: addi_apci_3120: move AI (*do_cmdtest) to main driver

2014-11-04 Thread H Hartley Sweeten
Move apci3120_ai_cmdtest() from the included source file to the main driver source file. Tidy up Step 4 a bit during the move. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 64 - drivers

[PATCH 096/108] staging: comedi: addi_apci_3120: move apci3120_addon_write() to driver

2014-11-04 Thread H Hartley Sweeten
Move this function from the included source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 14 -- drivers/staging/comedi/drivers/addi_apci_3120.c

[PATCH 100/108] staging: comedi: addi_apci_3120: use async->events to report hardware error

2014-11-04 Thread H Hartley Sweeten
In apci3120_interrupt_dma(), use the async->events to report the hardware error and allow the core to (*cancel) the command instead of calling the (*cancel) function directly. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-d

[PATCH 094/108] staging: comedi: addi_apci_3120: add copyright information

2014-11-04 Thread H Hartley Sweeten
Remove the copyright info from the included source file and add it to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 42 -- drivers/staging/comedi/drivers

[PATCH 103/108] staging: comedi: addi_apci_3120: use comedi_bytes_to_samples()

2014-11-04 Thread H Hartley Sweeten
Use the helper function to calculate the number of samples in the DMA buffer. For aesthetics, tidy up the local variables. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 15 --- 1 file changed

[PATCH 104/108] staging: comedi: addi_apci_3120: move apci3120_interrupt_dma() to driver

2014-11-04 Thread H Hartley Sweeten
Move this function from the included source file to the main driver source file. Remove the now empty included source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 58 -- drivers

[PATCH 107/108] staging: comedi: addi_apci_3120: enable AI async commands

2014-11-04 Thread H Hartley Sweeten
The async command support should now work. Enable the hook up of the command support functions in apci3120_auto_attach(). Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi_apci_3120.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 099/108] staging: comedi: addi_apci_3120: move apci3120_ai_cmd() to driver

2014-11-04 Thread H Hartley Sweeten
Move this function from the included source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 56 -- drivers/staging/comedi/drivers/addi_apci_3120.c| 56

[PATCH 098/108] staging: comedi: addi_apci_3120: move apci3120_setup_dma() to driver

2014-11-04 Thread H Hartley Sweeten
Move this function from the included source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 51 -- drivers/staging/comedi/drivers/addi_apci_3120.c| 51

[PATCH 102/108] staging: comedi: addi_apci_3120: move apci3120_interrupt() to driver

2014-11-04 Thread H Hartley Sweeten
Move this function from the included source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 72 -- drivers/staging/comedi/drivers/addi_apci_3120.c| 72

[PATCH 083/108] staging: comedi: addi_apci_3120: move apci3120_reset() to driver source

2014-11-04 Thread H Hartley Sweeten
Move this function from the included hwdrv_apci31210.c source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 12 drivers/staging/comedi/drivers

[PATCH 108/108] staging: comedi: addi_apci_3120: absorb apci3120_ai_reset_fifo()

2014-11-04 Thread H Hartley Sweeten
This function is only called by apci3120_set_chanlist(). Absorb it. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi_apci_3120.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/comedi

[PATCH 082/108] staging: comedi: addi_apci_3120: tidy up apci3120_reset()

2014-11-04 Thread H Hartley Sweeten
upt sources and stop the timers. For aesthetics, change the return type to void. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 19 +-- 1 file changed, 5 insertions(+), 14 deletions(-) diff --g

[PATCH 106/108] staging: comedi: addi_apci_3120: switch DMA buffers after writing samples

2014-11-04 Thread H Hartley Sweeten
after writing the samples. The driver will detect the EOA event and not retart the DMA. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi_apci_3120.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git

[PATCH 078/108] staging: comedi: addi_apci_3120: use amcc_s5933.h defines

2014-11-04 Thread H Hartley Sweeten
Use the defines for the AMCC 5933 PCI controller registers and bits instead of creating private defines in this driver. Move the generic AGCSTS_TC_ENABLE define from this driver to the header. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi

[PATCH 086/108] staging: comedi: addi_apci_3120: rename private data 'ui_DmaActualBuffer'

2014-11-04 Thread H Hartley Sweeten
Rename this CamelCase member of the private data and convert it to a bit-field. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi-data/hwdrv_apci3120.c | 10 +- drivers/staging/comedi/drivers/addi_apci_3120.c | 2

[PATCH 027/108] staging: comedi: addi_apci_3120: introduce apci3120_ai_reset_fifo()

2014-11-04 Thread H Hartley Sweeten
xes a couple 8-bit reads. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 30 +- drivers/staging/comedi/drivers/addi_apci_3120.c| 6 + 2 files changed, 13 insertions(+), 23 deletions(-)

[PATCH 105/108] staging: comedi: addi_apci_3120: change params to apci3120_interrupt_dma()

2014-11-04 Thread H Hartley Sweeten
Pass the comedi_device and comedi_subdevice pointers to this function instead of the irq number and void pointer. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi_apci_3120.c | 7 +++ 1 file changed, 3 insertions(+), 4

[PATCH 101/108] staging: comedi: addi_apci_3120: move apci3120_cancel() to driver

2014-11-04 Thread H Hartley Sweeten
Move this function from the included source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 28 -- drivers/staging/comedi/drivers/addi_apci_3120.c| 28

[PATCH 097/108] staging: comedi: addi_apci_3120: move apci3120_init_dma() to driver

2014-11-04 Thread H Hartley Sweeten
Move this function from the included source file to the main driver source file. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- .../comedi/drivers/addi-data/hwdrv_apci3120.c | 32 -- drivers/staging/comedi/drivers/addi_apci_3120.c| 32

[PATCH v2 02/18] staging: comedi: addi_apci_2032: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'stop_count' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/addi_apci_2032.c | 17 +

[PATCH v2 03/18] staging: comedi: amplc_dio200_common: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'stopcount' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/amplc_dio200_common.c | 14 ++

[PATCH v2 01/18] staging: comedi: comedidev.h: add 'scans_done' member to comedi_async

2014-11-05 Thread H Hartley Sweeten
counting in the comedi drivers. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/comedi_buf.c | 1 + drivers/staging/comedi/comedidev.h | 2 ++ drivers/staging/comedi/drivers.c| 7 +++ 3 files changed, 10 insertions(+) diff --git a

[PATCH v2 04/18] staging: comedi: pcl711: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'ntrig' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcl711.c | 16 ++-- 1 file c

[PATCH v2 00/18] staging: comedi: make core track the number of scans

2014-11-05 Thread H Hartley Sweeten
cmd_running flag correctly H Hartley Sweeten (18): staging: comedi: comedidev.h: add 'scans_done' member to comedi_async staging: comedi: addi_apci_2032: use comedi_async 'scans_done' to detect EOA staging: comedi: amplc_dio200_common: use comedi_async 'scans_done'

[PATCH v2 05/18] staging: comedi: pcl812: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'ai_act_scan' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcl812.c | 9 + 1 file

[PATCH v2 11/18] staging: comedi: pcmuio: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'stop_count' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmuio.c | 14 +++--

[PATCH v2 10/18] staging: comedi: pcmmio: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'count' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmmio.c | 12 +++- 1 file c

[PATCH v2 06/18] staging: comedi: pcl816: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'ai_act_scan' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcl816.c | 9 + 1 file

[PATCH v2 09/18] staging: comedi: adv_pci1710: use comedi_async 'scans_done' to detect EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'ai_act_scan' and use the comedi_async 'scans_done' member to detect the end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/adv_pci1710.c | 26 +-

[PATCH v2 15/18] staging: comedi: usbduxsigma: use comedi_async 'scans_done' to detect AO EOA

2014-11-05 Thread H Hartley Sweeten
Remove the private data member 'ao_sample_count' and use the comedi_async 'scans_done' member to detect the analog output end-of-acquisition. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/driver

  1   2   3   4   5   6   7   8   9   10   >