[silicon_creator,ibex] Expand ibex unittest

Properly test the mcycles and rnd32 functions.

Signed-off-by: Amaury Pouly <[email protected]>

Author: pamaury

sw/device/silicon_creator/lib/drivers/BUILD

@@ -292,7 +292,7 @@ cc_test(
     name = "ibex_unittest",
     srcs = ["ibex_unittest.cc"],
     deps = [
-        ":ibex",
+        dual_cc_device_library_of(":ibex"),
         "//sw/device/silicon_creator/testing:rom_test",
         "@googletest//:gtest_main",
     ],

sw/device/silicon_creator/lib/drivers/ibex_unittest.cc

@@ -8,12 +8,22 @@
 
 #include "gtest/gtest.h"
 #include "sw/device/lib/base/mock_abs_mmio.h"
+#include "sw/device/silicon_creator/lib/base/mock_csr.h"
 #include "sw/device/silicon_creator/lib/base/mock_sec_mmio.h"
 #include "sw/device/silicon_creator/testing/rom_test.h"
 
 #include "hw/top/rv_core_ibex_regs.h"
 #include "hw/top_earlgrey/sw/autogen/top_earlgrey.h"
 
+namespace rom_test {
+/* This is somewhat of a hack: ibex_time_to_cycles is really just
+ * an alias for to_cpu_cycles so there is no point in testing it or
+ * creating a mock just for this. But we need the symbol for the
+ * linker to be happy.
+ */
+extern "C" uint64_t to_cpu_cycles(uint64_t usec) { abort(); }
+}  // namespace rom_test
+
 namespace ibex_unittest {
 namespace {
 
@@ -22,8 +32,47 @@ class IbexTest : public rom_test::RomTest {
   uint32_t base_ = TOP_EARLGREY_RV_CORE_IBEX_CFG_BASE_ADDR;
   rom_test::MockSecMmio sec_;
   rom_test::MockAbsMmio mmio_;
+  mock_csr::MockCsr csr_;
 };
 
+class MCycleTest : public IbexTest {};
+
+TEST_F(MCycleTest, MCycle32) {
+  EXPECT_CSR_READ(CSR_REG_MCYCLE, 978465);
+  EXPECT_EQ(ibex_mcycle32(), 978465);
+}
+
+TEST_F(MCycleTest, MCycleSimple) {
+  // Simple case where the two reads of MCYCLEH are consistent.
+  EXPECT_CSR_READ(CSR_REG_MCYCLEH, 0x42);
+  EXPECT_CSR_READ(CSR_REG_MCYCLE, 0xabcdef01);
+  EXPECT_CSR_READ(CSR_REG_MCYCLEH, 0x42);
+  EXPECT_EQ(ibex_mcycle(), 0x42abcdef01);
+}
+
+TEST_F(MCycleTest, MCycleComplex) {
+  // Complex case where the two reads of MCYCLEH are different.
+  EXPECT_CSR_READ(CSR_REG_MCYCLEH, 0x42);
+  EXPECT_CSR_READ(CSR_REG_MCYCLE, 0xabcdef01);
+  EXPECT_CSR_READ(CSR_REG_MCYCLEH, 0x43);
+  EXPECT_CSR_READ(CSR_REG_MCYCLEH, 0x44);
+  EXPECT_CSR_READ(CSR_REG_MCYCLE, 0x01234567);
+  EXPECT_CSR_READ(CSR_REG_MCYCLEH, 0x44);
+  EXPECT_EQ(ibex_mcycle(), 0x4401234567);
+}
+
+class RndTest : public IbexTest {};
+
+TEST_F(RndTest, Rnd32) {
+  EXPECT_ABS_READ32(base_ + RV_CORE_IBEX_RND_STATUS_REG_OFFSET,
+                    {{RV_CORE_IBEX_RND_STATUS_RND_DATA_VALID_BIT, false}});
+  EXPECT_ABS_READ32(base_ + RV_CORE_IBEX_RND_STATUS_REG_OFFSET,
+                    {{RV_CORE_IBEX_RND_STATUS_RND_DATA_VALID_BIT, true}});
+  EXPECT_ABS_READ32(base_ + RV_CORE_IBEX_RND_DATA_REG_OFFSET, 12345);
+
+  EXPECT_EQ(ibex_rnd32_read(), 12345);
+}
+
 class AddressTranslationTest : public IbexTest {};
 
 TEST_F(AddressTranslationTest, Slot0Sucess) {