Re: [PATCH 000/114] clk: convert drivers from deprecated round_rate() to determine_rate()
From: Manivannan Sadhasivam
Date: Mon Aug 11 2025 - 12:13:57 EST
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
--
மணிவண்ணன் சதாசிவம்