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

[Simon Glass]

Hi Mike,

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