[U-Boot] PATCH: Add "time" command

Mike Frysinger vapier at gentoo.org
Sun Oct 30 19:15:40 CET 2011


On Sunday 30 October 2011 11:55:15 Wolfgang Denk wrote:
> Mike Frysinger wrote:
> > > Add 'time <cmd> <args>' which executes <cmd> with <args> 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 


More information about the U-Boot mailing list