Muh print statements!!11!1

Author: multiplemonomials

tools/python/mbed_tools/build/_internal/memory_banks.py

@@ -130,6 +130,7 @@ def _print_mem_bank_summary(banks_by_type: BanksByType, configured_banks_by_type
     :param banks_by_type: Physical memory bank information
     :param configured_banks_by_type: Configured memory bank information
     """
+    print("Summary of available memory banks:")
     for bank_type in BANK_TYPES:
         banks = banks_by_type[bank_type]
 
@@ -142,18 +143,30 @@ def _print_mem_bank_summary(banks_by_type: BanksByType, configured_banks_by_type
             )
             continue
 
-        for _bank_index, (bank_name, bank_data) in enumerate(banks.items()):
+        print(f"Target {bank_type} banks: -----------------------------------------------------------")
+
+        for bank_index, (bank_name, bank_data) in enumerate(banks.items()):
             bank_size = bank_data["size"]
             bank_start = bank_data["start"]
 
             configured_size = configured_banks_by_type[bank_type][bank_name]["size"]
             configured_start_addr = configured_banks_by_type[bank_type][bank_name]["start"]
 
             # If the configured sizes are different, add info to the summary
+            configured_size_str = ""
+            configured_start_addr_str = ""
             if configured_size != bank_size:
-                f" (configured to {humanize.naturalsize(configured_size, binary=True)})"
+                configured_size_str = f" (configured to {humanize.naturalsize(configured_size, binary=True)})"
             if configured_start_addr != bank_start:
-                pass
+                configured_start_addr_str = f" (configured to 0x{configured_start_addr:08x})"
+
+            print(
+                f"{bank_index}. {bank_name}, "
+                f"start addr 0x{bank_start:08x}{configured_start_addr_str}, "
+                f"size {humanize.naturalsize(bank_size, binary=True)}{configured_size_str}"
+            )
+
+        print()
 
 
 def _generate_macros_for_memory_banks(banks_by_type: BanksByType, configured_banks_by_type: BanksByType) -> set[str]:

tools/python/mbed_tools/cli/cmsis_mcu_descr.py

@@ -18,6 +18,7 @@
 from __future__ import annotations
 
 import datetime
+import json
 import logging
 import os
 import pathlib
@@ -162,8 +163,12 @@ def find_unused() -> None:
     removable_mcus = sorted(available_mcu_names - used_mcu_names)
 
     if len(removable_mcus) == 0:
+        print("No MCU descriptions can be pruned, all are used.")
         return
 
+    print("The following MCU descriptions are not used and should be pruned from cmsis_mcu_descriptions.json5")
+    print("\n".join(removable_mcus))
+
     sys.exit(1)
 
 
@@ -181,8 +186,13 @@ def check_missing() -> None:
     # Are there any missing?
     missing_mcu_names = used_mcu_names - available_mcu_names
     if len(missing_mcu_names) == 0:
+        print("No missing MCUs, no work to do.")
         return
 
+    print(
+        "The following MCU descriptions are used by targets.json5 and need to be added to cmsis_mcu_descriptions.json5:"
+    )
+    print("\n".join(missing_mcu_names))
     sys.exit(1)
 
 
@@ -232,4 +242,6 @@ def fetch_missing() -> None:
         "just the 'memories' section as 'memory_banks' section from content below.\n"
         f"Otherwise add the whole following entries to {CMSIS_MCU_DESCRIPTIONS_JSON_PATH}:"
     )
+    print(json.dumps(missing_mcus_dict, indent=4, sort_keys=True))
+
     sys.exit(1)