scripts: runners: nrf: Add a new --dry-run parameter

In order to allow for users to invoke "west flash" without actual
hardware connected but still running the logic and pregeneration of
commands.

Signed-off-by: Carles Cufi <carles.cufi@nordicsemi.no>

Author: carlescufi

scripts/west_commands/runners/nrf_common.py

@@ -53,7 +53,7 @@ class NrfBinaryRunner(ZephyrBinaryRunner):
 
     def __init__(self, cfg, family, softreset, pinreset, dev_id, erase=False,
                  erase_mode=None, ext_erase_mode=None, reset=True,
-                 tool_opt=None, force=False, recover=False):
+                 tool_opt=None, force=False, recover=False, dry_run=False):
         super().__init__(cfg)
         self.hex_ = cfg.hex_file
         # The old --nrf-family options takes upper-case family names
@@ -67,6 +67,7 @@ def __init__(self, cfg, family, softreset, pinreset, dev_id, erase=False,
         self.reset = bool(reset)
         self.force = force
         self.recover = bool(recover)
+        self.dry_run = bool(dry_run)
 
         self.tool_opt = []
         if tool_opt is not None:
@@ -118,6 +119,10 @@ def do_add_parser(cls, parser):
                             choices=['none', 'ranges', 'all'],
                             help='Select the type of erase operation for the '
                                  'external non-volatile memory')
+        parser.add_argument('--dry-run', required=False,
+                            action='store_true',
+                            help='''Generate all the commands without actually
+                            executing them''')
 
         parser.set_defaults(reset=True)
 
@@ -139,7 +144,10 @@ def ensure_snr(self):
                 self.dev_id = [d.lstrip("0") for d in dev_id]
                 return
         if not dev_id or "*" in dev_id:
-            dev_id = self.get_board_snr(dev_id or "*")
+            if not self.dry_run:
+                dev_id = self.get_board_snr(dev_id or "*")
+            else:
+                dev_id = "DEVICEID" # for a dry run
         self.dev_id = dev_id.lstrip("0")
 
     @abc.abstractmethod

scripts/west_commands/runners/nrfjprog.py

@@ -19,11 +19,11 @@ class NrfJprogBinaryRunner(NrfBinaryRunner):
 
     def __init__(self, cfg, family, softreset, pinreset, dev_id, erase=False,
                  erase_mode=None, ext_erase_mode=None, reset=True, tool_opt=None,
-                 force=False, recover=False, qspi_ini=None):
+                 force=False, recover=False, qspi_ini=None, dry_run=False):
 
         super().__init__(cfg, family, softreset, pinreset, dev_id, erase,
                          erase_mode, ext_erase_mode, reset, tool_opt, force,
-                         recover)
+                         recover, dry_run)
 
         self.qspi_ini = qspi_ini
 
@@ -51,7 +51,7 @@ def do_create(cls, cfg, args):
                                     ext_erase_mode=args.ext_erase_mode,
                                     reset=args.reset, tool_opt=args.tool_opt,
                                     force=args.force, recover=args.recover,
-                                    qspi_ini=args.qspi_ini)
+                                    qspi_ini=args.qspi_ini, dry_run=args.dry_run)
     @classmethod
     def do_add_parser(cls, parser):
         super().do_add_parser(parser)

scripts/west_commands/runners/nrfutil.py

@@ -17,12 +17,12 @@ class NrfUtilBinaryRunner(NrfBinaryRunner):
 
     def __init__(self, cfg, family, softreset, pinreset, dev_id, erase=False,
                  erase_mode=None, ext_erase_mode=None, reset=True, tool_opt=None,
-                 force=False, recover=False,
-                 ext_mem_config_file=None):
+                 force=False, recover=False, ext_mem_config_file=None,
+                 dry_run=False):
 
         super().__init__(cfg, family, softreset, pinreset, dev_id, erase,
                          erase_mode, ext_erase_mode, reset, tool_opt, force,
-                         recover)
+                         recover, dry_run)
 
         self.ext_mem_config_file = ext_mem_config_file
 
@@ -54,7 +54,8 @@ def do_create(cls, cfg, args):
                                    ext_erase_mode=args.ext_erase_mode,
                                    reset=args.reset, tool_opt=args.tool_opt,
                                    force=args.force, recover=args.recover,
-                                   ext_mem_config_file=args.ext_mem_config_file)
+                                   ext_mem_config_file=args.ext_mem_config_file,
+                                   dry_run=args.dry_run)
 
     @classmethod
     def do_add_parser(cls, parser):
@@ -63,13 +64,13 @@ def do_add_parser(cls, parser):
                             dest='ext_mem_config_file',
                             help='path to an JSON file with external memory configuration')
 
-    def _exec(self, args):
+    def _exec(self, args, force=False):
         jout_all = []
 
         cmd = ['nrfutil', '--json', 'device'] + args
         self._log_cmd(cmd)
 
-        if self.dry_run:
+        if self.dry_run and not force:
             return {}
 
         with subprocess.Popen(cmd, stdout=subprocess.PIPE) as p:
@@ -147,7 +148,7 @@ def _append_batch(self, op, json_file):
         cmd += ['--core', op['core']] if op.get('core') else []
         cmd += ['--x-family', f'{self.family}']
         cmd += ['--x-append-batch', f'{json_file}']
-        self._exec(cmd)
+        self._exec(cmd, force=True)
 
     def _exec_batch(self):
         # Use x-append-batch to get the JSON from nrfutil itself