Messages in this thread | | | Date | Thu, 22 Mar 2012 11:23:26 +0800 | From | Shawn Guo <> | Subject | Re: [PATCH 2/2] clk: Move init fields from clk to clk_hw |
| |
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
| |