[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <801e16d1-7c41-dd80-3ed5-7e3e1b578d06@linaro.org>
Date: Tue, 4 Jul 2023 14:56:06 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To: Konrad Dybcio <konrad.dybcio@...aro.org>,
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>,
Philipp Zabel <p.zabel@...gutronix.de>
Cc: 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 2/2] clk: qcom: videocc-sm8350: Add SC8280XP support
On 03/07/2023 21:09, Konrad Dybcio wrote:
> SC8280XP, being a partial derivative of SM8350, shares almost the exact
> same videocc block. Extend the 8350 driver to support the bigger brother.
>
> The only notable changes are higher possible frequencies on some clocks
> and some switcheroo within the XO/sleep registers (probably due to some
> different board crystal configuration).
>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@...aro.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
Minor nits below
> ---
> drivers/clk/qcom/videocc-sm8350.c | 42 ++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 41 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/clk/qcom/videocc-sm8350.c b/drivers/clk/qcom/videocc-sm8350.c
> index b148877fc73d..581ad4440615 100644
> --- a/drivers/clk/qcom/videocc-sm8350.c
> +++ b/drivers/clk/qcom/videocc-sm8350.c
> @@ -41,6 +41,10 @@ static const struct pll_vco lucid_5lpe_vco[] = {
> { 249600000, 1750000000, 0 },
> };
>
> +static const struct pll_vco lucid_5lpe_vco_8280[] = {
Nit: _sc8280xp?
> + { 249600000, 1800000000, 0 },
> +};
> +
> static const struct alpha_pll_config video_pll0_config = {
> .l = 0x25,
> .alpha = 0x8000,
> @@ -159,6 +163,16 @@ static const struct freq_tbl ftbl_video_cc_mvs0_clk_src[] = {
> { }
> };
>
> +static const struct freq_tbl ftbl_video_cc_mvs0_clk_src_8280[] = {
And same here
> + F(720000000, P_VIDEO_PLL0_OUT_MAIN, 1, 0, 0),
> + F(1014000000, P_VIDEO_PLL0_OUT_MAIN, 1, 0, 0),
> + F(1098000000, P_VIDEO_PLL0_OUT_MAIN, 1, 0, 0),
> + F(1332000000, P_VIDEO_PLL0_OUT_MAIN, 1, 0, 0),
> + F(1599000000, P_VIDEO_PLL0_OUT_MAIN, 1, 0, 0),
> + F(1680000000, P_VIDEO_PLL0_OUT_MAIN, 1, 0, 0),
> + { }
> +};
> +
> static struct clk_rcg2 video_cc_mvs0_clk_src = {
> .cmd_rcgr = 0xb94,
> .mnd_width = 0,
> @@ -181,6 +195,15 @@ static const struct freq_tbl ftbl_video_cc_mvs1_clk_src[] = {
> { }
> };
>
> +static const struct freq_tbl ftbl_video_cc_mvs1_clk_src_8280[] = {
> + F(840000000, P_VIDEO_PLL1_OUT_MAIN, 1, 0, 0),
> + F(1098000000, P_VIDEO_PLL1_OUT_MAIN, 1, 0, 0),
> + F(1332000000, P_VIDEO_PLL1_OUT_MAIN, 1, 0, 0),
> + F(1600000000, P_VIDEO_PLL1_OUT_MAIN, 1, 0, 0),
> + F(1800000000, P_VIDEO_PLL1_OUT_MAIN, 1, 0, 0),
> + { }
> +};
> +
> static struct clk_rcg2 video_cc_mvs1_clk_src = {
> .cmd_rcgr = 0xbb4,
> .mnd_width = 0,
> @@ -499,6 +522,7 @@ static struct qcom_cc_desc video_cc_sm8350_desc = {
>
> static int video_cc_sm8350_probe(struct platform_device *pdev)
> {
> + u32 video_cc_xo_clk_cbcr = 0xeec;
> struct regmap *regmap;
> int ret;
>
> @@ -510,6 +534,21 @@ static int video_cc_sm8350_probe(struct platform_device *pdev)
> if (ret)
> return ret;
>
> + if (of_device_is_compatible(pdev->dev.of_node, "qcom,sc8280xp-videocc")) {
> + video_cc_sleep_clk_src.cmd_rcgr = 0xf38;
> + video_cc_sleep_clk.halt_reg = 0xf58;
> + video_cc_sleep_clk.clkr.enable_reg = 0xf58;
> + video_cc_xo_clk_src.cmd_rcgr = 0xf14;
> + video_cc_xo_clk_cbcr = 0xf34;
> +
> + video_pll0.vco_table = video_pll1.vco_table = lucid_5lpe_vco_8280;
> + /* No change, but assign it for completeness */
> + video_pll0.num_vco = video_pll1.num_vco = ARRAY_SIZE(lucid_5lpe_vco_8280);
> +
> + video_cc_mvs0_clk_src.freq_tbl = ftbl_video_cc_mvs0_clk_src_8280;
> + video_cc_mvs1_clk_src.freq_tbl = ftbl_video_cc_mvs1_clk_src_8280;
> + }
> +
> regmap = qcom_cc_map(pdev, &video_cc_sm8350_desc);
> if (IS_ERR(regmap)) {
> pm_runtime_put(&pdev->dev);
> @@ -525,7 +564,7 @@ static int video_cc_sm8350_probe(struct platform_device *pdev)
> * video_cc_xo_clk
> */
> regmap_update_bits(regmap, 0xe58, BIT(0), BIT(0));
> - regmap_update_bits(regmap, 0xeec, BIT(0), BIT(0));
> + regmap_update_bits(regmap, video_cc_xo_clk_cbcr, BIT(0), BIT(0));
>
> ret = qcom_cc_really_probe(pdev, &video_cc_sm8350_desc, regmap);
> pm_runtime_put(&pdev->dev);
> @@ -534,6 +573,7 @@ static int video_cc_sm8350_probe(struct platform_device *pdev)
> }
>
> static const struct of_device_id video_cc_sm8350_match_table[] = {
> + { .compatible = "qcom,sc8280xp-videocc" },
> { .compatible = "qcom,sm8350-videocc" },
> { }
> };
>
--
With best wishes
Dmitry
Powered by blists - more mailing lists