isp: separate writes by type

Author: eiln

src/isp.c

@@ -1,6 +1,7 @@
 /* SPDX-License-Identifier: MIT */
 
 #include "adt.h"
+#include "dart.h"
 #include "pmgr.h"
 #include "utils.h"
 
@@ -21,6 +22,27 @@ struct dart_tunables {
     u64 set;
 };
 
+static void isp_ctrr_init_t8020(u64 base, const struct dart_tunables *config, u32 length)
+{
+    /* DART error handler gets stuck w/o these */
+    write32(base + DART_T8020_ENABLED_STREAMS, 0x1);
+    write32(base + 0x2f0, 0x0);
+    write32(base + DART_T8020_STREAM_SELECT, 0xffffffff);
+    write32(base + DART_T8020_STREAM_COMMAND, DART_T8020_STREAM_COMMAND_INVALIDATE);
+
+    /* I think these lock CTRR? Configurable __TEXT read-only region? */
+    int count = length / sizeof(*config);
+    for (int i = 0; i < count; i++) {
+        u64 offset = config->offset & 0xffff;
+        u32 set = config->set & 0xffffffff;
+        mask32(base + offset, read32(base + offset), set);
+        config++;
+    }
+
+    write32(base + DART_T8020_TCR_OFF, DART_T8020_TCR_TRANSLATE_ENABLE);
+    write32(base + 0x13c, 0x20000);
+}
+
 int isp_init(void)
 {
     int err = 0;
@@ -37,6 +59,23 @@ int isp_init(void)
         return -1;
     }
 
+    enum dart_type_t type;
+    const char *type_s;
+    if (adt_is_compatible(adt, node, "dart,t8020")) {
+        type = DART_T8020;
+        type_s = "t8020";
+    } else if (adt_is_compatible(adt, node, "dart,t6000")) {
+        type = DART_T6000;
+        type_s = "t6000";
+    } else if (adt_is_compatible(adt, node, "dart,t8110")) {
+        type = DART_T8110;
+        type_s = "t8110";
+    } else {
+        printf("isp: dart %s is of an unknown type\n", dart_path);
+        return -1;
+    }
+    printf("isp: found dart type: %s\n", type_s);
+
     int dart_domain_count = 3; // TODO get from dt
     for (int index = 0; index < dart_domain_count; index++) {
         u64 base;
@@ -58,23 +97,15 @@ int isp_init(void)
         if (err < 0)
             goto out;
 
-        /* DART error handler gets stuck w/o these */
-        write32(base + DART_T8020_ENABLED_STREAMS, 0x1);
-        write32(base + 0x2f0, 0x0);
-        write32(base + DART_T8020_STREAM_SELECT, 0xffffffff);
-        write32(base + DART_T8020_STREAM_COMMAND, DART_T8020_STREAM_COMMAND_INVALIDATE);
-
-        /* I think these lock CTRR? Coproc __TEXT read-only region? */
-        int count = length / sizeof(*config);
-        for (int i = 0; i < count; i++) {
-            u64 offset = config->offset & 0xffff;
-            u32 set = config->set & 0xffffffff;
-            mask32(base + offset, read32(base + offset), set);
-            config++;
+        switch (type) {
+            case DART_T8020:
+                isp_ctrr_init_t8020(base, config, length);
+                break;
+            case DART_T8110:
+                break;
+            case DART_T6000:
+                break;
         }
-
-        write32(base + DART_T8020_TCR_OFF, DART_T8020_TCR_TRANSLATE_ENABLE);
-        write32(base + 0x13c, 0x20000);
     }
 
 out: