From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 5BF3F2E54BA; Mon, 11 Aug 2025 16:01:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754928063; cv=none; b=ZlepQsCnNHdekX2OFdvRUa0qK4smVfld6C1UCABGM8xhPN0oyUoeMT4ZOXg7eFoagE45NOIxK5MJk9OOQ3A5m7Rs7g3OFuRAS44fA9x1EKWpAcGvoMFn3auiXV9hcxcgmLvE9eprJOgShYmGYoyCIXKfS3k68t1Tmd1XhhZ8imk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754928063; c=relaxed/simple; bh=Mt5Z9iaxEJqunSSWFhe3foZePTxEE32AzKMZx3Qr8RQ=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=IdmCN5O2ghxDAYE2RCr3Q5lM49+T9HxG4kntuYayz4bWR2INTKwIdiQIH6/TrG/MRIYwHAdGZb35cZRuWXbgYlRd5WIv478L8pamFPjUOJMf06HM5xR9zReD+ap30NmASpSn8SB6Euy4KJZRNfBUTQaevmQZDV/2Pz+Q/GwmOjQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=MyiP+pGK; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="MyiP+pGK" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 16A6AC4CEED; Mon, 11 Aug 2025 16:00:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1754928062; bh=Mt5Z9iaxEJqunSSWFhe3foZePTxEE32AzKMZx3Qr8RQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=MyiP+pGKfJHAHSCkg/AuFQl20aYOw9VY+sW09ZDqrRnXl3qhO6kW3xwjReBD5Z32U rc5u4QiD9di8rs5Y26pPKqgz3ObtPPTdetcOMRnY4L3JBnCvV8biR9qNn6DSikeOF7 VhE5sUJSpEKSYYri6ra4/Vt6YfnG4ukgWhyQSV/dITF7k1RhjSrtVsfcRLEij9Eys6 q9OGM68dgjgR0McTiVcrG5fpKLeSFSRz3ApxW9cooonoB2N5qjQ2mVUArcBqG3wBQ8 ehA3INw6J09SgYdPezVfIC1CQjP1rhvwV1/tDG6KwuSpPtIw0jol1Ml6Et91NjCKhR E8uC2ag7Nqdxw== Date: Mon, 11 Aug 2025 21:30:30 +0530 From: Manivannan Sadhasivam To: bmasney@redhat.com Cc: Michael Turquette , Stephen Boyd , Sudeep Holla , Cristian Marussi , Chen Wang , Inochi Amaoto , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Paul Cercueil , Keguang Zhang , Taichi Sugaya , Takao Orito , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Jacky Huang , Shan-Chun Hung , Vladimir Zapolskiy , Piotr Wojtaszczyk , Paul Walmsley , Samuel Holland , Yixun Lan , Steen Hegelund , Daniel Machon , UNGLinuxDriver@microchip.com, Orson Zhai , Baolin Wang , Chunyan Zhang , Maxime Coquelin , Alexandre Torgue , Michal Simek , Maxime Ripard , Andreas =?utf-8?Q?F=C3=A4rber?= , Sven Peter , Janne Grunau , Alyssa Rosenzweig , Neal Gompa , Eugeniy Paltsev , Ray Jui , Scott Branden , Broadcom internal kernel review list , Max Filippov , Matthias Brugger , AngeloGioacchino Del Regno , Daniel Palmer , Romain Perier , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Bjorn Andersson , Geert Uytterhoeven , Heiko Stuebner , Andrea della Porta , Krzysztof Kozlowski , Sylwester Nawrocki , Chanwoo Choi , Alim Akhtar , Qin Jian , Viresh Kumar , Ulf Hansson , Luca Ceresoli , Alex Helms , Linus Walleij , Liviu Dudau , Lorenzo Pieralisi , Nobuhiro Iwamatsu , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, arm-scmi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, sophgo@lists.linux.dev, linux-mips@vger.kernel.org, imx@lists.linux.dev, linux-riscv@lists.infradead.org, spacemit@lists.linux.dev, linux-stm32@st-md-mailman.stormreply.com, patches@opensource.cirrus.com, linux-actions@lists.infradead.org, asahi@lists.linux.dev, linux-mediatek@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-samsung-soc@vger.kernel.org, soc@lists.linux.dev Subject: Re: [PATCH 000/114] clk: convert drivers from deprecated round_rate() to determine_rate() Message-ID: <3rhp73jjlpcoutaj56pjhrjtjxugf2nghiygax6s3n47biwfr7@pdk5zorat5dh> References: <20250811-clk-for-stephen-round-rate-v1-0-b3bf97b038dc@redhat.com> Precedence: bulk X-Mailing-List: linux-mips@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20250811-clk-for-stephen-round-rate-v1-0-b3bf97b038dc@redhat.com> On Mon, Aug 11, 2025 at 11:17:52AM GMT, Brian Masney via B4 Relay wrote: > The round_rate() clk ops is deprecated in the clk framework in favor > of the determine_rate() clk ops, so let's go ahead and convert the > various clk drivers using the Coccinelle semantic patch posted below. > I did a few minor cosmetic cleanups of the code in a few cases. > > This series is broken up into several categories: > > - Patch 1 to clk/at91/peripheral is a bug fix for a return value. > > - Patch 2 to clk/at91/peripheral wasn't straight forward like the other > drivers, so I'm explicitly calling this one out separately. > > - Patch 3 adds a determine_rate() op and keeps the round_rate() in place > since this is needed by drivers/clk/actions/owl-composite.c and keeps > the tree bisectable. The deprecated round_rate() op is dropped in patch > 113. > > - Patches 4-70 are for drivers where there is no clk submaintainer > listed in the MAINTAINERS file. Patch 19 includes a minor fix so that > Coccinelle can be ran against a particular source file. > > - Patches 71-110 are for drivers where this is an entry in MAINTAINERS > for the driver. > > - Patches 111-114 are for drivers that implement both round_rate() and > determine_rate(), so the round_rate() implementation is dropped. > > Once all of my conversion patches across the various trees in the kernel > have been merged, I will post a small series that removes the > round_rate() op from the clk core and the documentation. Here's the > other patch series that are currently in flight that need to be merged > before we can remove round_rate() from the core. > Please do not CC all reviewers/maintainers of individual clock controller drivers to all patches, but only to the respective clock controller patches. It just spams our inbox. CCing all of us in the cover letter is fine though. - Mani -- மணிவண்ணன் சதாசிவம்