| Message ID | 20220712125922.3461675-5-bryan.odonoghue@linaro.org |
|---|---|
| State | Accepted |
| Commit | 2c37d70c82e826e44163ab2c83f8b7b81e7db35a |
| Headers | show |
| Series | clk: qcom: gcc-msm8939: Align clock frequencies with downstream driver | expand |
On 12.07.2022 14:59, Bryan O'Donoghue wrote: > Reviewing the qcom msm8936.c clock frequency tables we see > > static struct clk_freq_tbl ftbl_gcc_camss_cpp_clk[] = { > F( 160000000, gpll0_out_main, 5, 0, 0), > F( 200000000, gpll0_out_main, 4, 0, 0), > F( 228570000, gpll0_out_main, 3.5, 0, 0), > F( 266670000, gpll0_out_main, 3, 0, 0), > F( 320000000, gpll0_out_main, 2.5, 0, 0), > F( 465000000, gpll2_out_main, 2, 0, 0), > F_END > }; > which is a super-set of the msm8916 original definitions. > Add in the missing frequency definitions now. > > Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org> Konrad > drivers/clk/qcom/gcc-msm8939.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/clk/qcom/gcc-msm8939.c b/drivers/clk/qcom/gcc-msm8939.c > index 2aad40dbef8e6..6b265b68524f2 100644 > --- a/drivers/clk/qcom/gcc-msm8939.c > +++ b/drivers/clk/qcom/gcc-msm8939.c > @@ -1143,6 +1143,9 @@ static struct clk_rcg2 csi1phytimer_clk_src = { > > static const struct freq_tbl ftbl_gcc_camss_cpp_clk[] = { > F(160000000, P_GPLL0, 5, 0, 0), > + F(200000000, P_GPLL0, 4, 0, 0), > + F(228570000, P_GPLL0, 3.5, 0, 0), > + F(266670000, P_GPLL0, 3, 0, 0), > F(320000000, P_GPLL0, 2.5, 0, 0), > F(465000000, P_GPLL2, 2, 0, 0), > { }
diff --git a/drivers/clk/qcom/gcc-msm8939.c b/drivers/clk/qcom/gcc-msm8939.c index 2aad40dbef8e6..6b265b68524f2 100644 --- a/drivers/clk/qcom/gcc-msm8939.c +++ b/drivers/clk/qcom/gcc-msm8939.c @@ -1143,6 +1143,9 @@ static struct clk_rcg2 csi1phytimer_clk_src = { static const struct freq_tbl ftbl_gcc_camss_cpp_clk[] = { F(160000000, P_GPLL0, 5, 0, 0), + F(200000000, P_GPLL0, 4, 0, 0), + F(228570000, P_GPLL0, 3.5, 0, 0), + F(266670000, P_GPLL0, 3, 0, 0), F(320000000, P_GPLL0, 2.5, 0, 0), F(465000000, P_GPLL2, 2, 0, 0), { }
Reviewing the qcom msm8936.c clock frequency tables we see static struct clk_freq_tbl ftbl_gcc_camss_cpp_clk[] = { F( 160000000, gpll0_out_main, 5, 0, 0), F( 200000000, gpll0_out_main, 4, 0, 0), F( 228570000, gpll0_out_main, 3.5, 0, 0), F( 266670000, gpll0_out_main, 3, 0, 0), F( 320000000, gpll0_out_main, 2.5, 0, 0), F( 465000000, gpll2_out_main, 2, 0, 0), F_END }; which is a super-set of the msm8916 original definitions. Add in the missing frequency definitions now. Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> --- drivers/clk/qcom/gcc-msm8939.c | 3 +++ 1 file changed, 3 insertions(+)