[U-Boot] [RFC PATCH v2 11/15] bootstage: Define an optional microsecond timer

Simon Glass
2012-01-08T17:33:34Z

Hi Mike,

On Sun, Jan 8, 2012 at 12:30 AM, Mike Frysinger <vapier@gentoo.org> 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() ?
> -mike

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.

Regards,
Simon