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

Simon Glass sjg at chromium.org
Sun Jan 8 18:33:34 CET 2012


Hi Mike,

On Sun, Jan 8, 2012 at 12:30 AM, Mike Frysinger <vapier at 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


More information about the U-Boot mailing list