[U-Boot] [PATCH 0/4] Accurate boot time measurement

Simon Glass sjg at chromium.org
Sun May 15 23:34:49 CEST 2011


On Sun, May 15, 2011 at 3:03 AM, Graeme Russ <graeme.russ at gmail.com> wrote:

> Couple of thoughts:
>  - Macro the definition of show_boot_progress() so it accepts a (const
>    char *) argument if CONFIG_BOOTSTAGE is defined
>  - Change BOOTSTAGE_COUNT to CONFIG_MAX_BOOTSTAGE_RECORDS
>  - Any call to show_boot_progress() with a non-null second argument
>    causes it to be logged in the next available bootstage record
>  - After the last bootstage record is filled, no further calls to
>    show_boot_progress() cause logging to the bootstage records
>  - Instead of CONFIG_BOOTSTAGE_REPORT, create a command which dumps the
>    contents of the bootstage records
>
> No this is going to result in a big patch (every instance of
> show_boot_progress() will need to be touched to at least add NULL as the
> second parameter - Maintainers can add better text later ;)
>

Thanks for your thoughts. It can be done, but my bigger question is this:

> If there is a strong appetite for that sort of patch I'm happy to do it.

If I get some +ve responses then I will do it, otherwise I think I should
assume people want this left as is.

Regards,
Simon


>
> Regards,
>
> Graeme
>


More information about the U-Boot mailing list