From nobody Fri Sep 25 20:47:54 2026 Received: from mail-oi1-f177.google.com (mail-oi1-f177.google.com [209.85.167.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6B33233E360 for ; Tue, 8 Sep 2026 16:01:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.177 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788883301; cv=none; b=oMrHHVkqM71zaEe6kENuZ1gwefqGRptwaGoxKkRuhy5KIaQORT16s6kDBL1o9GD8JkU9+lYvfpQnwqAHWi5/NP4B5M+o/wvB5UCQwTgB4/OI//q0HDAyepJjr8cCA2nuEaqRW+Maqdi+7ju0+Prl99hzkF8xEGGJ9E2nMYWw3Wo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788883301; c=relaxed/simple; bh=fSnUC/Uevm5Vbrih7n+FM4vcyD9bqX8J6oiWP1kp/DY=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=MQmYkT+arp/1J3V00qpEOBOGLNLumFmk9zo8jyhm7SQg8MP7nisq5299rP8qgxWGWsvm/jQI2yKH76Exb57g2B4QirykdZapbTHRSuISdeHdrI9O33950TDQT3kOZM9PwFa41W9z9ltbuoIfz80YyqK8w6hUOwu8qRys8m/Lznc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=eNV+3LFb; arc=none smtp.client-ip=209.85.167.177 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="eNV+3LFb" Received: by mail-oi1-f177.google.com with SMTP id 5614622812f47-4b28d7c729fso3138503b6e.0 for ; Tue, 08 Sep 2026 09:01:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788883298; x=1789488098; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=1HavqBuvOK9+7hkcuCpVB9Cv/dvjoerCr3RPaWhvFOI=; b=eNV+3LFb0X8EySGhpt/G4eUKLZBtSxeBr0h2RzFyu8qL1fHSnhBwr6u/Gdrky1Ed15 OCrGbQ8L37t4w4Qs5LoAEJyB3Gcti0jnLzTxxIHWzyFBwVQ8Eea/mLEpKgCeCAORuB1s 8ytcmAp0iXmH2Q7HjJ7wnMlWplnKWslbMGq4Xhqu3Qy96TNEYiAXThejrhsZrTRBzYSp 5N4dSm7ORo3AGLkh2lmem5UF3m3fMCQ+EPaIHKxb3XfVcuK3fAgPiFQDh47YxYpJzmNm YdmUYZl/6Cu9+SngJizmQTXBT/1UlCsBU6qBSn4wfLRxeH/aaim72RIEI9Pd/bI0agAG rnKQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788883298; x=1789488098; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=1HavqBuvOK9+7hkcuCpVB9Cv/dvjoerCr3RPaWhvFOI=; b=g+rLP9jxUgGuqcG6HtwGh0sBzyrgjyLuRElxlmXUUt5y9xIXY6ttFrjZ+F9J6MUUs+ Vfy8PsG0j4ZdstxlwjJEc3UJrZQpnM2EZ4wzgZ1TRNAcneATx67jTVcGn+0x+pYb98KV 64yZ102cCRZ8zTLO7oWbjOKkWT05fyyvqcb4DK3ioYS8CitZZ05B8NGH1W8RrAslJGAy txd2clHL1x/z5sxqmRregIAKKsIfwtueUa4Le1rQebLlp98mzzvR/7hx6RLFpkXEmsQx H/JbCa5NlwDUfmCwzc8lHBKtDxpNbQ0FR3V9nbGe5oeJXFGDsBwcTkrGHqoMAX1WfO8b CJfA== X-Forwarded-Encrypted: i=1; AKwUvByN8sL2BVP0u63sFTERf/CWML2yBvBa/zIHayLDD0Q9QWG+GoQMoteDiDh5bd1A5brBslUpKckMOvg7s8c=@vger.kernel.org X-Gm-Message-State: AFuF++m5sGRuV9ftC1ptcJ5H3ZkZSnFS/vy9m8tZDzeLbdHpZgnk7svO Vkm1i40BkRxfjdEeDbLPoIiZRMu8TYKB5FtnFXBtrrBfH7BXljYZVK4+ X-Gm-Gg: AYBFou0fy9e1AlQ2KN0lAVB6XhB4uCaAnZyIfI7wZ+lMF8c6TvfPzUL1uevQ9cQEM1M w5sRFnxacBqIXizfMowyZW93ouTotLcbwxzkaEMdB+YDyXYU/6LNMj1oXDzBcAWL5LtGaCOQnZZ w+dIb6z6rymSvqn8453a5VzHweH8YFKob3R4LC0cux9Y21o7OFLBJYTWu+oi27WFAPup0xwAMrY ID6NogGB/5pJwDcHnHgNjduAuGgZWt15dP7ZP9J1Hp+UULh/Xm3yNBiEc+hB/zXhb/zKujVjfet d2r7vIhTWA424Yvz4Ud0FQ1Rg4K5wtl1WNGTNhh9frkOcLWKT29JZ0hMk1ljKTUU6O9+8m0m8es R7mgVsMJYzFu/tOCbbhCGmZYG4Btt3VzZ+3YiER20m2Ry41jcsVYeMPal2CJM90R0/7P08ZKuNG xE3Gufpacu6G3USsUnzEZPR/roOz/cKQs9qqi/QOUDiaNTGjqFKRvrTBzbUm+j428VQVr/7g1hP wK8760U X-Received: by 2002:a05:6820:6ac3:b0:6b0:4170:59ad with SMTP id 006d021491bc7-6b6fc7d1f64mr19096852eaf.24.1788883295449; Tue, 08 Sep 2026 09:01:35 -0700 (PDT) Received: from kernel ([45.251.35.112]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-3339a534df1sm43084171eec.7.2026.09.08.09.01.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Sep 2026 09:01:34 -0700 (PDT) From: Mohamad Raizudeen To: herbert@gondor.apana.org.au, clabbe@baylibre.com, davem@davemloft.net Cc: skhan@linuxfoundation.org, me@brighamcampbell.com, jkoolstra@xs4all.nl, linux-crypto@vger.kernel.org, linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, Mohamad Raizudeen , stable@vger.kernel.org Subject: [PATCH] crypto: amlogic: Fix IRQ handler return value and fallthrough logic Date: Tue, 8 Sep 2026 21:31:20 +0530 Message-ID: <20260908160120.4805-1-raizudeen.kerneldev@gmail.com> X-Mailer: git-send-email 2.53.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" In irqreturn_t_meson_irq_handler(), when an interrupt matches a flow but the status register is empty, the driver prints an error but doesn't return. It falls through the loop and incorrectly prints an `unknown irq` message. Fix this by returning IRQ_HANDLED immediately after the error print. Additionally, the handler returns IRQ_HANDLED for genuinely unknown interrupts. This masks false interrupts and prevents the kernel from detecting interrupts. Return IRQ_NONE instead for unhandled interrupts. Cc: stable@vger.kernel.org Fixes: 48fe583fe5417 ("crypto: amlogic - Add crypto accelerator for amlogic= GXL") Signed-off-by: Mohamad Raizudeen --- drivers/crypto/amlogic/amlogic-gxl-core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/crypto/amlogic/amlogic-gxl-core.c b/drivers/crypto/aml= ogic/amlogic-gxl-core.c index 6cb33949915f..302b398405e2 100644 --- a/drivers/crypto/amlogic/amlogic-gxl-core.c +++ b/drivers/crypto/amlogic/amlogic-gxl-core.c @@ -38,11 +38,12 @@ static irqreturn_t meson_irq_handler(int irq, void *dat= a) return IRQ_HANDLED; } dev_err(mc->dev, "%s %d Got irq for flow %d but ctrl is empty\n", __fun= c__, irq, flow); + return IRQ_HANDLED;=20 } } =20 dev_err(mc->dev, "%s %d from unknown irq\n", __func__, irq); - return IRQ_HANDLED; + return IRQ_NONE; } =20 static struct meson_alg_template mc_algs[] =3D { --=20 2.53.0