linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/4] soc: qcom: Add support for MSM8226 RPM
@ 2021-05-02 12:20 Bartosz Dudziak
  2021-05-02 12:20 ` [PATCH 1/4] dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible Bartosz Dudziak
                   ` (3 more replies)
  0 siblings, 4 replies; 10+ messages in thread
From: Bartosz Dudziak @ 2021-05-02 12:20 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Michael Turquette, Stephen Boyd,
	Rob Herring, Kathiravan T, linux-arm-msm, linux-clk, devicetree,
	linux-kernel
  Cc: Bartosz Dudziak

Add support for Resource Power Manager (RPM) and RPM Clock Controller
(RPMCC) in MSM8226 SoC.

Bartosz Dudziak (4):
  dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible
  soc: qcom: smd-rpm: Add MSM8226 compatible
  dt-bindings: clock: qcom: rpmcc: Document MSM8226 compatible
  clk: qcom: smd-rpmcc: Add support for MSM8226 rpm clocks

 .../devicetree/bindings/clock/qcom,rpmcc.txt  |  1 +
 .../bindings/soc/qcom/qcom,smd-rpm.yaml       |  1 +
 drivers/clk/qcom/clk-smd-rpm.c                | 74 +++++++++++++++++++
 drivers/soc/qcom/smd-rpm.c                    |  1 +
 4 files changed, 77 insertions(+)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH 1/4] dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible
  2021-05-02 12:20 [PATCH 0/4] soc: qcom: Add support for MSM8226 RPM Bartosz Dudziak
@ 2021-05-02 12:20 ` Bartosz Dudziak
  2021-05-06 15:21   ` Rob Herring
  2021-05-02 12:20 ` [PATCH 2/4] " Bartosz Dudziak
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 10+ messages in thread
From: Bartosz Dudziak @ 2021-05-02 12:20 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Michael Turquette, Stephen Boyd,
	Rob Herring, Kathiravan T, linux-arm-msm, linux-clk, devicetree,
	linux-kernel
  Cc: Bartosz Dudziak

Add the dt-binding for the rpm on the Qualcomm MSM8226 SoC platform.

Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
---
 Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm.yaml
index 2684f22a1d..e6e4cb3d0d 100644
--- a/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm.yaml
+++ b/Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm.yaml
@@ -32,6 +32,7 @@ properties:
     enum:
       - qcom,rpm-apq8084
       - qcom,rpm-ipq6018
+      - qcom,rpm-msm8226
       - qcom,rpm-msm8916
       - qcom,rpm-msm8974
       - qcom,rpm-msm8976
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 2/4] soc: qcom: smd-rpm: Add MSM8226 compatible
  2021-05-02 12:20 [PATCH 0/4] soc: qcom: Add support for MSM8226 RPM Bartosz Dudziak
  2021-05-02 12:20 ` [PATCH 1/4] dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible Bartosz Dudziak
