2017-10-25 00:29:03 +08:00
|
|
|
# Copyright (c) 2017 Linaro Limited.
|
|
|
|
#
|
|
|
|
# SPDX-License-Identifier: Apache-2.0
|
|
|
|
|
|
|
|
'''Runner for openocd.'''
|
|
|
|
|
|
|
|
from os import path
|
|
|
|
|
scripts: runner: use arguments, not environment vars
The various runners (flash/debug scripts) use environment variables to
take arguments. This is legacy behavior which is not desirable.
Use command line arguments instead.
Note: this leaves more general environment variables with publicly
documented behavior in place for now, for compatibility, e.g.:
ZEPHYR_FLASH_OVER_DFU, OPENSDA_FW, ESP_IDF_PATH, PYOCD_DAPARG
For example, when using dfu-util to flash arduino_101, instead of
setting DFUUTIL_PID, DFUUTIL_ALT, and DFUUTIL_IMG environment
variables, have the script invocation look like this:
python3 .../zephyr_flash_debug.py dfu-util flash \
[common arguments omitted] \
--pid=8087:0aba --alt=x86_app \
--img=.../build/zephyr/zephyr.bin
Make similar changes for other runners (openocd, etc.) and
targets (debug, debugserver).
To implement this in the scripts:
- have the individual scripts/support/runner/some-runner.py files
register their own command line arguments
- teach them to construct instances from arguments, not the
environment
- have zephyr_flash_debug.py request runners to register command
line argument parsers, and handle arguments
In the build system:
- add a new board_runner_args() extension function that board.cmake
files can use to add to the zephyr_flash_debug.py command line
- adjust cmake/flash/CMakeLists.txt to invoke with arguments
- add new helper include files for each runner (like
boards/common/dfu-util.board.cmake, etc.), which add default
options as needed and then add on overrides from
board_runner_args() calls
- update board.cmake files to use the new includes and extension
This implied some tweaking when using openocd to make the CMake string
escaping and unescaping work properly.
Signed-off-by: Marti Bolivar <marti@opensourcefoundries.com>
2017-11-17 06:45:38 +08:00
|
|
|
from .core import ZephyrBinaryRunner
|
2017-10-25 00:29:03 +08:00
|
|
|
|
|
|
|
DEFAULT_OPENOCD_TCL_PORT = 6333
|
|
|
|
DEFAULT_OPENOCD_TELNET_PORT = 4444
|
|
|
|
DEFAULT_OPENOCD_GDB_PORT = 3333
|
|
|
|
|
|
|
|
|
|
|
|
class OpenOcdBinaryRunner(ZephyrBinaryRunner):
|
|
|
|
'''Runner front-end for openocd.'''
|
|
|
|
|
|
|
|
def __init__(self, openocd_config,
|
scripts: runner: use arguments, not environment vars
The various runners (flash/debug scripts) use environment variables to
take arguments. This is legacy behavior which is not desirable.
Use command line arguments instead.
Note: this leaves more general environment variables with publicly
documented behavior in place for now, for compatibility, e.g.:
ZEPHYR_FLASH_OVER_DFU, OPENSDA_FW, ESP_IDF_PATH, PYOCD_DAPARG
For example, when using dfu-util to flash arduino_101, instead of
setting DFUUTIL_PID, DFUUTIL_ALT, and DFUUTIL_IMG environment
variables, have the script invocation look like this:
python3 .../zephyr_flash_debug.py dfu-util flash \
[common arguments omitted] \
--pid=8087:0aba --alt=x86_app \
--img=.../build/zephyr/zephyr.bin
Make similar changes for other runners (openocd, etc.) and
targets (debug, debugserver).
To implement this in the scripts:
- have the individual scripts/support/runner/some-runner.py files
register their own command line arguments
- teach them to construct instances from arguments, not the
environment
- have zephyr_flash_debug.py request runners to register command
line argument parsers, and handle arguments
In the build system:
- add a new board_runner_args() extension function that board.cmake
files can use to add to the zephyr_flash_debug.py command line
- adjust cmake/flash/CMakeLists.txt to invoke with arguments
- add new helper include files for each runner (like
boards/common/dfu-util.board.cmake, etc.), which add default
options as needed and then add on overrides from
board_runner_args() calls
- update board.cmake files to use the new includes and extension
This implied some tweaking when using openocd to make the CMake string
escaping and unescaping work properly.
Signed-off-by: Marti Bolivar <marti@opensourcefoundries.com>
2017-11-17 06:45:38 +08:00
|
|
|
openocd='openocd', search=None,
|
|
|
|
elf_name=None,
|
|
|
|
pre_cmd=None, load_cmd=None, verify_cmd=None, post_cmd=None,
|
2017-10-25 00:29:03 +08:00
|
|
|
tcl_port=DEFAULT_OPENOCD_TCL_PORT,
|
|
|
|
telnet_port=DEFAULT_OPENOCD_TELNET_PORT,
|
|
|
|
gdb_port=DEFAULT_OPENOCD_GDB_PORT,
|
|
|
|
gdb=None, tui=None, debug=False):
|
|
|
|
super(OpenOcdBinaryRunner, self).__init__(debug=debug)
|
|
|
|
self.openocd_config = openocd_config
|
|
|
|
|
|
|
|
search_args = []
|
scripts: runner: use arguments, not environment vars
The various runners (flash/debug scripts) use environment variables to
take arguments. This is legacy behavior which is not desirable.
Use command line arguments instead.
Note: this leaves more general environment variables with publicly
documented behavior in place for now, for compatibility, e.g.:
ZEPHYR_FLASH_OVER_DFU, OPENSDA_FW, ESP_IDF_PATH, PYOCD_DAPARG
For example, when using dfu-util to flash arduino_101, instead of
setting DFUUTIL_PID, DFUUTIL_ALT, and DFUUTIL_IMG environment
variables, have the script invocation look like this:
python3 .../zephyr_flash_debug.py dfu-util flash \
[common arguments omitted] \
--pid=8087:0aba --alt=x86_app \
--img=.../build/zephyr/zephyr.bin
Make similar changes for other runners (openocd, etc.) and
targets (debug, debugserver).
To implement this in the scripts:
- have the individual scripts/support/runner/some-runner.py files
register their own command line arguments
- teach them to construct instances from arguments, not the
environment
- have zephyr_flash_debug.py request runners to register command
line argument parsers, and handle arguments
In the build system:
- add a new board_runner_args() extension function that board.cmake
files can use to add to the zephyr_flash_debug.py command line
- adjust cmake/flash/CMakeLists.txt to invoke with arguments
- add new helper include files for each runner (like
boards/common/dfu-util.board.cmake, etc.), which add default
options as needed and then add on overrides from
board_runner_args() calls
- update board.cmake files to use the new includes and extension
This implied some tweaking when using openocd to make the CMake string
escaping and unescaping work properly.
Signed-off-by: Marti Bolivar <marti@opensourcefoundries.com>
2017-11-17 06:45:38 +08:00
|
|
|
if search is not None:
|
|
|
|
search_args = ['-s', search]
|
2017-10-25 00:29:03 +08:00
|
|
|
self.openocd_cmd = [openocd] + search_args
|
|
|
|
self.elf_name = elf_name
|
|
|
|
self.load_cmd = load_cmd
|
|
|
|
self.verify_cmd = verify_cmd
|
|
|
|
self.pre_cmd = pre_cmd
|
|
|
|
self.post_cmd = post_cmd
|
|
|
|
self.tcl_port = tcl_port
|
|
|
|
self.telnet_port = telnet_port
|
|
|
|
self.gdb_port = gdb_port
|
|
|
|
self.gdb_cmd = [gdb] if gdb is not None else None
|
scripts: runner: use arguments, not environment vars
The various runners (flash/debug scripts) use environment variables to
take arguments. This is legacy behavior which is not desirable.
Use command line arguments instead.
Note: this leaves more general environment variables with publicly
documented behavior in place for now, for compatibility, e.g.:
ZEPHYR_FLASH_OVER_DFU, OPENSDA_FW, ESP_IDF_PATH, PYOCD_DAPARG
For example, when using dfu-util to flash arduino_101, instead of
setting DFUUTIL_PID, DFUUTIL_ALT, and DFUUTIL_IMG environment
variables, have the script invocation look like this:
python3 .../zephyr_flash_debug.py dfu-util flash \
[common arguments omitted] \
--pid=8087:0aba --alt=x86_app \
--img=.../build/zephyr/zephyr.bin
Make similar changes for other runners (openocd, etc.) and
targets (debug, debugserver).
To implement this in the scripts:
- have the individual scripts/support/runner/some-runner.py files
register their own command line arguments
- teach them to construct instances from arguments, not the
environment
- have zephyr_flash_debug.py request runners to register command
line argument parsers, and handle arguments
In the build system:
- add a new board_runner_args() extension function that board.cmake
files can use to add to the zephyr_flash_debug.py command line
- adjust cmake/flash/CMakeLists.txt to invoke with arguments
- add new helper include files for each runner (like
boards/common/dfu-util.board.cmake, etc.), which add default
options as needed and then add on overrides from
board_runner_args() calls
- update board.cmake files to use the new includes and extension
This implied some tweaking when using openocd to make the CMake string
escaping and unescaping work properly.
Signed-off-by: Marti Bolivar <marti@opensourcefoundries.com>
2017-11-17 06:45:38 +08:00
|
|
|
self.tui_arg = ['-tui'] if tui else []
|
2017-10-25 00:29:03 +08:00
|
|
|
|
2017-11-17 02:32:00 +08:00
|
|
|
@classmethod
|
|
|
|
def name(cls):
|
|
|
|
return 'openocd'
|
2017-10-25 00:29:03 +08:00
|
|
|
|
scripts: runner: use arguments, not environment vars
The various runners (flash/debug scripts) use environment variables to
take arguments. This is legacy behavior which is not desirable.
Use command line arguments instead.
Note: this leaves more general environment variables with publicly
documented behavior in place for now, for compatibility, e.g.:
ZEPHYR_FLASH_OVER_DFU, OPENSDA_FW, ESP_IDF_PATH, PYOCD_DAPARG
For example, when using dfu-util to flash arduino_101, instead of
setting DFUUTIL_PID, DFUUTIL_ALT, and DFUUTIL_IMG environment
variables, have the script invocation look like this:
python3 .../zephyr_flash_debug.py dfu-util flash \
[common arguments omitted] \
--pid=8087:0aba --alt=x86_app \
--img=.../build/zephyr/zephyr.bin
Make similar changes for other runners (openocd, etc.) and
targets (debug, debugserver).
To implement this in the scripts:
- have the individual scripts/support/runner/some-runner.py files
register their own command line arguments
- teach them to construct instances from arguments, not the
environment
- have zephyr_flash_debug.py request runners to register command
line argument parsers, and handle arguments
In the build system:
- add a new board_runner_args() extension function that board.cmake
files can use to add to the zephyr_flash_debug.py command line
- adjust cmake/flash/CMakeLists.txt to invoke with arguments
- add new helper include files for each runner (like
boards/common/dfu-util.board.cmake, etc.), which add default
options as needed and then add on overrides from
board_runner_args() calls
- update board.cmake files to use the new includes and extension
This implied some tweaking when using openocd to make the CMake string
escaping and unescaping work properly.
Signed-off-by: Marti Bolivar <marti@opensourcefoundries.com>
2017-11-17 06:45:38 +08:00
|
|
|
@classmethod
|
|
|
|
def do_add_parser(cls, parser):
|
|
|
|
# Options for flashing:
|
|
|
|
parser.add_argument('--cmd-pre-load',
|
|
|
|
help='Command to run before flashing')
|
|
|
|
parser.add_argument('--cmd-load',
|
|
|
|
help='''Command to load/flash binary
|
|
|
|
(required when flashing)''')
|
|
|
|
parser.add_argument('--cmd-verify',
|
|
|
|
help='''Command to verify flashed binary''')
|
|
|
|
parser.add_argument('--cmd-post-verify',
|
|
|
|
help='Command to run after verification')
|
|
|
|
|
|
|
|
# Options for debugging:
|
|
|
|
parser.add_argument('--tui', default=False, action='store_true',
|
|
|
|
help='if given, GDB uses -tui')
|
|
|
|
parser.add_argument('--tcl-port', default=DEFAULT_OPENOCD_TCL_PORT,
|
|
|
|
help='openocd TCL port, defaults to 6333')
|
|
|
|
parser.add_argument('--telnet-port',
|
|
|
|
default=DEFAULT_OPENOCD_TELNET_PORT,
|
|
|
|
help='openocd telnet port, defaults to 4444')
|
|
|
|
parser.add_argument('--gdb-port', default=DEFAULT_OPENOCD_GDB_PORT,
|
|
|
|
help='openocd gdb port, defaults to 3333')
|
|
|
|
|
|
|
|
@classmethod
|
|
|
|
def create_from_args(cls, args):
|
|
|
|
openocd_config = path.join(args.board_dir, 'support', 'openocd.cfg')
|
|
|
|
|
|
|
|
return OpenOcdBinaryRunner(
|
|
|
|
openocd_config,
|
|
|
|
openocd=args.openocd, search=args.openocd_search,
|
|
|
|
elf_name=args.kernel_elf,
|
|
|
|
pre_cmd=args.cmd_pre_load,
|
|
|
|
load_cmd=args.cmd_load, verify_cmd=args.cmd_verify,
|
|
|
|
post_cmd=args.cmd_post_verify,
|
|
|
|
tcl_port=args.tcl_port, telnet_port=args.telnet_port,
|
|
|
|
gdb_port=args.gdb_port, gdb=args.gdb, tui=args.tui,
|
|
|
|
debug=args.verbose)
|
2017-10-25 00:29:03 +08:00
|
|
|
|
2017-11-16 23:41:39 +08:00
|
|
|
def do_run(self, command, **kwargs):
|
2017-10-25 00:29:03 +08:00
|
|
|
if command == 'flash':
|
|
|
|
self.do_flash(**kwargs)
|
|
|
|
elif command == 'debug':
|
|
|
|
self.do_debug(**kwargs)
|
|
|
|
else:
|
|
|
|
self.do_debugserver(**kwargs)
|
|
|
|
|
|
|
|
def do_flash(self, **kwargs):
|
|
|
|
if self.load_cmd is None:
|
|
|
|
raise ValueError('Cannot flash; load command is missing')
|
|
|
|
if self.verify_cmd is None:
|
|
|
|
raise ValueError('Cannot flash; verify command is missing')
|
|
|
|
|
|
|
|
pre_cmd = []
|
|
|
|
if self.pre_cmd is not None:
|
|
|
|
pre_cmd = ['-c', self.pre_cmd]
|
|
|
|
|
|
|
|
post_cmd = []
|
|
|
|
if self.post_cmd is not None:
|
|
|
|
post_cmd = ['-c', self.post_cmd]
|
|
|
|
|
|
|
|
cmd = (self.openocd_cmd +
|
|
|
|
['-f', self.openocd_config,
|
|
|
|
'-c', 'init',
|
|
|
|
'-c', 'targets'] +
|
|
|
|
pre_cmd +
|
|
|
|
['-c', 'reset halt',
|
|
|
|
'-c', self.load_cmd,
|
|
|
|
'-c', 'reset halt',
|
|
|
|
'-c', self.verify_cmd] +
|
|
|
|
post_cmd +
|
|
|
|
['-c', 'reset run',
|
|
|
|
'-c', 'shutdown'])
|
|
|
|
self.check_call(cmd)
|
|
|
|
|
|
|
|
def do_debug(self, **kwargs):
|
|
|
|
if self.gdb_cmd is None:
|
|
|
|
raise ValueError('Cannot debug; no gdb specified')
|
|
|
|
if self.elf_name is None:
|
|
|
|
raise ValueError('Cannot debug; no .elf specified')
|
|
|
|
|
|
|
|
server_cmd = (self.openocd_cmd +
|
scripts: runner: use arguments, not environment vars
The various runners (flash/debug scripts) use environment variables to
take arguments. This is legacy behavior which is not desirable.
Use command line arguments instead.
Note: this leaves more general environment variables with publicly
documented behavior in place for now, for compatibility, e.g.:
ZEPHYR_FLASH_OVER_DFU, OPENSDA_FW, ESP_IDF_PATH, PYOCD_DAPARG
For example, when using dfu-util to flash arduino_101, instead of
setting DFUUTIL_PID, DFUUTIL_ALT, and DFUUTIL_IMG environment
variables, have the script invocation look like this:
python3 .../zephyr_flash_debug.py dfu-util flash \
[common arguments omitted] \
--pid=8087:0aba --alt=x86_app \
--img=.../build/zephyr/zephyr.bin
Make similar changes for other runners (openocd, etc.) and
targets (debug, debugserver).
To implement this in the scripts:
- have the individual scripts/support/runner/some-runner.py files
register their own command line arguments
- teach them to construct instances from arguments, not the
environment
- have zephyr_flash_debug.py request runners to register command
line argument parsers, and handle arguments
In the build system:
- add a new board_runner_args() extension function that board.cmake
files can use to add to the zephyr_flash_debug.py command line
- adjust cmake/flash/CMakeLists.txt to invoke with arguments
- add new helper include files for each runner (like
boards/common/dfu-util.board.cmake, etc.), which add default
options as needed and then add on overrides from
board_runner_args() calls
- update board.cmake files to use the new includes and extension
This implied some tweaking when using openocd to make the CMake string
escaping and unescaping work properly.
Signed-off-by: Marti Bolivar <marti@opensourcefoundries.com>
2017-11-17 06:45:38 +08:00
|
|
|
['-f', self.openocd_config,
|
|
|
|
'-c', 'tcl_port {}'.format(self.tcl_port),
|
2017-10-25 00:29:03 +08:00
|
|
|
'-c', 'telnet_port {}'.format(self.telnet_port),
|
|
|
|
'-c', 'gdb_port {}'.format(self.gdb_port),
|
|
|
|
'-c', 'init',
|
|
|
|
'-c', 'targets',
|
|
|
|
'-c', 'halt'])
|
|
|
|
|
|
|
|
gdb_cmd = (self.gdb_cmd + self.tui_arg +
|
|
|
|
['-ex', 'target remote :{}'.format(self.gdb_port),
|
|
|
|
self.elf_name])
|
|
|
|
|
|
|
|
self.run_server_and_client(server_cmd, gdb_cmd)
|
|
|
|
|
|
|
|
def do_debugserver(self, **kwargs):
|
|
|
|
cmd = (self.openocd_cmd +
|
|
|
|
['-f', self.openocd_config,
|
|
|
|
'-c', 'init',
|
|
|
|
'-c', 'targets',
|
|
|
|
'-c', 'reset halt'])
|
|
|
|
self.check_call(cmd)
|