Changed weak target family descriptors to references.

- This causes the g_families[] entries to be NULL if the family descriptor
  is not defined. So the g_families terminator was changed to all Fs
  and init_families() updated appropriately.
- This changed saves quite a bit of .rodata.

Author: flit

source/target/target_family.c

@@ -43,31 +43,21 @@ const target_family_descriptor_t g_sw_sysresetreq_family = {
     .soft_reset_type = SYSRESETREQ, 
 };
 
-//Weakly define family
-__attribute__((weak))
-const target_family_descriptor_t g_nxp_kinetis_kseries = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_nxp_kinetis_lseries = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_nxp_kinetis_k32w_series = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_nxp_mimxrt = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_nxp_rapid_iot = {0};
-__attribute__((weak))
-const target_family_descriptor_t  g_nordic_nrf51  = {0};
-__attribute__((weak))
-const target_family_descriptor_t  g_nordic_nrf52  = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_realtek_rtl8195am  = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_ti_family  = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_wiznet_family  = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_renesas_family  = {0};
-__attribute__((weak))
-const target_family_descriptor_t g_toshiba_tz_family  = {0};
+// Weak references to family definitions.
+extern __WEAK const target_family_descriptor_t g_nxp_kinetis_kseries;
+extern __WEAK const target_family_descriptor_t g_nxp_kinetis_lseries;
+extern __WEAK const target_family_descriptor_t g_nxp_kinetis_k32w_series;
+extern __WEAK const target_family_descriptor_t g_nxp_mimxrt;
+extern __WEAK const target_family_descriptor_t g_nxp_rapid_iot;
+extern __WEAK const target_family_descriptor_t g_nordic_nrf51;
+extern __WEAK const target_family_descriptor_t g_nordic_nrf52;
+extern __WEAK const target_family_descriptor_t g_realtek_rtl8195am;
+extern __WEAK const target_family_descriptor_t g_ti_family;
+extern __WEAK const target_family_descriptor_t g_wiznet_family;
+extern __WEAK const target_family_descriptor_t g_renesas_family;
+extern __WEAK const target_family_descriptor_t g_toshiba_tz_family;
+
+#define FAMILY_LIST_TERMINATOR ((const target_family_descriptor_t *)(0xffffffff))
 
 __attribute__((weak))
 const target_family_descriptor_t *g_families[] = { 
@@ -86,30 +76,34 @@ const target_family_descriptor_t *g_families[] = {
     &g_wiznet_family,
     &g_renesas_family,
     &g_toshiba_tz_family,
-    0 // list terminator 
-}; 
+    FAMILY_LIST_TERMINATOR // list terminator
+};
 
 __attribute__((weak))
 const target_family_descriptor_t *g_target_family = NULL;
 
 
 void init_family(void)
 {
-    uint8_t index = 0;
-    uint16_t family_id = get_family_id();
-    if (g_target_family != NULL){ //already set
+    // Check if the family is already set.
+    if (g_target_family != NULL) {
         return;
     }
-    
-    while (g_families[index]!=0) {
-        if (g_families[index]->family_id && (g_families[index]->family_id == family_id)) {
+
+    // Scan families table looking for matching family ID.
+    uint8_t index = 0;
+    uint16_t family_id = get_family_id();
+
+    while (g_families[index] != FAMILY_LIST_TERMINATOR) {
+        if ((g_families[index] != NULL) && (g_families[index]->family_id == family_id)) {
             g_target_family = g_families[index];
             break;
         }
         index++;
     }
-    
-    if(g_target_family == NULL){ //default family
+
+    // Last resort is to use a default family.
+    if (g_target_family == NULL) {
         g_target_family = &g_hw_reset_family;
     }
 }
@@ -138,11 +132,11 @@ uint8_t target_set_state(TARGET_RESET_STATE state)
                     swd_set_soft_reset(g_target_family->soft_reset_type);
                 }
                 return swd_set_target_state_sw(state);
-            }else {
+            } else {
                 return 1;
             } 
         }
-    }else{
+    } else {
         return 0;
     }
 }
@@ -151,7 +145,7 @@ void swd_set_target_reset(uint8_t asserted)
 {
     if (g_target_family && g_target_family->swd_set_target_reset) {
         g_target_family->swd_set_target_reset(asserted);
-    }else {
+    } else {
         (asserted) ? PIN_nRESET_OUT(0) : PIN_nRESET_OUT(1);
     }
 }