<feed xmlns='http://www.w3.org/2005/Atom'>
<title>pm24.git/drivers/staging/comedi, branch v3.14</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<id>https://git.kobert.dev/pm24.git/atom?h=v3.14</id>
<link rel='self' href='https://git.kobert.dev/pm24.git/atom?h=v3.14'/>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/'/>
<updated>2014-02-07T19:08:46Z</updated>
<entry>
<title>staging: comedi: usbduxsigma: fix unaligned dereferences</title>
<updated>2014-02-07T19:08:46Z</updated>
<author>
<name>Ian Abbott</name>
<email>abbotti@mev.co.uk</email>
</author>
<published>2014-01-21T17:39:05Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=791771e4e0587652e193ac8920e14911045a1dc8'/>
<id>urn:sha1:791771e4e0587652e193ac8920e14911045a1dc8</id>
<content type='text'>
There are a couple of dereferences such as `*(uint32_t
*)(devpriv-&gt;insn_buf + 1)` that are unaligned as `devpriv-&gt;insn_buf` is
of type `uint8_t *`.  This works on x86 architecture but may not be
supported on other architectures.  Call `get_unalign()` to perform the
unaligned dereferences.

Signed-off-by: Ian Abbott &lt;abbotti@mev.co.uk&gt;
Cc: Bernd Porr &lt;mail@berndporr.me.uk&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>staging: comedi: fix too early cleanup in comedi_auto_config()</title>
<updated>2014-02-07T19:08:46Z</updated>
<author>
<name>Ian Abbott</name>
<email>abbotti@mev.co.uk</email>
</author>
<published>2014-01-21T17:06:34Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=f1ffdfcc52744ed264041741020364cfbad9c0de'/>
<id>urn:sha1:f1ffdfcc52744ed264041741020364cfbad9c0de</id>
<content type='text'>
`comedi_auto_config()` is usually called from the probe routine of a
low-level comedi driver to allocate and auto-configure a comedi device.
Part of this involves calling the low-level driver's `auto_attach()`
handler, and if that is successful, `comedi_device_postconfig()` tries
to complete the configuration of the comedi device.  If either of those
fail, `comedi_device_detach()` is called to clean up, and
`comedi_release_hardware_device()` is called to remove the dynamically
allocated comedi device.

Unfortunately, `comedi_device_detach()` clears the `hw_dev` member of
the `struct comedi_device` (indirectly via `comedi_clear_hw_dev()`), and
that stops `comedi_release_hardware_device()` finding the comedi device
associated with the hardware device, so the comedi device won't be
removed properly.

Since `comedi_release_hardware_device()` also calls
`comedi_device_detach()` (assuming it finds the comedi device associated
with the hardware device), the fix is to remove the direct call to
`comedi_device_detach()` from `comedi_auto_config()` and let the call to
`comedi_release_hardware_device()` take care of it.

Signed-off-by: Ian Abbott &lt;abbotti@mev.co.uk&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>staging: comedi: adv_pci1710: fix analog output readback value</title>
<updated>2014-02-07T17:19:30Z</updated>
<author>
<name>H Hartley Sweeten</name>
<email>hsweeten@visionengravers.com</email>
</author>
<published>2014-02-05T21:59:53Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=1e85c1ea1ff2a60659e790ef8ec76c7339445841'/>
<id>urn:sha1:1e85c1ea1ff2a60659e790ef8ec76c7339445841</id>
<content type='text'>
The last value written to a analog output channel is cached in the
private data of this driver for readback.

Currently, the wrong value is cached in the (*insn_write) functions.
The current code stores the data[n] value for readback afer the loop
has written all the values. At this time 'n' points past the end of
the data array.

Fix the functions by using a local variable to hold the data being
written to the analog output channel. This variable is then used
after the loop is complete to store the readback value. The current
value is retrieved before the loop in case no values are actually
written..

Signed-off-by: H Hartley Sweeten &lt;hsweeten@visionengravers.com&gt;
Reviewed-by: Ian Abbott &lt;abbotti@mev.co.uk&gt;
Cc: stable &lt;stable@vger.kernel.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: comedi: fix extra whitespace style issues in ni_mio_common.c.</title>
<updated>2014-01-13T23:04:52Z</updated>
<author>
<name>Chase Southwood</name>
<email>chase.southwood@yahoo.com</email>
</author>
<published>2014-01-11T04:03:12Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=69ba83585a22091455fdbf7f8b3ddc4788b401e2'/>
<id>urn:sha1:69ba83585a22091455fdbf7f8b3ddc4788b401e2</id>
<content type='text'>
This patch for ni_mio_common.c removes extra whitespace causing
checkpatch.pl warnings.

