14940a52ab
commit 57a1592784d622ecee0b71940c65429173996b33 upstream.
There are 2 issues with interrupt handling in the mxc4005 driver:
1. mxc4005_set_trigger_state() writes MXC4005_REG_INT_MASK1_BIT_DRDYE
(0x01) to INT_MASK1 to enable the interrupt, but to disable the interrupt
it writes ~MXC4005_REG_INT_MASK1_BIT_DRDYE which is 0xfe, so it enables
all other interrupt sources in the INT_SRC1 register. On the MXC4005 this
is not an issue because only bit 0 of the register is used. On the MXC6655
OTOH this is a problem since bit7 is used as TC (Temperature Compensation)
disable bit and writing 1 to this disables Temperature Compensation which
should only be done when running self-tests on the chip.
Write 0 instead of ~MXC4005_REG_INT_MASK1_BIT_DRDYE to disable
the interrupts to fix this.
2. The datasheets for the MXC4005 / MXC6655 do not state what the reset
value for the INT_MASK0 and INT_MASK1 registers is and since these are
write only we also cannot learn this from the hw. Presumably the reset
value for both is all 0, which means all interrupts disabled.
Explicitly set both registers to 0 from mxc4005_chip_init() to ensure
both masks are actually set to 0.
Fixes: 79846e33aac1 ("iio: accel: mxc4005: add support for mxc6655")
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20240326113700.56725-2-hdegoede@redhat.com
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
||
|---|---|---|
| .. | ||
| accel | ||
| adc | ||
| addac | ||
| afe | ||
| amplifiers | ||
| buffer | ||
| chemical | ||
| common | ||
| dac | ||
| dummy | ||
| frequency | ||
| gyro | ||
| health | ||
| humidity | ||
| imu | ||
| light | ||
| magnetometer | ||
| multiplexer | ||
| orientation | ||
| position | ||
| potentiometer | ||
| potentiostat | ||
| pressure | ||
| proximity | ||
| resolver | ||
| temperature | ||
| trigger | ||
| iio_core.h | ||
| iio_core_trigger.h | ||
| industrialio-buffer.c | ||
| industrialio-configfs.c | ||
| industrialio-core.c | ||
| industrialio-event.c | ||
| industrialio-sw-device.c | ||
| industrialio-sw-trigger.c | ||
| industrialio-trigger.c | ||
| industrialio-triggered-event.c | ||
| inkern.c | ||
| Kconfig | ||
| Makefile | ||
| TODO | ||