pcl818: Fix write instruction for AO subdevice
authorIan Abbott <abbotti@mev.co.uk>
Tue, 20 Mar 2012 14:40:57 +0000 (14:40 +0000)
committerIan Abbott <abbotti@mev.co.uk>
Fri, 23 Mar 2012 14:11:45 +0000 (14:11 +0000)
commite0da48cb4559df480136b671a6e84ac36ca6a8ad
tree2e7ecd49c8e7b42858d3549c50cd928337301abc
parent755dbfe700494e3e34a0f282b399471d12be45cc
pcl818: Fix write instruction for AO subdevice

Max Korovkin reported that analog output was not working on Advantech
PCI818L and found the cause to be an operator precedence error in
pcl818_ao_insn_write().  The effect of the bug was that the function
wrote to absolute I/O port addresses 6 and 7 (part of the ISA DMA
controller) instead of the port addresses assigned to the board.

The same bug was reported by Kudlaty on 2010-11-25, but was not
investigated further at the time.

The same bug was already fixed by Roel Kluin in the "staging" version of
this driver in the 2.6.31 kernel back in May 2009 (commit
f764df88f86c8db669bd7bfac230b9d2689ae322 in Linus's linux-2.6 git
repository).

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
comedi/drivers/pcl818.c