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

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to