fix failed spi flash detection

fallback to 4 pages when SPI IC is unknown some code styling remove some unused code
RfidResearchGroup · Nov 18, 2024 · 68e354d · 68e354d
1 parent 52c46db
commit 68e354d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 17 deletions.
diff --git a/common_arm/flashmem.c b/common_arm/flashmem.c
@@ -455,35 +455,24 @@ void Flashmem_print_info(void) {
 }
 
 //read spi flash JEDEC ID and fill the global variable spi_flash_p64k
-bool FlashDetect(bool flash_init) {
+bool FlashDetect(void) {
     flash_device_type_t flash_device = {0};
 
-    if (flash_init) {
-        if (!FlashInit()) {
-            if (g_dbglevel > 3) Dbprintf("FlashDetect() FlashInit fail");
-            return false;
-        }
-    }
-
     if (!Flash_ReadID(&flash_device, true)) {
         if (g_dbglevel > 3) Dbprintf("Flash_ReadID failed");
         return false;
     }
 
-    uint32_t identifier = (flash_device.manufacturer_id <<16) + (flash_device.device_id <<8) +  flash_device.device_id2;
+    uint32_t identifier = (flash_device.manufacturer_id << 16) + (flash_device.device_id <<8 ) + flash_device.device_id2;
     int i = 0;
-    for (; i < ARRAYLEN(SpiFlashTable); i++) {
+    for (; i < ARRAYLEN(SpiFlashTable)-1; i++) {
         if (SpiFlashTable[i].identifier == identifier) {
             break;
         }
     }
 
     spi_flash_p64k = SpiFlashTable[i].pages64;
 
-    if (flash_init) {
-        FlashStop();
-    }
-
     return true;
 }
 
@@ -503,7 +492,7 @@ bool FlashInit(void) {
 
 #ifndef AS_BOOTROM
     if (spi_flash_p64k == 0) {
-        if (!FlashDetect(false)) {
+        if (!FlashDetect()) {
             return false;
         }
     }

diff --git a/common_arm/flashmem.h b/common_arm/flashmem.h
@@ -151,7 +151,6 @@ typedef struct spi_flash_s {
     const char    *desc;
 } spi_flash_t;
 
-// spi_flash_t is expected to be NULL terminated
 const static spi_flash_t SpiFlashTable[] = {
     // Manufacturer: Puya
     { 0x856015, 32, "P25Q16H" },
@@ -165,6 +164,7 @@ const static spi_flash_t SpiFlashTable[] = {
     { 0xEF4016, 64, "W25Q32BV" },
 
     { 0xEF7022,  4, "W25Q02JV" },
+    // (default) last record
     { 0x000000,  4, "Unknown!" }
 };
 
@@ -174,7 +174,7 @@ const static spi_flash_t SpiFlashTable[] = {
 
 extern uint8_t spi_flash_p64k;
 
-bool FlashDetect(bool);
+bool FlashDetect(void);
 
 #endif // #ifndef AS_BOOTROM