]> git.karo-electronics.de Git - linux-beck.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)
commitf52666772d579233b5aa3e1fb57187bbec80fc50
tree7d46f92bed39784a8685106418a61fba387d4308
parenta866058f8e40a30c62027990c91aa71008617207
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 1caf7cb46135
("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