[PATCH] clk: qcom: videocc-sm8750: Constify qcom_cc_desc

Krzysztof Kozlowski posted 1 patch 1 week, 4 days ago
drivers/clk/qcom/videocc-sm8750.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] clk: qcom: videocc-sm8750: Constify qcom_cc_desc
Posted by Krzysztof Kozlowski 1 week, 4 days ago
Static 'struct qcom_cc_desc' is not modified by drivers and can be made
const for code safety.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

---

My standard commit, one more time. I wonder when I will run out of the
same commits doing the same...
---
 drivers/clk/qcom/videocc-sm8750.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/qcom/videocc-sm8750.c b/drivers/clk/qcom/videocc-sm8750.c
index 0acf3104d702..823aca2bdd34 100644
--- a/drivers/clk/qcom/videocc-sm8750.c
+++ b/drivers/clk/qcom/videocc-sm8750.c
@@ -416,7 +416,7 @@ static struct qcom_cc_driver_data video_cc_sm8750_driver_data = {
 	.clk_regs_configure = clk_sm8750_regs_configure,
 };
 
-static struct qcom_cc_desc video_cc_sm8750_desc = {
+static const struct qcom_cc_desc video_cc_sm8750_desc = {
 	.config = &video_cc_sm8750_regmap_config,
 	.clks = video_cc_sm8750_clocks,
 	.num_clks = ARRAY_SIZE(video_cc_sm8750_clocks),
-- 
2.51.0
Re: [PATCH] clk: qcom: videocc-sm8750: Constify qcom_cc_desc
Posted by Taniya Das 1 week, 4 days ago

On 12/8/2025 7:36 AM, Krzysztof Kozlowski wrote:
> Static 'struct qcom_cc_desc' is not modified by drivers and can be made
> const for code safety.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> 
> ---
> 
> My standard commit, one more time. I wonder when I will run out of the
> same commits doing the same...
> ---
>  drivers/clk/qcom/videocc-sm8750.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/qcom/videocc-sm8750.c b/drivers/clk/qcom/videocc-sm8750.c
> index 0acf3104d702..823aca2bdd34 100644
> --- a/drivers/clk/qcom/videocc-sm8750.c
> +++ b/drivers/clk/qcom/videocc-sm8750.c
> @@ -416,7 +416,7 @@ static struct qcom_cc_driver_data video_cc_sm8750_driver_data = {
>  	.clk_regs_configure = clk_sm8750_regs_configure,
>  };
>  
> -static struct qcom_cc_desc video_cc_sm8750_desc = {
> +static const struct qcom_cc_desc video_cc_sm8750_desc = {
>  	.config = &video_cc_sm8750_regmap_config,
>  	.clks = video_cc_sm8750_clocks,
>  	.num_clks = ARRAY_SIZE(video_cc_sm8750_clocks),

Reviewed-by: Taniya Das <taniya.das@oss.qualcomm.com>

-- 
Thanks,
Taniya Das
Re: [PATCH] clk: qcom: videocc-sm8750: Constify qcom_cc_desc
Posted by Imran Shaik 1 week, 4 days ago

On 08-12-2025 07:36, Krzysztof Kozlowski wrote:
> Static 'struct qcom_cc_desc' is not modified by drivers and can be made
> const for code safety.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> 
> ---
> 
> My standard commit, one more time. I wonder when I will run out of the
> same commits doing the same...
> ---
>   drivers/clk/qcom/videocc-sm8750.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> 

Reviewed-by: Imran Shaik <imran.shaik@oss.qualcomm.com>

Thanks,
Imran