[U-Boot] [PATCH v6 0/2] timer: Add High Precision Event Timers (HPET) support

Add HPET driver as an alternative timer for x86 (default is TSC). HPET counter has constant frequency and does not need calibration. This change also makes TSC timer driver optional on x86. New HPET driver can also be selected as the early timer on x86.
v6: Added TSC-specific timer functions to use instead of early timer in the code that specifically needs TSC.
v5: Using readq() and writeq() for main counter access.
v3: Added early timer choice in x86 Kconfig.
Ivan Gorinov (2): x86: Add TSC-specific timer functions timer: Add High Precision Event Timers (HPET) support
arch/Kconfig | 1 + arch/x86/Kconfig | 21 ++++ arch/x86/cpu/coreboot/timestamp.c | 2 +- arch/x86/cpu/quark/mrc_util.c | 13 ++- arch/x86/dts/hpet.dtsi | 7 ++ arch/x86/include/asm/u-boot-x86.h | 2 +- common/Kconfig | 1 + drivers/timer/Kconfig | 9 ++ drivers/timer/Makefile | 1 + drivers/timer/hpet_timer.c | 209 ++++++++++++++++++++++++++++++++++++++ drivers/timer/tsc_timer.c | 39 +++++-- 11 files changed, 288 insertions(+), 17 deletions(-) create mode 100644 arch/x86/dts/hpet.dtsi create mode 100644 drivers/timer/hpet_timer.c

Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Add tsc_rate_mhz() function and use it in the code that specifically needs to get TSC rate regardless of currently selected early timer.
Signed-off-by: Ivan Gorinov ivan.gorinov@intel.com --- arch/x86/cpu/coreboot/timestamp.c | 2 +- arch/x86/cpu/quark/mrc_util.c | 13 ++++++------- arch/x86/include/asm/u-boot-x86.h | 2 +- drivers/timer/tsc_timer.c | 33 ++++++++++++++++++++++++--------- 4 files changed, 32 insertions(+), 18 deletions(-)
diff --git a/arch/x86/cpu/coreboot/timestamp.c b/arch/x86/cpu/coreboot/timestamp.c index b382795..05bb214 100644 --- a/arch/x86/cpu/coreboot/timestamp.c +++ b/arch/x86/cpu/coreboot/timestamp.c @@ -78,7 +78,7 @@ int timestamp_add_to_bootstage(void) if (name) { bootstage_add_record(0, name, BOOTSTAGEF_ALLOC, tse->entry_stamp / - get_tbclk_mhz()); + tsc_rate_mhz()); } }
diff --git a/arch/x86/cpu/quark/mrc_util.c b/arch/x86/cpu/quark/mrc_util.c index fac2d72..4794395 100644 --- a/arch/x86/cpu/quark/mrc_util.c +++ b/arch/x86/cpu/quark/mrc_util.c @@ -57,19 +57,18 @@ void mrc_post_code(uint8_t major, uint8_t minor) void delay_n(uint32_t ns) { /* 1000 MHz clock has 1ns period --> no conversion required */ - uint64_t final_tsc = rdtsc(); + uint64_t start_tsc = rdtsc(); + uint64_t ticks;
- final_tsc += ((get_tbclk_mhz() * ns) / 1000); - - while (rdtsc() < final_tsc) - ; + ticks = (tsc_rate_mhz() * ns) / 1000; + while (rdtsc() - start_tsc < ticks); }
/* Delay number of microseconds */ -void delay_u(uint32_t ms) +void delay_u(uint32_t us) { /* 64-bit math is not an option, just use loops */ - while (ms--) + while (us--) delay_n(1000); }
diff --git a/arch/x86/include/asm/u-boot-x86.h b/arch/x86/include/asm/u-boot-x86.h index 187fe5f..de68120 100644 --- a/arch/x86/include/asm/u-boot-x86.h +++ b/arch/x86/include/asm/u-boot-x86.h @@ -29,7 +29,7 @@ int cleanup_before_linux(void); void timer_isr(void *); typedef void (timer_fnc_t) (void); int register_timer_isr (timer_fnc_t *isr_func); -unsigned long get_tbclk_mhz(void); +unsigned long tsc_rate_mhz(void); void timer_set_base(uint64_t base); int i8254_init(void);
diff --git a/drivers/timer/tsc_timer.c b/drivers/timer/tsc_timer.c index 9296de6..98cbf12 100644 --- a/drivers/timer/tsc_timer.c +++ b/drivers/timer/tsc_timer.c @@ -277,15 +277,12 @@ success: return delta / 1000; }
-/* Get the speed of the TSC timer in MHz */ -unsigned notrace long get_tbclk_mhz(void) -{ - return get_tbclk() / 1000000; -} - static ulong get_ms_timer(void) { - return (get_ticks() * 1000) / get_tbclk(); + if (gd->arch.clock_rate == 0) + return 0; + + return (rdtsc() * 1000) / gd->arch.clock_rate; }
ulong get_timer(ulong base) @@ -295,7 +292,10 @@ ulong get_timer(ulong base)
ulong notrace timer_get_us(void) { - return get_ticks() / get_tbclk_mhz(); + if (gd->arch.clock_rate == 0) + return 0; + + return (rdtsc() * 1000000) / gd->arch.clock_rate; }
ulong timer_get_boot_us(void) @@ -308,7 +308,7 @@ void __udelay(unsigned long usec) u64 now = get_ticks(); u64 stop;
- stop = now + usec * get_tbclk_mhz(); + stop = now + ((u64)usec * gd->arch.clock_rate) / 1000000;
while ((int64_t)(stop - get_ticks()) > 0) #if defined(CONFIG_QEMU) && defined(CONFIG_SMP) @@ -355,6 +355,14 @@ static void tsc_timer_ensure_setup(void) } }
+/* Get the speed of the TSC timer in MHz */ +unsigned notrace long tsc_rate_mhz(void) +{ + tsc_timer_ensure_setup(); + + return gd->arch.clock_rate / 1000000; +} + static int tsc_timer_probe(struct udevice *dev) { struct timer_dev_priv *uc_priv = dev_get_uclass_priv(dev); @@ -365,6 +373,13 @@ static int tsc_timer_probe(struct udevice *dev) return 0; }
+int timer_init(void) +{ + tsc_timer_ensure_setup(); + + return 0; +} + unsigned long notrace timer_early_get_rate(void) { tsc_timer_ensure_setup();

On Thu, 2018-04-12 at 15:12 -0700, Ivan Gorinov wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Add tsc_rate_mhz() function and use it in the code that specifically needs to get TSC rate regardless of currently selected early timer.
void delay_n(uint32_t ns) { /* 1000 MHz clock has 1ns period --> no conversion required */
- uint64_t final_tsc = rdtsc();
- uint64_t start_tsc = rdtsc();
- uint64_t ticks;
- final_tsc += ((get_tbclk_mhz() * ns) / 1000);
- while (rdtsc() < final_tsc)
;
- ticks = (tsc_rate_mhz() * ns) / 1000;
- while (rdtsc() - start_tsc < ticks);
I would rather preserve existing style.
}
/* Delay number of microseconds */ -void delay_u(uint32_t ms) +void delay_u(uint32_t us) { /* 64-bit math is not an option, just use loops */
- while (ms--)
- while (us--) delay_n(1000);
}
This is a separate change.

On Fri, Apr 20, 2018 at 06:25:08AM -0600, Andy Shevchenko wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Add tsc_rate_mhz() function and use it in the code that specifically needs to get TSC rate regardless of currently selected early timer.
void delay_n(uint32_t ns) { /* 1000 MHz clock has 1ns period --> no conversion required */
- uint64_t final_tsc = rdtsc();
- uint64_t start_tsc = rdtsc();
- uint64_t ticks;
- final_tsc += ((get_tbclk_mhz() * ns) / 1000);
- while (rdtsc() < final_tsc)
;
- ticks = (tsc_rate_mhz() * ns) / 1000;
- while (rdtsc() - start_tsc < ticks);
I would rather preserve existing style.
OK. Existing style does not correctly handle overflow, but for a 64-bit counter it's a very unlikely event.

On Fri, 2018-04-20 at 11:00 -0700, Ivan Gorinov wrote:
On Fri, Apr 20, 2018 at 06:25:08AM -0600, Andy Shevchenko wrote:
- while (rdtsc() < final_tsc)
;
- while (rdtsc() - start_tsc < ticks);
I would rather preserve existing style.
OK. Existing style does not correctly handle overflow, but for a 64-bit counter it's a very unlikely event.
You didn't get me. I'm just talking about style, not about functionality.
See above for lines I left in this reply.

Hi Ivan,
On Fri, Apr 13, 2018 at 6:12 AM, Ivan Gorinov ivan.gorinov@intel.com wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Thanks for working on this. But get_tbclk() is one API provided by the timer library. The get_tbclk_mhz() is something that is implemented by the TSC timer driver, so can we get rid of the get_tbclk_mhz() and use the get_tbclk() instead in coreboot/timestamp.c and quark/mrc_util.c?
Add tsc_rate_mhz() function and use it in the code that specifically needs to get TSC rate regardless of currently selected early timer.
Signed-off-by: Ivan Gorinov ivan.gorinov@intel.com
arch/x86/cpu/coreboot/timestamp.c | 2 +- arch/x86/cpu/quark/mrc_util.c | 13 ++++++------- arch/x86/include/asm/u-boot-x86.h | 2 +- drivers/timer/tsc_timer.c | 33 ++++++++++++++++++++++++--------- 4 files changed, 32 insertions(+), 18 deletions(-)
diff --git a/arch/x86/cpu/coreboot/timestamp.c b/arch/x86/cpu/coreboot/timestamp.c index b382795..05bb214 100644 --- a/arch/x86/cpu/coreboot/timestamp.c +++ b/arch/x86/cpu/coreboot/timestamp.c @@ -78,7 +78,7 @@ int timestamp_add_to_bootstage(void) if (name) { bootstage_add_record(0, name, BOOTSTAGEF_ALLOC, tse->entry_stamp /
get_tbclk_mhz());
tsc_rate_mhz()); } }
diff --git a/arch/x86/cpu/quark/mrc_util.c b/arch/x86/cpu/quark/mrc_util.c index fac2d72..4794395 100644 --- a/arch/x86/cpu/quark/mrc_util.c +++ b/arch/x86/cpu/quark/mrc_util.c @@ -57,19 +57,18 @@ void mrc_post_code(uint8_t major, uint8_t minor) void delay_n(uint32_t ns) { /* 1000 MHz clock has 1ns period --> no conversion required */
uint64_t final_tsc = rdtsc();
uint64_t start_tsc = rdtsc();
uint64_t ticks;
final_tsc += ((get_tbclk_mhz() * ns) / 1000);
while (rdtsc() < final_tsc)
;
ticks = (tsc_rate_mhz() * ns) / 1000;
while (rdtsc() - start_tsc < ticks);
}
/* Delay number of microseconds */ -void delay_u(uint32_t ms) +void delay_u(uint32_t us) { /* 64-bit math is not an option, just use loops */
while (ms--)
while (us--) delay_n(1000);
}
diff --git a/arch/x86/include/asm/u-boot-x86.h b/arch/x86/include/asm/u-boot-x86.h index 187fe5f..de68120 100644 --- a/arch/x86/include/asm/u-boot-x86.h +++ b/arch/x86/include/asm/u-boot-x86.h @@ -29,7 +29,7 @@ int cleanup_before_linux(void); void timer_isr(void *); typedef void (timer_fnc_t) (void); int register_timer_isr (timer_fnc_t *isr_func); -unsigned long get_tbclk_mhz(void); +unsigned long tsc_rate_mhz(void); void timer_set_base(uint64_t base); int i8254_init(void);
diff --git a/drivers/timer/tsc_timer.c b/drivers/timer/tsc_timer.c index 9296de6..98cbf12 100644 --- a/drivers/timer/tsc_timer.c +++ b/drivers/timer/tsc_timer.c @@ -277,15 +277,12 @@ success: return delta / 1000; }
-/* Get the speed of the TSC timer in MHz */ -unsigned notrace long get_tbclk_mhz(void) -{
return get_tbclk() / 1000000;
-}
static ulong get_ms_timer(void) {
return (get_ticks() * 1000) / get_tbclk();
if (gd->arch.clock_rate == 0)
return 0;
return (rdtsc() * 1000) / gd->arch.clock_rate;
}
ulong get_timer(ulong base) @@ -295,7 +292,10 @@ ulong get_timer(ulong base)
ulong notrace timer_get_us(void) {
return get_ticks() / get_tbclk_mhz();
if (gd->arch.clock_rate == 0)
return 0;
return (rdtsc() * 1000000) / gd->arch.clock_rate;
}
ulong timer_get_boot_us(void) @@ -308,7 +308,7 @@ void __udelay(unsigned long usec) u64 now = get_ticks(); u64 stop;
stop = now + usec * get_tbclk_mhz();
stop = now + ((u64)usec * gd->arch.clock_rate) / 1000000; while ((int64_t)(stop - get_ticks()) > 0)
#if defined(CONFIG_QEMU) && defined(CONFIG_SMP) @@ -355,6 +355,14 @@ static void tsc_timer_ensure_setup(void) } }
+/* Get the speed of the TSC timer in MHz */ +unsigned notrace long tsc_rate_mhz(void) +{
tsc_timer_ensure_setup();
return gd->arch.clock_rate / 1000000;
+}
static int tsc_timer_probe(struct udevice *dev) { struct timer_dev_priv *uc_priv = dev_get_uclass_priv(dev); @@ -365,6 +373,13 @@ static int tsc_timer_probe(struct udevice *dev) return 0; }
+int timer_init(void) +{
tsc_timer_ensure_setup();
return 0;
+}
unsigned long notrace timer_early_get_rate(void) { tsc_timer_ensure_setup(); --
Regards, Bin

Hi Ivan,
On Mon, Apr 23, 2018 at 3:38 PM, Bin Meng bmeng.cn@gmail.com wrote:
Hi Ivan,
On Fri, Apr 13, 2018 at 6:12 AM, Ivan Gorinov ivan.gorinov@intel.com wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Thanks for working on this. But get_tbclk() is one API provided by the timer library. The get_tbclk_mhz() is something that is implemented by the TSC timer driver, so can we get rid of the get_tbclk_mhz() and use the get_tbclk() instead in coreboot/timestamp.c and quark/mrc_util.c?
Further request on TSC timer driver clean up, in order to make HPET work with either TSC or HPET being the U-Boot (early) timer, we should:
- Remove get_tbclk_mhz() in tsc_timer.c - Remove get_timer() in tsc_timer.c - Remove timer_get_us() in tsc_timer.c - Remove timer_get_boot_us() in tsc_timer.c - Move __udelay() implementation in tsc_timer.c to arch/x86/cpu/qemu/qemu.c, as this is the QEMU specific support. For other platforms, use the default one provided in lib/time.c
Regards, Bin

Hi Bin,
On Mon, Apr 23, 2018 at 01:38:05AM -0600, Bin Meng wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Thanks for working on this. But get_tbclk() is one API provided by the timer library. The get_tbclk_mhz() is something that is implemented by the TSC timer driver, so can we get rid of the get_tbclk_mhz() and use the get_tbclk() instead in coreboot/timestamp.c and quark/mrc_util.c?
The Coreboot timestamp code and Quark MRC specifically use rdtsc(). We can replace it with timer_early_get_count() or provide a function to get the TSC frequency even when another early timer is selected.
Add tsc_rate_mhz() function and use it in the code that specifically needs to get TSC rate regardless of currently selected early timer.
Signed-off-by: Ivan Gorinov ivan.gorinov@intel.com
arch/x86/cpu/coreboot/timestamp.c | 2 +- arch/x86/cpu/quark/mrc_util.c | 13 ++++++------- arch/x86/include/asm/u-boot-x86.h | 2 +- drivers/timer/tsc_timer.c | 33 ++++++++++++++++++++++++--------- 4 files changed, 32 insertions(+), 18 deletions(-)
diff --git a/arch/x86/cpu/coreboot/timestamp.c b/arch/x86/cpu/coreboot/timestamp.c index b382795..05bb214 100644 --- a/arch/x86/cpu/coreboot/timestamp.c +++ b/arch/x86/cpu/coreboot/timestamp.c @@ -78,7 +78,7 @@ int timestamp_add_to_bootstage(void) if (name) { bootstage_add_record(0, name, BOOTSTAGEF_ALLOC, tse->entry_stamp /
get_tbclk_mhz());
tsc_rate_mhz()); } }
diff --git a/arch/x86/cpu/quark/mrc_util.c b/arch/x86/cpu/quark/mrc_util.c index fac2d72..4794395 100644 --- a/arch/x86/cpu/quark/mrc_util.c +++ b/arch/x86/cpu/quark/mrc_util.c @@ -57,19 +57,18 @@ void mrc_post_code(uint8_t major, uint8_t minor) void delay_n(uint32_t ns) { /* 1000 MHz clock has 1ns period --> no conversion required */
uint64_t final_tsc = rdtsc();
uint64_t start_tsc = rdtsc();
uint64_t ticks;
final_tsc += ((get_tbclk_mhz() * ns) / 1000);
while (rdtsc() < final_tsc)
;
ticks = (tsc_rate_mhz() * ns) / 1000;
while (rdtsc() - start_tsc < ticks);
}
/* Delay number of microseconds */ -void delay_u(uint32_t ms) +void delay_u(uint32_t us) { /* 64-bit math is not an option, just use loops */
while (ms--)
while (us--) delay_n(1000);
}
diff --git a/arch/x86/include/asm/u-boot-x86.h b/arch/x86/include/asm/u-boot-x86.h index 187fe5f..de68120 100644 --- a/arch/x86/include/asm/u-boot-x86.h +++ b/arch/x86/include/asm/u-boot-x86.h @@ -29,7 +29,7 @@ int cleanup_before_linux(void); void timer_isr(void *); typedef void (timer_fnc_t) (void); int register_timer_isr (timer_fnc_t *isr_func); -unsigned long get_tbclk_mhz(void); +unsigned long tsc_rate_mhz(void); void timer_set_base(uint64_t base); int i8254_init(void);
diff --git a/drivers/timer/tsc_timer.c b/drivers/timer/tsc_timer.c index 9296de6..98cbf12 100644 --- a/drivers/timer/tsc_timer.c +++ b/drivers/timer/tsc_timer.c @@ -277,15 +277,12 @@ success: return delta / 1000; }
-/* Get the speed of the TSC timer in MHz */ -unsigned notrace long get_tbclk_mhz(void) -{
return get_tbclk() / 1000000;
-}
static ulong get_ms_timer(void) {
return (get_ticks() * 1000) / get_tbclk();
if (gd->arch.clock_rate == 0)
return 0;
return (rdtsc() * 1000) / gd->arch.clock_rate;
}
ulong get_timer(ulong base) @@ -295,7 +292,10 @@ ulong get_timer(ulong base)
ulong notrace timer_get_us(void) {
return get_ticks() / get_tbclk_mhz();
if (gd->arch.clock_rate == 0)
return 0;
return (rdtsc() * 1000000) / gd->arch.clock_rate;
}
ulong timer_get_boot_us(void) @@ -308,7 +308,7 @@ void __udelay(unsigned long usec) u64 now = get_ticks(); u64 stop;
stop = now + usec * get_tbclk_mhz();
stop = now + ((u64)usec * gd->arch.clock_rate) / 1000000; while ((int64_t)(stop - get_ticks()) > 0)
#if defined(CONFIG_QEMU) && defined(CONFIG_SMP) @@ -355,6 +355,14 @@ static void tsc_timer_ensure_setup(void) } }
+/* Get the speed of the TSC timer in MHz */ +unsigned notrace long tsc_rate_mhz(void) +{
tsc_timer_ensure_setup();
return gd->arch.clock_rate / 1000000;
+}
static int tsc_timer_probe(struct udevice *dev) { struct timer_dev_priv *uc_priv = dev_get_uclass_priv(dev); @@ -365,6 +373,13 @@ static int tsc_timer_probe(struct udevice *dev) return 0; }
+int timer_init(void) +{
tsc_timer_ensure_setup();
return 0;
+}
unsigned long notrace timer_early_get_rate(void) { tsc_timer_ensure_setup(); --

Hi Ivan,
On Tue, Apr 24, 2018 at 7:56 AM, Ivan Gorinov ivan.gorinov@intel.com wrote:
Hi Bin,
On Mon, Apr 23, 2018 at 01:38:05AM -0600, Bin Meng wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Thanks for working on this. But get_tbclk() is one API provided by the timer library. The get_tbclk_mhz() is something that is implemented by the TSC timer driver, so can we get rid of the get_tbclk_mhz() and use the get_tbclk() instead in coreboot/timestamp.c and quark/mrc_util.c?
The Coreboot timestamp code and Quark MRC specifically use rdtsc(). We can replace it with timer_early_get_count() or provide a function to get the TSC frequency even when another early timer is selected.
Good catch. Yes, we should fix coreboot timestamp code and Quark MRC codes to not explicitly call rdtsc.
Another driver that explicitly calls rdtsc() is hw_watchdog_reset() in watchdog/tangier_wdt.c driver. We need fix that too.
Regards, Bin

Hi Ivan,
On Tue, Apr 24, 2018 at 4:41 PM, Bin Meng bmeng.cn@gmail.com wrote:
Hi Ivan,
On Tue, Apr 24, 2018 at 7:56 AM, Ivan Gorinov ivan.gorinov@intel.com wrote:
Hi Bin,
On Mon, Apr 23, 2018 at 01:38:05AM -0600, Bin Meng wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Thanks for working on this. But get_tbclk() is one API provided by the timer library. The get_tbclk_mhz() is something that is implemented by the TSC timer driver, so can we get rid of the get_tbclk_mhz() and use the get_tbclk() instead in coreboot/timestamp.c and quark/mrc_util.c?
The Coreboot timestamp code and Quark MRC specifically use rdtsc(). We can replace it with timer_early_get_count() or provide a function to get the TSC frequency even when another early timer is selected.
Good catch. Yes, we should fix coreboot timestamp code and Quark MRC codes to not explicitly call rdtsc.
Further checking the coreboot timestamp codes, I think we may have to leave the coreboot timestamp codes as it is now.
We have the codes blow: void timestamp_add_now(enum timestamp_id id) { timestamp_add(id, rdtsc()); }
We cannot replace rdtsc() with timer_early_get_count(), because this timestamp_add_now() is called both before and after DM initialization. If the HPET is selected as the early timer and TSC is selected as the normal timer, the timestamp numbers are meaningless to compare against each other.
Another driver that explicitly calls rdtsc() is hw_watchdog_reset() in watchdog/tangier_wdt.c driver. We need fix that too.
Simon, another issue is the bootstage support. So far the timer_get_boot_us() is not implemented by DM timer APIs. timer_get_boot_us() is implemented per timer driver if CONFIG_SYS_TIMER_COUNTER is not defined. Note CONFIG_SYS_TIMER_COUNTER is non-DM stuff. That means the bootstage support is bounded by a specific timer driver, instead of a generic library. To me this overall timer support is somehow fragmentary.
Regards, Bin

Hi Bin,
On 25 April 2018 at 21:42, Bin Meng bmeng.cn@gmail.com wrote:
Hi Ivan,
On Tue, Apr 24, 2018 at 4:41 PM, Bin Meng bmeng.cn@gmail.com wrote:
Hi Ivan,
On Tue, Apr 24, 2018 at 7:56 AM, Ivan Gorinov ivan.gorinov@intel.com wrote:
Hi Bin,
On Mon, Apr 23, 2018 at 01:38:05AM -0600, Bin Meng wrote:
Coreboot timestamp functions and Quark memory reference code use get_tbclk() to get TSC frequency. This will not work if another early timer is selected.
Thanks for working on this. But get_tbclk() is one API provided by the timer library. The get_tbclk_mhz() is something that is implemented by the TSC timer driver, so can we get rid of the get_tbclk_mhz() and use the get_tbclk() instead in coreboot/timestamp.c and quark/mrc_util.c?
The Coreboot timestamp code and Quark MRC specifically use rdtsc(). We can replace it with timer_early_get_count() or provide a function to get the TSC frequency even when another early timer is selected.
Good catch. Yes, we should fix coreboot timestamp code and Quark MRC codes to not explicitly call rdtsc.
Further checking the coreboot timestamp codes, I think we may have to leave the coreboot timestamp codes as it is now.
We have the codes blow: void timestamp_add_now(enum timestamp_id id) { timestamp_add(id, rdtsc()); }
We cannot replace rdtsc() with timer_early_get_count(), because this timestamp_add_now() is called both before and after DM initialization. If the HPET is selected as the early timer and TSC is selected as the normal timer, the timestamp numbers are meaningless to compare against each other.
Another driver that explicitly calls rdtsc() is hw_watchdog_reset() in watchdog/tangier_wdt.c driver. We need fix that too.
Simon, another issue is the bootstage support. So far the timer_get_boot_us() is not implemented by DM timer APIs. timer_get_boot_us() is implemented per timer driver if CONFIG_SYS_TIMER_COUNTER is not defined. Note CONFIG_SYS_TIMER_COUNTER is non-DM stuff. That means the bootstage support is bounded by a specific timer driver, instead of a generic library. To me this overall timer support is somehow fragmentary.
Yes I agree. I'm open to ideas and patches :-)
Regards, Simon

Add HPET driver as an alternative timer for x86 (default is TSC). HPET counter has constant frequency and does not need calibration. New HPET driver can also be selected as the early timer on x86.
HPET can be selected as the tick timer in the Device Tree "chosen" node:
/include/ "hpet.dtsi"
...
chosen { tick-timer = "/hpet"; };
Signed-off-by: Ivan Gorinov ivan.gorinov@intel.com --- arch/Kconfig | 1 + arch/x86/Kconfig | 21 +++++ arch/x86/dts/hpet.dtsi | 7 ++ common/Kconfig | 1 + drivers/timer/Kconfig | 9 ++ drivers/timer/Makefile | 1 + drivers/timer/hpet_timer.c | 209 +++++++++++++++++++++++++++++++++++++++++++++ drivers/timer/tsc_timer.c | 8 ++ 8 files changed, 257 insertions(+) create mode 100644 arch/x86/dts/hpet.dtsi create mode 100644 drivers/timer/hpet_timer.c
diff --git a/arch/Kconfig b/arch/Kconfig index e599e7a..f904fab 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -105,6 +105,7 @@ config X86 select PCI select TIMER select X86_TSC_TIMER + imply HPET_TIMER imply BLK imply DM_ETH imply DM_GPIO diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index 5c23b2c..8f64b95 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -119,6 +119,27 @@ source "arch/x86/cpu/tangier/Kconfig"
# architecture-specific options below
+choice + prompt "Select which timer to use early" + depends on TIMER_EARLY + default X86_EARLY_TIMER_TSC + +config X86_EARLY_TIMER_TSC + bool "TSC" + depends on X86_TSC_TIMER + help + This selects x86 Time Stamp Counter (TSC) as the early timer. + See CONFIG_TIMER_EARLY for the early timer description. + +config X86_EARLY_TIMER_HPET + bool "HPET" + depends on HPET_TIMER + help + This selects High Precision Event Timers as the early timer. + Early HPET base address is specified by CONFIG_HPET_ADDRESS. + +endchoice + config AHCI default y
diff --git a/arch/x86/dts/hpet.dtsi b/arch/x86/dts/hpet.dtsi new file mode 100644 index 0000000..a74f739 --- /dev/null +++ b/arch/x86/dts/hpet.dtsi @@ -0,0 +1,7 @@ +/ { + hpet: hpet@fed00000 { + compatible = "hpet-x86"; + u-boot,dm-pre-reloc; + reg = <0xfed00000 0x1000>; + }; +}; diff --git a/common/Kconfig b/common/Kconfig index 03eeeb2..b02384c 100644 --- a/common/Kconfig +++ b/common/Kconfig @@ -2,6 +2,7 @@ menu "Boot timing"
config BOOTSTAGE bool "Boot timing and reporting" + select TIMER_EARLY help Enable recording of boot time while booting. To use it, insert calls to bootstage_mark() with a suitable BOOTSTAGE_ID from diff --git a/drivers/timer/Kconfig b/drivers/timer/Kconfig index 2c96896..26743b7 100644 --- a/drivers/timer/Kconfig +++ b/drivers/timer/Kconfig @@ -65,6 +65,15 @@ config X86_TSC_TIMER help Select this to enable Time-Stamp Counter (TSC) timer for x86.
+config HPET_TIMER + bool "High Precision Event Timers (HPET) support" + depends on TIMER + default y if X86 + help + Select this to enable High Precision Event Timers (HPET) on x86. + HPET main counter increments at constant rate and does not need + calibration. + config OMAP_TIMER bool "Omap timer support" depends on TIMER diff --git a/drivers/timer/Makefile b/drivers/timer/Makefile index a6e7832..557fecc 100644 --- a/drivers/timer/Makefile +++ b/drivers/timer/Makefile @@ -8,6 +8,7 @@ obj-y += timer-uclass.o obj-$(CONFIG_ALTERA_TIMER) += altera_timer.o obj-$(CONFIG_SANDBOX_TIMER) += sandbox_timer.o obj-$(CONFIG_X86_TSC_TIMER) += tsc_timer.o +obj-$(CONFIG_HPET_TIMER) += hpet_timer.o obj-$(CONFIG_OMAP_TIMER) += omap-timer.o obj-$(CONFIG_AST_TIMER) += ast_timer.o obj-$(CONFIG_STI_TIMER) += sti-timer.o diff --git a/drivers/timer/hpet_timer.c b/drivers/timer/hpet_timer.c new file mode 100644 index 0000000..0914dd5 --- /dev/null +++ b/drivers/timer/hpet_timer.c @@ -0,0 +1,209 @@ +/* + * Copyright (c) 2017 Intel Corporation + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <dm.h> +#include <malloc.h> +#include <timer.h> +#include <asm/cpu.h> +#include <asm/io.h> +#include <asm/u-boot-x86.h> + +#define HPET_PERIOD_REG 0x004 +#define HPET_CONFIG_REG 0x010 +#define HPET_MAIN_COUNT 0x0f0 + +#define ENABLE_CNF 1 + +#define HPET_MAX_PERIOD 100000000 + +struct hpet_timer_priv { + void *regs; +}; + +/* + * Returns HPET clock frequency in HZ + * (rounding to the nearest integer), + * or 0 if HPET is not available. + */ +static inline u32 get_clock_frequency(void *regs) +{ + u64 d = 1000000000000000ull; + u32 period; + + period = readl(regs + HPET_PERIOD_REG); + if (period == 0) + return 0; + if (period > HPET_MAX_PERIOD) + return 0; + + d += period / 2; + + return d / period; +} + +/* Reset and start the main counter */ +static void start_main_counter(void *regs) +{ + u32 config; + + config = readl(regs + HPET_CONFIG_REG); + config &= ~ENABLE_CNF; + writel(config, regs + HPET_CONFIG_REG); + writeq(0, regs + HPET_MAIN_COUNT); + config |= ENABLE_CNF; + writel(config, regs + HPET_CONFIG_REG); +} + +/* Read the main counter, repeat if 32-bit rollover happens */ +static u64 read_main_counter(void *regs) +{ + u64 t, t0; + + t = readq(regs + HPET_MAIN_COUNT); + do { + t0 = t; + t = readq(regs + HPET_MAIN_COUNT); + } while ((t >> 32) != (t0 >> 32)); + + return t; +} + +static int hpet_timer_get_count(struct udevice *dev, u64 *count) +{ + struct hpet_timer_priv *priv = dev_get_priv(dev); + + *count = read_main_counter(priv->regs); + + return 0; +} + +static int hpet_timer_probe(struct udevice *dev) +{ + struct timer_dev_priv *uc_priv = dev_get_uclass_priv(dev); + struct hpet_timer_priv *priv = dev_get_priv(dev); + u32 rate; + + rate = get_clock_frequency(priv->regs); + if (rate == 0) + return -ENODEV; + + start_main_counter(priv->regs); + + uc_priv->clock_rate = rate; + + return 0; +} + +#ifdef CONFIG_X86_EARLY_TIMER_HPET + +static void *early_regs = (void *)CONFIG_HPET_ADDRESS; +static u32 clock_rate; + +unsigned long notrace timer_early_get_rate(void) +{ + return get_clock_frequency(early_regs); +} + +u64 notrace timer_early_get_count(void) +{ + return read_main_counter(early_regs); +} + +int timer_init(void) +{ + clock_rate = get_clock_frequency(early_regs); + if (clock_rate == 0) + return -ENODEV; + + start_main_counter(early_regs); + + return 0; +} + +ulong notrace timer_get_us(void) +{ + u32 period; + u64 d; + + period = readl(early_regs + HPET_PERIOD_REG); + if (period == 0) + return 0; + if (period > HPET_MAX_PERIOD) + return 0; + + d = read_main_counter(early_regs); + + /* + * Multiplication overflow + * at 2^64 femtoseconds + * (more than 5 hours) + */ + + d *= period; + + return d / 1000000000; +} + +ulong timer_get_boot_us(void) +{ + return timer_get_us(); +} + +/* + * Maximum delay is 2^32 HPET ticks, + * ~5 minutes with 14.318 MHz clock. + */ +void __udelay(unsigned long usec) +{ + u32 c, c0, ticks; + u32 period; + + c0 = readl(early_regs + HPET_MAIN_COUNT); + + period = readl(early_regs + HPET_PERIOD_REG); + if (period == 0) + return; + if (period > HPET_MAX_PERIOD) + return; + + ticks = ((u64)usec * 1000000000) / period; + do { + c = readl(early_regs + HPET_MAIN_COUNT); + } while (c - c0 < ticks); +} + +#endif /* CONFIG_X86_EARLY_TIMER_HPET */ + +static int hpet_timer_ofdata_to_platdata(struct udevice *dev) +{ + struct hpet_timer_priv *priv = dev_get_priv(dev); + + priv->regs = map_physmem(devfdt_get_addr(dev), 0x1000, MAP_NOCACHE); + + return 0; +} + +static const struct timer_ops hpet_timer_ops = { + .get_count = hpet_timer_get_count, +}; + +static const struct udevice_id hpet_timer_ids[] = { + { .compatible = "hpet-x86", }, + { .compatible = "intel,ce4100-hpet", }, + { } +}; + +U_BOOT_DRIVER(hpet_timer) = { + .name = "hpet_timer", + .id = UCLASS_TIMER, + .of_match = hpet_timer_ids, + .ofdata_to_platdata = hpet_timer_ofdata_to_platdata, + .priv_auto_alloc_size = sizeof(struct hpet_timer_priv), + .probe = hpet_timer_probe, + .ops = &hpet_timer_ops, + .flags = DM_FLAG_PRE_RELOC, +}; diff --git a/drivers/timer/tsc_timer.c b/drivers/timer/tsc_timer.c index 98cbf12..8fc7130 100644 --- a/drivers/timer/tsc_timer.c +++ b/drivers/timer/tsc_timer.c @@ -277,6 +277,8 @@ success: return delta / 1000; }
+#if defined(CONFIG_X86_EARLY_TIMER_TSC) + static ulong get_ms_timer(void) { if (gd->arch.clock_rate == 0) @@ -322,6 +324,8 @@ void __udelay(unsigned long usec) #endif }
+#endif /* CONFIG_X86_EARLY_TIMER_TSC */ + static int tsc_timer_get_count(struct udevice *dev, u64 *count) { u64 now_tick = rdtsc(); @@ -373,6 +377,8 @@ static int tsc_timer_probe(struct udevice *dev) return 0; }
+#if defined(CONFIG_X86_EARLY_TIMER_TSC) + int timer_init(void) { tsc_timer_ensure_setup(); @@ -392,6 +398,8 @@ u64 notrace timer_early_get_count(void) return rdtsc() - gd->arch.tsc_base; }
+#endif + static const struct timer_ops tsc_timer_ops = { .get_count = tsc_timer_get_count, };
participants (4)
-
Andy Shevchenko
-
Bin Meng
-
Ivan Gorinov
-
Simon Glass