From: Simon Glass <sjg@chromium.org>
Date: Sun, 31 Jul 2022 02:57:05 +0000 (-0600)
Subject: dtoc: Tidy up fdt_tests RunTestCoverage() args
X-Git-Tag: v2025.01-rc5-pxa1908~1315^2~6
X-Git-Url: http://git.dujemihanovic.xyz/img/static/git-favicon.png?a=commitdiff_plain;h=25980791b1848df7df6c273dc5578837cc84f853;p=u-boot.git

dtoc: Tidy up fdt_tests RunTestCoverage() args

Pass the options args in rather than using the global various. Use snake
case and fix up comments to make pylint happy.

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

diff --git a/tools/dtoc/test_fdt.py b/tools/dtoc/test_fdt.py
index 3baf4437cd..ec257552ef 100755
--- a/tools/dtoc/test_fdt.py
+++ b/tools/dtoc/test_fdt.py
@@ -768,10 +768,14 @@ class TestFdtUtil(unittest.TestCase):
             tools.outdir= old_outdir
 
 
-def RunTestCoverage():
-    """Run the tests and check that we get 100% coverage"""
+def run_test_coverage(build_dir):
+    """Run the tests and check that we get 100% coverage
+
+    Args:
+        build_dir (str): Directory containing the build output
+    """
     test_util.run_test_coverage('tools/dtoc/test_fdt.py', None,
-            ['tools/patman/*.py', '*test_fdt.py'], options.build_dir)
+            ['tools/patman/*.py', '*test_fdt.py'], build_dir)
 
 
 def RunTests(args):
@@ -811,4 +815,4 @@ if options.test:
     ret_code = RunTests(args)
     sys.exit(ret_code)
 elif options.test_coverage:
-    RunTestCoverage()
+    run_test_coverage(options.build_dir)