From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Sun, 30 Oct 2011 14:15:40 -0400 Subject: [U-Boot] PATCH: Add "time" command In-Reply-To: <20111030155515.31F46180985A@gemini.denx.de> References: <4EAC146C.8060104@logicpd.com> <201110301113.25446.vapier@gentoo.org> <20111030155515.31F46180985A@gemini.denx.de> Message-ID: <201110301415.41556.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sunday 30 October 2011 11:55:15 Wolfgang Denk wrote: > Mike Frysinger wrote: > > > Add 'time ' which executes with and shows the > > > execution time in seconds. Requires get_timer(). > > > > NAK: we already have common/cmd_time.c and CONFIG_CMD_TIME > > But common/cmd_time.c is unportable and only available on ARM due so > using a non standard timer API (get_timer_masked() etc. i think the idea was to use the more exact API like will be available with Graeme's proposed rewrite ... > I wonder if we should swap the implementations. extending the current code to support non-ARM makes sense, but adding a new code base like the original patch does not -mike -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. Url : http://lists.denx.de/pipermail/u-boot/attachments/20111030/6706434c/attachment.pgp