All of lore.kernel.org
 help / color / mirror / Atom feed
From: Graeme Russ <graeme.russ@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/7] Bootgraph.pl instrumentation support for UBoot
Date: Tue, 13 Sep 2011 22:01:56 +1000	[thread overview]
Message-ID: <4E6F4634.8080304@gmail.com> (raw)
In-Reply-To: <CAPnjgZ0KrdqSv9YJ+bv4pnbYhNmDRpRr3yT4OnFWUYDdhpcytw@mail.gmail.com>

Hi Simon,

On 13/09/11 21:52, Simon Glass wrote:
> Hi Graeme,
> 
> On Mon, Sep 12, 2011 at 10:24 PM, Graeme Russ <graeme.russ@gmail.com> wrote:
>> Hi Simon,
>>
>> On Tue, Sep 13, 2011 at 2:34 PM, Simon Glass <sjg@chromium.org> wrote:
>>> Hi Andrew,
>>>
>>> On Sat, Sep 10, 2011 at 5:40 AM, Andrew Murray <amurray@theiet.org> wrote:
>>>> On 1 September 2011 00:53, Andrew Murray <amurray@theiet.org> wrote:
>>>>>

[snip]

>>  - Bootgraph
>>  - Unified timer API (nanosecond would be nice)
>>  - initcall
>>  - 'pre-console' output buffer
>>  - timestamped printf()
>>
>> Looking forward to opening up these cans of worms again :)
> 
> Bravery is to be encouraged. Biting off what seems like the smallest,
> what is the status of pre-console output?

I made the requested changes, the pre-console buffer only gets compiled in
if it is explicitly configured and it works a treat

I see no reason not to include it (along with the pre-console printf()
squelching)

The only related patch that was under discussion was 'flagifying' a couple
of global variables which opened up a question regarding the impact on API
versioning - Wolfgang resolved this with 'gd does not impact API version'
so the 'flagify' patch could also be added to the mix, but that's not
really that critical and could come in later after the dust settles.

Unified timer API would be the next 'least contentious' - There is
universal agreement on the why, but not on the how. I'll reopen this again
'soon'

The initcall API is the most contentious... Asbestos suit is still on the
clothes line drying ;)

Regards,

Graeme

  reply	other threads:[~2011-09-13 12:01 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-31 22:20 [U-Boot] [PATCH 0/7] Bootgraph.pl instrumentation support for UBoot Andrew Murray
2011-08-31 22:20 ` [U-Boot] [PATCH 1/7] Add bootgraph.pl script for generating a boot graph SVG file Andrew Murray
2011-08-31 22:20 ` [U-Boot] [PATCH 2/7] Add macros for recording init calls during UBoot execution Andrew Murray
2011-08-31 22:50   ` Mike Frysinger
2011-08-31 23:40     ` Andrew Murray
2011-08-31 22:20 ` [U-Boot] [PATCH 3/7] Add timing information to printf's for use with bootgraph.pl Andrew Murray
2011-08-31 22:47   ` Mike Frysinger
2011-08-31 23:30     ` Andrew Murray
2011-08-31 23:38     ` Graeme Russ
2011-08-31 23:42       ` Andrew Murray
2011-09-07 21:10     ` Wolfgang Denk
2011-09-09  0:50       ` Mike Frysinger
2011-09-09  7:00         ` Wolfgang Denk
2011-08-31 22:20 ` [U-Boot] [PATCH 4/7] Add bootgraph instrumentation for ARM boards Andrew Murray
2011-08-31 22:20 ` [U-Boot] [PATCH 5/7] Add bootgraph instrumentation for bootm command Andrew Murray
2011-08-31 22:21 ` [U-Boot] [PATCH 6/7] Add bootgraph instrumentation for UBoot commands Andrew Murray
2011-08-31 22:21 ` [U-Boot] [PATCH 7/7] Add documentation for bootgraph.pl Andrew Murray
2011-08-31 22:47 ` [U-Boot] [PATCH 0/7] Bootgraph.pl instrumentation support for UBoot Mike Frysinger
2011-08-31 23:12   ` Simon Glass
2011-08-31 23:25     ` Andrew Murray
2011-08-31 23:32       ` Graeme Russ
2011-08-31 23:39         ` Simon Glass
2011-08-31 23:53           ` Andrew Murray
2011-09-10 12:40             ` Andrew Murray
2011-09-13  4:34               ` Simon Glass
2011-09-13  5:24                 ` Graeme Russ
2011-09-13 11:52                   ` Simon Glass
2011-09-13 12:01                     ` Graeme Russ [this message]
2011-09-13 12:22                       ` Simon Glass
2011-09-10 12:37 ` [U-Boot] [PATCH v2 1/7] Add bootgraph.pl script for generating a boot graph SVG file Andrew Murray
2011-09-10 12:37   ` [U-Boot] [PATCH v2 2/7] Add macros for recording init calls during UBoot execution Andrew Murray
2011-09-10 13:17     ` Andrew Murray
2011-09-18  2:16       ` Mike Frysinger
2011-09-10 12:37   ` [U-Boot] [PATCH v2 3/7] Add timing information to printf's for use with bootgraph.pl Andrew Murray
2011-09-18  2:10     ` Mike Frysinger
2011-09-10 12:37   ` [U-Boot] [PATCH v2 4/7] Add bootgraph instrumentation for ARM boards Andrew Murray
2011-09-18  2:11     ` Mike Frysinger
2011-10-06 21:37     ` Wolfgang Denk
2011-09-10 12:37   ` [U-Boot] [PATCH v2 5/7] Add bootgraph instrumentation for bootm command Andrew Murray
2011-09-10 12:37   ` [U-Boot] [PATCH v2 6/7] Add bootgraph instrumentation for UBoot commands Andrew Murray
2011-09-18  2:11     ` Mike Frysinger
2011-10-06 21:35     ` Wolfgang Denk
2011-09-10 12:37   ` [U-Boot] [PATCH v2 7/7] Add documentation for bootgraph.pl Andrew Murray
2011-10-06 21:38   ` [U-Boot] [PATCH v2 1/7] Add bootgraph.pl script for generating a boot graph SVG file Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E6F4634.8080304@gmail.com \
    --to=graeme.russ@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.