Re: [PATCH v3 04/14] clk: qcom: cpu-8996: fix the init clock rate

[Konrad Dybcio <konrad.dybcio@linaro.org>]

On 13.01.2023 13:05, Dmitry Baryshkov wrote:
> Current multiplier (60) results in CPU getting the rate which is
> unlisted in the CPU frequency tables (60 * 19.2 = 1152 MHz). This
> results in warnings from the cpufreq during startup.
> 
> Change PLL programming (l = 54) to init CPU clocks to start with the
> frequency of 54 * 19.2 = 1036.8 MHz which is supported by both power and
> performance clusters from all speed bins.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  drivers/clk/qcom/clk-cpu-8996.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/qcom/clk-cpu-8996.c b/drivers/clk/qcom/clk-cpu-8996.c
> index ed8cb558e1aa..d51965fda56d 100644
> --- a/drivers/clk/qcom/clk-cpu-8996.c
> +++ b/drivers/clk/qcom/clk-cpu-8996.c
> @@ -102,7 +102,7 @@ static const u8 alt_pll_regs[PLL_OFF_MAX_REGS] = {
>  /* PLLs */
>  
>  static const struct alpha_pll_config hfpll_config = {
> -	.l = 60,
> +	.l = 54,
>  	.config_ctl_val = 0x200d4828,
>  	.config_ctl_hi_val = 0x006,
>  	.test_ctl_val = 0x1c000000,