>On Thursday, April 17, 2014 1:33 AM, Chase Southwood 
><chase.southw...@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.southw...@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
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to