On Wed, Nov 02, 2022 at 09:58:49AM +0530, Viresh Kumar wrote: > 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. > Good catch! "static" is not needed here. Thanks, Mani > -- > viresh -- மணிவண்ணன் சதாசிவம்