[PATCH] clk: qcom: milos: Constify 'struct qcom_cc_desc'

Krzysztof Kozlowski posted 1 patch 1 month, 2 weeks ago
drivers/clk/qcom/camcc-milos.c   | 2 +-
drivers/clk/qcom/dispcc-milos.c  | 2 +-
drivers/clk/qcom/videocc-milos.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
[PATCH] clk: qcom: milos: Constify 'struct qcom_cc_desc'
Posted by Krzysztof Kozlowski 1 month, 2 weeks ago
'struct qcom_cc_desc' is passed to qcom_cc_map() and
qcom_cc_really_probe() only as pointer to const, so make the memory
const for safety.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

My standard commit... I just keep repeating the same over and over
again.
---
 drivers/clk/qcom/camcc-milos.c   | 2 +-
 drivers/clk/qcom/dispcc-milos.c  | 2 +-
 drivers/clk/qcom/videocc-milos.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/clk/qcom/camcc-milos.c b/drivers/clk/qcom/camcc-milos.c
index 75bd939f7dd1..0077c9c9249f 100644
--- a/drivers/clk/qcom/camcc-milos.c
+++ b/drivers/clk/qcom/camcc-milos.c
@@ -2124,7 +2124,7 @@ static struct qcom_cc_driver_data cam_cc_milos_driver_data = {
 	.num_clk_cbcrs = ARRAY_SIZE(cam_cc_milos_critical_cbcrs),
 };
 
-static struct qcom_cc_desc cam_cc_milos_desc = {
+static const struct qcom_cc_desc cam_cc_milos_desc = {
 	.config = &cam_cc_milos_regmap_config,
 	.clks = cam_cc_milos_clocks,
 	.num_clks = ARRAY_SIZE(cam_cc_milos_clocks),
diff --git a/drivers/clk/qcom/dispcc-milos.c b/drivers/clk/qcom/dispcc-milos.c
index 602d3a498d33..95b6dd89d9ae 100644
--- a/drivers/clk/qcom/dispcc-milos.c
+++ b/drivers/clk/qcom/dispcc-milos.c
@@ -937,7 +937,7 @@ static struct qcom_cc_driver_data disp_cc_milos_driver_data = {
 	.clk_regs_configure = disp_cc_milos_clk_regs_configure,
 };
 
-static struct qcom_cc_desc disp_cc_milos_desc = {
+static const struct qcom_cc_desc disp_cc_milos_desc = {
 	.config = &disp_cc_milos_regmap_config,
 	.clks = disp_cc_milos_clocks,
 	.num_clks = ARRAY_SIZE(disp_cc_milos_clocks),
diff --git a/drivers/clk/qcom/videocc-milos.c b/drivers/clk/qcom/videocc-milos.c
index 998301e0ba88..acc9df295d4f 100644
--- a/drivers/clk/qcom/videocc-milos.c
+++ b/drivers/clk/qcom/videocc-milos.c
@@ -366,7 +366,7 @@ static struct qcom_cc_driver_data video_cc_milos_driver_data = {
 	.num_clk_cbcrs = ARRAY_SIZE(video_cc_milos_critical_cbcrs),
 };
 
-static struct qcom_cc_desc video_cc_milos_desc = {
+static const struct qcom_cc_desc video_cc_milos_desc = {
 	.config = &video_cc_milos_regmap_config,
 	.clks = video_cc_milos_clocks,
 	.num_clks = ARRAY_SIZE(video_cc_milos_clocks),
-- 
2.48.1
Re: [PATCH] clk: qcom: milos: Constify 'struct qcom_cc_desc'
Posted by Bjorn Andersson 4 weeks, 1 day ago
On Wed, 20 Aug 2025 14:48:22 +0200, Krzysztof Kozlowski wrote:
> 'struct qcom_cc_desc' is passed to qcom_cc_map() and
> qcom_cc_really_probe() only as pointer to const, so make the memory
> const for safety.
> 
> 

Applied, thanks!

[1/1] clk: qcom: milos: Constify 'struct qcom_cc_desc'
      commit: d923b9682e12bb63b8a1353110e67869a9bd0ba8

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>
Re: [PATCH] clk: qcom: milos: Constify 'struct qcom_cc_desc'
Posted by Dmitry Baryshkov 1 month, 1 week ago
On Wed, Aug 20, 2025 at 02:48:22PM +0200, Krzysztof Kozlowski wrote:
> 'struct qcom_cc_desc' is passed to qcom_cc_map() and
> qcom_cc_really_probe() only as pointer to const, so make the memory
> const for safety.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> My standard commit... I just keep repeating the same over and over
> again.
> ---
>  drivers/clk/qcom/camcc-milos.c   | 2 +-
>  drivers/clk/qcom/dispcc-milos.c  | 2 +-
>  drivers/clk/qcom/videocc-milos.c | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>


-- 
With best wishes
Dmitry