west: add native_sim flash command

Add 'west flash' support which in the case of native_sim just start the
built application.
Reuse existing runner and rename it to be more generic as it does more
than just debugging now.

Fixes zephyrproject-rtos#36706

Signed-off-by: Anas Nashif <anas.nashif@intel.com>

boards/posix/native_posix/board.cmake

@@ -2,5 +2,6 @@
 
 set(SUPPORTED_EMU_PLATFORMS native)
 
-board_set_debugger_ifnset(native_gdb)
-board_finalize_runner_args(native_gdb)
+board_set_debugger_ifnset(native)
+board_set_flasher_ifnset(native)
+board_finalize_runner_args(native)

boards/posix/native_sim/board.cmake

@@ -2,5 +2,6 @@
 
 set(SUPPORTED_EMU_PLATFORMS native)
 
-board_set_debugger_ifnset(native_gdb)
-board_finalize_runner_args(native_gdb)
+board_set_debugger_ifnset(native)
+board_set_flasher_ifnset(native)
+board_finalize_runner_args(native)

boards/posix/nrf_bsim/board.cmake

@@ -2,5 +2,6 @@
 
 set(SUPPORTED_EMU_PLATFORMS native)
 
-board_set_debugger_ifnset(native_gdb)
-board_finalize_runner_args(native_gdb)
+board_set_debugger_ifnset(native)
+board_set_flasher_ifnset(native)
+board_finalize_runner_args(native)

scripts/west_commands/runners/__init__.py

@@ -40,7 +40,7 @@ def _import_runner_module(runner_name):
     'linkserver',
     'mdb',
     'misc',
-    'native_gdb',
+    'native',
     'nios2',
     'nrfjprog',
     'nrfutil',

scripts/west_commands/runners/native_gdb.py → scripts/west_commands/runners/native.py

@@ -1,36 +1,47 @@
 # Copyright (c) 2023 Nordic Semiconductor
 # SPDX-License-Identifier: Apache-2.0
 
-"""This file provides a ZephyrBinaryRunner that launches GDB."""
+"""This file provides a ZephyrBinaryRunner that launches GDB and enables
+flashing (running) a native application."""
 
 import argparse
 from runners.core import ZephyrBinaryRunner, RunnerCaps, RunnerConfig
 
-class NativeGDBBinaryRunner(ZephyrBinaryRunner):
+class NativeSimBinaryRunner(ZephyrBinaryRunner):
     """Runs the ELF binary under GDB."""
 
     @classmethod
     def name(cls):
-        return 'native_gdb'
+        return 'native'
 
     @classmethod
     def capabilities(cls):
-        return RunnerCaps(commands={'debug'})
+        return RunnerCaps(commands={'debug', 'flash'})
 
     @classmethod
     def do_add_parser(cls, parser: argparse.ArgumentParser):
         pass
 
     @classmethod
     def do_create(cls, cfg: RunnerConfig, args: argparse.Namespace) -> ZephyrBinaryRunner:
-        return NativeGDBBinaryRunner(cfg)
+        return NativeSimBinaryRunner(cfg)
 
     def do_run(self, command: str, **kwargs):
-        assert command == 'debug'
-
+        if command == 'flash':
+            self.do_flash(**kwargs)
+        elif command == 'debug':
+            self.do_debug(**kwargs)
+        else:
+            assert False
+
+    def do_flash(self, **kwargs):
+        cmd = self.cfg.exe_file
+        self.check_call(cmd)
+
+    def do_debug(self, **kwargs):
         # Clues to debug missing RunnerConfig values (in context of `west debug`):
         #   build/zephyr/runners.yaml is missing `gdb` or `elf_file`.
-        #   board.cmake should have `board_finalize_runner_args(native_gdb)`.
+        #   board.cmake should have `board_finalize_runner_args(native)`.
         #   build/CMakeCache.txt should have `CMAKE_GDB`.
 
         if self.cfg.gdb is None:

scripts/west_commands/tests/test_imports.py

@@ -31,7 +31,7 @@ def test_runner_imports():
                     'mdb-nsim',
                     'mdb-hw',
                     'misc-flasher',
-                    'native_gdb',
+                    'native',
                     'nios2',
                     'nrfjprog',
                     'nrfutil',