From 7c498b904337e54799cb992adc22272be72fbc19 Mon Sep 17 00:00:00 2001
From: James Buren <braewoods+rb@braewoods.net>
Date: Fri, 30 Oct 2020 02:14:43 +0000
Subject: iriver_flash: revise valid_bootloaders and detect_valid_bootloader

The main change is revising how the checksums are guarded by macros.
But both are also converted to static linkage so they can be better
optimized by GCC. I also change the types around to reflect how the
space the data types actually need. Furthermore I make use of C99
changes to how variables can be declared to move them closer to where
they are used.

Change-Id: I0b21b655f3f4a7c4bbd4365a384a551e75451159
---
 apps/plugins/iriver_flash.c | 36 ++++++++++++++++--------------------
 1 file changed, 16 insertions(+), 20 deletions(-)

(limited to 'apps/plugins/iriver_flash.c')

diff --git a/apps/plugins/iriver_flash.c b/apps/plugins/iriver_flash.c
index 449bf349a8..a9cf62d3c4 100644
--- a/apps/plugins/iriver_flash.c
+++ b/apps/plugins/iriver_flash.c
@@ -66,11 +66,11 @@ enum sections {
 static volatile uint16_t* FB = (uint16_t*)0x00000000; /* Flash base address */
 #endif
 
-#ifdef IRIVER_H100
+#ifdef IRIVER_H100 /* iRiver H110/H115 */
 #define MODEL "h100"
-#elif defined(IRIVER_H120)
+#elif defined(IRIVER_H120) /* iRiver H120/H140 */
 #define MODEL "h120"
-#elif defined(IRIVER_H300)
+#elif defined(IRIVER_H300) /* iRiver H320/H340 */
 #define MODEL "h300"
 #endif
 
@@ -307,33 +307,29 @@ static off_t load_firmware_file(const char* filename, uint32_t* out_checksum)
     return len;
 }
 
-unsigned long valid_bootloaders[][2] = {
-    /* Size-8   CRC32 */
-#ifdef IRIVER_H120 /* Iriver H120/H140 checksums */
+static const uint32_t valid_bootloaders[][2] = {
+    /* Size-8, CRC32 */
+#ifdef IRIVER_H100 /* iRiver H110/H115 */
+    { 48760, 0x2efc3323 }, /* 7-pre4 */
+    { 56896, 0x0cd8dad4 }, /* 7-pre5 */
+#elif defined(IRIVER_H120) /* iRiver H120/H140 */
     { 63788, 0x08ff01a9 }, /* 7-pre3, improved failsafe functions */
     { 48764, 0xc674323e }, /* 7-pre4. Fixed audio thump & remote bootup */
     { 56896, 0x167f5d25 }, /* 7-pre5, various ATA fixes */
+#elif defined(IRIVER_H300) /* iRiver H320/H340 */
 #endif
-#ifdef IRIVER_H100
-    { 48760, 0x2efc3323 }, /* 7-pre4 */
-    { 56896, 0x0cd8dad4 }, /* 7-pre5 */
-#endif
-    { 0,     0 }
+    {}
 };
 
-
-bool detect_valid_bootloader(const unsigned char *addr, int len)
+/* check if the bootloader is a known good one */
+static bool detect_valid_bootloader(const uint8_t* pAddr, uint32_t len)
 {
-    int i;
-    unsigned long crc32;
-
-    /* Try to scan through all valid bootloaders. */
-    for (i = 0; valid_bootloaders[i][0]; i++)
+    for (size_t i = 0; valid_bootloaders[i][0]; i++)
     {
-        if (len > 0 && len != (long)valid_bootloaders[i][0])
+        if (len != valid_bootloaders[i][0])
             continue;
 
-        crc32 = rb->crc_32(addr, valid_bootloaders[i][0], 0xffffffff);
+        uint32_t crc32 = rb->crc_32(pAddr, valid_bootloaders[i][0], 0xffffffff);
         if (crc32 == valid_bootloaders[i][1])
             return true;
     }
-- 
cgit v1.2.3