[rtems commit] bsp/riscv: Rework clock driver
Sebastian Huber
sebh at rtems.org
Fri Jun 29 09:57:43 UTC 2018
Module: rtems
Branch: master
Commit: cdfed94f347a112638b914876de8069e10adf175
Changeset: http://git.rtems.org/rtems/commit/?id=cdfed94f347a112638b914876de8069e10adf175
Author: Sebastian Huber <sebastian.huber at embedded-brains.de>
Date: Mon Jun 25 11:12:24 2018 +0200
bsp/riscv: Rework clock driver
Use device tree provided timebase frequency. Do not write to read-only
mtime register.
Update #3433.
---
bsps/include/bsp/fatal.h | 7 +-
bsps/riscv/riscv/clock/clockdrv.c | 108 +++++++++++++++++++------------
bsps/riscv/riscv/include/bsp.h | 22 -------
bsps/riscv/riscv/include/dev/irq/clint.h | 53 +++++++++++++++
4 files changed, 125 insertions(+), 65 deletions(-)
diff --git a/bsps/include/bsp/fatal.h b/bsps/include/bsp/fatal.h
index ca2e39a..12d33a2 100644
--- a/bsps/include/bsp/fatal.h
+++ b/bsps/include/bsp/fatal.h
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2012, 2016 embedded brains GmbH. All rights reserved.
+ * Copyright (c) 2012, 2018 embedded brains GmbH. All rights reserved.
*
* embedded brains GmbH
* Dornierstr. 4
@@ -137,7 +137,10 @@ typedef enum {
ATSAM_FATAL_PIO_CONFIGURE_IT,
/* i.MX fatal codes */
- IMX_FATAL_GENERIC_TIMER_FREQUENCY = BSP_FATAL_CODE_BLOCK(12)
+ IMX_FATAL_GENERIC_TIMER_FREQUENCY = BSP_FATAL_CODE_BLOCK(12),
+
+ /* RISC-V fatal codes */
+ RISCV_FATAL_NO_TIMEBASE_FREQUENCY_IN_DEVICE_TREE = BSP_FATAL_CODE_BLOCK(13)
} bsp_fatal_code;
RTEMS_NO_RETURN static inline void
diff --git a/bsps/riscv/riscv/clock/clockdrv.c b/bsps/riscv/riscv/clock/clockdrv.c
index 340e158..8be2037 100644
--- a/bsps/riscv/riscv/clock/clockdrv.c
+++ b/bsps/riscv/riscv/clock/clockdrv.c
@@ -7,8 +7,7 @@
*/
/*
- * riscv_generic Clock driver
- *
+ * Copyright (c) 2018 embedded brains GmbH
* COPYRIGHT (c) 2015 Hesham Alatary <hesham at alumni.york.ac.uk>
*
* Redistribution and use in source and binary forms, with or without
@@ -33,75 +32,102 @@
* SUCH DAMAGE.
*/
-#include <rtems.h>
+#include <rtems/timecounter.h>
+
#include <bsp.h>
+#include <bsp/fatal.h>
+#include <bsp/fdt.h>
#include <bsp/irq.h>
-#include <rtems/score/riscv-utility.h>
-#include <rtems/score/cpu.h>
-#include <rtems/timecounter.h>
-/* The number of clock cycles before generating a tick timer interrupt. */
-#define TTMR_NUM_OF_CLOCK_TICKS_INTERRUPT 1000
-#define RISCV_CLOCK_CYCLE_TIME_NANOSECONDS 1
+#include <dev/irq/clint.h>
-static struct timecounter riscv_generic_tc;
+#include <libfdt.h>
-/* CPU counter */
-static CPU_Counter_ticks cpu_counter_ticks;
+#define CLINT ((volatile clint_regs *) 0x02000000)
-/* This prototype is added here to Avoid warnings */
+/* This is defined in dev/clock/clockimpl.h */
void Clock_isr(void *arg);
-static void riscv_generic_clock_at_tick(void)
+static struct timecounter riscv_clock_tc;
+
+static uint32_t riscv_clock_interval;
+
+static void riscv_clock_at_tick(void)
{
- REG(MTIME_MM) = 0;
- REG(MTIMECMP_MM) = TTMR_NUM_OF_CLOCK_TICKS_INTERRUPT;
+ volatile clint_regs *clint;
+ uint64_t cmp;
+
+ clint = CLINT;
+
+ cmp = clint->mtimecmp[0].val_64;
+ cmp += riscv_clock_interval;
- cpu_counter_ticks += TTMR_NUM_OF_CLOCK_TICKS_INTERRUPT * 10000;
+#if __riscv_xlen == 32
+ clint->mtimecmp[0].val_32[0] = 0xffffffff;
+ clint->mtimecmp[0].val_32[1] = (uint32_t) (cmp >> 32);
+ clint->mtimecmp[0].val_32[0] = (uint32_t) cmp;
+#elif __riscv_xlen == 64
+ clint->mtimecmp[0].val_64 = cmp;
+#endif
}
-static void riscv_generic_clock_handler_install(proc_ptr new_isr)
+static void riscv_clock_handler_install(proc_ptr new_isr)
{
- rtems_status_code sc = RTEMS_SUCCESSFUL;
_CPU_ISR_install_vector(RISCV_MACHINE_TIMER_INTERRUPT,
new_isr,
NULL);
-
- if (sc != RTEMS_SUCCESSFUL) {
- rtems_fatal_error_occurred(0xdeadbeef);
- }
}
-static uint32_t riscv_generic_get_timecount(struct timecounter *tc)
+static uint32_t riscv_clock_get_timecount(struct timecounter *tc)
{
- uint32_t ticks_since_last_timer_interrupt = REG(MTIME_MM);
+ volatile clint_regs *clint;
- return cpu_counter_ticks + ticks_since_last_timer_interrupt;
+ clint = CLINT;
+ return clint->mtime.val_32[0];
}
CPU_Counter_ticks _CPU_Counter_read(void)
{
- return riscv_generic_get_timecount(NULL);
+ return riscv_clock_get_timecount(NULL);
+}
+
+static uint32_t riscv_clock_get_timebase_frequency(const void *fdt)
+{
+ int node;
+ const uint32_t *val;
+ int len;
+
+ node = fdt_path_offset(fdt, "/cpus");
+ val = fdt_getprop(fdt, node, "timebase-frequency", &len);
+ if (val == NULL || len < 4) {
+ bsp_fatal(RISCV_FATAL_NO_TIMEBASE_FREQUENCY_IN_DEVICE_TREE);
+ }
+
+ return fdt32_to_cpu(*val);
}
-static void riscv_generic_clock_initialize(void)
+static void riscv_clock_initialize(void)
{
- uint32_t mtimecmp = TTMR_NUM_OF_CLOCK_TICKS_INTERRUPT;
- uint64_t frequency = (1000000000 / RISCV_CLOCK_CYCLE_TIME_NANOSECONDS);
+ const char *fdt;
+ uint32_t tb_freq;
+ uint64_t us_per_tick;
+
+ fdt = bsp_fdt_get();
+ tb_freq = riscv_clock_get_timebase_frequency(fdt);
+ us_per_tick = rtems_configuration_get_microseconds_per_tick();
+ riscv_clock_interval = (uint32_t) ((tb_freq * us_per_tick) / 1000000);
- REG(MTIME_MM) = 0;
- REG(MTIMECMP_MM) = TTMR_NUM_OF_CLOCK_TICKS_INTERRUPT;
+ riscv_clock_at_tick();
/* Enable mtimer interrupts */
set_csr(mie, MIP_MTIP);
- set_csr(mip, MIP_MTIP);
/* Initialize timecounter */
- riscv_generic_tc.tc_get_timecount = riscv_generic_get_timecount;
- riscv_generic_tc.tc_counter_mask = 0xffffffff;
- riscv_generic_tc.tc_frequency = frequency;
- riscv_generic_tc.tc_quality = RTEMS_TIMECOUNTER_QUALITY_CLOCK_DRIVER;
- rtems_timecounter_install(&riscv_generic_tc);
+ riscv_clock_tc.tc_get_timecount = riscv_clock_get_timecount;
+ riscv_clock_tc.tc_counter_mask = 0xffffffff;
+ riscv_clock_tc.tc_frequency = tb_freq;
+ riscv_clock_tc.tc_quality = RTEMS_TIMECOUNTER_QUALITY_CLOCK_DRIVER;
+ rtems_timecounter_install(&riscv_clock_tc);
}
CPU_Counter_ticks _CPU_Counter_difference(
@@ -112,11 +138,11 @@ CPU_Counter_ticks _CPU_Counter_difference(
return second - first;
}
-#define Clock_driver_support_at_tick() riscv_generic_clock_at_tick()
+#define Clock_driver_support_at_tick() riscv_clock_at_tick()
-#define Clock_driver_support_initialize_hardware() riscv_generic_clock_initialize()
+#define Clock_driver_support_initialize_hardware() riscv_clock_initialize()
#define Clock_driver_support_install_isr(isr) \
- riscv_generic_clock_handler_install(isr)
+ riscv_clock_handler_install(isr)
#include "../../../shared/dev/clock/clockimpl.h"
diff --git a/bsps/riscv/riscv/include/bsp.h b/bsps/riscv/riscv/include/bsp.h
index 6298966..5ec916e 100644
--- a/bsps/riscv/riscv/include/bsp.h
+++ b/bsps/riscv/riscv/include/bsp.h
@@ -46,28 +46,6 @@ extern "C" {
#define BSP_FDT_IS_SUPPORTED
-/**
- * @defgroup riscv_generic Register Definitions
- *
- * @ingroup riscv_generic
- *
- * @brief Shared register definitions for RISC-V systems.
- *
- * @{
- */
-
-/**
- * @name Register Macros
- *
- * @{
- */
-
-#define REG(x) (*((volatile unsigned long *) (x)))
-#define BIT(n) (1 << (n))
-
-#define MTIME_MM 0x000000000200bff8
-#define MTIMECMP_MM 0x0000000002004000
-
#ifdef __cplusplus
}
#endif
diff --git a/bsps/riscv/riscv/include/dev/irq/clint.h b/bsps/riscv/riscv/include/dev/irq/clint.h
new file mode 100644
index 0000000..6be3145
--- /dev/null
+++ b/bsps/riscv/riscv/include/dev/irq/clint.h
@@ -0,0 +1,53 @@
+/*
+ * Copyright (c) 2018 embedded brains GmbH
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+
+#include <stdint.h>
+
+#ifndef DEV_IRQ_CLINT_H
+#define DEV_IRQ_CLINT_H
+
+#ifdef __cplusplus
+extern "C" {
+#endif /* __cplusplus */
+
+typedef struct {
+ uint32_t msip[4096];
+ union {
+ uint64_t val_64;
+ uint32_t val_32[2];
+ } mtimecmp[2048];
+ uint32_t reserved_8000[4094];
+ union {
+ uint64_t val_64;
+ uint32_t val_32[2];
+ } mtime;
+ uint32_t reserved_c000[4096];
+} clint_regs;
+
+#ifdef __cplusplus
+}
+#endif /* __cplusplus */
+
+#endif /* DEV_IRQ_CLINT_H */
More information about the vc
mailing list