The check for managed bootloader support should be in regions and we need a try/catch when we call it.

Moved "bootloader_not_supported" check to where it was and handle that exception at only one place.

Removed ram/rom size info for realtek from targets.json. THe info we have is not correct.

was Not handling config exceptions from regions and ram_regions property

adding rom-ram info for REALTEK_RTL8195AM

Author: aashishc1988

platform/mbed_stats.c

@@ -131,10 +131,14 @@ void mbed_stats_sys_get(mbed_stats_sys_t *stats)
 
 #if defined(MBED_SYS_STATS_ENABLED)
     stats->os_version = MBED_VERSION;
+#if defined(MBED_RAM_START) && defined(MBED_RAM_SIZE)
     stats->ram_start[0] = MBED_RAM_START;
     stats->ram_size[0] = MBED_RAM_SIZE;
+#endif
+#if defined(MBED_ROM_START) && defined(MBED_ROM_SIZE)
     stats->rom_start[0] = MBED_ROM_START;
     stats->rom_size[0] = MBED_ROM_SIZE;
+#endif
 #if defined(MBED_RAM1_START) && defined(MBED_RAM1_SIZE)
     stats->ram_start[1] = MBED_RAM1_START;
     stats->ram_size[1] = MBED_RAM1_SIZE;

tools/config/__init__.py

@@ -630,7 +630,7 @@ def get_all_active_memories(self, memory_list):
         # Counter to keep track of ROM/RAM memories supported by target
         active_memory_counter = 0
         # Find which memory we are dealing with, RAM/ROM
-        active_memory = 'RAM' if any('RAM' in mem_list for mem_list in memory_list) else 'ROM'
+        active_memory = 'ROM' if any('ROM' in mem_list for mem_list in memory_list) else 'RAM'
 
         try:
             cmsis_part = self._get_cmsis_part()
@@ -699,9 +699,9 @@ def ram_regions(self):
 
     @property
     def regions(self):
-        """Generate a list of regions from the config"""
         if not getattr(self.target, "bootloader_supported", False):
             raise ConfigException("Bootloader not supported on this target.")
+        """Generate a list of regions from the config"""
         if  ((self.target.bootloader_img or self.target.restrict_size) and
              (self.target.mbed_app_start or self.target.mbed_app_size)):
             raise ConfigException(

tools/toolchains/__init__.py

@@ -724,19 +724,26 @@ def add_regions(self):
         """Add regions to the build profile, if there are any.
         """
         if self.config.has_regions:
-            regions = list(self.config.regions)
-            self.notify.info("Using ROM region%s %s in this build." % (
-                "s" if len(regions) > 1 else "",
-                ", ".join(r.name for r in regions)
-            ))
-            self._add_all_regions(regions, "MBED_APP")
+            try:
+                regions = list(self.config.regions)
+                self.notify.info("Using ROM region%s %s in this build." % (
+                    "s" if len(regions) > 1 else "",
+                    ", ".join(r.name for r in regions)
+                ))
+                self._add_all_regions(regions, "MBED_APP")
+            except ConfigException:
+                pass           
+        
         if self.config.has_ram_regions:
-            regions = list(self.config.ram_regions)
-            self.notify.info("Using RAM region%s %s in this build." % (
-                "s" if len(regions) > 1 else "",
-                ", ".join(r.name for r in regions)
-            ))
-            self._add_all_regions(regions, "MBED_RAM")
+            try:
+                regions = list(self.config.ram_regions)
+                self.notify.info("Using RAM region%s %s in this build." % (
+                    "s" if len(regions) > 1 else "",
+                    ", ".join(r.name for r in regions)
+                ))
+                self._add_all_regions(regions, "MBED_RAM")
+            except ConfigException:
+                pass           
 
         Region = namedtuple("Region", "name start size")