Hi Mike, On Sun, Jan 8, 2012 at 11:57 AM, Mike Frysinger <vap...@gentoo.org> wrote: > On Sunday 08 January 2012 12:33:34 Simon Glass wrote: >> On Sun, Jan 8, 2012 at 12:30 AM, Mike Frysinger wrote: >> > On Saturday 10 December 2011 16:08:03 Simon Glass wrote: >> >> --- a/lib/time.c >> >> +++ b/lib/time.c >> >> >> >> +ulong __timer_get_boot_us(void) >> >> +{ >> >> + static ulong base_time; >> >> + >> >> + /* >> >> + * We can't implement this properly. Return 0 on the first call >> >> and + * larger values after that. >> >> + */ >> >> + if (base_time) >> >> + return get_timer(base_time) * 1000; >> >> + base_time = get_timer(0); >> >> + return 0; >> >> +} >> >> + >> >> +ulong timer_get_boot_us(void) >> >> + __attribute__((weak, alias("__timer_get_boot_us"))); >> > >> > isn't this duplicating existing post_time_ms() ? >> >> Yes I am aware of the post stuff, but I have already expended great >> effort integrating the boot progress stuff and I don't want to do >> everything at once! >> >> There is some scope to join bootstage and post, but for now they are >> separate things with separate CONFIGs, and you don't have to enable >> one to get the other. > > ok. i'd change the "bootstage:" part of your commit message though to > something like "timer: add microsecond boot func".
OK done. Regards, Simon > -mike _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot