Hi, On 03/12/2013 10:54 AM, Thomas Abraham wrote: > This change is fine but one point that should be considered is that on > non-dt platforms, the memory allocation of clk_table cannot really be > justified just because few clocks require two or more aliases. > > Instead, the optional alias passed for divider/mux register functions > can actually be a list of alias names, the list being terminated by a > zero-length string. The clock register helper functions can then loop > through that list and register all the aliases. Heiko's approach look much more clean to me and even for Exynos platform which has currently about 350 clocks and is going to be dt-only starting from 3.10 we are talking about 1.5kB of memory. Other (s3c64/24xx) platforms have much less clocks (70...150). IMHO all clocks aliases should be registered as it is done in Heiko's patch series. And the clocks table could just be freed on non-dt platforms if it is a real issue and anyone is concerned about it. Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |