scripts: west_commands: Fix for flash into ram.

When users try to debug while flashing into ram,
JLink/Linkserver would fail to flash straight to ram.
Each with their own issues. Both would reset after
load which would flush the ram and point the PC to flash.
Linkserver treated the ram as inaccessible, and the added
memory region in board.cmake has the incorrect size for the
region and is already included inside linkserver.

Signed-off-by: Emilio Benavente <emilio.benavente@nxp.com>

Author: EmilioCBen

boards/nxp/mimxrt1170_evk/board.cmake

@@ -6,10 +6,7 @@
 
 if(CONFIG_SOC_MIMXRT1176_CM7 OR CONFIG_SECOND_CORE_MCUX)
  board_runner_args(pyocd "--target=mimxrt1170_cm7")
- board_runner_args(jlink "--device=MIMXRT1176xxxA_M7" "--reset-after-load")
- # ITCM is not defined in RT1170's LinkServer device file
- board_runner_args(linkserver "--override=/device/memory/-=\{\"location\":\"0x00000000\",\
-                               \"size\":\"0x00040000\",\"type\":\"RAM\"\}")
+ board_runner_args(jlink "--device=MIMXRT1176xxxA_M7")
 
  if(${BOARD_REVISION} STREQUAL "A")
   board_runner_args(linkserver  "--device=MIMXRT1176xxxxx:MIMXRT1170-EVK")

scripts/west_commands/run_common.py

@@ -439,7 +439,7 @@ def do_run_common_image(command, user_args, user_runner_args, used_cmds,
     # the board has enabled this functionality, check if the board should be
     # reset or not. If this is not specified in the board/soc file, leave it up to
     # the runner's default configuration to decide if a reset should occur.
-    if runner_cls.capabilities().reset and '--no-reset' not in final_argv:
+    if runner_cls.capabilities().reset and '--no-reset' not in final_argv and len(board_image_count):
         if board_image_count is not None:
             reset = True
 

scripts/west_commands/runners/linkserver.py

@@ -150,7 +150,10 @@ def do_run(self, command, **kwargs):
                            ['-ex', f'target remote {self.gdb_host}:{self.gdb_port}'])
 
                 if command == 'debug':
-                    gdb_cmd += [ '-ex', 'load', '-ex', 'monitor reset']
+                    # If the flash node points to ram, linserver treats
+                    # the ram as inaccessible and does not flash.
+                    gdb_cmd += ['-ex', 'set mem inaccessible-by-default off']
+                    gdb_cmd += ['-ex', 'monitor reset', '-ex', 'load']
 
                 if command == 'attach':
                     linkserver_cmd += ['--attach']