Signed-off-by: Chase Southwood &lt;chase.southwood@yahoo.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: comedi: fix indentation coding style issue in ni_mio_common.c.</title>
<updated>2014-01-13T23:04:52Z</updated>
<author>
<name>Chase Southwood</name>
<email>chase.southwood@yahoo.com</email>
</author>
<published>2014-01-11T04:02:57Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=b9ede3154a4342434c155a7e4daa3915fc9769b1'/>
<id>urn:sha1:b9ede3154a4342434c155a7e4daa3915fc9769b1</id>
<content type='text'>
This patch for ni_mio_common.c fixes several indentation warnings from
checkpatch.pl.

Signed-off-by: Chase Southwood &lt;chase.southwood@yahoo.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: comedi: fix numerous brace coding style issues in ni_mio_common.c.</title>
<updated>2014-01-13T23:04:52Z</updated>
<author>
<name>Chase Southwood</name>
<email>chase.southwood@yahoo.com</email>
</author>
<published>2014-01-11T04:02:33Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=bc4615561fc3df0826ac9a93876770cdbf673a80'/>
<id>urn:sha1:bc4615561fc3df0826ac9a93876770cdbf673a80</id>
<content type='text'>
This patch for ni_mio_common.c removes many unneccesary braces to fix
checkpatch.pl warnings.

Signed-off-by: Chase Southwood &lt;chase.southwood@yahoo.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>comedi: Humusoft MF634 and MF624 DAQ cards driver</title>
<updated>2014-01-13T23:03:12Z</updated>
<author>
<name>Rostislav Lisovy</name>
<email>lisovy@gmail.com</email>
</author>
<published>2014-01-09T22:46:46Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=04b565021a83ae1940f1f9b801d3ce5b5fc8ee1e'/>
<id>urn:sha1:04b565021a83ae1940f1f9b801d3ce5b5fc8ee1e</id>
<content type='text'>
This patch adds Comedi driver for Humusoft MF634 (PCIe) and
MF624 (PCI) data acquisition cards. The legacy card Humusoft
MF614 is not supported. More info about the cards may be found
at http://humusoft.cz/produkty/datacq/
The driver was tested with both cards. Everything seems to work
properly. Just the basic functionality of the card (DIO, ADC, DAC)
is supported by this driver.

Signed-off-by: Rostislav Lisovy &lt;lisovy@gmail.com&gt;
Reviewed-by: Ian Abbott &lt;abbotti@mev.co.uk&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: comedi: amcc_s5933: "no space before tabs" coding style fixes.</title>
<updated>2014-01-11T20:41:56Z</updated>
<author>
<name>Michal Kwiatkowski</name>
<email>michaelflowersky@geekingspree.com</email>
</author>
<published>2014-01-09T22:58:55Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=509e42e1d35350cc5c1bbf1800a0d743be16a63a'/>
<id>urn:sha1:509e42e1d35350cc5c1bbf1800a0d743be16a63a</id>
<content type='text'>
Fixed a coding style issues.

Signed-off-by: Michal Kwiatkowski &lt;michaelflowersky@geekingspree.com&gt;
Reviewed-by: Ian Abbott &lt;abbotti@mev.co.uk&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Staging: comedi: fix spacing coding style issue in 8255.c.</title>
<updated>2014-01-11T20:41:56Z</updated>
<author>
<name>Chase Southwood</name>
<email>chase.southwood@yahoo.com</email>
</author>
<published>2014-01-10T03:59:41Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=b281539839ef8eacc79f4e12e5ac13202de13d0b'/>
<id>urn:sha1:b281539839ef8eacc79f4e12e5ac13202de13d0b</id>
<content type='text'>
This patch for 8255.c fixes a spacing warning found by checkpatch.pl.

Signed-off-by: Chase Southwood &lt;chase.southwood@yahoo.com&gt;
Reviewed-by: Ian Abbott &lt;abbotti@mev.co.uk&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>staging: comedi: usbduxsigma: removing unneccesay attached info</title>
<updated>2014-01-09T18:51:54Z</updated>
<author>
<name>Bernd Porr</name>
<email>mail@berndporr.me.uk</email>
</author>
<published>2014-01-07T21:43:43Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=2783980525df12b9c49e8b4baaff06abc7f2f8f2'/>
<id>urn:sha1:2783980525df12b9c49e8b4baaff06abc7f2f8f2</id>
<content type='text'>
Comedi core now reports that a device has been attached so that
the driver itself won't need to do it any longer. The driver now
just outputs the offset of the ADC converter which is a soft indicator
of the health of the board and also the user can grep this value
from the kernel log easier for debugging purposes.

Signed-off-by: Bernd Porr &lt;mail@berndporr.me.uk&gt;
Reviewed-by: Ian Abbott &lt;abbotti@mev.co.uk&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
</feed>
