iio: adc: bcm_iproc_adc: swap primary and secondary isr handler's
authorRaveendra Padasalagi <raveendra.padasalagi@broadcom.com>
Tue, 16 May 2017 06:52:42 +0000 (12:22 +0530)
committerJonathan Cameron <jic23@kernel.org>
Sun, 21 May 2017 12:01:17 +0000 (13:01 +0100)
commitf7d86ecf83cb66d3c4c6ac4edb1dd50c0919aa2b
treef7dc3da6807be4b53e57b66cc7d71c842de59604
parent4eecbe81885180c9f6217ecfd679b1f285967218
iio: adc: bcm_iproc_adc: swap primary and secondary isr handler's

The third argument of devm_request_threaded_irq() is the primary
handler. It is called in hardirq context and checks whether the
interrupt is relevant to the device. If the primary handler returns
IRQ_WAKE_THREAD, the secondary handler (a.k.a. handler thread) is
scheduled to run in process context.

bcm_iproc_adc.c uses the secondary handler as the primary one
and the other way around. So this patch fixes the same, along with
re-naming the secondary handler and primary handler names properly.

Tested on the BCM9583XX iProc SoC based boards.

Fixes: 4324c97ecedc ("iio: Add driver for Broadcom iproc-static-adc")
Reported-by: Pavel Roskin <plroskin@gmail.com>
Signed-off-by: Raveendra Padasalagi <raveendra.padasalagi@broadcom.com>
Cc: stable@vger.kernel.org
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/adc/bcm_iproc_adc.c