Hi Brian,
On Tue, 12 Aug 2025 at 00:18, Brian Masney via B4 Relay
<devnull+bmasney.redhat.com@kernel.org> wrote:
>
> From: Brian Masney <bmasney@redhat.com>
>
> The round_rate() clk ops is deprecated, so migrate this driver from
> round_rate() to determine_rate() using the Coccinelle semantic patch
> on the cover letter of this series.
>
> Signed-off-by: Brian Masney <bmasney@redhat.com>
> ---
> drivers/clk/mstar/clk-msc313-cpupll.c | 18 ++++++++++--------
> 1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/clk/mstar/clk-msc313-cpupll.c b/drivers/clk/mstar/clk-msc313-cpupll.c
> index a93e2dba09d3523199263494efd2bf6987bb4156..3e643be02fe2a79c043c1b79a5e95c869b757ba1 100644
> --- a/drivers/clk/mstar/clk-msc313-cpupll.c
> +++ b/drivers/clk/mstar/clk-msc313-cpupll.c
> @@ -140,20 +140,22 @@ static unsigned long msc313_cpupll_recalc_rate(struct clk_hw *hw, unsigned long
> parent_rate);
> }
>
> -static long msc313_cpupll_round_rate(struct clk_hw *hw, unsigned long rate,
> - unsigned long *parent_rate)
> +static int msc313_cpupll_determine_rate(struct clk_hw *hw,
> + struct clk_rate_request *req)
> {
> - u32 reg = msc313_cpupll_regforfrequecy(rate, *parent_rate);
> - long rounded = msc313_cpupll_frequencyforreg(reg, *parent_rate);
> + u32 reg = msc313_cpupll_regforfrequecy(req->rate, req->best_parent_rate);
> + long rounded = msc313_cpupll_frequencyforreg(reg, req->best_parent_rate);
>
> /*
> * This is my poor attempt at making sure the resulting
> * rate doesn't overshoot the requested rate.
> */
> - for (; rounded >= rate && reg > 0; reg--)
> - rounded = msc313_cpupll_frequencyforreg(reg, *parent_rate);
> + for (; rounded >= req->rate && reg > 0; reg--)
> + rounded = msc313_cpupll_frequencyforreg(reg, req->best_parent_rate);
>
> - return rounded;
> + req->rate = rounded;
> +
> + return 0;
> }
>
> static int msc313_cpupll_set_rate(struct clk_hw *hw, unsigned long rate, unsigned long parent_rate)
> @@ -168,7 +170,7 @@ static int msc313_cpupll_set_rate(struct clk_hw *hw, unsigned long rate, unsigne
>
> static const struct clk_ops msc313_cpupll_ops = {
> .recalc_rate = msc313_cpupll_recalc_rate,
> - .round_rate = msc313_cpupll_round_rate,
> + .determine_rate = msc313_cpupll_determine_rate,
> .set_rate = msc313_cpupll_set_rate,
> };
>
>
> --
> 2.50.1
>
>
Only visually inspected but seems correct.
Reviewed-by: Daniel Palmer <daniel@thingy.jp>
Thanks,
Daniel