On Tue, Mar 20, 2012 at 11:40:21PM +0100, Sascha Hauer wrote: > On Tue, Mar 20, 2012 at 01:14:51PM -0700, Saravana Kannan wrote: ... > > Ah, I see. That's a lot of functions calls. I think it would be much > > more efficient if you just have an array and loop over it. With my > > patch, you can just call a single register function for all these > > clocks. > > I was curious and gave it a try. I registered a fixed clock and 100 > gates as child clocks. I tried both DEFINE_CLK_GATE and > clk_register_gate. It turned out there was no difference in speed. > While there is no difference in speed efficiency, I think we will gain space efficiency if we can have Saravana's patch to copy the clk_hw fields and make the static array __initdata as you mentioned. -- Regards, Shawn -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html