clk: qcom: videocc-sm8550: switch to clk_lucid_ole_pll_configure

Instead of manually specifying the RINGOSC_CAL_L and CAL_L values in the
alpha_pll_config.l field, use the proper clk_lucid_ole_pll_configure()
function to configure the PLL.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Link: https://lore.kernel.org/r/20231016084356.1301854-2-dmitry.baryshkov@linaro.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>

diff --git a/drivers/clk/qcom/videocc-sm8550.c b/drivers/clk/qcom/videocc-sm8550.c
index 3bb136ec31b1641f68247f209cf2999f6a5af905..f3c9dfaee968fb73ea1a5366c95f24c51a212a2c 100644 (file)
--- a/drivers/clk/qcom/videocc-sm8550.c
+++ b/drivers/clk/qcom/videocc-sm8550.c
@@ -36,8 +36,7 @@ static const struct pll_vco lucid_ole_vco[] = {
 };
 
 static const struct alpha_pll_config video_cc_pll0_config = {
-       /* .l includes RINGOSC_CAL_L_VAL, CAL_L_VAL, L_VAL fields */
-       .l = 0x44440025,
+       .l = 0x25,
        .alpha = 0x8000,
        .config_ctl_val = 0x20485699,
        .config_ctl_hi_val = 0x00182261,
@@ -68,8 +67,7 @@ static struct clk_alpha_pll video_cc_pll0 = {
 };
 
 static const struct alpha_pll_config video_cc_pll1_config = {
-       /* .l includes RINGOSC_CAL_L_VAL, CAL_L_VAL, L_VAL fields */
-       .l = 0x44440036,
+       .l = 0x36,
        .alpha = 0xb000,
        .config_ctl_val = 0x20485699,
        .config_ctl_hi_val = 0x00182261,
@@ -427,8 +425,8 @@ static int video_cc_sm8550_probe(struct platform_device *pdev)
                return PTR_ERR(regmap);
        }
 
-       clk_lucid_evo_pll_configure(&video_cc_pll0, regmap, &video_cc_pll0_config);
-       clk_lucid_evo_pll_configure(&video_cc_pll1, regmap, &video_cc_pll1_config);
+       clk_lucid_ole_pll_configure(&video_cc_pll0, regmap, &video_cc_pll0_config);
+       clk_lucid_ole_pll_configure(&video_cc_pll1, regmap, &video_cc_pll1_config);
 
        /*
         * Keep clocks always enabled: