MIRRORS/linux-yocto
1
0
Fork 0
mirror of git://git.yoctoproject.org/linux-yocto.git synced 2025-12-17 16:18:25 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

iio: xilinx-ams: Unmask interrupts after updating alarms

Browse Source 
commit feb500c7ae upstream.

To convert level-triggered alarms into edge-triggered IIO events, alarms
are masked when they are triggered. To ensure we catch subsequent
alarms, we then periodically poll to see if the alarm is still active.
If it isn't, we unmask it. Active but masked alarms are stored in
current_masked_alarm.

If an active alarm is disabled, it will remain set in
current_masked_alarm until ams_unmask_worker clears it. If the alarm is
re-enabled before ams_unmask_worker runs, then it will never be cleared
from current_masked_alarm. This will prevent the alarm event from being
pushed even if the alarm is still active.

Fix this by recalculating current_masked_alarm immediately when enabling
or disabling alarms.

Fixes: d5c70627a7 ("iio: adc: Add Xilinx AMS driver")
Signed-off-by: Sean Anderson <sean.anderson@linux.dev>
Reviewed-by: O'Griofa, Conall <conall.ogriofa@amd.com>
Tested-by: Erim, Salih <Salih.Erim@amd.com>
Acked-by: Erim, Salih <Salih.Erim@amd.com>
Link: https://patch.msgid.link/20250715002847.2035228-1-sean.anderson@linux.dev
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Sean Anderson 2025-07-14 20:28:47 -04:00 committed by Greg Kroah-Hartman
parent b9bf012d7b
commit b62607e4f4
1 changed files with 25 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

45
drivers/iio/adc/xilinx-ams.c
View File

@ -389,6 +389,29 @@ static void ams_update_pl_alarm(struct ams *ams, unsigned long alarm_mask)
ams_pl_update_reg(ams, AMS_REG_CONFIG3, AMS_REGCFG3_ALARM_MASK, cfg);
}
static void ams_unmask(struct ams *ams)
{
unsigned int status, unmask;
status = readl(ams->base + AMS_ISR_0);
/* Clear those bits which are not active anymore */
unmask = (ams->current_masked_alarm ^ status) & ams->current_masked_alarm;
/* Clear status of disabled alarm */
unmask |= ams->intr_mask;
ams->current_masked_alarm &= status;
/* Also clear those which are masked out anyway */
ams->current_masked_alarm &= ~ams->intr_mask;
/* Clear the interrupts before we unmask them */
writel(unmask, ams->base + AMS_ISR_0);
ams_update_intrmask(ams, ~AMS_ALARM_MASK, ~AMS_ALARM_MASK);
}
static void ams_update_alarm(struct ams *ams, unsigned long alarm_mask)
{
unsigned long flags;
@ -401,6 +424,7 @@ static void ams_update_alarm(struct ams *ams, unsigned long alarm_mask)
spin_lock_irqsave(&ams->intr_lock, flags);
ams_update_intrmask(ams, AMS_ISR0_ALARM_MASK, ~alarm_mask);
ams_unmask(ams);
spin_unlock_irqrestore(&ams->intr_lock, flags);
}
@ -1035,28 +1059,9 @@ static void ams_handle_events(struct iio_dev *indio_dev, unsigned long events)
static void ams_unmask_worker(struct work_struct *work)
{
struct ams *ams = container_of(work, struct ams, ams_unmask_work.work);
unsigned int status, unmask;
spin_lock_irq(&ams->intr_lock);
status = readl(ams->base + AMS_ISR_0);
/* Clear those bits which are not active anymore */
unmask = (ams->current_masked_alarm ^ status) & ams->current_masked_alarm;
/* Clear status of disabled alarm */
unmask |= ams->intr_mask;
ams->current_masked_alarm &= status;
/* Also clear those which are masked out anyway */
ams->current_masked_alarm &= ~ams->intr_mask;
/* Clear the interrupts before we unmask them */
writel(unmask, ams->base + AMS_ISR_0);
ams_update_intrmask(ams, ~AMS_ALARM_MASK, ~AMS_ALARM_MASK);
ams_unmask(ams);
spin_unlock_irq(&ams->intr_lock);
/* If still pending some alarm re-trigger the timer */