[PATCH 2/2 v4] staging: comedi: addi_apci_1564: tidy up apci1564_reset()

Chase Southwood chase.southwood at yahoo.com
Wed Apr 23 07:08:52 UTC 2014


>On Thursday, April 17, 2014 1:33 AM, Chase Southwood <chase.southwood at yahoo.com> wrote:

>The reset function for this driver is a bit of a mess; clean it up to
>ensure that it is functioning properly.
>
>Signed-off-by: Chase Southwood <chase.southwood at yahoo.com>
>---
>2: *Changed order of register accesses for digital input registers back to
>original ordering.
>*Removed read of digital output status register and reordered the accesses
>of digital output registers to agree with the order found in addi_apci_2032.
>*Fixed copy/paste error in the counter register reset lines.
>
>3: *Returned the reset callback to the boardinfo...oops.
>*Don't write APCI1564_DO_IRQ_REG (it's read only).
>*Don't clear counter reload values...counters have already been
>disabled.
>
>4: Leave this in hwdrv_apci1564.c for now (it can be moved after separating
>this driver from addi_common.c)
>
>NB: This patch has been renamed in this version.  It was previously known
>as "staging: comedi: addi_apci_1564: fixup and absorb apci1564_reset()"
>


Ian and Hartley,

Just following up to see if you have any comments on this version of this patch.
PATCH 1/2 from this set was added to staging-next today, and I'd like to know
where this one stands before I work on separating this driver from addi_common.c.
I'd be happy to make any more changes that are desired, or if it's just better to
drop the patch for now, that's ok too.  Just want to make sure it's taken care of
before I move on!

Thanks a lot,
Chase


More information about the devel mailing list