staging: comedi: cb_pcidas: cleanup dac08_write()
authorH Hartley Sweeten <hartleys@visionengravers.com>
Wed, 27 Jun 2012 01:12:15 +0000 (18:12 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Jul 2012 22:23:56 +0000 (15:23 -0700)
commitb436356dcc67a665da07441ab113e9a5652073e5
treebd023d8c60e1a2db680208f5aee32be9c3e09266
parentf66faa576f084bed49d4a34e0bc2289ff0d07a70
staging: comedi: cb_pcidas: cleanup dac08_write()

Reverse the logic of the test against the cached value so that
the function only has one exit point.

Make the logic of the dac write a bit clearer by create a local
variable for the address used in the outw() calls and masking the
value to write and setting the enable bits before doing the outw()
calls. Add a comment just to make sure it's clear.

Also, add a comment about the 'return 1;'.  This should be
insn->n which is the number of data parameters used to do the
dac08_write_insn() but the insn is not a parameter to this
function.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/cb_pcidas.c