clk: qcom: apss-ipq-pll: Use stromer plus ops for stromer plus pll
authorVaradarajan Narayanan <quic_varada@quicinc.com>
Fri, 20 Oct 2023 06:19:33 +0000 (11:49 +0530)
committerBjorn Andersson <andersson@kernel.org>
Sat, 21 Oct 2023 15:48:39 +0000 (08:48 -0700)
The set rate and determine rate operations are different between
Stromer and Stromer Plus PLLs. Since the programming sequence is
different, the PLLs dont get configured properly and random,
inexplicable crash/freeze is seen. Hence, use stromer plus ops
for ipq_pll_stromer_plus.

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Acked-by: Stephen Boyd <sboyd@kernel.org>
Fixes: c7ef7fbb1ccf ("clk: qcom: apss-ipq-pll: add support for IPQ5332")
Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com>
Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com>
Link: https://lore.kernel.org/r/c86ecaa23dc4f39650bcf4a3bd54a617a932e4fd.1697781921.git.quic_varada@quicinc.com
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/clk/qcom/apss-ipq-pll.c

index e170331858cc14f833d8e85f36edc8b4f10c2dfd..18c4ffe153d6c6fc81fe128ddbe5a62ad835e1ed 100644 (file)
@@ -68,7 +68,7 @@ static struct clk_alpha_pll ipq_pll_stromer_plus = {
                                .fw_name = "xo",
                        },
                        .num_parents = 1,
-                       .ops = &clk_alpha_pll_stromer_ops,
+                       .ops = &clk_alpha_pll_stromer_plus_ops,
                },
        },
 };