summaryrefslogtreecommitdiff
path: root/drivers/mfd/sun4i-gpadc.c
diff options
context:
space:
mode:
authorAidan MacDonald <aidanmacdonald.0x0@gmail.com>2022-11-12 18:18:32 +0300
committerLee Jones <lee@kernel.org>2022-12-07 16:28:15 +0300
commitc79e387389d5add7cb967d2f7622c3bf5550927b (patch)
treec6df53d1c69881a5ef54b1a039e77f0bd4162b79 /drivers/mfd/sun4i-gpadc.c
parent963cd957694b28e227709608467595d58fcf1605 (diff)
downloadlinux-c79e387389d5add7cb967d2f7622c3bf5550927b.tar.xz
mfd: stpmic1: Fix swapped mask/unmask in irq chip
The usual behavior of mask registers is writing a '1' bit to disable (mask) an interrupt; similarly, writing a '1' bit to an unmask register enables (unmasks) an interrupt. Due to a longstanding issue in regmap-irq, mask and unmask registers were inverted when both kinds of registers were present on the same chip, ie. regmap-irq actually wrote '1's to the mask register to enable an IRQ and '1's to the unmask register to disable an IRQ. This was fixed by commit e8ffb12e7f06 ("regmap-irq: Fix inverted handling of unmask registers") but the fix is opt-in via mask_unmask_non_inverted = true because it requires manual changes for each affected driver. The new behavior will become the default once all drivers have been updated. The STPMIC1 has a normal mask register with separate set and clear registers. The driver intends to use the set & clear registers with regmap-irq and has compensated for regmap-irq's inverted behavior, and should currently be working properly. Thus, swap mask_base and unmask_base, and opt in to the new non-inverted behavior. Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com> Signed-off-by: Lee Jones <lee@kernel.org> Link: https://lore.kernel.org/r/20221112151835.39059-16-aidanmacdonald.0x0@gmail.com
Diffstat (limited to 'drivers/mfd/sun4i-gpadc.c')
0 files changed, 0 insertions, 0 deletions