From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Kaehlcke Date: Thu, 25 Feb 2010 19:15:29 +0100 Subject: [U-Boot] [PATCH 2/2 rev2] ep93xx: Refactoring of timer code In-Reply-To: <4B869D53.1040207@windriver.com> References: <7ee75976e78e2f82b4163fe1ff4233a850d4393c.1266966938.git.matthias@kaehlcke.net> <20100223232209.GR20201@darwin> <4B869D53.1040207@windriver.com> Message-ID: <20100225181528.GA3628@darwin> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tom, El Thu, Feb 25, 2010 at 09:54:59AM -0600 Tom ha dit: > Matthias Kaehlcke wrote: > >ep93xx: Refactoring of the timer code, including the following changes > > > > * use a free running timer instead of a periodical one > > * use unsigned long long for total number of ticks > > * hold the timer state in a structure instead of separate variables > > * increment the timer counter instead of decrementing it > > * remove unused function udelay_masked() > > * remove unused function set_timer() > > > >Signed-off-by: Matthias Kaehlcke > > This set has been applied to ARM. i was/am working on a new version of the patch, taking into account your remarks about the unit of TIMER_FREQ and fixing some issues discussed with Alessandro Rubini off-list, who worked on a similar patch. what do you prefer, a patch based on the ones you just applied, or revert them and get one with all fixes (only one patch, cause the function clk_to_systicks() fixed in the first patch of the set is removed) i apologize for not having taken the time to notify about this and causing you overhead :( -- Matthias Kaehlcke Embedded Linux Developer Barcelona Someone has said that it requires less mental effort to condemn than to think (Emma Goldman) .''`. using free software / Debian GNU/Linux | http://debian.org : :' : `. `'` gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4 `-