]> git.baikalelectronics.ru Git - kernel.git/commit
staging:iio:ad7606: Run trigger handler only once per trigger event
authorLars-Peter Clausen <lars@metafoo.de>
Wed, 19 Oct 2016 17:07:06 +0000 (19:07 +0200)
committerJonathan Cameron <jic23@kernel.org>
Sun, 23 Oct 2016 18:34:17 +0000 (19:34 +0100)
commitd4c4606d8967a0802c66f6c9ca9a3edcfeebdd61
tree7d46f92bed39784a8685106418a61fba387d4308
parent86368ba3492fbd307b328ec7c9086e3e41f1a3cb
staging:iio:ad7606: Run trigger handler only once per trigger event

Currently the ad7606 driver installs the same function for the hard-irq and
threaded trigger handlers. This was introduced in commit 91f520d3431a
("staging:iio:adc:ad7606 Convert to new channel registration method Update
Add missing call to iio_trigger_notify_done() Set pollfunc top and bottom
half handler"). Unfortunately the commit message does not mention why this
was done and Michael does not remember either.

Since the trigger handler function is idempotent (set a GPIO to 1) running
it twice does not do any harm, but is simply not necessary either. So set
the threaded trigger handler for the driver to NULL.

While we are at it also remove the function description comment that does
no say anything that can't be derived from the function name itself.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/adc/ad7606_ring.c