Re: [PATCH v2 2/2] clk: qcom: apss-ipq-pll: Add IPQ5210 support

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Missing 'const' qualifier on 'ipq5210_pll_data' structure.
--

commit 82121884055bac497befa1efc2a14f014cd9e6e1
Author: Varadarajan Narayanan <varadarajan.narayanan@oss.qualcomm.com>

clk: qcom: apss-ipq-pll: Add IPQ5210 support

The Application Processor Subsystem on the IPQ5210 platform sources
its clock from the Huayra PLL. Add the configuration data necessary
to set it up.

> diff --git a/drivers/clk/qcom/apss-ipq-pll.c b/drivers/clk/qcom/apss-ipq-pll.c
> index 3a8987fe7008a..569ac76c75dc3 100644
> --- a/drivers/clk/qcom/apss-ipq-pll.c
> +++ b/drivers/clk/qcom/apss-ipq-pll.c

[ ... ]

> @@ -140,6 +152,12 @@ static const struct apss_pll_data ipq5018_pll_data = {
>  	.pll_config = &ipq5018_pll_config,
>  };
>  
> +static struct apss_pll_data ipq5210_pll_data = {

[Severity: Low]
This isn't a bug, but should this structure be marked as const to match
the other apss_pll_data structs in this file? Without the const qualifier,
the structure gets placed in the mutable data section rather than rodata.

> +	.pll_type = CLK_ALPHA_PLL_TYPE_HUAYRA,
> +	.pll = &ipq_pll_huayra,
> +	.pll_config = &ipq5210_pll_config,
> +};
> +
>  static const struct apss_pll_data ipq5332_pll_data = {
>  	.pll_type = CLK_ALPHA_PLL_TYPE_STROMER_PLUS,
>  	.pll = &ipq_pll_stromer_plus,

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260701-apss-clk-v2-0-776d054ab8c3@oss.qualcomm.com?part=2