On 09/09/2013 12:53 AM, Wolfgang Denk wrote: > Dear Rob Herring, > > In message <1378671174-18535-1-git-send-email-robherri...@gmail.com> you > wrote: >> From: Rob Herring <rob.herr...@calxeda.com> >> >> Much of the ARM timer code is re-implemented for each platform yet it >> is all pretty much the same code. >> >> This series introduces a common implementation of timer functions and >> simplifies the platform code down to 2 or 3 config defines. It is >> intended for platforms with 32-bit freerunning timers. I've converted >> a couple of platforms as an example, but there are many more still that >> can be converted. This probably could be extended to work with 16-bit >> timers as well. >> >> I've compiled all ARM boards, but only tested on highbank. > > Why do we need any ARM specific code at all? What's the difference > between an ARM and - say - a PowerPC system in this respect, and why > do we need different code for both? > > If we consolidate code, it should always be done in a cross-platform > way, unless platform specifc code really cannot be avoided (which I > fail to see yet here).
I agree, but it appeared that at least some PPC platforms implement timers differently using a timer interrupt. I haven't really looked at other architectures. At least moving the code to a common spot is trivial. Aside from the location, do you have any comments on the implementation? Rob _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot