Hi Mike,
On Sun, Jan 8, 2012 at 11:57 AM, Mike Frysinger vapier@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