Quoting Taniya Das (2022-02-02 09:42:13)
> diff --git a/drivers/clk/qcom/clk-alpha-pll.c b/drivers/clk/qcom/clk-alpha-pll.c
> index 4406cf609aae..4e2e93cd8c8b 100644
> --- a/drivers/clk/qcom/clk-alpha-pll.c
> +++ b/drivers/clk/qcom/clk-alpha-pll.c
> @@ -812,12 +812,25 @@ static int alpha_pll_huayra_set_rate(struct clk_hw *hw, unsigned long rate,
> return 0;
> }
>
> -static long alpha_pll_huayra_round_rate(struct clk_hw *hw, unsigned long rate,
> - unsigned long *prate)
> +static int alpha_pll_huayra_determine_rate(struct clk_hw *hw,
> + struct clk_rate_request *req)
> {
> + struct clk_hw *parent_hw;
> + unsigned long rrate, prate;
> u32 l, a;
>
> - return alpha_huayra_pll_round_rate(rate, *prate, &l, &a);
> + parent_hw = clk_hw_get_parent(hw);
The clk_rate_request should already have the parent_hw pointer set in
it. See clk_core_init_rate_req(). So there's no need to
clk_hw_get_parent() again here.
> + if (!parent_hw)
> + return -EINVAL;
> +
> + prate = clk_hw_get_rate(parent_hw);
And low and behold the parent rate is also prepopulated in 'req'. Just
use that.
> + rrate = alpha_huayra_pll_round_rate(req->rate, prate, &l, &a);
> +
> + req->best_parent_hw = parent_hw;
Remove.
> + req->best_parent_rate = prate;
Remove.
> + req->rate = rrate;
Keep.
req->rate = alpha_pll_huayra_round_rate(req->rate, prate, &l, &a);
> +
> + return 0;
> }
>
> static int trion_pll_is_enabled(struct clk_alpha_pll *pll,
