From: Simon Glass <sjg@chromium.org>
Date: Wed, 17 Jul 2024 15:57:11 +0000 (+0100)
Subject: qconfig: Move operation check into parse_args()
X-Git-Url: http://git.dujemihanovic.xyz/posts?a=commitdiff_plain;h=c57d406134d6c1cd6a5471df04ef27b4ff28d1ff;p=u-boot.git

qconfig: Move operation check into parse_args()

Put the check for an operation being provided into the parse_args()
function, to reduce the size of main().

Signed-off-by: Simon Glass <sjg@chromium.org>
---

diff --git a/tools/qconfig.py b/tools/qconfig.py
index dd24e4b398..408807931f 100755
--- a/tools/qconfig.py
+++ b/tools/qconfig.py
@@ -1552,7 +1552,13 @@ doc/develop/moveconfig.rst for documentation.'''
                       help='show any build errors as boards are built')
     parser.add_argument('configs', nargs='*')
 
-    return parser, parser.parse_args()
+    args = parser.parse_args()
+    if not any((args.force_sync, args.build_db, args.imply, args.find,
+                args.scan_source, args.test)):
+        parser.print_usage()
+        sys.exit(1)
+
+    return parser, args
 
 
 def imply(args):
@@ -1660,11 +1666,6 @@ def do_tests():
 def main():
     """Main program"""
     parser, args = parse_args()
-    if not any((args.force_sync, args.build_db, args.imply, args.find,
-                args.scan_source, args.test)):
-        parser.print_usage()
-        sys.exit(1)
-
     check_top_directory()
 
     # prefix the option name with CONFIG_ if missing