commit a68488ccea32bb39b28cd669e0943c336c8d06bd
author Simon Glass <sjg@chromium.org> Thu Feb 09 16:53:47 2023 -0700
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Feb 10 22:01:45 2023 +0000
tree f5ad9bf975b8aabd50fec312bbb036676d9ce561
parent d20333a9c3bf1b15d520faccc3e5a551a0374c82

scripts: crosfw: Switch target and -b args

The target argument is hardly ever used. Move it to -T and use the
board as the positional parameter.

BUG=b:268384689
TEST=crosfw coreboot
See that it builds OK

Change-Id: Id927ccc8a92c9d73d9161ba12f9cb71229811d54
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4237214
Commit-Queue: Simon Glass <sjg@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>

scripts/crosfw.py

diff --git a/scripts/crosfw.py b/scripts/crosfw.py
index 76e9792..0f9ddda 100644
--- a/scripts/crosfw.py
+++ b/scripts/crosfw.py
@@ -234,13 +234,6 @@
     """
     parser = commandline.ArgumentParser(description=__doc__)
     parser.add_argument(
-        "-b",
-        "--board",
-        type=str,
-        default=default_board,
-        help="Select board to build (daisy/peach_pit/link)",
-    )
-    parser.add_argument(
         "-B",
         "--build",
         action="store_false",
@@ -275,6 +268,13 @@
         help="Enable trace support",
     )
     parser.add_argument(
+        "-T",
+        "--target",
+        nargs="?",
+        default="all",
+        help="Select target to build",
+    )
+    parser.add_argument(
         "-V",
         "--verified",
         action="store_true",
@@ -289,7 +289,10 @@
         help="Display U-Boot image size",
     )
     parser.add_argument(
-        "target", nargs="?", default="all", help="The target to work on"
+        "board",
+        type=str,
+        default=default_board,
+        help="Select board to build (daisy/peach_pit/link)",
     )
     return parser.parse_args(argv)