From nobody Fri Jun 19 09:30:49 2026 Received: from TY3P286CU002.outbound.protection.outlook.com (mail-japaneastazon11010068.outbound.protection.outlook.com [52.101.229.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id ABC0B14F70; Sat, 25 Apr 2026 15:51:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.229.68 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777132306; cv=fail; b=bQxrjJLmqYC+A6+Xz+V+kWuIAbyvvg4zJ/4hmGzpvvlVXtUfa7QWJ53voUvcVxpyC+hoslXCPFl8JfqvLpTE+fAceKMh4hfyRlpwnFsQ66GYxSxQ52IhUOmOX0Y9/537WfIcqEW50WERghsHRa0YUIP5LlIJcI86BAoRmrT6qjA= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777132306; c=relaxed/simple; bh=gUxeMeW4Hfw1f8k8xWKVHEqwLKy/QKjZhZydgZfYiOQ=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=Kr9axot3XX3gyOWqpACO+3253sT9Y5SwSrCekmgiUeJSAM8m21cyExpeLg6wrWGwjo+GD1pk4GhQVvNuf6hfxEa3CVqoDvl5MTRk5kYt9K7xMX4pSYxdBvwFdtsTXm1ARgDQjNE0+AvqSuZmMUPRB6osvwa31RsMLbdoeyBOXFs= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=bp.renesas.com; spf=pass smtp.mailfrom=bp.renesas.com; dkim=pass (1024-bit key) header.d=bp.renesas.com header.i=@bp.renesas.com header.b=CX3smKdd; arc=fail smtp.client-ip=52.101.229.68 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=bp.renesas.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bp.renesas.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=bp.renesas.com header.i=@bp.renesas.com header.b="CX3smKdd" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=WB8UvyNMDDzR5pPiK31BCqsH/7g5nJx8x+C844WiIcW1IMNpqUCYpmG2EYfONA8aP/Y+NS0ub2bbx519FJPdtJ0zRSr6ZPkIO+iItYi3Q6BCDnnW3I1rQgkuOT+MhZrYnyiTlHUPNW3AgKc+ZzVI4PlZ83wLfyzT6dmLwGGjWvpHIKK8TWspK3UsI1DZWW9oWNyws5Hxna91iqaTmaqfGk3gTPZ4fTrJvYnXhsNx1xmc2HyTOgWfx6TJFRjbPWlNOXpWaNObS3Z8MLDOCirE603e3ZxIx9bFcjkLvYCS6Hlo7AZ2YLcaRF9vQF7tTQOQOapVuwTPSa4xB3H648Li7g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=SgLPNY1mwTRtNHDgct6ujuu84+CN1chjenKtp8sAv2A=; b=iPB89I3GvhCu1Lk2Zg7ofJHVgGdVHTFe2Dn1LurGICFdRQ0Z9k+F0RtyfF9BtZF19B98Hl72Oq1vGTbW+/jvLTk6VjP7M7MbEUkjEe3bpr5QXhpb4K1PnNxHGYR5w/nG/eX0eRmg09or3qjI2ksHfEiPIQ4ggKz0svvrs4FcD7LJ5sD6VNk7dveAxSqNZAzPuwr8yM/5Ssccv8G/hpmJOkXGTcLkQU0ppl6ymqkAHXxJgKCoV7oiKeBo3gVcV9F3rfCo53p7TVPdjzQmuUam5rzzsMs9ewNWwtOnxEKXih1ehD21mjZ1KY5V2jSslBuLiPVhkorpX1TOT+fQ+l9qCQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=bp.renesas.com; dmarc=pass action=none header.from=bp.renesas.com; dkim=pass header.d=bp.renesas.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bp.renesas.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=SgLPNY1mwTRtNHDgct6ujuu84+CN1chjenKtp8sAv2A=; b=CX3smKddvxnV9iMlvrGKPZGJtKlQtLvT4U9bNAd8QGa3CN0yRI0BiEWtZ0FYGQPApCPJ6E1Mf4i4TuO6CFaybzUdFCkJeWtmmwoaSrhTYaZ+XeVwJhKHySe7jYmyjhQZsPRj8WYxiHo9Pr8c61PA7mrjxKTbG19d6YdCXsglWaU= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bp.renesas.com; Received: from TY6PR01MB17377.jpnprd01.prod.outlook.com (2603:1096:405:35b::6) by TYCPR01MB7315.jpnprd01.prod.outlook.com (2603:1096:400:f1::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9846.24; Sat, 25 Apr 2026 15:51:42 +0000 Received: from TY6PR01MB17377.jpnprd01.prod.outlook.com ([fe80::f373:26d6:86c4:6aa3]) by TY6PR01MB17377.jpnprd01.prod.outlook.com ([fe80::f373:26d6:86c4:6aa3%6]) with mapi id 15.20.9846.022; Sat, 25 Apr 2026 15:51:42 +0000 From: John Madieu To: alexandre.belloni@bootlin.com Cc: ryan@bluewatersys.com, akpm@linux-foundation.org, m.grzeschik@pengutronix.de, Denis.Osterland@diehl.com, linux-rtc@vger.kernel.org, linux-kernel@vger.kernel.org, biju.das.jz@bp.renesas.com, john.madieu@gmail.com, John Madieu Subject: [PATCH 1/2] rtc: isl1208: Fix returning errno as irqreturn_t in IRQ handler Date: Sat, 25 Apr 2026 15:49:58 +0000 Message-Id: <20260425154959.2796261-2-john.madieu.xa@bp.renesas.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20260425154959.2796261-1-john.madieu.xa@bp.renesas.com> References: <20260425154959.2796261-1-john.madieu.xa@bp.renesas.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: PA7P264CA0117.FRAP264.PROD.OUTLOOK.COM (2603:10a6:102:34c::20) To OS7PR01MB17371.jpnprd01.prod.outlook.com (2603:1096:604:43e::6) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: TY6PR01MB17377:EE_|TYCPR01MB7315:EE_ X-MS-Office365-Filtering-Correlation-Id: 815cc159-34af-43be-c21f-08dea2e28aaf X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|376014|1800799024|366016|38350700014|56012099003|22082099003|18002099003; X-Microsoft-Antispam-Message-Info: jbB6D8ia+7GbxYfHVVEf8E1KT1tFnE8wjImGlkGPLqcKV+RKcFxcBAbEReH7PwkaK+GDhy/0QRmYr4E2uuALI3RZ3vqei8SvORHGDsqWf6r/s/PpOiBe1FPjvTjC1rsG+XAJyomeiB9kY9mIpIrsg2Sz33nGdn2nkKyL6FHWXoft0T0MZ3NlSY7/lrxQTnR9Iewo4gJDiX+urAUJBTMm/ULBgMj2pV3f7g/1jlQkfDuGvKj0PSCunKmwVO/fAGcVoE8Q2BSokJ44mwMK9/AxSI8I3c7KpjmtcVPF7SmTrJdtD0p03WI+9AL2DHi2csFmR4bOdFqIg/Y+wPBjOnsVTOeXY/riFYv0Qqmm2nJWk/LjZOtOv2sZ6QYyswHm6CDRlqfOISjt9bO/k1+njVXTJR4rwRcmE17gwCoCxNFN/tffix6wnn5ONvbC1hz6sWiE2EiDaqqn+/Daj3Nkaz+yoxT5QEEOx3tCEGgRf80j6tr+u0IkxDFl1izriVweM9wqyl6kW3Skf5Keb+9cCagAukksEsaUdqm2mslEoAywY5jCNdXhEhUIVdaS9OHR2/K0gk5HglzhMgRvwMoKPyR6Ex52g4gRIxQkjTgflxCpRsjGBPrwtY23If21xrP8Wvjvk5oVxSAlDiaCUaJ8REWeVhgPO/zhUyJj1VvRl88vO77klOM6z57tDvcJQiMhp2JSwwfdqXIwoXQT3/5sQIbnhzHsobh9SiZVHuUwsC22q4KdSPjkq3uV37tgIUvAjR1UtlVU22Kyq+zxPBy7I9+bq9sjRKVGFiffZCOGmfkW6LQ= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TY6PR01MB17377.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(376014)(1800799024)(366016)(38350700014)(56012099003)(22082099003)(18002099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?xaEPHCtH43QuVrFZGIH5QqweIoL1mW/vF1CGwQXp9cEwMKFEIkbS+NmNubC0?= =?us-ascii?Q?xGMGzXKNkxGbBzqT3TuRcl13yJkbn5dEcddK9ZPZ1Lna9NGkwEIduQgp+St4?= =?us-ascii?Q?09L4EQzkpUIS8QdIT4WFdx8rpty9qd2X6/Pd2zvOKC73S/Fv8+DUfaI5ENTw?= =?us-ascii?Q?nhWHM0hgFc175OblndBlSP+Q2f3+yunMB+Aa8TeWXTxCk4BXiD/8aYMf02fD?= =?us-ascii?Q?ajKZWehF0VITkVlACY+p/YQ4jOwqJLuUjud1g4alyCVOMBd9SHKGu5il6ubZ?= =?us-ascii?Q?CykIkQinR0fmkzgmJAr6UI3JeejcKZZW31LgUNvK3YeWJw7WJZAjqZlVXWg3?= =?us-ascii?Q?cUtv/nT7hLhvRJScztoUWty0ey2pn3GCvVpLdX7MpyJzNvuYD79gyivRNW6F?= =?us-ascii?Q?6QwyCA4YZVd1rc/9zxMyQf+EGBI1xujno176zh/7yFwX3/AP1nL1LqBBzIYL?= =?us-ascii?Q?XziYWKfVpMHAU5fB7aEam3w249KuYQBeEqG85NKWfan3PathLLp96crfEUng?= =?us-ascii?Q?X1hv6MThn4RHg74CvXoqT1+FAMy/yYj+v+UIf30EqN4a18BaYBnCnzO9us41?= =?us-ascii?Q?2Fw1wxX//cbxCUJxg0sGgUweoxCMNtQJOBtP0W2pqS6A8Ozho40DjKx2n3XH?= =?us-ascii?Q?En1i+Gc+X1gbEer7WDXIkkLxEKZGXyh84B69Wekl2HeBKWT28deDNQpeQnyT?= =?us-ascii?Q?Ldw6WGyn4/xVNb9ZvKqhGQNGBBunQQnqNHKFE5wsBHL+OaGov7zG7WmC0S2T?= =?us-ascii?Q?NuCBlyIS2/RE/5mjVKFqWrLuUcscXPMCzXoqcdmzphAYc52EwOP7VTKYF2cM?= =?us-ascii?Q?9Ml5fcRwQxqjs+Z5SE3HxcHfCJzviCh/8C9DU2doAskhPLEAnbWeXiHMBbls?= =?us-ascii?Q?al4M0gwL5uejhdCfyDVekbu9bNIdR8t9abmT6i8L+539Hwi2JERuMLurDOSN?= =?us-ascii?Q?U+eUGl0dkhd/4jRjSHF9YGVC/SDyniboAN6U3VQD5OGmNP2Lv0KO/AYvcaK3?= =?us-ascii?Q?4V0SnvmEi9OsG7hjIPlKvDvur7T55BmaN2uBHKi5ZPw1n6NDQgMQoPJJVIiE?= =?us-ascii?Q?dglna/xoQX+LIBAzuq6rj3yC7NXnCP6+Lwx8AFIMNzudFvE+wx4d5LgF/mLX?= =?us-ascii?Q?NsNK9irvNQZA8ce3HZT9jS9OyQGo8G8BORV7zIHsZUoIHDevEBRZbNFaCIsL?= =?us-ascii?Q?MagNWiS0EfTDipfVUdQY26hQQ6zsnxb1grNetGXdPG0TeXHPQQCobxVU2W5s?= =?us-ascii?Q?rozOcUhwz38ljAD4/xHY1wsC9jkqhNqwcQhabn/69m8vAxNkMNmNVW6E07C1?= =?us-ascii?Q?o2A5u1z/reaDOIcBxbsRCjnm1359WUIAtpOrAL8SLuWduDf/O8dIuMfnGtsp?= =?us-ascii?Q?vFlaVj2ZsXx4Kw1lRDYpCbIdlv/L2sZg0cdJHtSqcDpuPo80D5ubXAjm+SJ/?= =?us-ascii?Q?XIIKQUElyFtF2jNhWfWfMjUyXJKBuuvvnEj6dHi3qaA7JqYT+Whg8f8dRRY7?= =?us-ascii?Q?DHi11SERsc4WvYLS1bPaCsrtZwnQbhlwEGsrHI1GWk2WewlzLzwfnEaoCiPU?= =?us-ascii?Q?ulJIP8kYl8g8GhD2oZwxypEAB2nek6zGd7IbyvHAZxw2SK3DhI4SWQtHS1Hk?= =?us-ascii?Q?S3wcxtRgy57eAeUVUQ4AkMu0bLbBSFwrLIbCZEys23Zj9ddWAONKQdJ0Wba/?= =?us-ascii?Q?73qx0sry6dpIuIR/TAQTu9gJiNXgPqryRulsK6YiFIutIabQ65nsMsd4U6nA?= =?us-ascii?Q?oN89pFOeIeGGDa4PgDUVeF5QIvRsuo8=3D?= X-OriginatorOrg: bp.renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: 815cc159-34af-43be-c21f-08dea2e28aaf X-MS-Exchange-CrossTenant-AuthSource: OS7PR01MB17371.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Apr 2026 15:51:40.8113 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 53d82571-da19-47e4-9cb4-625a166a4a2a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: qxm8JCLCo/7NcN9hdN683tkcS3BtKqOnjNrrq1dXZ2v+QKskkxhnHgM1regGNbuUvpHqEDolOHvHY54fej0ETobni5hrJKhlc9TPzX9ZZR0= X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYCPR01MB7315 Content-Type: text/plain; charset="utf-8" isl1208_rtc_interrupt() is of irqreturn_t type but two paths return a negative i2c errno instead of an IRQ_* value: - The SR-poll loop on timeout: `return sr;` - The post-alarm cleanup path: `return err;` genirq's note_interrupt() casts the return to unsigned int and flags any value above IRQ_HANDLED|IRQ_WAKE_THREAD as a bogus return, logging "irq event N: bogus return value X" each time it happens. Return IRQ_NONE when the SR read failed (no progress, can't claim the interrupt) and IRQ_HANDLED when toggle_alarm failed. Fixes: cf044f0ed526 ("drivers/rtc/rtc-isl1208.c: add alarm support") Signed-off-by: John Madieu --- drivers/rtc/rtc-isl1208.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/rtc/rtc-isl1208.c b/drivers/rtc/rtc-isl1208.c index f71a6bb77b2a..c93998c53e7a 100644 --- a/drivers/rtc/rtc-isl1208.c +++ b/drivers/rtc/rtc-isl1208.c @@ -654,7 +654,7 @@ isl1208_rtc_interrupt(int irq, void *data) if (time_after(jiffies, timeout)) { dev_err(&client->dev, "%s: reading SR failed\n", __func__); - return sr; + return IRQ_NONE; } } =20 @@ -666,7 +666,7 @@ isl1208_rtc_interrupt(int irq, void *data) /* Disable the alarm */ err =3D isl1208_rtc_toggle_alarm(client, 0); if (err) - return err; + return IRQ_HANDLED; =20 fsleep(275); =20 --=20 2.25.1 From nobody Fri Jun 19 09:30:49 2026 Received: from TY3P286CU002.outbound.protection.outlook.com (mail-japaneastazon11010068.outbound.protection.outlook.com [52.101.229.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BBE3934E762; Sat, 25 Apr 2026 15:51:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.229.68 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777132309; cv=fail; b=OEuKgJJX5aCg5oY0C4Qh1qBNdWgtVXxrJK+uyL6tj2+6OzSWNO41h5haeTEgtRIRVP0bV/rrpR/qgPJbF/sA65qc6wBhqC++cEEYO5o5UX/z9GyRkHjkvw496UKV2PxPpULaskvZ0uBinZqjHhxHITozWOzFrJxzyiPfNUV4SPM= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777132309; c=relaxed/simple; bh=CM4wAzX+r6s+6EDSYxgeT54472zsMUUpOxkQhborR1U=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=turDRIYsII9n59jOcFcz1fFRp1+Aw7NYhXh4E+gLECpJY9dwOhKsvbw63KVKnHZ7s4RT9OPQH98bagvrPNCN0OWhXOnHonEfgdVXr2acniaJI/HXxF5siU3N3H8UtMV8ungjFeHHmtReUmtyn+04d9f3xh35RygmdvB/MEXR60g= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=bp.renesas.com; spf=pass smtp.mailfrom=bp.renesas.com; dkim=pass (1024-bit key) header.d=bp.renesas.com header.i=@bp.renesas.com header.b=r1Tdv5lw; arc=fail smtp.client-ip=52.101.229.68 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=bp.renesas.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bp.renesas.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=bp.renesas.com header.i=@bp.renesas.com header.b="r1Tdv5lw" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=kuiV2FUgfxNmaBAecTizd7BQRGnz5wMaZp+dFBfdhbjGaeD3pB6dpbPv+zzevBkCrIhcJEMQC9UjARtyhU3lj4eZUjMJIW1cuvp4qsJzL8CwWYcMM6MXH55t5iK6l0Wb9Yh8kmFtchGwdEQP1b68v1z/0shIT8dUECirIRVXSSsyYuKSHxf2Fli9EIgk6RTdYM8+2fh+ddy7JqvC/0P0pzCDt7jK7g59Qe9nDMbIUkQ48qBTO/9lAPXFO0exh2pA9/OfqyVyZLUrxan7LnZlbhgN7ApLe/sB6LhJTJuyUvneEUsOxOerBu+KCP/BMxvHJ8ssAh0Scp0cCkF6dBjBhA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=XFJJFHTRyhmbloymNaKNtGQ4bzz6gtQVghHc4aWsHFk=; b=Be+yutiCWVDkJEDcpLiOW1fJoYIVb7vnHicQXlJRAUcgji0b8PAsb8nCr9pj1HslXgaoYnJlF8We4d7MhpPm4Ke2Bw5wfK4SYZ2u2WBOIgJYN8CRMUUX1vbIS1nIpDr+MXQm7kAQvzK9lFbEYSmkFJbwr0Vc7MpMP6jj2D+AQeC+ubF4rEz6VZHYBeWn0ucwdgxJq4OJ8O54VcdLQaV35tX5Q0DxXoBHViFcTyQCoOg4LeOtJ8zdSxxtxs4S0F7Lqu5J4lSZ4DsYznTu0EM8Df+3B2w/zhh1d+VNAWsGujN/l+s+L/WMdT+GwAZYRl0JEzlC77j17vTkghvUyZTAkw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=bp.renesas.com; dmarc=pass action=none header.from=bp.renesas.com; dkim=pass header.d=bp.renesas.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bp.renesas.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=XFJJFHTRyhmbloymNaKNtGQ4bzz6gtQVghHc4aWsHFk=; b=r1Tdv5lw1LVEv91/1dXkdHUXonldThMIro5inCGsEi8cFQ3WmnYxes4D7gBmAmOG1eylwXBJRpuq0y9vwkZQHuX0VSV3hpJwzkypQjjQ4926mMKWfEp1MBhDbmwpFS/AAqTyuTAHTj7I0q70m5m/wER4GbjbM5MUBBcch/w1azE= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bp.renesas.com; Received: from TY6PR01MB17377.jpnprd01.prod.outlook.com (2603:1096:405:35b::6) by TYCPR01MB7315.jpnprd01.prod.outlook.com (2603:1096:400:f1::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9846.24; Sat, 25 Apr 2026 15:51:45 +0000 Received: from TY6PR01MB17377.jpnprd01.prod.outlook.com ([fe80::f373:26d6:86c4:6aa3]) by TY6PR01MB17377.jpnprd01.prod.outlook.com ([fe80::f373:26d6:86c4:6aa3%6]) with mapi id 15.20.9846.022; Sat, 25 Apr 2026 15:51:45 +0000 From: John Madieu To: alexandre.belloni@bootlin.com Cc: ryan@bluewatersys.com, akpm@linux-foundation.org, m.grzeschik@pengutronix.de, Denis.Osterland@diehl.com, linux-rtc@vger.kernel.org, linux-kernel@vger.kernel.org, biju.das.jz@bp.renesas.com, john.madieu@gmail.com, John Madieu Subject: [PATCH 2/2] rtc: isl1208: Balance enable_irq_wake() with disable_irq_wake() on cleanup Date: Sat, 25 Apr 2026 15:49:59 +0000 Message-Id: <20260425154959.2796261-3-john.madieu.xa@bp.renesas.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20260425154959.2796261-1-john.madieu.xa@bp.renesas.com> References: <20260425154959.2796261-1-john.madieu.xa@bp.renesas.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: PA7P264CA0117.FRAP264.PROD.OUTLOOK.COM (2603:10a6:102:34c::20) To OS7PR01MB17371.jpnprd01.prod.outlook.com (2603:1096:604:43e::6) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: TY6PR01MB17377:EE_|TYCPR01MB7315:EE_ X-MS-Office365-Filtering-Correlation-Id: 7bbab269-519e-4d2e-e929-08dea2e28d2a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|376014|1800799024|366016|38350700014|56012099003|22082099003|18002099003; X-Microsoft-Antispam-Message-Info: bbLFwHgy/2d4BkC+SUwprs4ve66XfZz7Vh4rOBDpSbPuSIloj1mbhVCNJY4M1v6hC5KA9+p5T5yaQ5kaDXxnXh9HqHCm1+OuiqCyT7mQdNq12nikYOw0Gyl+z2ciHZ1fymso6LRsjZCug2XruSW1mOREKjjyQr8uqmUqQUMiZEIqFDvc0l9qA1CNDmXiGbF5n5aaJlyH8cu9ywrxkLmQwaMaxfp/njEB6zPWpVqmew0sUelmCYkNePcQUW06PB5D5p0c4JgAPKjVwehC11HxznFxCnlxmtwBqiKK9opneh301NBrGB2xgvARxkp6EAtCwRxy9u5M35N7IPa2GdgUBZmT4RZt7F/rc8jldvAsnmyC9DwDX3vTU8RMAh++VoWyYBu+hRv0D+B3sI/A3YskTQc/3R4GbaCV4fUZcCxBh4v9aMChdOwR0QwSbjXt5N24Sgn3BAg50YnfymQszfPBd57tPDK1oOZ1AoHbCduTpLfb5ZN0qvK4bTY+Mf7jnLA7F3Qn5w4MKacTGninbzwPXJwvTPXld6fZcDXf7I/Oj+Jek1ATGv5CU6X1MtVcMMgeAVERIp8l8NKd7NaJUGFsf65SZJNX6s+qMGLA2XBI77RWERu/8cBI9zrjxD6eMY9cCAlF3I3N+0Zs4gHkWg5vMmV8nUu/s11deeidc2rEedBXNNjbSrTCTQvcIVDJu8gghh9FUZFlB6dFyOaLgkmyIUkM6FrZAUpmVM+VbieuHbwQJGzyt6KywTVLbEh6MBAR6EF11f5dkaQBMbe+9lTF2M9ZvVfA6N0suNTFBDBxUW4= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TY6PR01MB17377.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(376014)(1800799024)(366016)(38350700014)(56012099003)(22082099003)(18002099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?bpguTUGHD7WYMSbK4KWDd9aHLTlCArzYpx/CPxndkTD4DEe4rfZWzT3eLpN/?= =?us-ascii?Q?2wDNtAUEM5ny/KWkFw+rX6jdr6+OLpl1zvx+qEKbbKBHTNee+6QoCzFywOLm?= =?us-ascii?Q?nh6XQcddgffZBK+7XU1zxWJ4ifmwOJb9mrs9r1G2od4CeyjjYulSV+PvK+j7?= =?us-ascii?Q?i+2b9yMkEOTGYT+NSYa9P0Cj4VCTk03c6GY3tL8BW/WtOOLQsOr7+vPyLCxO?= =?us-ascii?Q?/J2latV81kmtrtftb2469WXayfDaCgHDFWhV8jjWZehP0i1sjNjCbdUk0nEe?= =?us-ascii?Q?8sdWCBpsHsdbS6ywCNg0t3yTuuolu6NfFOV0UcpopowmDG6lmsMIpmQol3Cr?= =?us-ascii?Q?9EVSIgP/MVPPxwlhjoV166P09L5lo3cAWXtE2OO2MLXFajHYwSW4scYZ6FVL?= =?us-ascii?Q?I0Brr2zlSV4heiZefpL7hKPP2Hp3Sof+38VQCKICOsgy31igI3WXRM/pmEpx?= =?us-ascii?Q?Gi206vnarcsS/9JN4PnVIVCmeSzQhVVcyWp6Ew2qvgrqXUF0Dlqw3GJ4jSdI?= =?us-ascii?Q?eFVvIII3cmVOh+eSBHOchv2QzFJ6ahjYzRG8V6EoXEJEXVI4Rn/LaA2XQqN6?= =?us-ascii?Q?ex0H3JgbkS27Fns3FtDx1+AbAdNquwvq3/cjZ7puwTK56k8ftgDxdpj0Q1KT?= =?us-ascii?Q?DkSb2YLq4knMuNOMuZyPpXAJpBdxvD5E4FZ3vkTNAwpa2ybWt78/S5Xwutz4?= =?us-ascii?Q?pxa8ObQcclWKD9YhvKv6GZk58Ka9EFVmQuZ4WiJmzVW03zDb92OZ9TwNkPyY?= =?us-ascii?Q?2MyWMNAJRi+UKMfVcGGebaLOzpvPBZ0PBSU7eVDnKWXW9I7c5eHRmkJ2pPmO?= =?us-ascii?Q?539KRYGVkqtMOCVKBcQyfebx+JOGPbqsueN3ltYM3/PPTssNdbg1PI5iHSce?= =?us-ascii?Q?4XgGkz2miuvILdIOG+s5yoW2cuTUFLfv+ntvMDXG+XjaYF7QVsH9Y5oShZOo?= =?us-ascii?Q?tBFWzX7P4fcA5nC9r62BuDO9jhCyt+QGEXMNujgaxO4hTGjvV+r+6vtDgOHD?= =?us-ascii?Q?52BVew5DKQIs3V+OyOWhxSDzPy2iB14VvASQuzUMm1+otDOqsfkyYysVvufo?= =?us-ascii?Q?LITW0uQ6tl2/ZUvFitTpTbqHO+YbiOZjP1n6mBRlIk9MdcT3bMfPfU2IUUL/?= =?us-ascii?Q?syIELMNjMQvjl38X4zXSWgXnZ4O8ZoCyi+sj1ZbL6vpFP/bp/jeLzSiMAccN?= =?us-ascii?Q?nP8CswHTj9O2J/nja0KgVdaTe5ZAWxZv/pzc7pvnl4OAAEwQhEGmXiCZAT3h?= =?us-ascii?Q?aY4weSySw6oTvv1/1oY0oLfMk4rIpy/bmFwm/fK+bHTvGujZ93c171hhiMVe?= =?us-ascii?Q?VhQA5VniJ+okz5KmR9yDpZ+/i7Fkm+nTc/l0IHZcVFOelWIdcks6JEaaBWhY?= =?us-ascii?Q?T0XkE1xee/d8YFj8eSIVQ7MIt+D5HqMuM83jDwt8OZUOIf6/3V3Bb2li4W7k?= =?us-ascii?Q?UnabqGq1PfwctItl2e3nfbxw5GnjFGg+zOQhJO23zNNff5hkqeIILINYGOse?= =?us-ascii?Q?ZTQm5o8/FOLUCgTWWcoh4QDdHlZsfg9TGGGeU/d81q6clp2lI3chJSyOewm1?= =?us-ascii?Q?deglwvBAfd5GR86rWcqkKTtMO7SyoEqcOD6RzzYUbt6oef5sGzE8MaHS4hMf?= =?us-ascii?Q?T3OYtyV7znfRsYYdzOBZH27hHwuuT5zXBvPG4Q3gr5NwFAQeDI67+vE/J2bs?= =?us-ascii?Q?wd4dEBbuPqqWR1iOaH5giwYYigGeYV5jow0sr0mMsp9lV5Sn/kye2J3qRkhD?= =?us-ascii?Q?SHn+V1UILz1bGM9uutQPwNYwmJ3gRP4=3D?= X-OriginatorOrg: bp.renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7bbab269-519e-4d2e-e929-08dea2e28d2a X-MS-Exchange-CrossTenant-AuthSource: OS7PR01MB17371.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Apr 2026 15:51:45.2056 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 53d82571-da19-47e4-9cb4-625a166a4a2a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: DtoHuc8tG7BDGLv0TtlloekeJx/7txxDcTdPRklDI7MwK0jiHxxM03AEC9m6v+AaT8QSZEnvc2Q3eClhUq9na4GkN1NrSdpN4h+KUxqkmdE= X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYCPR01MB7315 Content-Type: text/plain; charset="utf-8" isl1208_setup_irq() calls enable_irq_wake() after a successful IRQ request, but the driver has no remove path that balances it. The driver is devm-only, so on unbind devm releases the IRQ - but enable_irq_wake() is not undone by IRQ release, so the wake count for that IRQ stays incremented. Each rebind therefore leaks one wake reference; the leak doubles for the chip variant that has a separate evdet IRQ, since isl1208_setup_irq() is then called twice during probe. Register a devm action that calls disable_irq_wake() per IRQ. While at it, check enable_irq_wake()'s return value: on failure, propagate the error rather than silently registering a disable action for an IRQ whose wake state was never enabled. Fixes: 9ece7cd833a3 ("rtc: isl1208: Add "evdet" interrupt source for isl121= 9") Signed-off-by: John Madieu --- drivers/rtc/rtc-isl1208.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/drivers/rtc/rtc-isl1208.c b/drivers/rtc/rtc-isl1208.c index c93998c53e7a..1de80fc9c9c9 100644 --- a/drivers/rtc/rtc-isl1208.c +++ b/drivers/rtc/rtc-isl1208.c @@ -822,6 +822,11 @@ static const struct nvmem_config isl1208_nvmem_config = =3D { .reg_write =3D isl1208_nvmem_write, }; =20 +static void isl1208_disable_irq_wake_action(void *data) +{ + disable_irq_wake((unsigned long)data); +} + static int isl1208_setup_irq(struct i2c_client *client, int irq) { int rc =3D devm_request_threaded_irq(&client->dev, irq, NULL, @@ -831,7 +836,15 @@ static int isl1208_setup_irq(struct i2c_client *client= , int irq) client); if (!rc) { device_init_wakeup(&client->dev, true); - enable_irq_wake(irq); + rc =3D enable_irq_wake(irq); + if (rc) + return rc; + + rc =3D devm_add_action_or_reset(&client->dev, + isl1208_disable_irq_wake_action, + (void *)(unsigned long)irq); + if (rc) + return rc; } else { dev_err(&client->dev, "Unable to request irq %d, no alarm support\n", --=20 2.25.1