On Thu, Oct 7, 2010 at 2:33 PM, Yong Shen <yong.s...@linaro.org> wrote: > >> > # Object file lists. >> > -obj-y := cpu.o mm.o clock-mx51.o devices.o >> > +obj-y := cpu.o mm.o clock-mx51.o devices.o cpu_wp-mx51.o >> >> By hardcoding cpu_wp-mx51 here, you are making the assumption that even if >> cpufreq is turned off, you'll still need the entire WP table to set the >> cpu frequency. Why can't this be handled transparently in the clock code? >> If >> the clock code does the calculations, every board doesn't need to first >> setup >> calls to get_cpu_wp. >> > > It has history. Not only cpufreq needs this table, other freescale drivers > also need it, like dvfs and busfreq drivers. So it is better to be there. I > do agree clock should do its calculation by itself. > Agree about the rest.
Let's modify that history and do it the correct way then :) When we come to the dvfs and bus drivers we'll fix them too. Regards, Amit _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev
