diff options
| author | Brian Masney <bmasney@redhat.com> | 2026-01-08 16:16:39 -0500 |
|---|---|---|
| committer | Brian Masney <bmasney@redhat.com> | 2026-01-22 10:48:26 -0500 |
| commit | 56c1cfb488cc17944c200edad96191a70a3783ba (patch) | |
| tree | 92536047e4d855fe0e8a05fa72a5260c45f29b44 | |
| parent | 2532795a6d6bb9791d713ffa9d9433f293b45b14 (diff) | |
clk: versaclock3: convert from divider_round_rate() to divider_determine_rate()
The divider_round_rate() function is now deprecated, so let's migrate
to divider_determine_rate() instead so that this deprecated API can be
removed.
Note that when the main function itself was migrated to use
determine_rate, this was mistakenly converted to:
req->rate = divider_round_rate(...)
This is invalid in the case when an error occurs since it can set the
rate to a negative value.
Fixes: 9e3372b2ebac ("clk: versaclock3: convert from round_rate() to determine_rate()")
Signed-off-by: Brian Masney <bmasney@redhat.com>
| -rw-r--r-- | drivers/clk/clk-versaclock3.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/clk/clk-versaclock3.c b/drivers/clk/clk-versaclock3.c index 1849863dbd67..27b6cf70f3ae 100644 --- a/drivers/clk/clk-versaclock3.c +++ b/drivers/clk/clk-versaclock3.c @@ -523,11 +523,8 @@ static int vc3_div_determine_rate(struct clk_hw *hw, return 0; } - req->rate = divider_round_rate(hw, req->rate, &req->best_parent_rate, - div_data->table, - div_data->width, div_data->flags); - - return 0; + return divider_determine_rate(hw, req, div_data->table, div_data->width, + div_data->flags); } static int vc3_div_set_rate(struct clk_hw *hw, unsigned long rate, |
