From nobody Fri Oct 3 08:52:33 2025 Received: from OS0P286CU011.outbound.protection.outlook.com (mail-japanwestazon11010003.outbound.protection.outlook.com [52.101.228.3]) (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 8CB932EBDDE; Wed, 3 Sep 2025 08:28:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.228.3 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888116; cv=fail; b=Ikgf8CPNDc2g8esMp0ghSFgYtRylOmiOAMpooPQ7WmWJDx6qjQSlxqKs/JPmrSnj9HdsRSLTEQc19EqDcXLps74CJT+EWmMtN0639+0KLr7q+QtsM2GlBCpbzBnezZgM2DnK7spH+w4dzsFv21D12cf8Phv0palWt4ISBTeFy3w= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888116; c=relaxed/simple; bh=bjrh8H472uFl71DY0cjpnpnWSyGXEPjO2viCs9T5ZNE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=A23nepJb0WGIdEYQpSbx7AjlUNF45r7BVgglNclvsS99y2NZCNR2nFhbPU51a1OVBCDhEKtA4Oi+oe977VDFqD05DnyOICVFH+YpGwdTVOnwXu1jmC3cITTjx7MoP/PbGDXG4Nti9PpzRc1CZjfrYmC8ezIEE+7ikI3EaqKBZvo= 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=cUs4k13u; arc=fail smtp.client-ip=52.101.228.3 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="cUs4k13u" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=EJ8GXA8+dXxvKJPV0nK7gidCisby1XYpnrs/7XHsv8COmYjyeF2GWN2m2TKwDM0e2j+DOVIOYaRxe6uinlx9uvTgjBlEvRGKjkiuP+zYbTPQMtD2mlqFVxoACWUDgnX3sKMs99LQMebf7f2L2wEiua+b20Qq+MGT5gvAjsOxg84DEw8x3LGJ2b8XQZiy0lWYK63IDxrN62QXwMHZxb2XWhGIvYIZ86wqTa0ehbHsTlVt2arH1CNXGdZv8oXjB4GE0MYiZcSOxhDJTHBNX8FRX3az9AtmdYeaTpP4xPMt7TlPHe4/AxfLEJMy8LBLRwQwKNHUPAlg5SBoN9+DOZCuww== 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=uJ9cdDJC9XlrXrpcduxhMhx1O4RqqbvdEYIj0qAlBfA=; b=R/6glz+2Mjr1xA3zeGlUFRSxUTDpcW6AyvhKMz3G0G7uH6jgdpgCR7Tmd2CQCWTasMfFFk8LrQLJcasq6ONAS0rAhg33aJgbX0vGHJCsqBN29OpOIZ4CtMBtdnoeP4O9KKW1WQWwWZcod1hZYh8oDYRb2n8A8Wc7PtKT1ekkxgv9X/KnaFIaAkZ41nt0+cFv7xFjxmm5NZT0DX3v0Ko7srVSubPWxrgnM1xGX1ud9jAqUZKKDwY5ghYyHZjz4XUflUYqANPquuQ/hnhyDvtxOqPsYrNshkJ/BbmAzNdII6+j2VPTuHMRsM16V+nkp9vRSK5J1Kq13y+Ip24Igs6JqA== 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=uJ9cdDJC9XlrXrpcduxhMhx1O4RqqbvdEYIj0qAlBfA=; b=cUs4k13uyycFjeca25Lb3E61Y5srwIUFHvBENcAneSSJvGkjzOppF5dkVDqAGc9EdrPNEh17xt7SPowXwGP/EW9JtvVt8PZfqq2Z5A4vv+uytg6qVwQ82h+YW6AZG1Fs1pC9oVP9G7G9hMP4xk76nVeYVl+5KP38YHIp1j/k2bg= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bp.renesas.com; Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::6) by OSCPR01MB14678.jpnprd01.prod.outlook.com (2603:1096:604:39f::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9094.17; Wed, 3 Sep 2025 08:28:32 +0000 Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31]) by TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31%5]) with mapi id 15.20.9073.026; Wed, 3 Sep 2025 08:28:32 +0000 From: Tommaso Merciai To: tomm.merciai@gmail.com Cc: linux-renesas-soc@vger.kernel.org, biju.das.jz@bp.renesas.com, Tommaso Merciai , Geert Uytterhoeven , Michael Turquette , Stephen Boyd , Philipp Zabel , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 1/5] clk: renesas: rzg2l: Simplify rzg2l_cpg_assert() and rzg2l_cpg_deassert() Date: Wed, 3 Sep 2025 10:27:50 +0200 Message-ID: <20250903082757.115778-2-tommaso.merciai.xr@bp.renesas.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> References: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: FR3P281CA0058.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:4b::19) To TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::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: TYCPR01MB11947:EE_|OSCPR01MB14678:EE_ X-MS-Office365-Filtering-Correlation-Id: 6b12d9a1-9527-48ca-140d-08ddeac3ddc6 X-LD-Processed: 53d82571-da19-47e4-9cb4-625a166a4a2a,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|366016|376014|52116014|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?4iVacUxz7bj1rydFy3wrBkYhNjs6JJp82H6J5nGzBM3PAmU9NYPP6KjDe5j1?= =?us-ascii?Q?yowtBDdf7xjQQ7az2X7QqJN8CuPVRRddwmxh2fdGL+QNZkuv1FutbGoR0282?= =?us-ascii?Q?Gyvkpo1BTnonqH2k1rG9mrp16vLKj/j2Y8nsPpQ+DS4QWIJN9JZpnDqKIOPU?= =?us-ascii?Q?hqA5GKcivf+yx6+4e0meIJ4Ij10Xzlwzb+9SDdl2tAKropW2jhxajNgcq72I?= =?us-ascii?Q?/qHr0LItUS/YAzhsjDcRJ909in/+QIEPIPfuaCHB9N7fH3Gc6PFsVN0w9Tmm?= =?us-ascii?Q?AHNHDUeTcRNQ8iCZ6Krr4eOGWzpaQ+TNzmjkII3gfcczP1EG2BwGXT1aejyP?= =?us-ascii?Q?gkaqRM1G8iv6w2M5Zf6XDkW8pbUqYNIB8CgEtbFp67kcFtFbMk5QBLhCBEof?= =?us-ascii?Q?Ol7x6oFfl4SSb2LGCGIPV7eTkABUMp8KDfQ+RarCBkvMgP1gUtBY99uKr4Wp?= =?us-ascii?Q?XFlWr2u6H7JeEFg6/YTQwZ0sjmJRfBkg4vc5U9Khrd+GC1VYLd4henwi3WAo?= =?us-ascii?Q?TWHvpZ26IliBESyNm735R+EVwfNZJjXityeRo/wcZ9RxwBTRip5XfT0lzPLX?= =?us-ascii?Q?dBmPz+Jnqs03HF8D+4wUnT6xB4hFHSHHxl7+Fybv/9D77hSwTbKAyWM8gIWL?= =?us-ascii?Q?5lLNfv/9KvkllLCHkRGxkjiA9hOJNswX/EsgVYu4XjimT+o5U6/mMBfHz3KI?= =?us-ascii?Q?obQxxECfCX/Ffqb5+/fWIRHwceeJ/BKydFkJWPSnzAQ/ClwGdLkK7UtQjnsM?= =?us-ascii?Q?3CvlVZAWFogTi6DG2W7rDDL71u5oNULT5h7t81VoT/6qqrtu7R3gAD1EpupM?= =?us-ascii?Q?wvyW5rC91lD4r1gIzWDjbMePCtomMX3hf3INLBpfZjm9X5xXffqAsOxUpbTV?= =?us-ascii?Q?I6wIb+PpaNUQPzxRTTViUrfFy9SscHko9GAFhoX+8sfkRoomFASBrjea+jCp?= =?us-ascii?Q?IlFN9Q5kBxFjks70IquOC4v58ppsbsMUVaT+NWmvHqxAOnjDeJpo3fpVfJUu?= =?us-ascii?Q?iPy4q+rBpm/3PSWw4It1KwbynMxKRUnXvoFXZCdLAN8lCtX+pdQxrAARYxVJ?= =?us-ascii?Q?GbO52TGRU38K43UnzkQrfdgaSlerFVqS8P7xgYa2tpT2sKNJepZJEfAp2HLt?= =?us-ascii?Q?NHdYmuER0MEZ+uuGHOSKRWmH+PJp6K6d2T6Trnb2WMxqH2BoIw8fnBibyb72?= =?us-ascii?Q?rmaqxutOVJmVZYD25MFpEirSfz8IBsVdU467j4SGwa5z5DfQ4hrvw3XXeBFN?= =?us-ascii?Q?ydi/7pVyPCcAq6vzqTaI/sq2EM2G3cwqhWguckl09edARHnz1fZTqHxf0ydW?= =?us-ascii?Q?l7zUHGDGGyjmd2Y0I5cwY11PzFqBM7aP/2WoTNtSusX4XcPYhLkGxcLsf/Ut?= =?us-ascii?Q?8S0yYG73mMkp7mEdjNmCbmGU0vNCG1atuHkJ1hGkElEyGtLINMjMkzMer+GQ?= =?us-ascii?Q?vW48VbC5fWdhb+UNMzBXIEcW7309bEjyxz/wnlKtanYaPE1xhxyvRg=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYCPR01MB11947.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(366016)(376014)(52116014)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?gvhZ+JItXPIm/Tm42eORk0yFHIbAhgpoAxW2xvUvM3bQ31OIvhscTkP087Ss?= =?us-ascii?Q?LVppV358gdWKNbr47cmBEI4FhF3aqFx6BHZ314vltD3ct2yNZMaaSE5NYhR0?= =?us-ascii?Q?rgwTjg+RueAtbfBCJkTHZLiF0PIJ+g5j9pFJF0mzhigGVz8+29D9n7apKMT3?= =?us-ascii?Q?LB1KQerrKMhLRPHq4+FYK4JZfMEfdFUFGGPBFiZpqoWtslPkL6DBWCUPt6Vl?= =?us-ascii?Q?gMox5wEPbz9b+ZNAS3sCqFJotMNUyLKtCSis4+gLYo3iCkT+1hlsyw4vcuom?= =?us-ascii?Q?7k3Iwpg2GK/tgmqTiSozr1YNCLWILauETQwLyuY/9GVxZdQzP+XPz3ch1Rg8?= =?us-ascii?Q?6cko2EBusxZ9tPuxpwXUYnqA1Qi865LrGMmJjA7gtQj8ufAY5ksZjRnIS1+L?= =?us-ascii?Q?QE91iRf49iFzvJbzGIN/27uU/ao0UJO9s9114FiKvI79K3xpQhDYPzbu7jLo?= =?us-ascii?Q?YEl66y0CUgFVbQ6J76FGkkKemN6th+FHKerHeA3q6mqNKQS6f1BnUw4oFio7?= =?us-ascii?Q?bp0GuXyXNhLZ2DSdni51nGs0Tegftgpelmgp/0+sexGhGFLCSexOgk+XdOhU?= =?us-ascii?Q?l3fYA4LGUhp8GT589mRyNERhEiQNzITjwYDUs5HELeBonLuE6e+V6RT52Zco?= =?us-ascii?Q?lJQ8p50TgHn9NMC5mKleACgHwvhkBemV2qASLnSFpC+nChQWpMJqJqo3EW1v?= =?us-ascii?Q?jQcjF6rzEnLHkhC1IBsbF8pyYhisqsxBcFbAj4leiEldz48zqEGqKhHMQFfm?= =?us-ascii?Q?oqh7k/N7xv4hmLYj+8e/S0jAJ+ZaUDMe8RSylQ9nC/VdUkh03WntgqAQGfT/?= =?us-ascii?Q?XOJEw5Dg+yO+qpQDSVAA60QXGaEbvvsd7Km83z/r8z8FuHd4G5zSaVufOze+?= =?us-ascii?Q?yt6qwtEGKGSJBds/jB8s+p+7RCAdjESsBvS4jdUjSU2vRC/dTS6StfQlkWlj?= =?us-ascii?Q?4FGAV+97cz370FYu1VVg6Y3qYdtJk9EIQgrSTzxh8kZ9vkOzohtG/IUNH9b5?= =?us-ascii?Q?OtPvS/2fPicbn1TCps3OJSpRTrQy6C/JRAp+mITuQJBxxJ/Vb9NmMW/4lxam?= =?us-ascii?Q?js9QYurjQTC9qfBzC1F7xoAaHhp7As9lYfzPEwA3HZKHMC29d9iDzmQ7P0rN?= =?us-ascii?Q?y6mfAYBac1CZznSK/gdEFeNhiZON9jJj9Wykql2Vg85t5kjdr2hDPfpuE6Ae?= =?us-ascii?Q?7I8HhYn3m8bM4iMAWzROhf+YAc/YFXgxnd3TCL5UDk2tmPSRZ2Jr9IBO9i1I?= =?us-ascii?Q?tgiZGU4rWYOFzW3izNIzKobb0YLT+58jiNRzs1SM/zqhjy7NimfC9ld1VMCt?= =?us-ascii?Q?TpWCjg0aqYT5h3JiTieNZrJX25z0hV1yLW+pjH7m8kwZKdAUpbhw/vHlgN6Q?= =?us-ascii?Q?ScrANHaCXP0MHAEZrWjFJSjn5MDdAYWklaiGs+RZTp1KmPrzyc7buU3SLdQ3?= =?us-ascii?Q?OR4rMLxBLAQKl4L2snU1jJa/RtG3XVajgLcVFa1E9gDfxKeggqjBXF47kqFq?= =?us-ascii?Q?nRcPvEV/JvxmkG/X2a/X01B6c4d4nbWgSEFOY3/CCD/NX91KrGng4PTh1S7B?= =?us-ascii?Q?4qhMjdo4ZJpGCiGCi+X4Ius0lX0kiFkdb2MHJ4acwEo8HHsZ/RHvkEZZqLdo?= =?us-ascii?Q?ebbDlxwENCY6DXiSeXRn8Ak=3D?= X-OriginatorOrg: bp.renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6b12d9a1-9527-48ca-140d-08ddeac3ddc6 X-MS-Exchange-CrossTenant-AuthSource: TYCPR01MB11947.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2025 08:28:32.1179 (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: SyMakHNKIQTKHNa1o9zWCC2OcNirwRiMW5QNREEu6UmXUarOk2UhSGfgISFUBeVZkHxsT02w3QMFH5nef1AcYqKtUNGOvOjAZNzvpfprCtqHFKXTBjW44vnhHMXNYSPs X-MS-Exchange-Transport-CrossTenantHeadersStamped: OSCPR01MB14678 Content-Type: text/plain; charset="utf-8" Combine common code from rzg2l_cpg_assert() and rzg2l_cpg_deassert() into a new __rzg2l_cpg_assert() helper to avoid code duplication. This reduces maintenance effort and improves code clarity. Reviewed-by: Geert Uytterhoeven Signed-off-by: Tommaso Merciai --- v1->v2: - Collected GUytterhoeven tag - Simplify polling condition in __rzg2l_cpg_assert() drivers/clk/renesas/rzg2l-cpg.c | 45 ++++++++++++--------------------- 1 file changed, 16 insertions(+), 29 deletions(-) diff --git a/drivers/clk/renesas/rzg2l-cpg.c b/drivers/clk/renesas/rzg2l-cp= g.c index b508823f9723c..1143ce0afed47 100644 --- a/drivers/clk/renesas/rzg2l-cpg.c +++ b/drivers/clk/renesas/rzg2l-cpg.c @@ -1638,8 +1638,8 @@ rzg2l_cpg_register_mod_clk(const struct rzg2l_mod_clk= *mod, =20 #define rcdev_to_priv(x) container_of(x, struct rzg2l_cpg_priv, rcdev) =20 -static int rzg2l_cpg_assert(struct reset_controller_dev *rcdev, - unsigned long id) +static int __rzg2l_cpg_assert(struct reset_controller_dev *rcdev, + unsigned long id, bool assert) { struct rzg2l_cpg_priv *priv =3D rcdev_to_priv(rcdev); const struct rzg2l_cpg_info *info =3D priv->info; @@ -1647,9 +1647,13 @@ static int rzg2l_cpg_assert(struct reset_controller_= dev *rcdev, u32 mask =3D BIT(info->resets[id].bit); s8 monbit =3D info->resets[id].monbit; u32 value =3D mask << 16; + int ret; =20 - dev_dbg(rcdev->dev, "assert id:%ld offset:0x%x\n", id, CLK_RST_R(reg)); + dev_dbg(rcdev->dev, "%s id:%ld offset:0x%x\n", + assert ? "assert" : "deassert", id, CLK_RST_R(reg)); =20 + if (!assert) + value |=3D mask; writel(value, priv->base + CLK_RST_R(reg)); =20 if (info->has_clk_mon_regs) { @@ -1664,37 +1668,20 @@ static int rzg2l_cpg_assert(struct reset_controller= _dev *rcdev, } =20 return readl_poll_timeout_atomic(priv->base + reg, value, - value & mask, 10, 200); + assert =3D=3D !!(value & mask), + 10, 200); +} + +static int rzg2l_cpg_assert(struct reset_controller_dev *rcdev, + unsigned long id) +{ + return __rzg2l_cpg_assert(rcdev, id, true); } =20 static int rzg2l_cpg_deassert(struct reset_controller_dev *rcdev, unsigned long id) { - struct rzg2l_cpg_priv *priv =3D rcdev_to_priv(rcdev); - const struct rzg2l_cpg_info *info =3D priv->info; - unsigned int reg =3D info->resets[id].off; - u32 mask =3D BIT(info->resets[id].bit); - s8 monbit =3D info->resets[id].monbit; - u32 value =3D (mask << 16) | mask; - - dev_dbg(rcdev->dev, "deassert id:%ld offset:0x%x\n", id, - CLK_RST_R(reg)); - - writel(value, priv->base + CLK_RST_R(reg)); - - if (info->has_clk_mon_regs) { - reg =3D CLK_MRST_R(reg); - } else if (monbit >=3D 0) { - reg =3D CPG_RST_MON; - mask =3D BIT(monbit); - } else { - /* Wait for at least one cycle of the RCLK clock (@ ca. 32 kHz) */ - udelay(35); - return 0; - } - - return readl_poll_timeout_atomic(priv->base + reg, value, - !(value & mask), 10, 200); + return __rzg2l_cpg_assert(rcdev, id, false); } =20 static int rzg2l_cpg_reset(struct reset_controller_dev *rcdev, --=20 2.43.0 From nobody Fri Oct 3 08:52:33 2025 Received: from OS0P286CU011.outbound.protection.outlook.com (mail-japanwestazon11010062.outbound.protection.outlook.com [52.101.228.62]) (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 AC0CE2EBBBC; Wed, 3 Sep 2025 08:28:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.228.62 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888125; cv=fail; b=kG+rwGEB0vta5d9n9qaxDhyZCoTS/g/VKsvQUGf22/RVUFFcMFzkM6iYqtqxmTzcQxZGPsQ4QOLrLCtV+jb36DUPMcrwKx23/2EW5kka3rNDzwIKgYegih5nJz41GgQGoVYoLVUdhj8Q9XvAZtAdg+TgfMeC8/ZPdlaMDKiZhf4= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888125; c=relaxed/simple; bh=VlSDBVZXQtOXbMnB2gErEIvA4Wx74SSrasKP9VzzvgM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=Jfw0Gh/6zImGw/ftxh6nn1CZxZN5kG+fIA1mAY8IIwBpP7ewwO7HXp2x6BsRSMV5yeZwhT/yAafLjaPPMlO0IYzuGbecYLsbKHu2LLen8IRRqJBLsSV5N5Dpg5pb2l+CwGTEwxQXBz7ETVMm9C5LYzzqFCmsrFAPDR+3ky9f3AU= 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=loTN2pUp; arc=fail smtp.client-ip=52.101.228.62 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="loTN2pUp" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ysRn3gkew5NgcH/euzEv7Mx/8R4xROWDpGXLJ8tSt7/ZCAy+SB0JK9/rUv+w9sOacgCPhI1O9nR3+FS3OHaNY66LrUyYYlVeU2dLx3uNZwu+Iy1OXdn6ekdWWcLeG7gw61k3bmiojEm+emMnOARk5UataK2rq7B4uesXeb9OI/WryRVb0EGAs7uxmu9bXU29ZzczsqiZhnIClNso5y2wIvWcx8HBtube/LRkNnkP+B0Wv5QDYiBiz1oVZJlfN6/LhviEni97pmH1pIfAa5N0osvWN8LkHC4vyj2JWfekAvmVlTgZDi0Ti8ik+e0xcuJwo+xhcpphdiauEZPYlKz/QQ== 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=1TUMm7+HiU3+cilrRoEHS4p2qr6fWj11XDXtMnNCgNY=; b=N1w5dLTDlR2p7uWX0bMv9XntHI6Q/OzU8erBwH05E6apqE800kEPSzZjvCNox/ZdnLF0EksfgDuo4h9/h+ouuNqmmIlarMuk3Dmlux6LzwKoC9cb9cH8zVHR0DuSCbEiojMKuQQAgy6hnEJDXlvd41g4N7wrgTboOycnvWtYrWMcadHU0b5yZYEg6nq++4wf8XOS1tehlZP3WzDrvNAh1djS3sVnt0GTM0pkZOYBKBcCLEGASrlhNSuV8smhHUCTni0nIYHFrFMu/uyPnT9OFxW7sF+cjxkqrvhForLAnInSNu6q3ES2MTGjvsuQ137N6Tpp7uuyCStBKIsF96eWGA== 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=1TUMm7+HiU3+cilrRoEHS4p2qr6fWj11XDXtMnNCgNY=; b=loTN2pUpyGyUK1WLz75fDIZAr/K25/NReMQFhyzUZqYYJ2AHvqRVsbrEV4HStr0vhfr9qKLYW1WAFiKAlJHVR5C01davJApSg/dmXF0/EU3KBS/Y8FblMSicDzby9zs7c3iQU0IfH5E5kVtls7c+fPyNHVyh4x5jAVj/ZWlqE8c= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bp.renesas.com; Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::6) by OSZPR01MB8530.jpnprd01.prod.outlook.com (2603:1096:604:18b::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9073.27; Wed, 3 Sep 2025 08:28:40 +0000 Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31]) by TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31%5]) with mapi id 15.20.9073.026; Wed, 3 Sep 2025 08:28:36 +0000 From: Tommaso Merciai To: tomm.merciai@gmail.com Cc: linux-renesas-soc@vger.kernel.org, biju.das.jz@bp.renesas.com, Tommaso Merciai , Geert Uytterhoeven , Michael Turquette , Stephen Boyd , Philipp Zabel , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 2/5] clk: renesas: rzg2l: Re-assert reset on deassert timeout Date: Wed, 3 Sep 2025 10:27:51 +0200 Message-ID: <20250903082757.115778-3-tommaso.merciai.xr@bp.renesas.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> References: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: FR3P281CA0058.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:4b::19) To TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::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: TYCPR01MB11947:EE_|OSZPR01MB8530:EE_ X-MS-Office365-Filtering-Correlation-Id: cbff585d-0c7b-4ed1-245a-08ddeac3e06d X-LD-Processed: 53d82571-da19-47e4-9cb4-625a166a4a2a,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|1800799024|376014|366016|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?m6MU7hfdIQAerrYT/n19epihaDjw3X76720TnwPX1Uk7r289ldtECE+RmZJc?= =?us-ascii?Q?037O7+5hnD3ONLgSi4BaahbUeutfbFhFtEpWIFRHH6bVf9oQF2PUxbyiwU12?= =?us-ascii?Q?NeN8pvhdXZE9Hg1WaTi3ezcLAtX8idVKAzxGuaHT5Vas9BoIPL8dVHq1TTjV?= =?us-ascii?Q?okNCIanJfhSJ88fSlbkYudqAt9LTYkKRtlTfUT7ksyqlzg8SlwtQtgQSYBvC?= =?us-ascii?Q?5yfpQAAEbcoTJe3vN05UUeRkdR8fxEKxYbfQeWjX17tX1ZEaYMzHS9p5NXRc?= =?us-ascii?Q?fb4MNYTOrjxjICVLcDy3C2i7dDeSqxvhF6NUF0RHHaj69V5bHAThi4xZr9NF?= =?us-ascii?Q?7W/JvwSEoFa3zp71oGiOw35yc89tjdb8wZ8wf2vy36nWefR3olBW8sAPKvFa?= =?us-ascii?Q?Cf93lkSmNMUFwmCfxBn690AQJG5XRlxNcCNSAmUkMsw669PahpgGLx2DvKSQ?= =?us-ascii?Q?nWtNQF9h105wlXSxLlj7+y71SVQq/gjrxgMq38JQpcMQEJcNObMSrz+fUhv2?= =?us-ascii?Q?Oukn1219sC9qS8/2lQiEEinzfV5L6QUme5n0nH5j8TVQQsTjMvhHCSb4/os7?= =?us-ascii?Q?Q4DHHye5Glr4YrJlNibwiQEU6J4SNt6F7odsBD/FarGcPGPOltBxZLrpMHBr?= =?us-ascii?Q?TVJdgn8iLJWy2dYK6tYMMm17+IjiAsoYcTwhyc1jZAEWirIIOkb+xnARYJ4s?= =?us-ascii?Q?TnYUHo0+yLwtERM6Fg7fdlB0Mpm3n88FHAQFLE6RZWYckScsk3h03wE7qHLD?= =?us-ascii?Q?A08CNTi73DW8E53KuxJGS3GrpSLMHSFGcTKC4pc8m17cmFmgaOKO9hMGryqc?= =?us-ascii?Q?47MC8JnGUYaY8wxrgfEKBFNqCJZo4NrJR+qzsfOaXtzZ16xJqcxSPZKxO5fu?= =?us-ascii?Q?D3raXiGPEWmoRawcHcwbBQDpRVTJIYeEvPHKzZv6OYwZRCv6a1HATMBzHTFu?= =?us-ascii?Q?9KC9k0/iuFBaR7Isy4e+zMCHvmbgs3LMlVYnn7Mh1DWxHzbu1H/EovQbnduf?= =?us-ascii?Q?2ItASzuE/dMAmMFnzEoVLJWhx/Y8t3dWahg0aZ1Fn8ykbyjd+27Af/aLIxnn?= =?us-ascii?Q?HS86fdN0SIbkUCFqFPnCSBPRj7PF5VlXQaQs0LAHUnDWWJdoEfwnwXR8iBGc?= =?us-ascii?Q?K0JpYKYNK+HL2dmo2cohqo7EOJpyQJtbjOlx78Y/ZALHXsppbvdvUIyY9abJ?= =?us-ascii?Q?fgyKtVtVi0yMREAhHc/w/lu/JnNUhp6/rOhO6l7ByMJtgxuL9C3gBWvdRIZ5?= =?us-ascii?Q?Pldv8rf3Zrl0V7skpJ+v0drpYxecfAVsSv6JUwWfffYsRXKSxhm4yCnL07vO?= =?us-ascii?Q?EFbJ3pjVChHokwWiYoevGeT/StW3If5fwiwAGp5b99Rin2ZyHsAXWIilFv7s?= =?us-ascii?Q?vOTcjl4Npr3OY7lZ3XHb0QR7a0ys7PUmq/PaDUtCKJEwUtz4PfG0ASc9BItA?= =?us-ascii?Q?Q2Q5zjbDWwXNEiBPr2f7XL/5O3zI5I4pRYVrn/5w9qb+yiHw2SQ2EA=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYCPR01MB11947.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(1800799024)(376014)(366016)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?NS0qSE0bR23aD0OlLRQhf2oPiglHtSd3LjATlyV/N5mMcGtuSUN/sn60wJnp?= =?us-ascii?Q?seSvkpq2LfT2EJEoSyee9PN840bH1FcHzz38SOq/Sux7IZHzhXqZAm47jB61?= =?us-ascii?Q?swRjwf91BDMYF8iIJaL06lRLBURoUjEqPFyjAEHRHZie/jDE8+NeyqDrE7C7?= =?us-ascii?Q?nz8fXymQJHKdbWwQp0D9256g//8vrwwhErDpn0LFYNm4/lHoeL2E8Yz0/l6A?= =?us-ascii?Q?iQbbPE46VVgySBnpNaAVbRJ/cZ2DIaiypcN9uqMinkQdkG3/NRs7t4WqYzfG?= =?us-ascii?Q?5sDRJEBeL7GhIMmUUahyv/uIYktGsA7f9vOdQeege0Pk3YZicJ9jVFJK59BB?= =?us-ascii?Q?DkA7EPRbpcT82S3BmWnl49F8ibQDbldylc9Cb1MSzQtg+x+rzdpDzr4WsiYZ?= =?us-ascii?Q?MYO8gmFlkEHnUdxvRchz/eOAVv/O6WIU8vKnsVOnPR/en+XqDQDeSJJEGV74?= =?us-ascii?Q?apx7BJ5RB/HJj5WlViEzBOXHny0uu3bkOljSYJahnM+3sqWvJG6FL208qXJI?= =?us-ascii?Q?xNBPUoyDer2qjAAAHbDb/pAuD7sFATOLcHMiMnseLsnk8l20Os9tLmFEp+BA?= =?us-ascii?Q?aQ5iyjRnmKVkvcOHVmOH5elvO+gkfQeFAEkB/vtwtbOffqCrJMh27NXjd122?= =?us-ascii?Q?ucdXJHUxeX9eBrq5j7watc5N/XraqmL4oajS0bSqg9bFXpdOZd/eVZgTZbCu?= =?us-ascii?Q?Khaa12Sid0U/Ug0sCc/rm6QBmmIYjR7nae30JV4TgGgbsFnlipVRttfIVZQr?= =?us-ascii?Q?C2yMDLDykFPR9JX88genEaufy0mz0vvE5OF7Iv2MbUt1zNWBS4zvTJt6PmF2?= =?us-ascii?Q?8f6QaA9R4kEj1pQBzQ2VSC2f6Ex7Vn8d35OiySCWxGCv1p6KvEReGGZRm9A/?= =?us-ascii?Q?HmVehqpV2iRB6dshz1nXoGKPP17aAVsxBzowwSx0vjpnOkAsv7q5Ki+jr6Wc?= =?us-ascii?Q?3QGehkpDuoaMFIVHLvvwdmHjeLDcJ7c8a13sSXCuyAHEWV58OzaCx7w5hBkO?= =?us-ascii?Q?UJs73elhNdSJ6XWOW9586uF+AhmHV4g3oepdvVb9k95fmy7sQqu98ziFuntg?= =?us-ascii?Q?c6q+FccUYWXa8nGp/3UFSh1lT2h9vrDJtJYVU4CnyyOemHr4UZOAbfm8eTwF?= =?us-ascii?Q?reMK5uCdNm/XW0Czw4wihc0BcWaJyb0KDpsQnEHG/ZoVfbjx3DlArqrwTWHC?= =?us-ascii?Q?kmPTPLYHUgYb4WsUInTR2OTbVSetoLxCBiDtzJdx9IoQOzd5XhWijQEqK6gm?= =?us-ascii?Q?DqoCFpiFVf54yjbvpBkfC54fFaveJWByaWauLGwaH/3u8ybQ9FbZl9KbAMNp?= =?us-ascii?Q?RnDU66pgrmBh3uorAAYAf9VxbKq2n0h7ZA+AAPj4pxAN+UCechRMU9WEb8hX?= =?us-ascii?Q?/vN/Ldj6clLdDoQum9tcMJC2dbXK1VAnSisp4ziE7FSLe4+NRgPEWTTB3UE7?= =?us-ascii?Q?92go4gm6/fRqNWc6Q/693g1UWuM+MVRLSYrtpRrG0JkHx6iVBEGVva053us2?= =?us-ascii?Q?4VtvmtYDXwrTIL8L5afN6Inav6OOYFJaYBVYbUsP+IejzNUK5N2UHQuewD5A?= =?us-ascii?Q?vJ/rMwyRR3i6x4W/sWdzVAr7WEnxXchYL8vsdhLtEjwxw6Sk8+XDxeyz91vS?= =?us-ascii?Q?xNLsIAl3wrL+5+RqGG/Yqp4=3D?= X-OriginatorOrg: bp.renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: cbff585d-0c7b-4ed1-245a-08ddeac3e06d X-MS-Exchange-CrossTenant-AuthSource: TYCPR01MB11947.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2025 08:28:36.5287 (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: uG6jFU1iVTINmrpREPUVM8a41Kq2I0OAMFGcdJ8vELSA+hgPL0ADIhGUlr6sYXmsGurRIMJ36NyVopEOjlK4/1Lfva8bKr3A83aY7hmyteVDRfcLSZmukV0HVSiLrJGf X-MS-Exchange-Transport-CrossTenantHeadersStamped: OSZPR01MB8530 Content-Type: text/plain; charset="utf-8" Prevent issues during reset deassertion by re-asserting the reset if a timeout occurs when trying to deassert. This ensures the reset line is in a known state and improves reliability for hardware that may not immediately clear the reset monitor bit. Reviewed-by: Geert Uytterhoeven Signed-off-by: Tommaso Merciai --- v1->v2: - Collected GUytterhoeven tag - Removed dev_warn() in __rzg2l_cpg_assert() drivers/clk/renesas/rzg2l-cpg.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/drivers/clk/renesas/rzg2l-cpg.c b/drivers/clk/renesas/rzg2l-cp= g.c index 1143ce0afed47..8e9fea7f7dccb 100644 --- a/drivers/clk/renesas/rzg2l-cpg.c +++ b/drivers/clk/renesas/rzg2l-cpg.c @@ -1667,9 +1667,15 @@ static int __rzg2l_cpg_assert(struct reset_controlle= r_dev *rcdev, return 0; } =20 - return readl_poll_timeout_atomic(priv->base + reg, value, - assert =3D=3D !!(value & mask), - 10, 200); + ret =3D readl_poll_timeout_atomic(priv->base + reg, value, + assert =3D=3D !!(value & mask), + 10, 200); + if (ret && !assert) { + value =3D mask << 16; + writel(value, priv->base + CLK_RST_R(info->resets[id].off)); + } + + return ret; } =20 static int rzg2l_cpg_assert(struct reset_controller_dev *rcdev, --=20 2.43.0 From nobody Fri Oct 3 08:52:33 2025 Received: from OS0P286CU011.outbound.protection.outlook.com (mail-japanwestazon11010062.outbound.protection.outlook.com [52.101.228.62]) (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 C76452EBDC8; Wed, 3 Sep 2025 08:28:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.228.62 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888127; cv=fail; b=iy+K9PSAvEOBWsePGFyqV5sawUFqarzEK2tIRXlfPdSVsMe9ME2oKrsnqKhL2wCVR1Fs47+EykqHLpPfrnEXhXkW51/F1v2Rs+bCaPIQsUaWq87+RhoVTqOBPvZ343wPlg3IJL27FvIeYZbJGSeCWE4CIowgCQ+F3b5E+ndgv0Y= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888127; c=relaxed/simple; bh=svuEINQ/OwP0rEIcpfrhEBBvje/erXMJcQ+T5l3Kppw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=CgM7xh/AuJhcZRPCRMRFnO99WzAswOSjBGW1csKVSYPCpuO+HCftP1vYVYDjP8Dn3EULxesujI8g+HoGsbMVC9l8OjK3pBHi2tW9j+/j3DWh7+/cwybn2zc5xlXgYq5QnCf/fjFXEAb3ikHeBlml08Lo0Gu/goh1+zRPUtaRB8E= 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=KKhJKpD2; arc=fail smtp.client-ip=52.101.228.62 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="KKhJKpD2" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=WBoUe1aMxqlvgNIxHWbp2naasEWW+whjyZTOkUaekkhbifPiLeiPogTdBatbRn2YD2Fwj6UXqfEFTH3YHchkjSLamnOqrjrjn2kbM51obSKa8ILPoj2sJKmL4gKwPkCdwMWWrl63TRBbWUMbg2AK9F/q3AwcS7i8t0gnFgS8IAQ/nGGvp03EHpp89nRaZYgvjXIkER/kWlRl1rnpx51HlLB1YKKV85tuQYTAlWYzI6CTzAX0PS3oswudO7IxSooFr0B4cqQn1Qp5yRdYW+ejyYrqcuAMVW+9xeZI3Cu/WVaWI9AKjq0kxEaNqzO/TgFP2BT07SnXGFZeO1g0XkkfOw== 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=7KN9OQXINA8N7hwnpoLwYlaZV7iZzNEkVCoPzuNoMes=; b=WIbN2PrKvMYE3+7Tr8aKDe/M7i/bZZ8IWAMvIXyBVt3faRAkCNhFnbhVZR5ZuUaZ0gXWDZt4WmZuZGAv3FQnBAihcSirOuE527P5G8CFuUKNN60dfaYWSfH+m/cVZSoZi/hCwanuZu2SpggCyu+ycVxJ3MsDyTOVlgXDfQZ45yI++OxwNdEL2AtlNqSMAdTgDnY25PbKoVn6nQGptsdRV2RSJ2LyjUnueI3n5FzxGuakxTUqFe+hVFPta/MifcKK9WGIivGlQuDqxzEHfmkn4jw5ZCiS232kT2eGRDUYne1ySNtRWuFIkvzZfSGQ4nl135Fz+9YGuKS5qViVXWzUwg== 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=7KN9OQXINA8N7hwnpoLwYlaZV7iZzNEkVCoPzuNoMes=; b=KKhJKpD2PhmtjauDyrGHeePWYurYgx+DmmXsCPQxb5+zI2d0/MIaXSBBQU5eORcwmsjzvKj10QOYki3DCWxE7uFQL0Px/Hjme/U664bAaXKfFsrZnujYlt+D3q4KvyP5wWvLIhpnOGYuZ/+Pqcb0lB/DV8Zmfr5pUKnz4/RXJxQ= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bp.renesas.com; Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::6) by OSZPR01MB8530.jpnprd01.prod.outlook.com (2603:1096:604:18b::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9073.27; Wed, 3 Sep 2025 08:28:41 +0000 Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31]) by TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31%5]) with mapi id 15.20.9073.026; Wed, 3 Sep 2025 08:28:41 +0000 From: Tommaso Merciai To: tomm.merciai@gmail.com Cc: linux-renesas-soc@vger.kernel.org, biju.das.jz@bp.renesas.com, Tommaso Merciai , Geert Uytterhoeven , Michael Turquette , Stephen Boyd , Philipp Zabel , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 3/5] clk: renesas: rzv2h: Re-assert reset on deassert timeout Date: Wed, 3 Sep 2025 10:27:52 +0200 Message-ID: <20250903082757.115778-4-tommaso.merciai.xr@bp.renesas.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> References: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: FR3P281CA0058.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:4b::19) To TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::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: TYCPR01MB11947:EE_|OSZPR01MB8530:EE_ X-MS-Office365-Filtering-Correlation-Id: 06f60607-ab8b-46e5-f23e-08ddeac3e315 X-LD-Processed: 53d82571-da19-47e4-9cb4-625a166a4a2a,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|1800799024|376014|366016|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?CgmvSvtNBPEuwolzx4PLoEhZO+NHEYZ0jhjKKyYZmmmuDci2Nhc7IKhRo7Fa?= =?us-ascii?Q?vawbtGVOZJ24cQ9apNfGqMD3D4q2oJLhBwpTCXpMngxxz9GMq8fDrpNHL1lh?= =?us-ascii?Q?7xvoRYpCjxS+f/ljNETwXgD6K2tISgaFnjfz009Crqo86ZvYoRHJVOJC6hEZ?= =?us-ascii?Q?YGTzrmL/D7Y0ATJNBfe4GObHdNyP6e8LTmLyS2FsSQV+W7TUGYe807W64Smb?= =?us-ascii?Q?REnt1MKcbEJO/Z5cmVlal2o7gzIUPcY+Us8IAyq50s1bCrYenYC3hOASkMGH?= =?us-ascii?Q?at+3GLjcR2z3+a2m/k8ntAVfQwjFxGPtxrQjqV2WYDezi5C7s3ogSSvyd7vG?= =?us-ascii?Q?Tp9ZQo3eb9D2Gx8sgKBqEner6b4MlUgd7bByLIi8urErPGYsXbHNQh/MSPqj?= =?us-ascii?Q?uAKMzWDBz4ynecCW+I4sSVrkVSrU97krrEjv0F0/NwCgHGPX+3ZZEjflsXWJ?= =?us-ascii?Q?PIHJvYnB/yJmNoSez6OXva0felqvgoz+FJDsmQjoRjSzKrUf3WLi4en7shLF?= =?us-ascii?Q?n+d5lkABEDvdfXgdGdsIdFGK9NZJtaIHzPmB38R6ceMGUsqXCQ1TJQKJMXR3?= =?us-ascii?Q?Q+UVETOs5+55lDxhZ4EydJE11ZLEzXvE/Y812349OiyRjgmf3spPkvCfJA0h?= =?us-ascii?Q?7ijghxFrxMjJtaWk6pZSoWQGZIYZl6T64PUi1/m9h+aFujoV43C8E/jdsNXu?= =?us-ascii?Q?7TjAmj0DA0UFCUaO/zqv7wSRr51QCuRp4nbrdm18mfb70zEtUK3azMCdxjkX?= =?us-ascii?Q?fE2cIPTITwIi4Zo1WKPsfMPiqxC7K9885fv1/boOd2QiTOmYpVeGDJbyznFE?= =?us-ascii?Q?R0/rh1oBQfeQARXKsSgiVJAlpZKRW/tMTgaU6JpFCJNv76I8Mrjshb+WAzOu?= =?us-ascii?Q?6d/jWZ17PJ0umDRXfBridyKZBHR+0ErmIzi4gQPGHfBexScujooyzbSffneC?= =?us-ascii?Q?zRLHIUNnWEdfpI2Pqcf8AS6SJI4msa4Z26XcLELBiRrue2Xi25XAIXl1vxGa?= =?us-ascii?Q?xLWPG2f0rSNFyqzbwOZuoN8D+ApXkAxzlITDUxapD3L+2ZWWlKd0OzGxCGFL?= =?us-ascii?Q?LLd/ZVg4pumTaxclfPXAmeF+EjLHV0c1UUYqATyzwjxJWdymUjvrnGuwZGua?= =?us-ascii?Q?cg3RDjuqKKH/swLT1D7d2ZrZ5Ys1FrZ1dop+iaa62P2p3EwmAH/yMeWoxAp7?= =?us-ascii?Q?fsJqIc/p1IYW34E5LmlBQIldhBJ5Rm/5Vj4zwL3a8FBn+F1mHR96GKWvlAdC?= =?us-ascii?Q?YqmHApfn54MTExKjzgxOSCwyi04RVCJOAQz4LduObx8Vg3BawWq0P2PGGVyL?= =?us-ascii?Q?J+9g7Ir3hkFyZ+7xScievnMIgv+35j3d1Qh3Sdse9xh36iwaQqJa5kHXMpkB?= =?us-ascii?Q?exvnp8rSF3qVYJD9fD63RqXDTsBNJ47m/b6N3yRe1JuS6XkOBOgCg3l+csVQ?= =?us-ascii?Q?qiSjWU3DfC01pO8HoK3RMRqDEugF9eFLCWQqUDAnwdU4bKpHGvVUpA=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYCPR01MB11947.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(1800799024)(376014)(366016)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?mojCU697peuX4fWb93J1MK89D9yKRdM71dQYCCfR4IJTBuJLyqcq+9zOh+js?= =?us-ascii?Q?tCW9DmezvUUkf8vXkqHTmqa6MTClBKvDEIjkUGEPptyihAQqOQTyFncwmHtm?= =?us-ascii?Q?8/YjGYw9iFfZqwMg8LRzdJh3hejqIn9N8Dm52Ln0yJH6/txb06nWAjhaJ7mz?= =?us-ascii?Q?xxTDeTdxvO/6zSuETx1p5Gn/KjsDslqvMsWqAWvPcnY0p+v+ipDplgNjAhDP?= =?us-ascii?Q?hqo2GRT4tImiAHL1VFj96+yLagrdmoVJ0Q5RBV7B8ZLiCXtO8r81O3283C8g?= =?us-ascii?Q?j7KyxA3lbiyzTCRxk10Krc8XHyNKFKrPSILSLvPyeiAoOQmCJH12xQEWlJoi?= =?us-ascii?Q?XhjxDdWEU+6YFOLI+alijSVqjBUZ5K9V+MsMMF3ox0M0gM5Ly1LKqYc5zBr8?= =?us-ascii?Q?/U/VohVnHCFwTK6EaJ7Oiwv8SjKTVSxG6ga5eu9iXMFLzoCIwd2wAs+sAr18?= =?us-ascii?Q?wjDEak1iNMvtb+ScRJbDGXthL1Yjq3o1hCkDKfWhRQbUUw0up4vBuTSlHDlz?= =?us-ascii?Q?O0O8/mns2XyVLuCStp1kMA4qyTO4ijuA+83W2QOYGrWB+B02QWW7pvnoZsYe?= =?us-ascii?Q?NVfSSdPrYz6bNA4iegalMTl4AK49WRz3e35G5PxUYeNToWMskcqIvDw85FWi?= =?us-ascii?Q?gj1SmBm4eoVPkpCDyjP9N/EP67qndN5TaMOHyhHztf4b9EQDs/Gd780c9h/s?= =?us-ascii?Q?6fhanUNktUHDZl4uZ4uce0b2VpqG3jDUW7BGe6EuRVB5Ca3S/FrRAnEmouYC?= =?us-ascii?Q?1DPa+sI50gYmueI8EZxBuxO9dCNBM+pTlXT7SPT6x9XrQCNlJVc7RVNjrlnb?= =?us-ascii?Q?jJ32n/+tqVmG/C/ssA9TMj6pZJTngv49Gkve5spcw/HhOd1TRk3WhKg1f7bD?= =?us-ascii?Q?rQI7Svxh9jltJL5XKaXIx7v1GuOT84qdR/Um8ocpY72spnSlU/goIS4t4H2N?= =?us-ascii?Q?LJTsi4gBP7hOGpucuWihpSK5Gu8VL9UsW3ctOqhiDhfTU0fsqApnEksnCVBk?= =?us-ascii?Q?E+llLpLgmYZtpYp5BUyF83CzFYoO/LFQyaLfCHUhJo213yPnb29aAJ8QzLWb?= =?us-ascii?Q?X9HE9TQ+wPxbt8ATVWbBA/k1lzY7jReiYJQeQhmENmhdsC+ehl5F4MidJn7Z?= =?us-ascii?Q?UCoooujl8wUgBumpZwHNsjUczjCxkNcLrczXyIPjqvuOSCe2M/cPGYH7cL/G?= =?us-ascii?Q?tGJGDNpmEPdwHwJ2IpmTbEf4aPqzlvDqQboWr7kZEIbjDO8sFA6afNYy5+CX?= =?us-ascii?Q?ggyCh2M95bshNmgaFPf8wlYBv4WHynWcPX24IgYEdYez1zqplu2pUwHp+fcv?= =?us-ascii?Q?dNFso/temEX5yETndlP6GYa+4xGv4WltMS7HnTras3olOM6WrzETs6kvw5vx?= =?us-ascii?Q?uLuy2Ej05jSekRRMqon9xmPiRipxN2C6QLMHal2qJgTAK3XHMnAIXI0aslnD?= =?us-ascii?Q?Twnka0CnD377jYqLnew6Hx+faJrIoiC8kxy+ttbHVmfFH8JbVBi8uyBiiOMY?= =?us-ascii?Q?1CXyC9E0ptEnSM947flGDdVWQoZMsTS26FuEA9bupm6s6NSi5Q37j9tChMRQ?= =?us-ascii?Q?SMUD1fZgXPbqPBq/vlWmYy86k/hF+tehILBSSQdYOxfZldu22luFC3D9Ahr5?= =?us-ascii?Q?t0D4qCgxN2AOMRCVyKA3uZY=3D?= X-OriginatorOrg: bp.renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: 06f60607-ab8b-46e5-f23e-08ddeac3e315 X-MS-Exchange-CrossTenant-AuthSource: TYCPR01MB11947.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2025 08:28:40.7349 (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: CvG71z7iH6EKt1mgFdWC/at6u7zoA9AwUvzWpEnfujyZ9+o4Lq4ZW5YnzJjjZsJTZPGb8th/cHeGf/a96uiMa46mt5+Y4irmLqlqpw+fCQuBZ/G8GZdEh8PPOcnHAU5Q X-MS-Exchange-Transport-CrossTenantHeadersStamped: OSZPR01MB8530 Content-Type: text/plain; charset="utf-8" Prevent issues during reset deassertion by re-asserting the reset if a timeout occurs when trying to deassert. This ensures the reset line is in a known state and improves reliability for hardware that may not immediately clear the reset monitor bit. Reviewed-by: Geert Uytterhoeven Signed-off-by: Tommaso Merciai --- v1->v2: - Collected GUytterhoeven tag - Removed dev_warn() in __rzg2l_cpg_assert() drivers/clk/renesas/rzv2h-cpg.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/drivers/clk/renesas/rzv2h-cpg.c b/drivers/clk/renesas/rzv2h-cp= g.c index 58ccbae0f9046..35c5ff38e231e 100644 --- a/drivers/clk/renesas/rzv2h-cpg.c +++ b/drivers/clk/renesas/rzv2h-cpg.c @@ -854,6 +854,7 @@ static int __rzv2h_cpg_assert(struct reset_controller_d= ev *rcdev, u32 mask =3D BIT(priv->resets[id].reset_bit); u8 monbit =3D priv->resets[id].mon_bit; u32 value =3D mask << 16; + int ret; =20 dev_dbg(rcdev->dev, "%s id:%ld offset:0x%x\n", assert ? "assert" : "deassert", id, reg); @@ -865,9 +866,15 @@ static int __rzv2h_cpg_assert(struct reset_controller_= dev *rcdev, reg =3D GET_RST_MON_OFFSET(priv->resets[id].mon_index); mask =3D BIT(monbit); =20 - return readl_poll_timeout_atomic(priv->base + reg, value, - assert ? (value & mask) : !(value & mask), - 10, 200); + ret =3D readl_poll_timeout_atomic(priv->base + reg, value, + assert ? (value & mask) : !(value & mask), + 10, 200); + if (ret && !assert) { + value =3D mask << 16; + writel(value, priv->base + GET_RST_OFFSET(priv->resets[id].reset_index)); + } + + return ret; } =20 static int rzv2h_cpg_assert(struct reset_controller_dev *rcdev, --=20 2.43.0 From nobody Fri Oct 3 08:52:33 2025 Received: from OS0P286CU011.outbound.protection.outlook.com (mail-japanwestazon11010062.outbound.protection.outlook.com [52.101.228.62]) (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 EBC8B2EC54D; Wed, 3 Sep 2025 08:28:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.228.62 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888129; cv=fail; b=kt6ilVJq6w6lHFm0nISMC0tZmahJ+xAHoUg8CrLvPCK+AZ9sK36ol7Ul5JIYVP4g0by6ZFBhSPI6c2c4ZUa8tlAieYbsJLN82PCnCbY75MW89GFGm9Q5lGdsKPdabmyU/7Ywgzf1Zm6fWryAa7Mc9FIupQI0ul268/Fbh96B8Vo= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888129; c=relaxed/simple; bh=rTYyKhrhdDNJGrg00+d/G1x8aYSyR9uB0OJnBtbbE5c=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=FAXjcn/S6oJ6z78PQsFRVinkN/ORbWUegfhDZ5JP0ZXzwvdtrsjRUDTn7yKVx4qAFUC+i8f2J/Ckq7W++7924G5hMgEGQRvnNe5Tkm1KhWiJdZEZtYmmKXnBlZH6Ou9w95vA7a0xeGvVljMouvCd7494MXNMF6sII2CWp1KsM5w= 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=U5fG+QHK; arc=fail smtp.client-ip=52.101.228.62 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="U5fG+QHK" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=I+sGWLMPUDI3dozjGI0XfigTJ+c+kApATEVfsJViBZr52g/F8SiOKkXVODWdHb5dguIe7MVL/+3hz/VwPRM3H9EgjG1pc0Y9EurrAJ6rEoKJ86S1sNGPSJQI/YCOoTwRyLJkrFlAW6LIiOGn1ZtyZU5CTne5ujQr8pm1Xvkc0x/Z6IYlRk08R6Taolmsb4kWgtpRbVGw5Hhw3YVGKgwSFogfJg+kZyTBRXLusDBpZnVZayZu1naglD6ixNIRe9E2ILAh8VfM+LZFVDiQnDf7SMACdOuG0TK1TO5PKVdoSedeBKdQGA0moq+hWFKam3sGroozcWwnRTSxwu7zs6dmGQ== 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=/JjmwcyjMiTrHYB50QeCDiq20/ad31AA4JAZ0Bn0NME=; b=hiNA+6rG7Ch6SNo3CBPZJgJaGjB8iUCuvj1HFnus348Y7bIFYWsYorxoqt/P0oDwC6tN6H7SguMohfS8LAOVAeobaCBYwq+cks2UqEAtXLL2pGiF32iY55H/NV6v6t5ylxQL9JhT0GfuryyUHtdiylp/QP6XElCXEo2R3BXsbpr5ceSZJ2I79rJecN3yVA0UZHAjZlvfUE6h7k6PRlKGBPf224ClOejHCCjpO+YQO9ckLfAKFkgAlwt693bUXu/6Qgnyj/iwKtP9uLAS9GKsWefOpgS5p0hGbRQKMDK85bG/CeGN/6diBYNiroIrp1e2O4A/ds0+WBVfNtEUrNOUYw== 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=/JjmwcyjMiTrHYB50QeCDiq20/ad31AA4JAZ0Bn0NME=; b=U5fG+QHKQzm6pq+5weiMsywmt/D8QE0ET75xpqFfpQ7UDgPb+LM2VLYj0D8Zn1RTywrivQDUvqelVuWwt23TqfL9B79BNOnNnqPRcr/F0fvK8Wp0PjzzGAR2gi849mABrQ+9cD6HTovhKHe88HirxHbBkntAfvUtpYLhVfUpAf0= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bp.renesas.com; Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::6) by OSZPR01MB8530.jpnprd01.prod.outlook.com (2603:1096:604:18b::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9073.27; Wed, 3 Sep 2025 08:28:45 +0000 Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31]) by TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31%5]) with mapi id 15.20.9073.026; Wed, 3 Sep 2025 08:28:45 +0000 From: Tommaso Merciai To: tomm.merciai@gmail.com Cc: linux-renesas-soc@vger.kernel.org, biju.das.jz@bp.renesas.com, Tommaso Merciai , Geert Uytterhoeven , Michael Turquette , Stephen Boyd , Philipp Zabel , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 4/5] clk: renesas: rzv2h: Simplify polling condition in __rzv2h_cpg_assert() Date: Wed, 3 Sep 2025 10:27:53 +0200 Message-ID: <20250903082757.115778-5-tommaso.merciai.xr@bp.renesas.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> References: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: FR3P281CA0058.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:4b::19) To TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::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: TYCPR01MB11947:EE_|OSZPR01MB8530:EE_ X-MS-Office365-Filtering-Correlation-Id: d2840342-4dac-4416-2e29-08ddeac3e594 X-LD-Processed: 53d82571-da19-47e4-9cb4-625a166a4a2a,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|1800799024|376014|366016|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?CYgGSnjsBR8SGo5xhT+Tz36vEXIceKhAAQQ19pNyP5NbyMJanCo0JXqalMGF?= =?us-ascii?Q?dA+Yeo/V5W9MMZGIWaRFlnlq2a34bBCRsHd1Yzb9rEMR9HVxDR8yet87fK/9?= =?us-ascii?Q?dxorZz+M+XPASD0+IRqMPXGB0DFwXpRgqmrK6KfXfiSVJUAM1jVkZF8YOl1q?= =?us-ascii?Q?6zidaQ10J4/tNGg7PLD8aFl0siE6aqOheqLW2ffxp4RACtEr5Pqi8Y5Yt8/g?= =?us-ascii?Q?zcKo8RgyXZ5vjjDQQ/SF8yLp+19czMcNnybBt9Yt1Z33jCs/ff0m8MsCBLB2?= =?us-ascii?Q?8Ah218g9r2edlqZJmrqOPZvnNAhwfGBXpoJg2x99H5S47ePNqAZ0V4HewwvX?= =?us-ascii?Q?Cni+MvdJdC65Bo3Gb8yFHWLcLo1sTmAmLhhYwBa1r5c5Tqlx16ABVVr/0n1E?= =?us-ascii?Q?Vkc4+F81AkICRGeaIhLhihzPrGPDbmFsKcBs4HNDCaB+5875bcppescTwqf3?= =?us-ascii?Q?7CuLUwkLfjCHoifU0ZAuL+5y7DOyjmeWNSsmXyaiBNYu80KsLcKRMPzAmknp?= =?us-ascii?Q?eQaVICnlffQLN7094myuVqd5l1m1peET2zZUgwMk6L2M5f1TIr4JqOR2RPup?= =?us-ascii?Q?R/aLTvDOY4uw48cVk6+lGUpE9A5lpwFrOAx/7bPrkqhpTxJsj2g0V0PK53XJ?= =?us-ascii?Q?2SDvyxgZGUwgMAYn8m+0NWXqoNyHqD2mBMZoMSR6aMhL/YnRpUvF760+l8/W?= =?us-ascii?Q?1Tt3nhbiaqxrw0ZJiO3UKr7a9gls4h/zOQHZgZxh/WrUJO2ODyebG9QYbeTx?= =?us-ascii?Q?ebA42L7OT9Kf8/YAkOd155QJzh2pU5P/KaG6fs7jZazCD8dXHYZRfOrRxCWe?= =?us-ascii?Q?5iVOgJnk8nu0x16wsvQikHGJDJkEO0djNta4xZSnD8Ip5HnWq81gvjqawTly?= =?us-ascii?Q?gi4Ao4UiYSo64agqJYmkN8R2+JtFYEKNkladtK8VL5YXIb2S4DIwyZpBf735?= =?us-ascii?Q?7ObOQagMnCnWlun7GVbERXgZYOw2w4Jh9UIcGzjU6lAglkPr5JkuMJCuGmyc?= =?us-ascii?Q?jGvxEz+vEpl8fMuyt1V22XtC45qiCDCHpf1y9zSorUvByy6P6X/BGuFL2Uia?= =?us-ascii?Q?4Z5IvBXrp58CfqRg5MB22wnRC9woBz9CFGbTzSpQsUgy2OKJ0GMDwiNAlcr9?= =?us-ascii?Q?n9PjGfAoAKx7APgF3qbd4gyYY6v0KgJQBKScf5QyfIZjAhHBOumfjmHMduDN?= =?us-ascii?Q?u0ghbX/tm5krAMaFM36CXhwV3Oy2hhKRlD7+r6CrL4ZU2LzLO4j8taQHhEpo?= =?us-ascii?Q?lNrBlXqlDICDjIWIuAVSdowhiNcmAqyN5/zRqUUXGwy2nGpwXJbNujbg5FxJ?= =?us-ascii?Q?wXVCWiHoFw9mMqGaJ7xl2DwqdUwvBGw1nDWqdW3NDvCN+OOrs7Xd3reeW+Jb?= =?us-ascii?Q?KT9AIUGiuBJvR70yxOfN27Vhkt8/u368d0QAyVJGpEilTZudFmApGOzsOeU2?= =?us-ascii?Q?DSINMlZF3mezA1DGwZpDYJjKQcTt5hcfK1n3r6EbWOzXrUL55gHlyA=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYCPR01MB11947.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(1800799024)(376014)(366016)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?bzn0Ui2Sfst8pwJM+uEh0siOjjv/cOZ1Cx0lC3HSm7AUen4N2OI2tM5PVcTM?= =?us-ascii?Q?ZDMxS/oF4s6QijnpzjWYF3lPLTZvhwLy+/UbdhfCB9beWU5/ecvOAYHEfO1E?= =?us-ascii?Q?JM7hAwBRLnPMX/qsFXFNtT3BfWE4A2PRWyu/dURONq90V6IjMA2QACfwWKJx?= =?us-ascii?Q?RpPsY9HoVM57mcQvIKspWsU2sHNrZZ1+WtB6w51kc/jRKgNHlL5CDIeWocMo?= =?us-ascii?Q?vIf352lNNcGiANDJotJVNCsz7Pr7Y/oWcpuHN2mR/ycuOyDj4Uo3Zjkq/loM?= =?us-ascii?Q?XXOQt9S/Rqqw5d1YvygbVVfSqiEXONaMv+BRCu4Wt4k4bWYGzn35gSVuHSpf?= =?us-ascii?Q?f+gAT6aVX0w0jx0ZvKc8G5pkhYaY58RjvvF/lucN7S+RDk86RVuTD7BnPS88?= =?us-ascii?Q?jdncrX+JAEzyZG2nM0sw09yCOzKDjQWEjMunjiYGUxW9SNGShZIy6OY4tgJu?= =?us-ascii?Q?k0IkG3OAk67MdW6HQ00gSku+Je/BQlW9HhFBwJXdDYJakbwZBDSqTo+1dFRj?= =?us-ascii?Q?MgzcuukXmtn1+WaOn6F5IgdNo8qxyYPi7rToYYKOJufBOHJgyxKUVIlG1iAY?= =?us-ascii?Q?nFg+rANJNk+lKWVSZvxtLu6TZQ2MBmiTRM7eDmJT7PeTH4jqxrrItXYWy0G+?= =?us-ascii?Q?N34f7G58vn452C46lyesJ1Pde4wG9O3ti9bDA+PxwrQ/gRSaNu9hUi9y/auw?= =?us-ascii?Q?TBTe9QEAw+B2lnpl7tGAkA4jWAZXcPm1KdB/gz6Pppk4cxOppqJqN/+9EqW2?= =?us-ascii?Q?V5gbwy+D1r749paR4daDptyUuyU1N0SMA0AJUvKY+GAYHl0O2T29nDZ/rH+8?= =?us-ascii?Q?TXlo4VSrom+dfKB4zEGQpDFaFhUGmzcDZ7BKLM1DrjYBCXan1Fmoi+c2y1YQ?= =?us-ascii?Q?WmOf/Aa1gAJEAs+fznCJQFFcHzgr+F7Y2Lp0PY8QqqGkAyj4PJxiR6JdALNO?= =?us-ascii?Q?pu1Wb7khWUd0JIAtSb95l7FjgpiyS3nVVbmsBJRuqILRrUYsNzmbqfEJPXg4?= =?us-ascii?Q?1YLUPsjU4ORB4ZbKarFqHY6JoIugVV6Fjbu/w9NlduXc5uxghDWSKJRxqrBB?= =?us-ascii?Q?PaZIsbFfDEDD1inXEYpKKDgPYYeGWb06Nyn0IjCHLtRkyENApHLodQoR6ShJ?= =?us-ascii?Q?nOnzyQngUFVVL5jzvFjE1JYttBl7PUwVDl0Xmn27WgjpoNJXXoAPy3zwa+0B?= =?us-ascii?Q?UM3P5ou3e1cpa8T1syTJFcYSbuWTQRPgqZ7IdCb9am5ezx7KO9Heq4YKVB8l?= =?us-ascii?Q?NRcAFLbdTv46Fq+IEdOEk+/WZWXBA+nbjF4BpEHLNISjzeAM+d4jidG5X4+d?= =?us-ascii?Q?8XtNUDq6V/zDaCy0VhnyD7Bm2xHx/HYSWW/oXuk77qp7/hRAaM9wtwZU+2An?= =?us-ascii?Q?AijmCqybLs2nGrEsxBKK0NXxvxrIKWMowH2KGigy7GvElxvmVsbaa81KqZxg?= =?us-ascii?Q?i5KEkgzssT3QrltplfmTQ/M+XJlpi7lKa0Y1DcwVbJxgDXjtusTj6dLXuzpz?= =?us-ascii?Q?zeg6N5sidMwys9nCI+k8iZ38Y4RjfVQhFsfGo6kso8n9NSVobzMnbNCC8K+A?= =?us-ascii?Q?C/W1ulE23laT3holASlgfalHmVvIBit35Ng0lDxR59Fs8Fuk2b82A1bOtaBJ?= =?us-ascii?Q?yCBd61u2jl2TpRQovYoxBuw=3D?= X-OriginatorOrg: bp.renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: d2840342-4dac-4416-2e29-08ddeac3e594 X-MS-Exchange-CrossTenant-AuthSource: TYCPR01MB11947.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2025 08:28:45.1652 (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: BYYTrX7423RJzvwWcin1xtbpI+HdjfqgUdD1Eb9bMSNZQBY9j2L38lviJUbc1aPYoCT25M/87iGq20C203a+ZxWhdywblF7xKIKJuyLDFAUA/N49t41yWt42iVZaxRr+ X-MS-Exchange-Transport-CrossTenantHeadersStamped: OSZPR01MB8530 Content-Type: text/plain; charset="utf-8" Replace the ternary operator with a direct boolean comparison to improve code readability and maintainability. The logic remains unchanged. Signed-off-by: Tommaso Merciai Reviewed-by: Geert Uytterhoeven --- v1->v2: - New patch drivers/clk/renesas/rzv2h-cpg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/renesas/rzv2h-cpg.c b/drivers/clk/renesas/rzv2h-cp= g.c index 35c5ff38e231e..0e8800906b89e 100644 --- a/drivers/clk/renesas/rzv2h-cpg.c +++ b/drivers/clk/renesas/rzv2h-cpg.c @@ -867,7 +867,7 @@ static int __rzv2h_cpg_assert(struct reset_controller_d= ev *rcdev, mask =3D BIT(monbit); =20 ret =3D readl_poll_timeout_atomic(priv->base + reg, value, - assert ? (value & mask) : !(value & mask), + assert =3D=3D !!(value & mask), 10, 200); if (ret && !assert) { value =3D mask << 16; --=20 2.43.0 From nobody Fri Oct 3 08:52:33 2025 Received: from OS0P286CU011.outbound.protection.outlook.com (mail-japanwestazon11010002.outbound.protection.outlook.com [52.101.228.2]) (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 240242ECEB2; Wed, 3 Sep 2025 08:28:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.228.2 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888137; cv=fail; b=clb5McqA8qP3GUyWkfdVFUtqzOJLVVZqyl7lAjehOXYT0yCYEdc0w9uNTtNsnum+Zl9FRjR3tP9SicCg6San8lvcr171ZMy6Ek3UxBXdPGheu+VCmYqc2fOgSq6NBAGGdkNTQeTzuHxV7/UFTDJriLPhRJX6LrLph7Xl8za+ntw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756888137; c=relaxed/simple; bh=dgNUL9QE4Jmz3LP43idVHybFUWWnTnM7MefrjxgIvwU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=KV+9SQujJMxkHNdw4jwnp2H7i/mzVRC8FHmhDJRH98sjg1rthls9cG4+PSfUuINuz3wZIo3pSqzMlui+7UHxa0nOivJWeNakybD7mUS4gCPmwPqYxrDMEbc0b276VfNS/w0+j+4/aN5SZ3WLToURr42YDZrBJnZU5O709VAJy9w= 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=LvAifYK7; arc=fail smtp.client-ip=52.101.228.2 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="LvAifYK7" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Ah59uscYtazilno7eiOPbOAKNaBvtTRO4RLZlvC3XhVC5RVMHGaIwYBEiyGartR7R4DfZKOXJG4/Sk7pqCQBNCSqJtlXkRg+QgwqrsL80fH+LLRoaWWSbBTcYY7BOOq23pLJw31Q1nUwcg0X6HPnc8NrIbwOCO/+f/UZs7Fui4CaUNS6BjzzCHXXlLRD6xp9xk2lPGSh2QRrsFcS3srm9IikNwSFUQ/mJ/44eI3ugtK28v7hvt3r362K/1vUJzYKbrkJ7HenSyPCZSh59RinS2h/TLE9+ZKgDrk+pM98U7fvuh7/+yanSRoBddR0AqvY6vY3B4mCrE4+o2s98XJhmA== 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=07VX0wpayWKUlVs+AvN1Azr24QZKESgYWTpa8pAhKm8=; b=Hsa8T+JkzHDnsAHMIIdkyM9FifaRj7pXOZIkL8mywvnaDI0YrffOseWEBKhFNWleFRNyLg9N2H2E71t3WIZY6Y3VbIW0Md+rXb8z4W6VEtwoQ89Xhc93VEJHb9LP8fgfwH9rx38Rzp6gv36WsKnzq+KSl+8uQCCPmmCJkdzlKHjw8MvFjlH8+hE/ooXBHEWHTl+eI22SRN4nNMuvsPaYGoXXTS36sD/eVdPKnWvDa0oca8RhyjLfhBQEBjvOFgIRUzDzeFDQ5at//vTKhnAntxiv6hp/nBIDbTbDfgf77+kl7mdKLDtulxmtTljSbkwmqOIT09HQFGgBnrCF1yPhmA== 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=07VX0wpayWKUlVs+AvN1Azr24QZKESgYWTpa8pAhKm8=; b=LvAifYK7CliTsF26p8WqmntPL7tCeJGNg6mDQC2XYNjUYhHMVFFZBgi1le8Hcl7XQpPnJ/MJp5QgVHuJBaKxkU7HuBnhSKp6+R1DkD6rG485Ct0wBYgbOEWDuGBHmFNBkIIgGC+lLCUw9b8mL18xc7z5b/nnOOtJ+cTCEC/VCc4= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=bp.renesas.com; Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::6) by OSZPR01MB8530.jpnprd01.prod.outlook.com (2603:1096:604:18b::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9073.27; Wed, 3 Sep 2025 08:28:52 +0000 Received: from TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31]) by TYCPR01MB11947.jpnprd01.prod.outlook.com ([fe80::63d8:fff3:8390:8d31%5]) with mapi id 15.20.9073.026; Wed, 3 Sep 2025 08:28:52 +0000 From: Tommaso Merciai To: tomm.merciai@gmail.com Cc: linux-renesas-soc@vger.kernel.org, biju.das.jz@bp.renesas.com, Tommaso Merciai , Geert Uytterhoeven , Michael Turquette , Stephen Boyd , Vinod Koul , Philipp Zabel , Wolfram Sang , Lad Prabhakar , Fabrizio Castro , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, dmaengine@vger.kernel.org Subject: [PATCH v2 5/5] dmaengine: sh: rz-dmac: Add runtime PM support Date: Wed, 3 Sep 2025 10:27:54 +0200 Message-ID: <20250903082757.115778-6-tommaso.merciai.xr@bp.renesas.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> References: <20250903082757.115778-1-tommaso.merciai.xr@bp.renesas.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: FR3P281CA0058.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:4b::19) To TYCPR01MB11947.jpnprd01.prod.outlook.com (2603:1096:400:3e1::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: TYCPR01MB11947:EE_|OSZPR01MB8530:EE_ X-MS-Office365-Filtering-Correlation-Id: 48992ef5-953b-4ef3-e292-08ddeac3e9eb X-LD-Processed: 53d82571-da19-47e4-9cb4-625a166a4a2a,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|1800799024|376014|366016|7416014|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?bsXXVj9QdStwgV47ZGWfIEJ3oif3C6A6AfrmiBOG9Ijd6m8VnQlTCxAYyIJY?= =?us-ascii?Q?KqE1IbBdUmCVhKb+B40L3+XND+skb1bVMPErJsMK72tJ5nEco+Im9rq6ysiH?= =?us-ascii?Q?/oe20MHtqrNUtWslQfE329KtagTY7VmdQyk9k6w6hS4BS4HRgq6eCdgDQRPa?= =?us-ascii?Q?tDp9l5nRLd42ihsXXAvTYEFihy14y5K9AqQkKcf6rtwQQSOtkbjMRwHRkfVj?= =?us-ascii?Q?ZLbyzJmWjdVaXgn/uKhVDP3F8jrcLxKw4fX/8pfP/rPhdwwyC4kfqMIGT4R0?= =?us-ascii?Q?Xxj+Z/Z4SMm4HCIPZ4T/q7VrRqCidASCq6DACXBE6YLlHcYnQp9p2c0Wl+At?= =?us-ascii?Q?l9gSodNYKDfqNm9U67XMQlGgiDKeugjbN+Pf5oHmMk4DXoe8+TDVoXXHhT+6?= =?us-ascii?Q?P6q7vmcgFbdNC5ryZoMxyIzd/EnPGbGP9Jnp5Y0ThNmB122wt2BGmU+DBb0h?= =?us-ascii?Q?7d4gV61jztjiNd84Z2UEmhx+j+yIV9WhhOZgXNqdgRnlnhddpp5Tix3Ivh4k?= =?us-ascii?Q?Bl7d+lYI/9WLqaPuh1YGcgUv38a1EPImuowKP9k0rP3MaQU5DbTgsxz2Z+V7?= =?us-ascii?Q?von51Ng9es9oaUHMk33cgJKU2Uht+FKKamaFtvnE1GgCPT70jz0YcU3BYaJV?= =?us-ascii?Q?VnR0Vbcb5itnicTPxn1MQR+1Cxu2lmBagDVHNwFqW1FSgBuzTw5U+qdUz26d?= =?us-ascii?Q?+bhr9Z52x3uMZZdrkMbz+SeDfBYAavhIfLj/Ia3AKGW5qtl8giD1uhc1tn4k?= =?us-ascii?Q?eRmD0CwQo/JzqK2NCpL32RKXsx60ph7/jBTjFQoFSVSFZ0qKKLY/Y+5BBUl6?= =?us-ascii?Q?dWSpzvY6jnOLsr9UQUEUO8RxfO6oxg3WKmtxalylu1/9hKuaojszGmgTi6q9?= =?us-ascii?Q?iSzFcndYTW94YSbXUpwHIYfc9poGm8qRADAnsq1KXhjUTpc1Ah22Fi6oodit?= =?us-ascii?Q?OxwxJn6TVK/GAnKzWc3+PT5n4Q++m8wpfHGLP6eZo4bi+G1gWjHk9lWAo/1c?= =?us-ascii?Q?6Fd5dwZIJE/+ClHjStrTs2opwYrLOV2HljlYpDNDVDEp+vEXg1Y16ReOQRgV?= =?us-ascii?Q?NQBOJJdxmnWF07WcSXN+/bk77WH84eatecSLTpjvE/EW+D2cB6mOzeHhBtaz?= =?us-ascii?Q?JyTtKDpimKmgwg0gfYnDfiyXi3qAIFF9lszOg79AzqHa6VUg4ZzgZQakTtA9?= =?us-ascii?Q?nEENZo9UY/uBhtlenzpEGMpR7uUKvIK6/FnTj6MKxMve0J/gRL03ecCpa26b?= =?us-ascii?Q?ZAWyFFt3RvU10kAmk1QpD08VcPCXYr7Wi4fy56X906hps9rDZPUJBK7/ZIgu?= =?us-ascii?Q?eYTtgkZPma2KRAdZ/QsdzMITOlH/ZW1pVU6TEx2+zk9xPFs8wLVDhHvamV7N?= =?us-ascii?Q?99C1o1gR1LOUi72J/9pgjxTkGkP7iKIEDiD1qCELm9v7pjp8U5K6QCeVROOK?= =?us-ascii?Q?fK9q+GY+vQ6U1+5uiC2+KWIWZOASD1HxzXpQVkZyTQFzKLbV2nG/eg=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYCPR01MB11947.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(1800799024)(376014)(366016)(7416014)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?272GXMGuauJ5d2i7BwyDkoXE2Du7P1/Xt+34QuDTe3ZyfnQEM+5J9Kuj9UdL?= =?us-ascii?Q?wTai0V8ewRsoT62/65yP/q63NR28uk6FcPPq94A+n6OVWyfuENvvYbxW6rQ7?= =?us-ascii?Q?1n6HBpzrMRFJPNjXrU0N+fHfFYGuPd44uVLze3GGSer5QAZg5gulrFimRDKa?= =?us-ascii?Q?YlUIgp3LVEqBwFl7DhJfpzBkFkCDW5+LVuexgOzn2y+xA1NSrOAmKM55wvNe?= =?us-ascii?Q?fhxqcHVutOYEjEzHn72b8POn6ae5LM8Ub7L0SMCIg3/AzvB1J8vXP7b82rmK?= =?us-ascii?Q?XeSA12IuQRbXziOA9WGBDba2B/NiVTEoxFDcowm+3XwH0jSpwJhvyuDksSyf?= =?us-ascii?Q?CRzhiulpjw/gLv4ry12YEYH7LCsNRIljGPGxTB5gGXJpXK/dWqjpw6z0KQXQ?= =?us-ascii?Q?DJQJDhpwzpbaPcm/RP0PxxfyncGHUliyOevmcY9HGHcPvkC/n1V2oXZhizyl?= =?us-ascii?Q?fXWIcXEVkXQpnteG1YaKlqVsgVV9LuJbsSWOb7hq0Tb51mCfQWAVmulgZegi?= =?us-ascii?Q?lxc/VngmL8F7OQYoVVZ/fbiJ73U6joptTD1OQUwv0aNOn6sdFWxEJ3pftJ0T?= =?us-ascii?Q?zln79YpwiS3zVtyp93ZcSZMUwHS+556tRgRLmK7E3lEyQ8Fv366Y1WMjbYEH?= =?us-ascii?Q?zB7RdFlB8d8KQLnWL3jqox3V2aydjKPeAja4VHRMEfDIzpvxYlYV7S/NdLzs?= =?us-ascii?Q?Vj2tSaGH2vNZNry6+RknyaLdZiOKARwtfw6wdM5+UVuVyiIkve2z3kAnMQS4?= =?us-ascii?Q?8fDrvvdFQj+MdYoUsl7m3wxuUftdCnzzMhYN9uwm3xIujIt4MrMTxkA6P3/x?= =?us-ascii?Q?NiIwTZg/pCRzjJHLxMI8VgWr+1zwuR++tkrp5RDjGJyVpF00YGNtGxd7ojwL?= =?us-ascii?Q?xu7j9tYaoo4QEQXX6b3eEbmJKyF9QIky6IWGVvuWy9wkmZ1jGqwUqPPHV4Vj?= =?us-ascii?Q?T3lweP891HUIsd0ms7DUHgBmTTMqmfsh0Szr+MmhNzEMBSKRcHq1BwZMXQ/5?= =?us-ascii?Q?GzbTiuJNX+RNbJJL53oyiRIJFdA6YtrVRQMZHlw8sIDaPjuAa57gpC4E31KQ?= =?us-ascii?Q?gd+dDk3zAIbzhcnDdSTrzU1/zei9IbqZkMDdy7RrAUkz/mA6tPK56EbwpR8w?= =?us-ascii?Q?crGOTA9FyoOM/Ht+O+Sz4/xUS54aUmfTl4jDYnR2PMXxkXluJRWDOR7ei7Ru?= =?us-ascii?Q?rNzgcp+UOO1EG5Qf9jmR8ti4lS0PeK5zom8xLIg3a/xqFsO62PAcGmkHjPL4?= =?us-ascii?Q?py/GFRLZqmib0OkINE50n/KT3rr0MoqXvOdwJ04gXc06hH7FPCYsNbLfUvem?= =?us-ascii?Q?S1k54PeYFX2nlGqbTSQBsPKLDsc7/HCeFTHNbptevuyRnvsdj8Wowo86YiRw?= =?us-ascii?Q?Qr9cFcFfYvQ/h+ddcd+8swmh/+gAla0BeRF1F+dm5CQ0Dd56IRCV3giTxHS4?= =?us-ascii?Q?3z6qq2fcq5z1GTkE+dpHT85w5kR3fIMVD0goP5Lm7iFzqwRYlNtgEcvHTpjn?= =?us-ascii?Q?0CAukRFweoJaZujz5NK/ejJBftG7qZLH9M/BD3/65nZ1Cmfy6Xsk9suYsWQj?= =?us-ascii?Q?1YCzVSM7c6d+nTo73ahtAxyEgjPl8pKoJwXSH71KntxXn5BIkTj2tecmOepw?= =?us-ascii?Q?DNZxBeDfkNBz+0k3ZElfzWY=3D?= X-OriginatorOrg: bp.renesas.com X-MS-Exchange-CrossTenant-Network-Message-Id: 48992ef5-953b-4ef3-e292-08ddeac3e9eb X-MS-Exchange-CrossTenant-AuthSource: TYCPR01MB11947.jpnprd01.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2025 08:28:52.2247 (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: 6MhP/uUYd88KLrxQmIncK1g7QVSKcnNNd/DVdnZMpkc/TqUAGYvtcFqddei/lwXLdMpyhA57+dyVsmhLHFTGKbIfSon/+qWTREwx8uATFhWpYXxcMcw8rDfGCUpC5wI3 X-MS-Exchange-Transport-CrossTenantHeadersStamped: OSZPR01MB8530 Content-Type: text/plain; charset="utf-8" Enable runtime power management in the rz-dmac driver by adding suspend and resume callbacks. This ensures the driver can correctly assert and deassert the reset control and manage power state transitions during suspend and resume. Adding runtime PM support allows the DMA controller to reduce power consumption when idle and maintain correct operation across system sleep states, addressing the previous lack of dynamic power management in the driver. Signed-off-by: Tommaso Merciai --- v1->v2: - No chanes drivers/dma/sh/rz-dmac.c | 57 +++++++++++++++++++++++++++++++++++----- 1 file changed, 50 insertions(+), 7 deletions(-) diff --git a/drivers/dma/sh/rz-dmac.c b/drivers/dma/sh/rz-dmac.c index 1f687b08d6b86..2f06bdb7ce3be 100644 --- a/drivers/dma/sh/rz-dmac.c +++ b/drivers/dma/sh/rz-dmac.c @@ -437,6 +437,17 @@ static int rz_dmac_xfer_desc(struct rz_dmac_chan *chan) * DMA engine operations */ =20 +static void rz_dmac_chan_init_all(struct rz_dmac *dmac) +{ + unsigned int i; + + rz_dmac_writel(dmac, DCTRL_DEFAULT, CHANNEL_0_7_COMMON_BASE + DCTRL); + rz_dmac_writel(dmac, DCTRL_DEFAULT, CHANNEL_8_15_COMMON_BASE + DCTRL); + + for (i =3D 0; i < dmac->n_channels; i++) + rz_dmac_ch_writel(&dmac->channels[i], CHCTRL_DEFAULT, CHCTRL, 1); +} + static int rz_dmac_alloc_chan_resources(struct dma_chan *chan) { struct rz_dmac_chan *channel =3D to_rz_dmac_chan(chan); @@ -970,10 +981,6 @@ static int rz_dmac_probe(struct platform_device *pdev) goto err_pm_disable; } =20 - ret =3D reset_control_deassert(dmac->rstc); - if (ret) - goto err_pm_runtime_put; - for (i =3D 0; i < dmac->n_channels; i++) { ret =3D rz_dmac_chan_probe(dmac, &dmac->channels[i], i); if (ret < 0) @@ -1028,8 +1035,6 @@ static int rz_dmac_probe(struct platform_device *pdev) channel->lmdesc.base_dma); } =20 - reset_control_assert(dmac->rstc); -err_pm_runtime_put: pm_runtime_put(&pdev->dev); err_pm_disable: pm_runtime_disable(&pdev->dev); @@ -1052,13 +1057,50 @@ static void rz_dmac_remove(struct platform_device *= pdev) channel->lmdesc.base, channel->lmdesc.base_dma); } - reset_control_assert(dmac->rstc); pm_runtime_put(&pdev->dev); pm_runtime_disable(&pdev->dev); =20 platform_device_put(dmac->icu.pdev); } =20 +static int rz_dmac_runtime_suspend(struct device *dev) +{ + struct rz_dmac *dmac =3D dev_get_drvdata(dev); + + return reset_control_assert(dmac->rstc); +} + +static int rz_dmac_runtime_resume(struct device *dev) +{ + struct rz_dmac *dmac =3D dev_get_drvdata(dev); + + return reset_control_deassert(dmac->rstc); +} + +static int rz_dmac_resume(struct device *dev) +{ + struct rz_dmac *dmac =3D dev_get_drvdata(dev); + int ret; + + ret =3D pm_runtime_force_resume(dev); + if (ret) + return ret; + + rz_dmac_chan_init_all(dmac); + + return 0; +} + +static const struct dev_pm_ops rz_dmac_pm_ops =3D { + /* + * TODO for system sleep/resume: + * - Wait for the current transfer to complete and stop the device, + * - Resume transfers, if any. + */ + NOIRQ_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, rz_dmac_resume) + RUNTIME_PM_OPS(rz_dmac_runtime_suspend, rz_dmac_runtime_resume, NULL) +}; + static const struct of_device_id of_rz_dmac_match[] =3D { { .compatible =3D "renesas,r9a09g057-dmac", }, { .compatible =3D "renesas,rz-dmac", }, @@ -1068,6 +1110,7 @@ MODULE_DEVICE_TABLE(of, of_rz_dmac_match); =20 static struct platform_driver rz_dmac_driver =3D { .driver =3D { + .pm =3D pm_ptr(&rz_dmac_pm_ops), .name =3D "rz-dmac", .of_match_table =3D of_rz_dmac_match, }, --=20 2.43.0