diff options
author | Anup Patel <anup.patel@wdc.com> | 2018-12-26 16:14:54 +0300 |
---|---|---|
committer | Anup Patel <anup@brainfault.org> | 2018-12-27 07:05:09 +0300 |
commit | 52ed1d17092bbe93a34802a55ff19cc7cbb7f85d (patch) | |
tree | d367bfc48a0c787fabd6bd830b9ed68513afaebd /platform/kendryte/k210 | |
parent | c1b6200653c51642e544c66fab8bb320969b818e (diff) | |
download | opensbi-52ed1d17092bbe93a34802a55ff19cc7cbb7f85d.tar.xz |
lib: Simplify sbi_platform timer_init() hooks
Instead of having separate timer_init() hooks for cold and
warm boot, this patch updates struct sbi_platform to have just
one timer_init() hook. The type of boot (cold or warm) is now
a boolean flag parameter for the updated timer_init() hook.
Signed-off-by: Anup Patel <anup.patel@wdc.com>
Diffstat (limited to 'platform/kendryte/k210')
-rw-r--r-- | platform/kendryte/k210/platform.c | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/platform/kendryte/k210/platform.c b/platform/kendryte/k210/platform.c index d11dd66..141c98e 100644 --- a/platform/kendryte/k210/platform.c +++ b/platform/kendryte/k210/platform.c @@ -66,9 +66,18 @@ static int k210_ipi_init(u32 hartid, bool cold_boot) return clint_warm_ipi_init(hartid); } -static int k210_cold_timer_init(void) +static int k210_timer_init(u32 hartid, bool cold_boot) { - return clint_cold_timer_init(CLINT_BASE_ADDR, K210_HART_COUNT); + int rc; + + if (cold_boot) { + rc = clint_cold_timer_init(CLINT_BASE_ADDR, + K210_HART_COUNT); + if (rc) + return rc; + } + + return clint_warm_timer_init(hartid); } static int k210_system_reboot(u32 type) @@ -107,11 +116,10 @@ struct sbi_platform platform = { .ipi_sync = clint_ipi_sync, .ipi_clear = clint_ipi_clear, - .cold_timer_init = k210_cold_timer_init, + .timer_init = k210_timer_init, .timer_value = clint_timer_value, .timer_event_stop = clint_timer_event_stop, .timer_event_start = clint_timer_event_start, - .warm_timer_init = clint_warm_timer_init, .system_reboot = k210_system_reboot, .system_shutdown = k210_system_shutdown |