diff options
author | Simon Glass | 2023-07-19 17:48:56 -0600 |
---|---|---|
committer | Simon Glass | 2023-07-24 09:34:11 -0600 |
commit | ea78233d8737d8e154a4b4429d11cc8ac4a836f8 (patch) | |
tree | 322afb535aa68d854f6c98cdca6ff14374360905 /tools | |
parent | 985d7ae4de9dfd8eaee190218f8c8744be42668d (diff) |
buildman: Adjust show_toolchain_prefix() to not return
This function does not need to return. Simplify the code by exiting
immediately.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/buildman/control.py | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/tools/buildman/control.py b/tools/buildman/control.py index f30b8beae82..5327929444b 100644 --- a/tools/buildman/control.py +++ b/tools/buildman/control.py @@ -121,10 +121,9 @@ def show_toolchain_prefix(brds, toolchains): for brd in board_selected.values(): tc_set.add(toolchains.Select(brd.arch)) if len(tc_set) != 1: - return 'Supplied boards must share one toolchain' + sys.exit('Supplied boards must share one toolchain') tchain = tc_set.pop() print(tchain.GetEnvArgs(toolchain.VAR_CROSS_COMPILE)) - return None def get_allow_missing(opt_allow, opt_no_allow, num_selected, has_branch): """Figure out whether to allow external blobs @@ -557,9 +556,7 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None, brds, args, col, options.boards, options.exclude) if options.print_prefix: - err = show_toolchain_prefix(brds, toolchains) - if err: - sys.exit(col.build(col.RED, err)) + show_toolchain_prefix(brds, toolchains) return 0 series = determine_series(selected, col, git_dir, options.count, |