On 25-10-22, 13:02, Manivannan Sadhasivam wrote: > for (i = 0; i < num_domains; i++) { > struct qcom_cpufreq_data *data = &qcom_cpufreq.data[i]; > + static struct clk_init_data init = {}; Why not make this part of a driver level structure instead of static ? And what will {} do ? Initialize to 0 ? static would have already done that. -- viresh