]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commit
iio: ad_sigma_delta: Properly handle SPI bus locking vs CS assertion
authorLars-Peter Clausen <lars@metafoo.de>
Tue, 19 Mar 2019 11:37:55 +0000 (13:37 +0200)
committerStefan Bader <stefan.bader@canonical.com>
Tue, 13 Aug 2019 12:11:36 +0000 (14:11 +0200)
commit85da82a8434782a86103b83ed86cd499310b5c5e
tree08658bf95deb88d133d76b67ec8f73e57325b778
parent355442e21c8c9e446364411286a473896ab7c162
iio: ad_sigma_delta: Properly handle SPI bus locking vs CS assertion

BugLink: https://bugs.launchpad.net/bugs/1837517
[ Upstream commit df1d80aee963480c5c2938c64ec0ac3e4a0df2e0 ]

For devices from the SigmaDelta family we need to keep CS low when doing a
conversion, since the device will use the MISO line as a interrupt to
indicate that the conversion is complete.

This is why the driver locks the SPI bus and when the SPI bus is locked
keeps as long as a conversion is going on. The current implementation gets
one small detail wrong though. CS is only de-asserted after the SPI bus is
unlocked. This means it is possible for a different SPI device on the same
bus to send a message which would be wrongfully be addressed to the
SigmaDelta device as well. Make sure that the last SPI transfer that is
done while holding the SPI bus lock de-asserts the CS signal.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Alexandru Ardelean <Alexandru.Ardelean@analog.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Khalid Elmously <khalid.elmously@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
drivers/iio/adc/ad_sigma_delta.c
include/linux/iio/adc/ad_sigma_delta.h