From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Sun, 8 Jan 2012 14:57:29 -0500 Subject: [U-Boot] [RFC PATCH v2 11/15] bootstage: Define an optional microsecond timer In-Reply-To: References: <1323551287-5351-1-git-send-email-sjg@chromium.org> <201201080330.28110.vapier@gentoo.org> Message-ID: <201201081457.31563.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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". -mike -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. URL: