Received-SPF: pass (google.com: domain of linux-kernel+bounces-131914-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Markus Burri <markus.burri@mt.com>
To: linux-kernel@vger.kernel.org
Cc: Markus Burri <markus.burri@mt.com>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Michael Hennerich <Michael.Hennerich@analog.com>,
	Jonathan Cameron <jic23@kernel.org>,
	linux-iio@vger.kernel.org
Subject: [PATCH v1] iio: adc: ad_sigma_delta: Clear pending interrupts before enable
Date: Thu,  4 Apr 2024 19:51:26 +0200
Message-Id: <20240404175126.110208-1-markus.burri@mt.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?Ul2duN86phhMzdxoM63VRaFR+bEU2eYEEYy1fTYpynZa4VyTUAUXaUkKSgWS?=
 =?us-ascii?Q?84KKLTSApNofiN7COhx/A1cZ5UbcE/owSQXbJYJ7UWlfWyIe4FFKluhqsqx9?=
 =?us-ascii?Q?12TQ249A8DDtiRd6gG8vtmr0uiL/9qEj+SC6Wb5P2VAgADYdZ6XP6SfKUXpU?=
 =?us-ascii?Q?CrrmvMb6I9KPxnp2XTMIoYL29t3tCVC+BW8kPiFsyaUqYDJpmI2IRJMnSqD1?=
 =?us-ascii?Q?HesMr8q2wTdzK53sqCc/87BWD6guq9G6WNUnJBbd9g+fWmDgLUe+Pd4R6/CK?=
 =?us-ascii?Q?QA6dqYoV0lNORjlofwfTXzOs5pQgmXMw6WsYAMLt8wHIOR9hvkOWAvdVYxcd?=
 =?us-ascii?Q?EdqJpZPIis93838JjC54kS8E2ZBVnPXUXFWXuTdljya/1sZwaayMsW2htbuT?=
 =?us-ascii?Q?NjOhkAq38uWEdtV7eHutp8dh9jfIt5SxZjLPE7GipjXzzoqudAHXxZdOCiT/?=
 =?us-ascii?Q?eZ+/yAAeIOvUgAOcuScXWpXLf3zkucVm3Nduw6I4Ayp6DbkXItRv4gFxTV1e?=
 =?us-ascii?Q?VCRjjvJDyTxt7GTeLprWCKTDPKXa973yaCxRyU+QLWcQtGdbJqB0NA6hsE0a?=
 =?us-ascii?Q?Y+7yAYMi+6oxHhFdPx044qMUOt+YdmuUyxxrCeEqShp4UITi7/C4Rc8OJpIt?=
 =?us-ascii?Q?qHU6lqUyvKhSYMSraYPDu7AwXnb2cEE4PB4V0br+Csi+xGP9f+jegdfkuxQr?=
 =?us-ascii?Q?FUFJ9bKp4dbGU39sL4X4jiJnfUFIAD0QpRarbdSly6WHtTaY7eu8irLKuYTK?=
 =?us-ascii?Q?R+XPL5AJEOKo78Hlw4akT4zsAjVkmwNlvn5a8WW6SbKYUvSd2NliwqEEDsAG?=
 =?us-ascii?Q?ZLKu36xrS4IJXwJ30Nrt5+CT+VRf7mcha9VCmjEqiyaAC/XmjhzZGUdkKIVN?=
 =?us-ascii?Q?55RzqM+o2chQ5xRO0ceSZuGNmtUv4n12j+XeRLF7Z/nLJnSgU7W5xSL22MmG?=
 =?us-ascii?Q?AzYTw0m1oY86r3AdNnb5+cXnplA7sggc8hcur5T7OrbhmdCviQcnv+Kxl5AQ?=
 =?us-ascii?Q?811swNmgR5FCR6BDCMvwpx8fVgApZUyZLh17L9bGseuoMCxm5MvOm8ipYXBL?=
 =?us-ascii?Q?6qtGlTEPYVaubm29Ysp0MyZqIDymhHjtHRY6xL0UswqnEllYCpkYm4mU5cL+?=
 =?us-ascii?Q?v0aB1YYbq9g/UPmm2wVRl2+H0vCXzaacL0ouuMJE7HRzhpM5fiNOtYY7HhAk?=
 =?us-ascii?Q?caak2F1C7qgWHC1nPEuH8NIojRSEgsOIwhAIKd5BLWeTmxfXowHhiUZW+S6y?=
 =?us-ascii?Q?BsxRIyz6ZrupbVD0SkXFrjerbQXH2CDP9Sg19VWJwoiaQutKb3lhEess2SJ/?=
 =?us-ascii?Q?72bOaHWOPfGFKJcX0A4f3IwcUrGYAB/cFIdpG5DKFcDL13vxY8TzgKMuDnUw?=
 =?us-ascii?Q?6LKoXFbr4BAfxZ2oFz4+4BitQ1FwOMvf8EFF7kvzSH2pVTvlL5tlZXbviUF5?=
 =?us-ascii?Q?HfXLQm2T/Hr+8SUhQhLbl/Rui3qlooMFgCi9LBpAqdeY7JJNcP5cSigBurJv?=
 =?us-ascii?Q?Hr1MwrbYmMKAJExnYGGswugUaT+oWy044i7obqEIQ2LT+4AJ5V4Foc8I5k+y?=
 =?us-ascii?Q?y4KV9ibMCLPlssd4LSVKT9++9WRUHVDD3nuMp4hX?=
X-OriginatorOrg: mt.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c161d3e0-b2ac-4554-2a14-08dc54d009f9
X-MS-Exchange-CrossTenant-AuthSource: DB9PR03MB7513.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Apr 2024 17:52:47.8090
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: fb4c0aee-6cd2-482f-a1a5-717e7c02496b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: unnyy/YZxH9fQlZnt3fLW5pkXYoZWH4BQFQfSz6R4ijM9/8RJt8SMIxA8JZnw/hS1FQoj3IQ4wemqYcLt97WUA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS4PR03MB8339

For device will enable and disable irq contiously like AD7195,
it use DOUT/RDY pin for both SPI transfer and data ready.
It will disable irq during SPI transfer, and re-eanble irq after SPI transfer.
That may cause irq status bit set to 1 during spi transfer.

When the active condition has been detected, the corresponding bit
remains set until cleared by software. Status flags are cleared
by writing a 1 to the corresponding bit position.

Signed-off-by: Markus Burri <markus.burri@mt.com>
---
 drivers/iio/adc/ad_sigma_delta.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/iio/adc/ad_sigma_delta.c b/drivers/iio/adc/ad_sigma_delta.c
index a602429cdde4..59544c39642a 100644
--- a/drivers/iio/adc/ad_sigma_delta.c
+++ b/drivers/iio/adc/ad_sigma_delta.c
@@ -515,6 +515,10 @@ static irqreturn_t ad_sd_trigger_handler(int irq, void *p)
 
 irq_handled:
 	iio_trigger_notify_done(indio_dev->trig);
+	/*
+	 * ACK pending interrupts from spi transfer.
+	 */
+	irq_gc_ack_set_bit(irq_get_irq_data(sigma_delta->spi->irq));
 	sigma_delta->irq_dis = false;
 	enable_irq(sigma_delta->spi->irq);
 
-- 
2.39.2