board: starfive: visionfive2: Order board detection logic to match config
authorE Shattow <e@freeshell.de>
Tue, 22 Apr 2025 06:49:17 +0000 (23:49 -0700)
committerLeo Yu-Chi Liang <ycliang@andestech.com>
Fri, 25 Apr 2025 09:04:09 +0000 (17:04 +0800)
Refactor inside-out EEPROM-checking logic to better match the board-seeking
callback and ordered list of targets from starfive_visionfive2_config since
the JH7110 OF_UPSTREAM migration.

Signed-off-by: E Shattow <e@freeshell.de>
Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com>
board/starfive/visionfive2/spl.c

index 9a3081e..0d4d5cc 100644 (file)
@@ -116,33 +116,29 @@ void board_init_f(ulong dummy)
 #if CONFIG_IS_ENABLED(LOAD_FIT)
 int board_fit_config_name_match(const char *name)
 {
-       const char *product_id;
-       u8 version;
-
-       product_id = get_product_id_from_eeprom();
-
-       /* Strip off prefix */
-       if (strncmp(name, "starfive/", 9))
-               return -EINVAL;
-       name += 9;
-       if (!strncmp(product_id, "FML13V01", 8) &&
-           !strcmp(name, "jh7110-deepcomputing-fml13v01")) {
+       if (!strcmp(name, "starfive/jh7110-deepcomputing-fml13v01") &&
+           !strncmp(get_product_id_from_eeprom(), "FML13V01", 8)) {
                return 0;
-       } else if (!strncmp(product_id, "VF7110", 6)) {
-               version = get_pcb_revision_from_eeprom();
-               if ((version == 'b' || version == 'B') &&
-                   !strcmp(name, "jh7110-starfive-visionfive-2-v1.3b"))
-                       return 0;
-
-               if ((version == 'a' || version == 'A') &&
-                   !strcmp(name, "jh7110-starfive-visionfive-2-v1.2a"))
-                       return 0;
-       } else if (!strncmp(product_id, "MARS", 4) &&
-                  !strcmp(name, "jh7110-milkv-mars")) {
+       } else if (!strcmp(name, "starfive/jh7110-milkv-mars") &&
+                  !strncmp(get_product_id_from_eeprom(), "MARS", 4)) {
                return 0;
-       } else if (!strncmp(product_id, "STAR64", 6) &&
-                  !strcmp(name, "jh7110-pine64-star64")) {
+       } else if ((!strcmp(name, "starfive/jh7110-pine64-star64")) &&
+                   !strncmp(get_product_id_from_eeprom(), "STAR64", 6)) {
                return 0;
+       } else if ((!strcmp(name, "starfive/jh7110-starfive-visionfive-2-v1.2a")) &&
+                   !strncmp(get_product_id_from_eeprom(), "VF7110", 6)) {
+               switch (get_pcb_revision_from_eeprom()) {
+               case 'a':
+               case 'A':
+                       return 0;
+               }
+       } else if ((!strcmp(name, "starfive/jh7110-starfive-visionfive-2-v1.2b")) &&
+                   !strncmp(get_product_id_from_eeprom(), "VF7110", 6)) {
+               switch (get_pcb_revision_from_eeprom()) {
+               case 'b':
+               case 'B':
+                       return 0;
+               }
        }
 
        return -EINVAL;