[PATCH 0/8] clk: qcom: explicitly include clk-provider.h

[PATCH 0/8] clk: qcom: explicitly include clk-provider.h

[Vinod Koul]

As disucssed in [1] it is a good idea to explicitly include clk-provider.h
for clk providers, so include this in rest of missing qcom clk providers.

[1]: lore.kernel.org/r/20211209082607.06929C004DD@smtp.kernel.org


Vinod Koul (8):
  clk: qcom: gcc-sm8350: explicitly include clk-provider.h
  clk: qcom: gcc-msm8994: explicitly include clk-provider.h
  clk: qcom: gcc-sm6350: explicitly include clk-provider.h
  clk: qcom: lpasscc-sc7280: explicitly include clk-provider.h
  clk: qcom: lpasscc-sdm845: explicitly include clk-provider.h
  clk: qcom: mmcc-apq8084: explicitly include clk-provider.h
  clk: qcom: q6sstop-qcs404: explicitly include clk-provider.h
  clk: qcom: turingcc-qcs404: explicitly include clk-provider.h

 drivers/clk/qcom/gcc-msm8994.c     | 1 +
 drivers/clk/qcom/gcc-sm6350.c      | 1 +
 drivers/clk/qcom/gcc-sm8350.c      | 1 +
 drivers/clk/qcom/lpasscc-sc7280.c  | 1 +
 drivers/clk/qcom/lpasscc-sdm845.c  | 1 +
 drivers/clk/qcom/mmcc-apq8084.c    | 1 +
 drivers/clk/qcom/q6sstop-qcs404.c  | 1 +
 drivers/clk/qcom/turingcc-qcs404.c | 1 +
 8 files changed, 8 insertions(+)

-- 
2.31.1

[PATCH 1/8] clk: qcom: gcc-sm8350: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/gcc-sm8350.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/gcc-sm8350.c b/drivers/clk/qcom/gcc-sm8350.c
index 6d0a9e2d5104..c3731f96c8e6 100644
--- a/drivers/clk/qcom/gcc-sm8350.c
+++ b/drivers/clk/qcom/gcc-sm8350.c
@@ -4,6 +4,7 @@
  * Copyright (c) 2020-2021, Linaro Limited
  */
 
+#include <linux/clk-provider.h>
 #include <linux/module.h>
 #include <linux/platform_device.h>
 #include <linux/regmap.h>
-- 
2.31.1

Re: [PATCH 1/8] clk: qcom: gcc-sm8350: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:37:56)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

[PATCH 2/8] clk: qcom: gcc-msm8994: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/gcc-msm8994.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/gcc-msm8994.c b/drivers/clk/qcom/gcc-msm8994.c
index 702a9bdc0559..71aa630fa4bd 100644
--- a/drivers/clk/qcom/gcc-msm8994.c
+++ b/drivers/clk/qcom/gcc-msm8994.c
@@ -2,6 +2,7 @@
 /* Copyright (c) 2013-2016, The Linux Foundation. All rights reserved.
  */
 
+#include <linux/clk-provider.h>
 #include <linux/kernel.h>
 #include <linux/init.h>
 #include <linux/err.h>
-- 
2.31.1

Re: [PATCH 2/8] clk: qcom: gcc-msm8994: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:37:57)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

[PATCH 3/8] clk: qcom: gcc-sm6350: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/gcc-sm6350.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/gcc-sm6350.c b/drivers/clk/qcom/gcc-sm6350.c
index 3236706771b1..a4f7fba70393 100644
--- a/drivers/clk/qcom/gcc-sm6350.c
+++ b/drivers/clk/qcom/gcc-sm6350.c
@@ -4,6 +4,7 @@
  * Copyright (c) 2021, Konrad Dybcio <konrad.dybcio@somainline.org>
  */
 
+#include <linux/clk-provider.h>
 #include <linux/module.h>
 #include <linux/platform_device.h>
 #include <linux/regmap.h>
-- 
2.31.1

Re: [PATCH 3/8] clk: qcom: gcc-sm6350: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:37:58)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

[PATCH 4/8] clk: qcom: lpasscc-sc7280: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/lpasscc-sc7280.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/lpasscc-sc7280.c b/drivers/clk/qcom/lpasscc-sc7280.c
index 89f1ad6631da..b39ee1c9647b 100644
--- a/drivers/clk/qcom/lpasscc-sc7280.c
+++ b/drivers/clk/qcom/lpasscc-sc7280.c
@@ -3,6 +3,7 @@
  * Copyright (c) 2021, The Linux Foundation. All rights reserved.
  */
 
+#include <linux/clk-provider.h>
 #include <linux/platform_device.h>
 #include <linux/pm_clock.h>
 #include <linux/pm_runtime.h>
-- 
2.31.1

Re: [PATCH 4/8] clk: qcom: lpasscc-sc7280: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:37:59)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

[PATCH 5/8] clk: qcom: lpasscc-sdm845: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/lpasscc-sdm845.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c
index 56d3e9928892..7040da952728 100644
--- a/drivers/clk/qcom/lpasscc-sdm845.c
+++ b/drivers/clk/qcom/lpasscc-sdm845.c
@@ -3,6 +3,7 @@
  * Copyright (c) 2018, The Linux Foundation. All rights reserved.
  */
 
+#include <linux/clk-provider.h>
 #include <linux/platform_device.h>
 #include <linux/module.h>
 #include <linux/of_address.h>
-- 
2.31.1

Re: [PATCH 5/8] clk: qcom: lpasscc-sdm845: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:38:00)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

[PATCH 6/8] clk: qcom: mmcc-apq8084: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/mmcc-apq8084.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/mmcc-apq8084.c b/drivers/clk/qcom/mmcc-apq8084.c
index fbfcf0006739..e9f971359155 100644
--- a/drivers/clk/qcom/mmcc-apq8084.c
+++ b/drivers/clk/qcom/mmcc-apq8084.c
@@ -3,6 +3,7 @@
  * Copyright (c) 2014-2015, The Linux Foundation. All rights reserved.
  */
 
+#include <linux/clk-provider.h>
 #include <linux/kernel.h>
 #include <linux/platform_device.h>
 #include <linux/module.h>
-- 
2.31.1

Re: [PATCH 6/8] clk: qcom: mmcc-apq8084: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:38:01)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

[PATCH 7/8] clk: qcom: q6sstop-qcs404: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/q6sstop-qcs404.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/q6sstop-qcs404.c b/drivers/clk/qcom/q6sstop-qcs404.c
index 507386bee07d..780074e05841 100644
--- a/drivers/clk/qcom/q6sstop-qcs404.c
+++ b/drivers/clk/qcom/q6sstop-qcs404.c
@@ -4,6 +4,7 @@
  */
 
 #include <linux/bitops.h>
+#include <linux/clk-provider.h>
 #include <linux/err.h>
 #include <linux/module.h>
 #include <linux/platform_device.h>
-- 
2.31.1

Re: [PATCH 7/8] clk: qcom: q6sstop-qcs404: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:38:02)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

[PATCH 8/8] clk: qcom: turingcc-qcs404: explicitly include clk-provider.h

[Vinod Koul]

Per Stephen, clk providers need to include clk-provider.h, so include in
this driver as well

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 drivers/clk/qcom/turingcc-qcs404.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/qcom/turingcc-qcs404.c b/drivers/clk/qcom/turingcc-qcs404.c
index 4543bda793f4..43184459228f 100644
--- a/drivers/clk/qcom/turingcc-qcs404.c
+++ b/drivers/clk/qcom/turingcc-qcs404.c
@@ -4,6 +4,7 @@
  */
 
 #include <linux/bitops.h>
+#include <linux/clk-provider.h>
 #include <linux/err.h>
 #include <linux/platform_device.h>
 #include <linux/module.h>
-- 
2.31.1

Re: [PATCH 8/8] clk: qcom: turingcc-qcs404: explicitly include clk-provider.h

[Stephen Boyd]

Quoting Vinod Koul (2021-12-15 03:38:03)
> Per Stephen, clk providers need to include clk-provider.h, so include in
> this driver as well
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@kernel.org>

Re: [PATCH 0/8] clk: qcom: explicitly include clk-provider.h

[Bjorn Andersson]

On Wed, 15 Dec 2021 17:07:55 +0530, Vinod Koul wrote:
> As disucssed in [1] it is a good idea to explicitly include clk-provider.h
> for clk providers, so include this in rest of missing qcom clk providers.
> 
> [1]: lore.kernel.org/r/20211209082607.06929C004DD@smtp.kernel.org
> 
> 
> Vinod Koul (8):
>   clk: qcom: gcc-sm8350: explicitly include clk-provider.h
>   clk: qcom: gcc-msm8994: explicitly include clk-provider.h
>   clk: qcom: gcc-sm6350: explicitly include clk-provider.h
>   clk: qcom: lpasscc-sc7280: explicitly include clk-provider.h
>   clk: qcom: lpasscc-sdm845: explicitly include clk-provider.h
>   clk: qcom: mmcc-apq8084: explicitly include clk-provider.h
>   clk: qcom: q6sstop-qcs404: explicitly include clk-provider.h
>   clk: qcom: turingcc-qcs404: explicitly include clk-provider.h
> 
> [...]

Applied, thanks!

[1/8] clk: qcom: gcc-sm8350: explicitly include clk-provider.h
      commit: 33aa94fd94d7c1ca1af77f398c194bd82ebc6a98
[2/8] clk: qcom: gcc-msm8994: explicitly include clk-provider.h
      commit: d7a49c8d2c67ab504b3959aafda290018d42b0e7
[3/8] clk: qcom: gcc-sm6350: explicitly include clk-provider.h
      commit: 1fc8887c04b28d294b71b802fca29a5fa667d7fa
[4/8] clk: qcom: lpasscc-sc7280: explicitly include clk-provider.h
      commit: 27f239a4c5e79a8e2f721b0ded24e1f34437d7f1
[5/8] clk: qcom: lpasscc-sdm845: explicitly include clk-provider.h
      commit: 3333607bdd4f1a64a13741a252f2eee1bb0a442c
[6/8] clk: qcom: mmcc-apq8084: explicitly include clk-provider.h
      commit: 5bcc2521ec7053b0a99e20bbf8f18d6001a78e6c
[7/8] clk: qcom: q6sstop-qcs404: explicitly include clk-provider.h
      commit: 737a2267581ac145db337081ae6fbf9f62feb47b
[8/8] clk: qcom: turingcc-qcs404: explicitly include clk-provider.h
      commit: 96ea2a4291343613a7c2da03c03f23dc259171ff

Best regards,
-- 
Bjorn Andersson <bjorn.andersson@linaro.org>