cleanup comments

Author: Grazfather

core/src/cpus/cortex_m.zig

@@ -578,17 +578,13 @@ var ram_vectors: [vector_count]usize align(256) = undefined;
 pub fn ram_image_entrypoint() linksection(".entry") callconv(.naked) void {
     asm volatile (
         \\
-        // Get address of vector table
+        // Set VTOR to point to ram table
         \\mov r0, %[_vector_table]
-        // Get address of VTOR register
         \\mov r1, %[_VTOR_ADDRESS]
-        // Set VTOR to point to ram table
         \\str r0, [r1]
-        // Load stack address and entry point
+        // Set up stack and jump to _start
         \\ldm r0!, {r1, r2}
-        // Set sp
         \\msr msp, r1
-        // Branch to _start (reset vector)
         \\bx r2
         :
         : [_vector_table] "r" (&startup_logic._vector_table),
@@ -630,7 +626,6 @@ pub const startup_logic = struct {
         }
 
         // Move vector table to RAM if requested
-        // TODO: Make sure that this isn't set for ram image
         if (interrupt.has_ram_vectors()) {
             // Copy vector table to RAM and set VTOR to point to it
 
@@ -689,6 +684,7 @@ fn is_ramimage() bool {
         return (std.mem.containsAtLeast(u8, board_name, 1, "ram image"));
     return false;
 }
+
 pub fn export_startup_logic() void {
     if (is_ramimage())
         @export(&ram_image_entrypoint, .{

port/raspberrypi/rp2xxx/build.zig

@@ -131,26 +131,22 @@ pub fn init(dep: *std.Build.Dependency) Self {
     };
 
     const bootrom_rp2040 = get_bootrom(b, &chip_rp2040, .w25q080);
-    // Create a "bootloader" 'import', but it's actucally used to embed the file.
-    // This file is just the flash xip bis?
-    // But then why is the PICO_NO_FLASH relevant in the bootloader?
     const rp2040_bootrom_imports = b.allocator.dupe(std.Build.Module.Import, &.{
         .{ .name = "bootloader", .module = b.createModule(.{ .root_source_file = bootrom_rp2040 }) },
     }) catch @panic("out of memory");
 
     return .{
         .chips = .{
             .rp2040 = chip_rp2040.derive(.{}),
-            // This should probably be a board? Here it's a mix of a flash-less pico, so rp2040 on its own, but with an external oscillator
+            // This should probably be a board? Here it's a mix of a flash-less pico, so rp2040 on
+            // its own, but with an external oscillator
             .rp2040_ram_image = chip_rp2040.derive(.{
                 .linker_script = b.path("rp2040_ram_image.ld"),
                 .board = .{
                     .name = "RaspberryPi Pico (ram image)",
                     .url = "https://www.raspberrypi.com/products/raspberry-pi-pico/",
-                    // Using pico2 because it doesn't need a bootloader, and we don't have flash for one
                     .root_source_file = b.path("src/boards/raspberry_pi_pico2.zig"),
                 },
-                // TODO: Support overriding the memory regions? It's only used for linker generation?
             }),
             .rp2350_arm = chip_rp2350_arm.derive(.{}),
             .rp2350_riscv = chip_rp2350_riscv.derive(.{}),