diff options
| author | Brian Masney <bmasney@redhat.com> | 2026-01-08 16:16:25 -0500 |
|---|---|---|
| committer | Brian Masney <bmasney@redhat.com> | 2026-01-22 10:48:25 -0500 |
| commit | 463b97bef0c9fb02b743d6b9f0d698cae81a1d9f (patch) | |
| tree | aaa471e76d7893815f8fa7f9139e01b1b66beaf1 | |
| parent | 7468ac0ce262640e928c71a5da6f6b42a89d453e (diff) | |
clk: bm1880: 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: 64613d7fb42f ("clk: bm1880: convert from round_rate() to determine_rate()")
Acked-by: Manivannan Sadhasivam <mani@kernel.org>
Signed-off-by: Brian Masney <bmasney@redhat.com>
| -rw-r--r-- | drivers/clk/clk-bm1880.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/clk/clk-bm1880.c b/drivers/clk/clk-bm1880.c index 536df8e828ea..1bdceb36fa87 100644 --- a/drivers/clk/clk-bm1880.c +++ b/drivers/clk/clk-bm1880.c @@ -625,10 +625,7 @@ static int bm1880_clk_div_determine_rate(struct clk_hw *hw, div->width, div->flags, val); } - req->rate = divider_round_rate(hw, req->rate, &req->best_parent_rate, - div->table, div->width, div->flags); - - return 0; + return divider_determine_rate(hw, req, div->table, div->width, div->flags); } static int bm1880_clk_div_set_rate(struct clk_hw *hw, unsigned long rate, |
