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@xxxxxxxxxx>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxx>
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),
> { }
