diff options
| author | Jonghwa Lee <jonghwa3.lee@samsung.com> | 2015-08-07 16:42:26 +0900 |
|---|---|---|
| committer | Seung-Woo Kim <sw0312.kim@samsung.com> | 2016-12-14 13:47:04 +0900 |
| commit | 0f2f3c13e883d7c1aaeea46f29112b43243cfc7b (patch) | |
| tree | df6a50314736aacf3e8d2d8fc3bf112144542740 /drivers/clocksource | |
| parent | d79533c104c68d59075b5cdd1cbe58512d5b8cfc (diff) | |
LOCAL \ clocksource: arch_timer: Use physical cnt instead of virtual one.
This fixes unsynchronized timestamp on kernel log which is occured by
unitialized virtual counter of each CPU. Even ARM commented that to use
virtual counter is mandatory on ARMv8, but it failed on EXYNOS5433.
Let it uses physical counter instead of virtual counter on EXYNOS5433.
Signed-off-by: Jonghwa Lee <jonghwa3.lee@samsung.com>
Diffstat (limited to 'drivers/clocksource')
| -rw-r--r-- | drivers/clocksource/arm_arch_timer.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c index 0aa135ddbf80..7c67e63cfe49 100644 --- a/drivers/clocksource/arm_arch_timer.c +++ b/drivers/clocksource/arm_arch_timer.c @@ -468,7 +468,7 @@ static void __init arch_counter_register(unsigned type) /* Register the CP15 based counter if we have one */ if (type & ARCH_CP15_TIMER) { - if (IS_ENABLED(CONFIG_ARM64) || arch_timer_use_virtual) + if (arch_timer_use_virtual) arch_timer_read_counter = arch_counter_get_cntvct; else arch_timer_read_counter = arch_counter_get_cntpct; @@ -737,6 +737,15 @@ static void __init arch_timer_of_init(struct device_node *np) arch_timer_c3stop = !of_property_read_bool(np, "always-on"); + /* FIXME : + * ARM recommended to use vcnt for clocksource of ARMv8. + * However, On Exynos5433, it seemed that they aren't intialized + * properly, each of counters' value are not synchronized. + * For Exynos5433, let it uses pcnt for representive clock source. + */ + if (of_machine_is_compatible("samsung,exynos5433")) + arch_timer_use_virtual = false; + /* * If we cannot rely on firmware initializing the timer registers then * we should use the physical timers instead. |