@ 2021-05-02 12:20 ` Bartosz Dudziak
  2021-05-02 12:20 ` [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document " Bartosz Dudziak
  2021-05-02 12:20 ` [PATCH 4/4] clk: qcom: smd-rpm: Add support for MSM8226 RPM clocks Bartosz Dudziak
  3 siblings, 0 replies; 10+ messages in thread
From: Bartosz Dudziak @ 2021-05-02 12:20 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Michael Turquette, Stephen Boyd,
	Rob Herring, Kathiravan T, linux-arm-msm, linux-clk, devicetree,
	linux-kernel
  Cc: Bartosz Dudziak

Add a compatible for the RPM on the Qualcomm MSM8226 platform.

Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
---
 drivers/soc/qcom/smd-rpm.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/soc/qcom/smd-rpm.c b/drivers/soc/qcom/smd-rpm.c
index b93218cb50..abd4178ec1 100644
--- a/drivers/soc/qcom/smd-rpm.c
+++ b/drivers/soc/qcom/smd-rpm.c
@@ -233,6 +233,7 @@ static void qcom_smd_rpm_remove(struct rpmsg_device *rpdev)
 static const struct of_device_id qcom_smd_rpm_of_match[] = {
 	{ .compatible = "qcom,rpm-apq8084" },
 	{ .compatible = "qcom,rpm-ipq6018" },
+	{ .compatible = "qcom,rpm-msm8226" },
 	{ .compatible = "qcom,rpm-msm8916" },
 	{ .compatible = "qcom,rpm-msm8936" },
 	{ .compatible = "qcom,rpm-msm8974" },
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document MSM8226 compatible
  2021-05-02 12:20 [PATCH 0/4] soc: qcom: Add support for MSM8226 RPM Bartosz Dudziak
  2021-05-02 12:20 ` [PATCH 1/4] dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible Bartosz Dudziak
  2021-05-02 12:20 ` [PATCH 2/4] " Bartosz Dudziak
@ 2021-05-02 12:20 ` Bartosz Dudziak
  2021-05-06 15:21   ` Rob Herring
  2021-06-02  7:39   ` Stephen Boyd
  2021-05-02 12:20 ` [PATCH 4/4] clk: qcom: smd-rpm: Add support for MSM8226 RPM clocks Bartosz Dudziak
  3 siblings, 2 replies; 10+ messages in thread
From: Bartosz Dudziak @ 2021-05-02 12:20 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Michael Turquette, Stephen Boyd,
	Rob Herring, Kathiravan T, linux-arm-msm, linux-clk, devicetree,
	linux-kernel
  Cc: Bartosz Dudziak

Add the dt-binding for the RPM Clock Controller on the MSM8226 SoC.

Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
---
 Documentation/devicetree/bindings/clock/qcom,rpmcc.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt b/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt
index b44a0622fb..6cf5a7ec2b 100644
--- a/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt
+++ b/Documentation/devicetree/bindings/clock/qcom,rpmcc.txt
@@ -12,6 +12,7 @@ Required properties :
 
 			"qcom,rpmcc-msm8660", "qcom,rpmcc"
 			"qcom,rpmcc-apq8060", "qcom,rpmcc"
+			"qcom,rpmcc-msm8226", "qcom,rpmcc"
 			"qcom,rpmcc-msm8916", "qcom,rpmcc"
 			"qcom,rpmcc-msm8936", "qcom,rpmcc"
 			"qcom,rpmcc-msm8974", "qcom,rpmcc"
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH 4/4] clk: qcom: smd-rpm: Add support for MSM8226 RPM clocks
  2021-05-02 12:20 [PATCH 0/4] soc: qcom: Add support for MSM8226 RPM Bartosz Dudziak
                   ` (2 preceding siblings ...)
  2021-05-02 12:20 ` [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document " Bartosz Dudziak
@ 2021-05-02 12:20 ` Bartosz Dudziak
  2021-06-02  7:40   ` Stephen Boyd
  3 siblings, 1 reply; 10+ messages in thread
From: Bartosz Dudziak @ 2021-05-02 12:20 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Michael Turquette, Stephen Boyd,
	Rob Herring, Kathiravan T, linux-arm-msm, linux-clk, devicetree,
	linux-kernel
  Cc: Bartosz Dudziak

Add rmp smd clocks, PMIC and bus clocks which are required on MSM8226
for clients to vote on.

Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
---
 drivers/clk/qcom/clk-smd-rpm.c | 74 ++++++++++++++++++++++++++++++++++
 1 file changed, 74 insertions(+)

diff --git a/drivers/clk/qcom/clk-smd-rpm.c b/drivers/clk/qcom/clk-smd-rpm.c
index 0e1dfa8948..bf73942e86 100644
--- a/drivers/clk/qcom/clk-smd-rpm.c
+++ b/drivers/clk/qcom/clk-smd-rpm.c
@@ -406,6 +406,79 @@ static const struct clk_ops clk_smd_rpm_branch_ops = {
 	.unprepare	= clk_smd_rpm_unprepare,
 };
 
+/* msm8226 */
+DEFINE_CLK_SMD_RPM(msm8226, pnoc_clk, pnoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 0);
+DEFINE_CLK_SMD_RPM(msm8226, snoc_clk, snoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 1);
+DEFINE_CLK_SMD_RPM(msm8226, cnoc_clk, cnoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 2);
+DEFINE_CLK_SMD_RPM(msm8226, mmssnoc_ahb_clk, mmssnoc_ahb_a_clk, QCOM_SMD_RPM_BUS_CLK, 3);
+DEFINE_CLK_SMD_RPM(msm8226, bimc_clk, bimc_a_clk, QCOM_SMD_RPM_MEM_CLK, 0);
+DEFINE_CLK_SMD_RPM(msm8226, gfx3d_clk_src, gfx3d_a_clk_src, QCOM_SMD_RPM_MEM_CLK, 1);
+DEFINE_CLK_SMD_RPM(msm8226, ocmemgx_clk, ocmemgx_a_clk, QCOM_SMD_RPM_MEM_CLK, 2);
+DEFINE_CLK_SMD_RPM_QDSS(msm8226, qdss_clk, qdss_a_clk, QCOM_SMD_RPM_MISC_CLK, 1);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, cxo_d0, cxo_d0_a, 1);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, cxo_d1, cxo_d1_a, 2);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, cxo_a0, cxo_a0_a, 4);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, cxo_a1, cxo_a1_a, 5);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, cxo_a2, cxo_a2_a, 6);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, diff_clk, diff_a_clk, 7);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, div_clk1, div_a_clk1, 11);
+DEFINE_CLK_SMD_RPM_XO_BUFFER(msm8226, div_clk2, div_a_clk2, 12);
+DEFINE_CLK_SMD_RPM_XO_BUFFER_PINCTRL(msm8226, cxo_d0_pin, cxo_d0_a_pin, 1);
+DEFINE_CLK_SMD_RPM_XO_BUFFER_PINCTRL(msm8226, cxo_d1_pin, cxo_d1_a_pin, 2);
+DEFINE_CLK_SMD_RPM_XO_BUFFER_PINCTRL(msm8226, cxo_a0_pin, cxo_a0_a_pin, 4);
+DEFINE_CLK_SMD_RPM_XO_BUFFER_PINCTRL(msm8226, cxo_a1_pin, cxo_a1_a_pin, 5);
+DEFINE_CLK_SMD_RPM_XO_BUFFER_PINCTRL(msm8226, cxo_a2_pin, cxo_a2_a_pin, 6);
+
+static struct clk_smd_rpm *msm8226_clks[] = {
+	[RPM_SMD_PNOC_CLK]		= &msm8226_pnoc_clk,
+	[RPM_SMD_PNOC_A_CLK]		= &msm8226_pnoc_a_clk,
+	[RPM_SMD_SNOC_CLK]		= &msm8226_snoc_clk,
+	[RPM_SMD_SNOC_A_CLK]		= &msm8226_snoc_a_clk,
+	[RPM_SMD_CNOC_CLK]		= &msm8226_cnoc_clk,
+	[RPM_SMD_CNOC_A_CLK]		= &msm8226_cnoc_a_clk,
+	[RPM_SMD_MMSSNOC_AHB_CLK]	= &msm8226_mmssnoc_ahb_clk,
+	[RPM_SMD_MMSSNOC_AHB_A_CLK]	= &msm8226_mmssnoc_ahb_a_clk,
+	[RPM_SMD_BIMC_CLK]		= &msm8226_bimc_clk,
+	[RPM_SMD_GFX3D_CLK_SRC]		= &msm8226_gfx3d_clk_src,
+	[RPM_SMD_GFX3D_A_CLK_SRC]	= &msm8226_gfx3d_a_clk_src,
+	[RPM_SMD_BIMC_A_CLK]		= &msm8226_bimc_a_clk,
+	[RPM_SMD_OCMEMGX_CLK]		= &msm8226_ocmemgx_clk,
+	[RPM_SMD_OCMEMGX_A_CLK]		= &msm8226_ocmemgx_a_clk,
+	[RPM_SMD_QDSS_CLK]		= &msm8226_qdss_clk,
+	[RPM_SMD_QDSS_A_CLK]		= &msm8226_qdss_a_clk,
+	[RPM_SMD_CXO_D0]		= &msm8226_cxo_d0,
+	[RPM_SMD_CXO_D0_A]		= &msm8226_cxo_d0_a,
+	[RPM_SMD_CXO_D1]		= &msm8226_cxo_d1,
+	[RPM_SMD_CXO_D1_A]		= &msm8226_cxo_d1_a,
+	[RPM_SMD_CXO_A0]		= &msm8226_cxo_a0,
+	[RPM_SMD_CXO_A0_A]		= &msm8226_cxo_a0_a,
+	[RPM_SMD_CXO_A1]		= &msm8226_cxo_a1,
+	[RPM_SMD_CXO_A1_A]		= &msm8226_cxo_a1_a,
+	[RPM_SMD_CXO_A2]		= &msm8226_cxo_a2,
+	[RPM_SMD_CXO_A2_A]		= &msm8226_cxo_a2_a,
+	[RPM_SMD_DIFF_CLK]		= &msm8226_diff_clk,
+	[RPM_SMD_DIFF_A_CLK]		= &msm8226_diff_a_clk,
+	[RPM_SMD_DIV_CLK1]		= &msm8226_div_clk1,
+	[RPM_SMD_DIV_A_CLK1]		= &msm8226_div_a_clk1,
+	[RPM_SMD_DIV_CLK2]		= &msm8226_div_clk2,
+	[RPM_SMD_DIV_A_CLK2]		= &msm8226_div_a_clk2,
+	[RPM_SMD_CXO_D0_PIN]		= &msm8226_cxo_d0_pin,
+	[RPM_SMD_CXO_D0_A_PIN]		= &msm8226_cxo_d0_a_pin,
+	[RPM_SMD_CXO_D1_PIN]		= &msm8226_cxo_d1_pin,
+	[RPM_SMD_CXO_D1_A_PIN]		= &msm8226_cxo_d1_a_pin,
+	[RPM_SMD_CXO_A0_PIN]		= &msm8226_cxo_a0_pin,
+	[RPM_SMD_CXO_A0_A_PIN]		= &msm8226_cxo_a0_a_pin,
+	[RPM_SMD_CXO_A1_PIN]		= &msm8226_cxo_a1_pin,
+	[RPM_SMD_CXO_A1_A_PIN]		= &msm8226_cxo_a1_a_pin,
+	[RPM_SMD_CXO_A2_PIN]		= &msm8226_cxo_a2_pin,
+	[RPM_SMD_CXO_A2_A_PIN]		= &msm8226_cxo_a2_a_pin,
+};
+
+static const struct rpm_smd_clk_desc rpm_clk_msm8226 = {
+	.clks = msm8226_clks,
+	.num_clks = ARRAY_SIZE(msm8226_clks),
+};
+
 /* msm8916 */
 DEFINE_CLK_SMD_RPM(msm8916, pcnoc_clk, pcnoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 0);
 DEFINE_CLK_SMD_RPM(msm8916, snoc_clk, snoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 1);
@@ -1060,6 +1133,7 @@ static const struct rpm_smd_clk_desc rpm_clk_sdm660 = {
 };
 
 static const struct of_device_id rpm_smd_clk_match_table[] = {
+	{ .compatible = "qcom,rpmcc-msm8226", .data = &rpm_clk_msm8226 },
 	{ .compatible = "qcom,rpmcc-msm8916", .data = &rpm_clk_msm8916 },
 	{ .compatible = "qcom,rpmcc-msm8936", .data = &rpm_clk_msm8936 },
 	{ .compatible = "qcom,rpmcc-msm8974", .data = &rpm_clk_msm8974 },
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH 1/4] dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible
  2021-05-02 12:20 ` [PATCH 1/4] dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible Bartosz Dudziak
@ 2021-05-06 15:21   ` Rob Herring
  0 siblings, 0 replies; 10+ messages in thread
From: Rob Herring @ 2021-05-06 15:21 UTC (permalink / raw)
  To: Bartosz Dudziak
  Cc: linux-clk, Rob Herring, Kathiravan T, linux-arm-msm,
	Stephen Boyd, linux-kernel, Andy Gross, Michael Turquette,
	Bjorn Andersson, devicetree

On Sun, 02 May 2021 14:20:24 +0200, Bartosz Dudziak wrote:
> Add the dt-binding for the rpm on the Qualcomm MSM8226 SoC platform.
> 
> Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
> ---
>  Documentation/devicetree/bindings/soc/qcom/qcom,smd-rpm.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document MSM8226 compatible
  2021-05-02 12:20 ` [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document " Bartosz Dudziak
@ 2021-05-06 15:21   ` Rob Herring
  2021-06-02  7:39   ` Stephen Boyd
  1 sibling, 0 replies; 10+ messages in thread
From: Rob Herring @ 2021-05-06 15:21 UTC (permalink / raw)
  To: Bartosz Dudziak
  Cc: Rob Herring, linux-clk, Bjorn Andersson, Kathiravan T,
	linux-arm-msm, devicetree, Stephen Boyd, linux-kernel,
	Andy Gross, Michael Turquette

On Sun, 02 May 2021 14:20:26 +0200, Bartosz Dudziak wrote:
> Add the dt-binding for the RPM Clock Controller on the MSM8226 SoC.
> 
> Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
> ---
>  Documentation/devicetree/bindings/clock/qcom,rpmcc.txt | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document MSM8226 compatible
  2021-05-02 12:20 ` [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document " Bartosz Dudziak
  2021-05-06 15:21   ` Rob Herring
@ 2021-06-02  7:39   ` Stephen Boyd
  1 sibling, 0 replies; 10+ messages in thread
From: Stephen Boyd @ 2021-06-02  7:39 UTC (permalink / raw)
  To: Andy Gross, Bartosz Dudziak, Bjorn Andersson, Kathiravan T,
	Michael Turquette, Rob Herring, devicetree, linux-arm-msm,
	linux-clk, linux-kernel
  Cc: Bartosz Dudziak

Quoting Bartosz Dudziak (2021-05-02 05:20:26)
> Add the dt-binding for the RPM Clock Controller on the MSM8226 SoC.
> 
> Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
> ---

Applied to clk-next

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 4/4] clk: qcom: smd-rpm: Add support for MSM8226 RPM clocks
  2021-05-02 12:20 ` [PATCH 4/4] clk: qcom: smd-rpm: Add support for MSM8226 RPM clocks Bartosz Dudziak
@ 2021-06-02  7:40   ` Stephen Boyd
  2021-06-05 10:57     ` Bartosz Dudziak
  0 siblings, 1 reply; 10+ messages in thread
From: Stephen Boyd @ 2021-06-02  7:40 UTC (permalink / raw)
  To: Andy Gross, Bartosz Dudziak, Bjorn Andersson, Kathiravan T,
	Michael Turquette, Rob Herring, devicetree, linux-arm-msm,
	linux-clk, linux-kernel
  Cc: Bartosz Dudziak

Quoting Bartosz Dudziak (2021-05-02 05:20:27)
> Add rmp smd clocks, PMIC and bus clocks which are required on MSM8226
> for clients to vote on.
> 
> Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
> ---
>  drivers/clk/qcom/clk-smd-rpm.c | 74 ++++++++++++++++++++++++++++++++++
>  1 file changed, 74 insertions(+)
> 
> diff --git a/drivers/clk/qcom/clk-smd-rpm.c b/drivers/clk/qcom/clk-smd-rpm.c
> index 0e1dfa8948..bf73942e86 100644
> --- a/drivers/clk/qcom/clk-smd-rpm.c
> +++ b/drivers/clk/qcom/clk-smd-rpm.c
> @@ -406,6 +406,79 @@ static const struct clk_ops clk_smd_rpm_branch_ops = {
>         .unprepare      = clk_smd_rpm_unprepare,
>  };
>  
> +/* msm8226 */

This doesn't apply anymore. Can you resend just this one patch please?
And base it on clk-next which I will push out shortly. Ideally a bunch
of clks can be reused instead of redefined.

> +DEFINE_CLK_SMD_RPM(msm8226, pnoc_clk, pnoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 0);
> +DEFINE_CLK_SMD_RPM(msm8226, snoc_clk, snoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 1);

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 4/4] clk: qcom: smd-rpm: Add support for MSM8226 RPM clocks
  2021-06-02  7:40   ` Stephen Boyd
@ 2021-06-05 10:57     ` Bartosz Dudziak
  0 siblings, 0 replies; 10+ messages in thread
From: Bartosz Dudziak @ 2021-06-05 10:57 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Andy Gross, Bjorn Andersson, Kathiravan T, Michael Turquette,
	Rob Herring, devicetree, linux-arm-msm, linux-clk, linux-kernel

On Wed, Jun 02, 2021 at 12:40:43AM -0700, Stephen Boyd wrote:
> Quoting Bartosz Dudziak (2021-05-02 05:20:27)
> > Add rmp smd clocks, PMIC and bus clocks which are required on MSM8226
> > for clients to vote on.
> > 
> > Signed-off-by: Bartosz Dudziak <bartosz.dudziak@snejp.pl>
> > ---
> >  drivers/clk/qcom/clk-smd-rpm.c | 74 ++++++++++++++++++++++++++++++++++
> >  1 file changed, 74 insertions(+)
> > 
> > diff --git a/drivers/clk/qcom/clk-smd-rpm.c b/drivers/clk/qcom/clk-smd-rpm.c
> > index 0e1dfa8948..bf73942e86 100644
> > --- a/drivers/clk/qcom/clk-smd-rpm.c
> > +++ b/drivers/clk/qcom/clk-smd-rpm.c
> > @@ -406,6 +406,79 @@ static const struct clk_ops clk_smd_rpm_branch_ops = {
> >         .unprepare      = clk_smd_rpm_unprepare,
> >  };
> >  
> > +/* msm8226 */
> 
> This doesn't apply anymore. Can you resend just this one patch please?
> And base it on clk-next which I will push out shortly. Ideally a bunch
> of clks can be reused instead of redefined.
> 
> > +DEFINE_CLK_SMD_RPM(msm8226, pnoc_clk, pnoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 0);
> > +DEFINE_CLK_SMD_RPM(msm8226, snoc_clk, snoc_a_clk, QCOM_SMD_RPM_BUS_CLK, 1);

I have send v2 of this one patch. I have reused the clocks from MSM8974 because
after the change my code happened to be the exacly same as msm8974_clks struct.


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2021-06-05 10:58 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-02 12:20 [PATCH 0/4] soc: qcom: Add support for MSM8226 RPM Bartosz Dudziak
2021-05-02 12:20 ` [PATCH 1/4] dt-bindings: soc: qcom: smd-rpm: Add MSM8226 compatible Bartosz Dudziak
2021-05-06 15:21   ` Rob Herring
2021-05-02 12:20 ` [PATCH 2/4] " Bartosz Dudziak
2021-05-02 12:20 ` [PATCH 3/4] dt-bindings: clock: qcom: rpmcc: Document " Bartosz Dudziak
2021-05-06 15:21   ` Rob Herring
2021-06-02  7:39   ` Stephen Boyd
2021-05-02 12:20 ` [PATCH 4/4] clk: qcom: smd-rpm: Add support for MSM8226 RPM clocks Bartosz Dudziak
2021-06-02  7:40   ` Stephen Boyd
2021-06-05 10:57     ` Bartosz Dudziak

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).