Skip to content

Commit

Permalink
runner: intel_adsp: no need to specify host in --pty if no port assign
Browse files Browse the repository at this point in the history
No need to specify host again in --pty when not specifying the port,
slightly reduce the input.

Signed-off-by: Enjia Mai <[email protected]>
  • Loading branch information
enjiamai authored and mbolivar-nordic committed Aug 17, 2022
1 parent d935ec3 commit ad8f211
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions scripts/west_commands/runners/intel_adsp.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ def do_add_parser(cls, parser):
help='the default basename of the key store in board.cmake')
parser.add_argument('--key',
help='specify where the signing key is')
parser.add_argument('--pty',
parser.add_argument('--pty', nargs='?', const="remote-host", type=str,
help=''''Capture the output of cavstool.py running on --remote-host \
and stream it remotely to west's standard output.''')

Expand Down Expand Up @@ -147,7 +147,10 @@ def flash(self, **kwargs):
#
# to get the result later separately.
if self.pty is not None:
self.log_cmd = ([f'{self.cavstool}','-s', f'{self.pty}', '-l'])
if self.pty == 'remote-host':
self.log_cmd = ([f'{self.cavstool}','-s', f'{self.remote_host}', '-l'])
else:
self.log_cmd = ([f'{self.cavstool}','-s', f'{self.pty}', '-l'])

self.logger.debug(f"rcmd: {self.log_cmd}")

Expand Down

0 comments on commit ad8f211

Please sign in to comment.