[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <411da19f-10f3-6dc1-a708-cdf06be9c4d8@linaro.org>
Date: Tue, 25 Jul 2023 10:02:28 +0200
From: Konrad Dybcio <konrad.dybcio@...aro.org>
To: Adam Skladowski <a39.skl@...il.com>
Cc: phone-devel@...r.kernel.org, ~postmarketos/upstreaming@...ts.sr.ht,
Bjorn Andersson <andersson@...nel.org>,
Andy Gross <agross@...nel.org>,
Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
AngeloGioacchino Del Regno <kholk11@...il.com>,
Marijn Suijten <marijn.suijten@...ainline.org>,
linux-arm-msm@...r.kernel.org, linux-clk@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 5/7] clk: qcom: hfpll: Add MSM8976 PLL data
On 23.07.2023 18:08, Adam Skladowski wrote:
> Add PLL configuration for MSM8976 SoC, this SoC offers 3 HFPLL.
> Small cluster offers two presets for 652-902Mhz range and 902Mhz-1.47Ghz.
> For simplicity only add second range as smaller frequencies can be obtained
> via apcs divider or safe parent this also saves us
> a hassle of reconfiguring VCO bit and config_val.
> A72 and CCI cluster only use single frequency range with their
> outputs/post_dividers/vco_bits being static.
>
> Signed-off-by: Adam Skladowski <a39.skl@...il.com>
> ---
[...]
> +static const struct hfpll_data msm8976_a72 = {
> + .mode_reg = 0x00,
> + .l_reg = 0x04,
> + .m_reg = 0x08,
> + .n_reg = 0x0c,
> + .user_reg = 0x10,
> + .config_reg = 0x14,
> + .config_val = 0x4e0405d,
> + .status_reg = 0x1c,
> + .lock_bit = 16,
> +
> + .l_val = 0x3e,
> + .user_val = 0x100109,
> + .min_rate = 940800000UL,
> + .max_rate = 1843200000UL,
2016000000?
[...]
> static const struct of_device_id qcom_hfpll_match_table[] = {
> { .compatible = "qcom,hfpll", &hdata },
> + { .compatible = "qcom,msm8976-hfpll-a53", &msm8976_a53 },
> + { .compatible = "qcom,msm8976-hfpll-a72", &msm8976_a72 },
> + { .compatible = "qcom,msm8976-hfpll-cci", &msm8976_cci },
.data = is missing
Konrad
Powered by blists - more mailing lists