[manuf] add delay to OTP writes

DO NOT MERGE; FOR TESTING ONLY Allow charge pump to recover on real silicon. Signed-off-by: Tim Trippel <[email protected]>
timothytrippel · Dec 18, 2024 · fc4d672 · fc4d672
1 parent 26f093b
commit fc4d672
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/sw/device/lib/testing/BUILD b/sw/device/lib/testing/BUILD
@@ -306,6 +306,7 @@ cc_library(
     target_compatible_with = [OPENTITAN_CPU],
     deps = [
         "//sw/device/lib/dif:otp_ctrl",
+        "//sw/device/lib/runtime:hart",
         "//sw/device/lib/runtime:ibex",
         "//sw/device/lib/runtime:log",
         "//sw/device/lib/testing/test_framework:check",

diff --git a/sw/device/lib/testing/otp_ctrl_testutils.c b/sw/device/lib/testing/otp_ctrl_testutils.c
@@ -5,6 +5,7 @@
 #include "sw/device/lib/testing/otp_ctrl_testutils.h"
 
 #include "sw/device/lib/dif/dif_otp_ctrl.h"
+#include "sw/device/lib/runtime/hart.h"
 #include "sw/device/lib/runtime/ibex.h"
 #include "sw/device/lib/runtime/log.h"
 #include "sw/device/lib/testing/test_framework/check.h"
@@ -173,6 +174,9 @@ status_t otp_ctrl_testutils_dai_write32(const dif_otp_ctrl_t *otp,
     if (read_data != buffer[i]) {
       return INTERNAL();
     }
+
+    // Wait for charge pump to recover.
+    busy_spin_micros(10000);  // 10ms
   }
   return OK_STATUS();
 }
@@ -194,6 +198,9 @@ status_t otp_ctrl_testutils_dai_write64(const dif_otp_ctrl_t *otp,
     if (read_data != buffer[i]) {
       return INTERNAL();
     }
+
+    // Wait for charge pump to recover.
+    busy_spin_micros(10000);  // 10ms
   }
   return OK_STATUS();
 }