Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-27 Thread Lee Jones
On Tue, 27 Nov 2012, Wolfgang Denk wrote: > Dear Lee Jones, > > In message <20121127085548.gc7...@gmail.com> you wrote: > > > > By the way, if Wolfgang didn't want these tracepoints in DT, then > > how was your implementations upstreamed into u-boot? > > Because I don;t manage a 100% review cov

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-27 Thread Wolfgang Denk
Dear Lee Jones, In message <20121127085548.gc7...@gmail.com> you wrote: > > By the way, if Wolfgang didn't want these tracepoints in DT, then > how was your implementations upstreamed into u-boot? Because I don;t manage a 100% review coverage over all submitted patches, i. e. it escaped my atten

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-27 Thread Lee Jones
Hi Simon, > >> On Tue, Nov 20, 2012 at 6:33 AM, Lee Jones wrote: > >> > Boottime is a tool which can be used for full system booting time > >> > measurement. Bootloader boot time is passed to the kernel component > >> > though ATAGS. The kernel-side driver then uses this information to > >> > pro

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-26 Thread Simon Glass
Hi Lee, On Mon, Nov 26, 2012 at 1:00 AM, Lee Jones wrote: > Hi Simon, > >> On Tue, Nov 20, 2012 at 6:33 AM, Lee Jones wrote: >> > Boottime is a tool which can be used for full system booting time >> > measurement. Bootloader boot time is passed to the kernel component >> > though ATAGS. The kern

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-26 Thread Lee Jones
Hi Simon, > On Tue, Nov 20, 2012 at 6:33 AM, Lee Jones wrote: > > Boottime is a tool which can be used for full system booting time > > measurement. Bootloader boot time is passed to the kernel component > > though ATAGS. The kernel-side driver then uses this information to > > provide full syste

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-25 Thread Simon Glass
Hi Lee, On Tue, Nov 20, 2012 at 6:33 AM, Lee Jones wrote: > Boottime is a tool which can be used for full system booting time > measurement. Bootloader boot time is passed to the kernel component > though ATAGS. The kernel-side driver then uses this information to > provide full system boot time

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-21 Thread Wolfgang Denk
Dear Lee Jones, In message <20121121172604.gb...@gmail.com> you wrote: > > > You appear to have a specific definition of log data in mind. It must > > be different to mine. > > I would say that a log entry would contain useful descriptive > information contained. It is _sometimes_ useful to dis

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-21 Thread Lee Jones
On Wed, 21 Nov 2012, Wolfgang Denk wrote: > Dear Lee Jones, > > In message <20121121150332.gc28...@gmail.com> you wrote: > > > > > Neither ATAGS not the device tree are intended nor designed for > > > passing logfile information. Yes, you can use them like that, and it > > > will actually work.

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-21 Thread Wolfgang Denk
Dear Lee Jones, In message <20121121150332.gc28...@gmail.com> you wrote: > > > Neither ATAGS not the device tree are intended nor designed for > > passing logfile information. Yes, you can use them like that, and it > > will actually work. > > ATAGs were exactly designed for this type of thing.

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-21 Thread Lee Jones
> > Yes, I intend to extend this functionality into Device Tree. > > That way it will be architecture and OS independent. > > > > > And forcing something upon a mechanism that was designed for a > > > completely different purpose, where you see right from the first > > > glance that it does not m

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-21 Thread Wolfgang Denk
Dear Lee Jones, In message <20121121095045.gi28...@gmail.com> you wrote: > > Yes, I intend to extend this functionality into Device Tree. > That way it will be architecture and OS independent. > > > And forcing something upon a mechanism that was designed for a > > completely different purpose, w

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-21 Thread Lee Jones
> > Boottime is a tool which can be used for full system booting time > > measurement. Bootloader boot time is passed to the kernel component > > though ATAGS. The kernel-side driver then uses this information to > > provide full system boot time diagnostics though debugfs. > > Aren't we convertin

Re: [U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-20 Thread Wolfgang Denk
Dear Lee Jones, In message <1353422034-28107-4-git-send-email-lee.jo...@linaro.org> you wrote: > Boottime is a tool which can be used for full system booting time > measurement. Bootloader boot time is passed to the kernel component > though ATAGS. The kernel-side driver then uses this information

[U-Boot] [PATCH 3/8] boottime: Add core boottime measurement support

2012-11-20 Thread Lee Jones
Boottime is a tool which can be used for full system booting time measurement. Bootloader boot time is passed to the kernel component though ATAGS. The kernel-side driver then uses this information to provide full system boot time diagnostics though debugfs. Based heavily on the original driver by