Re: [PATCH] clk: qcom: stop enforcing INTERCONNECT_CLK dependency for everybody

[Konrad Dybcio]

On 5/17/26 8:51 PM, Dmitry Baryshkov wrote:
> Only several (typically IPQ) platforms use INTERCONNECT_CLK to turn
> clocks into the interconnect providers. Stop inferring this dependency
> on everybody and set it only when required. As a safety measure, make
> sure that the coresponding clock description fields are not available if
> corresponding Kconfig option isn't enabled.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> ---

[...]

> @@ -395,6 +399,7 @@ config IPQ_GCC_5210
>  config IPQ_GCC_5332
>  	tristate "IPQ5332 Global Clock Controller"
>  	depends on ARM64 || COMPILE_TEST
> +	select QCOM_NEED_INTERCONNECT_CLK
>  	help
>  	  Support for the global clock controller on ipq5332 devices.
>  	  Say Y if you want to use peripheral devices such as UART, SPI,

There's a number of entries where there's spaces instead of tabs,
Krzysztof fixed that up for interconnect earlier today

[...]

> +#if IS_REACHABLE(CONFIG_INTERCONNECT_CLK)
>  	const struct qcom_icc_hws_data *icc_hws;
>  	size_t num_icc_hws;
> +#endif
>  	unsigned int icc_first_node_id;

this uint is also only present in drivers that use icc-clk

Konrad