drivers/clk/qcom/videocc-sm8750.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
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
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
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
© 2016 - 2025 Red Hat, Inc.