scripts: fix various typos

nashif · nashif · commit e836695cc865 · 2026-06-23T09:12:58.000-04:00
Fix various typos under scripts/

Signed-off-by: Anas Nashif &lt;anas.nashif@intel.com&gt;
diff --git a/scripts/ci/test_plan.py b/scripts/ci/test_plan.py
@@ -27,8 +27,9 @@
 if "ZEPHYR_BASE" not in os.environ:
     exit("$ZEPHYR_BASE environment variable undefined.")
 
-# These are globaly used variables. They are assigned in __main__ and are visible in further methods
-# however, pylint complains that it doesn't recognized them when used (used-before-assignment).
+# These are globally used variables. They are assigned in __main__ and are
+# visible in further methods however, pylint complains that it doesn't
+# recognized them when used (used-before-assignment).
 zephyr_base = Path(os.environ['ZEPHYR_BASE'])
 repository_path = zephyr_base
 repo_to_scan = Repo(zephyr_base)
diff --git a/scripts/ci/upload_test_results_es.py b/scripts/ci/upload_test_results_es.py
@@ -7,7 +7,7 @@
 This script uploads ``twister.json`` file to Elasticsearch index for reporting and analysis.
 see  https://kibana.zephyrproject.io/
 
-The script expects two evironment variables with the Elasticsearch server connection parameters:
+The script expects two environment variables with the Elasticsearch server connection parameters:
     `ELASTICSEARCH_SERVER`
     `ELASTICSEARCH_KEY`
 """
diff --git a/scripts/coccinelle/find_dev_usage.cocci b/scripts/coccinelle/find_dev_usage.cocci
@@ -3,7 +3,7 @@
 
 // Uses a python database (a dict) to find where const struct device
 // variable are being used in zephyr functions and, if it's being in place
-// of a void*, it will print an ERROR for loosing the const qualifier.
+// of a void*, it will print an ERROR for losing the const qualifier.
 // If it's being used on an unknown functions from an external module such
 // as a HAL, it will print a WARNING in order to check if the const qualifier
 // is not lost.
@@ -23,7 +23,7 @@ import pickle
 def check_and_report(F, f, D, nb_args, p):
     if f in f_void and int(nb_args) in f_void[f]:
         msg = "ERROR: in {} calling {} param with {}, \
-loosing const qualifier, please wrap".format(F, f, D)
+losing const qualifier, please wrap".format(F, f, D)
         coccilib.report.print_report(p[0], msg)
     elif f not in f_void and f not in f_other and not f.isupper():
         msg = "WARNING: in {} calling {} param with {}, \
diff --git a/scripts/coccinelle/find_functions.cocci b/scripts/coccinelle/find_functions.cocci
@@ -11,7 +11,7 @@
 // of the void* parameter in the first dict, and the actual number of
 // parameters in the second.
 // Then find_dev_usage.cocci can be used to verify if device instance
-// are not loosing their const qualifier.
+// are not losing their const qualifier.
 
 virtual patch
 virtual report
diff --git a/scripts/dts/gen_defines.py b/scripts/dts/gen_defines.py
@@ -833,7 +833,7 @@ def enum_macros(prop: edtlib.Property, macro: str):
     )
 
     for i, subval in enumerate(val):
-        # make sure the subval is a formated right.
+        # make sure the subval is formatted right.
         if isinstance(subval, str):
             subval = str2ident(subval)
         # DT_N_<node-id>_P_<prop-id>_IDX_<i>_EXISTS
diff --git a/scripts/dts/python-devicetree/tests/test_dtlib.py b/scripts/dts/python-devicetree/tests/test_dtlib.py
@@ -2305,7 +2305,7 @@ def test_names():
 /dts-v1/;
 
 / {
-	// A leading \ is accepted but ignored in node/propert names
+	// A leading \ is accepted but ignored in node/property names
 	\aA0,._+*#?- = &_, &{/aA0,._+@-};
 
 	// Names that overlap with operators and integer literals
diff --git a/scripts/dts/python-devicetree/tests/test_edtlib.py b/scripts/dts/python-devicetree/tests/test_edtlib.py
@@ -1447,7 +1447,7 @@ def test_dependencies():
     assert edt.get_node("/in-dir-1") in edt.get_node("/").required_by
 
 def test_child_dependencies():
-    '''Test dependencies relashionship with child nodes propagated to parent'''
+    '''Test dependencies relationship with child nodes propagated to parent'''
     with from_here():
         edt = edtlib.EDT("test.dts", ["test-bindings"])
 
diff --git a/scripts/footprint/pack_as_twister.py b/scripts/footprint/pack_as_twister.py
@@ -11,7 +11,7 @@
 for analysis, visualization, etc.
 
 The memory footprint input data files (rom.json, ram.json) are expected in directories
-sturctured as 'ZEPHYR_VERSION/APPLICATION/FEATURE/BOARD' under the input path(s).
+structured as 'ZEPHYR_VERSION/APPLICATION/FEATURE/BOARD' under the input path(s).
 The BOARD name itself can be in HWMv2 format as 'BOARD/SOC' or 'BOARD/SOC/VARIANT'
 with the corresponding sub-directories.
 
diff --git a/scripts/instrumentation/zaru.py b/scripts/instrumentation/zaru.py
@@ -91,7 +91,7 @@ def generate_reverse_symbol_lookup(addr_to_symbol):
     """Generate a reverse symbol lookup dict.
 
     Given a dict of ELF symbols indexed by the symbol address, return the same
-    dict but indexed by the symbols intead of by the symbol address, allowing
+    dict but indexed by the symbols instead of by the symbol address, allowing
     a reverse lookup, i.e. look up for a symbol address given a symbol.
     """
 
@@ -1047,7 +1047,7 @@ def trace(args):
     # 'data' (binary) file and 'metadata' file written in the TSDL, hence it's not possible to
     # specify an alternative path for 'data' or 'metadata' files. Thus here a temporary dir is
     # created and a copy of the 'metadata' is copied to it together with the binary stream extracted
-    # from the target, which is saved as 'data' file. The tempory dir is then passed to the
+    # from the target, which is saved as 'data' file. The temporary dir is then passed to the
     # babeltrace methods.
     with tempfile.TemporaryDirectory() as tmpdir:
         if args.verbose:
diff --git a/scripts/kconfig/kconfigfunctions.py b/scripts/kconfig/kconfigfunctions.py
@@ -1120,7 +1120,7 @@ def dt_highest_controller_irq_number(kconfig, _, path, irq_cell_name):
 
 def normalize_upper(kconf, _, string):
     """
-    Normalize the string, so that the string only contains alpha-numeric
+    Normalize the string, so that the string only contains alphanumeric
     characters or underscores. All non-alpha-numeric characters are replaced
     with an underscore, '_'.
     When string has been normalized it will be converted into upper case.
diff --git a/scripts/kconfig/kconfiglib.py b/scripts/kconfig/kconfiglib.py
@@ -4409,7 +4409,7 @@ def str_value(self):
                                 num2str(low), num2str(high)))
                 else:
                     # If the user value is well-formed and satisfies range
-                    # contraints, it is stored in exactly the same form as
+                    # constraints, it is stored in exactly the same form as
                     # specified in the assignment (with or without "0x", etc.)
                     val = self.user_value
                     use_defaults = False
diff --git a/scripts/list_hardware.py b/scripts/list_hardware.py
@@ -288,7 +288,7 @@ def dump_v2_archs(args):
                 NAME='NAME;' + arch['name'],
                 DIR='DIR;' + str(arch['path'].as_posix()),
                 HWM='HWM;' + arch['hwm'],
-                # Below is non exising for arch but is defined here to support
+                # Below is non existing for arch but is defined here to support
                 # common formatting string.
                 SERIES='',
                 FAMILY='',
@@ -301,7 +301,7 @@ def dump_v2_archs(args):
                 name=arch.get('name'),
                 dir=arch.get('path'),
                 hwm=arch.get('hwm'),
-                # Below is non exising for arch but is defined here to support
+                # Below is non existing for arch but is defined here to support
                 # common formatting string.
                 series='',
                 family='',
diff --git a/scripts/logging/dictionary/dictionary_parser/log_parser_v3.py b/scripts/logging/dictionary/dictionary_parser/log_parser_v3.py
@@ -4,7 +4,7 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-# Parsers are gonig to have very similar code.
+# Parsers are going to have very similar code.
 # So tell pylint not to care.
 # pylint: disable=duplicate-code
 
diff --git a/scripts/native_simulator/common/src/include/nsi_errno.h b/scripts/native_simulator/common/src/include/nsi_errno.h
@@ -103,7 +103,7 @@
 #define NSI_ERRNO_MID_EOVERFLOW       139 /**< Value overflow */
 #define NSI_ERRNO_MID_ECANCELED       140 /**< Operation canceled */
 
-/* Convert a errno value to the intermediate represetation to pass it to the other side */
+/* Convert a errno value to the intermediate representation to pass it to the other side */
 int nsi_errno_to_mid(int err);
 /* Convert a errno value from the intermediate representation into the local libC value */
 int nsi_errno_from_mid(int err);
diff --git a/scripts/profiling/stackcollapse.py b/scripts/profiling/stackcollapse.py
@@ -44,7 +44,7 @@ def collapse(buf, elf):
         func_trace = reversed(list(map(lambda a: addr_to_sym(a, elf), addrs)))
         prev_func = next(func_trace)
         line = prev_func
-        # merge dublicate functions
+        # merge duplicate functions
         for func in func_trace:
             if prev_func != func:
                 prev_func = func
diff --git a/scripts/pylib/pytest-twister-harness/tests/conftest.py b/scripts/pylib/pytest-twister-harness/tests/conftest.py
@@ -31,7 +31,7 @@ def zephyr_base() -> str:
 
 @pytest.fixture
 def twister_harness(zephyr_base) -> str:
-    """Retrun path to pytest-twister-harness src directory"""
+    """Return path to pytest-twister-harness src directory"""
     pytest_twister_harness_path = str(Path(zephyr_base) / 'scripts' / 'pylib' / 'pytest-twister-harness' / 'src')
     return pytest_twister_harness_path
 
diff --git a/scripts/pylib/twister/twisterlib/reports.py b/scripts/pylib/twister/twisterlib/reports.py
@@ -429,7 +429,7 @@ def create(
 
             for case in instance.testcases:
                 # freeform was set when no sub testcases were parsed, however,
-                # if we discover those at runtime, the fallback testcase wont be
+                # if we discover those at runtime, the fallback testcase won't be
                 # needed anymore and can be removed from the output, it does
                 # not have a status and would otherwise be reported as skipped.
                 if (
diff --git a/scripts/pylib/twister/twisterlib/runner.py b/scripts/pylib/twister/twisterlib/runner.py
@@ -1338,7 +1338,7 @@ def determine_testcases(self, results):
                 # Keep previous statuses and reasons
                 tc_info = tc_keeper.get(testcase_id, {})
                 if not tc_info and self.trace:
-                    # Also happens when Ztest uses macroses, eg. DEFINE_TEST_VARIANT
+                    # Also happens when Ztest uses macros, eg. DEFINE_TEST_VARIANT
                     logger.debug(f"Ztest case '{testcase_id}' discovered for "
                                  f"'{self.instance.testsuite.source_dir_rel}' "
                                  f"with {list(tc_keeper)}")
@@ -2081,9 +2081,9 @@ def run(self):
 
     def update_counting_before_pipeline(self):
         '''
-        Updating counting before pipeline is necessary because statically filterd
+        Updating counting before pipeline is necessary because statically filtered
         test instance never enter the pipeline. While some pipeline output needs
-        the static filter stats. So need to prepare them before pipline starts.
+        the static filter stats. So need to prepare them before pipeline starts.
         '''
         for instance in self.instances.values():
             if instance.status == TwisterStatus.FILTER and instance.reason != 'runtime filter':
diff --git a/scripts/pylib/twister/twisterlib/testinstance.py b/scripts/pylib/twister/twisterlib/testinstance.py
@@ -436,7 +436,7 @@ def get_elf_file(self) -> str:
         fns = glob.glob(os.path.join(build_dir, "zephyr", "*.elf"))
         fns.extend(glob.glob(os.path.join(build_dir, "testbinary")))
         blocklist = [
-                'remapped', # used for xtensa plaforms
+                'remapped', # used for xtensa platforms
                 'zefi', # EFI for Zephyr
                 'qemu', # elf files generated after running in qemu
                 '_pre']
diff --git a/scripts/tests/twister/test_twister_main.py b/scripts/tests/twister/test_twister_main.py
@@ -13,7 +13,7 @@
 
 @pytest.fixture
 def outdir(tmp_path) -> Path:
-    """Return twister output temporay directory."""
+    """Return twister output temporary directory."""
     outdir_path = tmp_path / "twister-out"
     return outdir_path
 
diff --git a/scripts/twister b/scripts/twister
@@ -163,7 +163,7 @@ pairs:
         filter = not ARCH in ["x86", "arc"]
 
     but unlike "filter", these cause platforms to be filtered already during the testplan
-    generation. While "filter" does not exclue platforms at the testplan generation, and instead
+    generation. While "filter" does not exclude platforms at the testplan generation, and instead
     relies on the result of running the build configuration stage. That is, to evaluate the filter
     expression, cmake is run for that target, and then the filter evaluated as a gate for the
     build and run steps.
diff --git a/scripts/utils/convert_guidelines.py b/scripts/utils/convert_guidelines.py
@@ -133,7 +133,7 @@ def convert_guidelines(args):
     )
     parser.add_argument(
         "-f", "--format", metavar="FORMAT", choices=supported_formats, required=True,
-        help="Format to convert guidlines to. Supported formats are: " + str(supported_formats)
+        help="Format to convert guidelines to. Supported formats are: " + str(supported_formats)
     )
     parser.add_argument(
         "-o", "--output", metavar="OUTPUT_FILE", type=Path, required=False,
diff --git a/scripts/west_commands/completion/west-completion.fish b/scripts/west_commands/completion/west-completion.fish
@@ -100,7 +100,7 @@ function __zephyr_west_use_subcommand
                     set -l prv_idx (math $idx - 1)
                     switch $tokens[$prv_idx]
                         # this option can be placed before subcommand and require a folder
-                        # if we don't do that the folder will be catched as a subcommand and
+                        # if we don't do that the folder will be caught as a subcommand and
                         # the subcommands will not be completed
                         case '-z' '--zephyr-base'
                             continue
diff --git a/scripts/west_commands/snippets_cmd.py b/scripts/west_commands/snippets_cmd.py
@@ -51,7 +51,7 @@ def do_add_parser(self, parser_adder):
                 west snippets --format="{name}: {dirs}"
 
             When no format string is given, snippets are listed with their
-            names and descriptions aligned in colums.
+            names and descriptions aligned in columns.
             '''),
         )
 

Original file line number	Diff line number	Diff line change
`@@ -833,7 +833,7 @@ def enum_macros(prop: edtlib.Property, macro: str):`
`833`	`833`	`)`
`834`	`834`
`835`	`835`	`for i, subval in enumerate(val):`
`836`		`- # make sure the subval is a formated right.`
	`836`	`+ # make sure the subval is formatted right.`
`837`	`837`	`if isinstance(subval, str):`
`838`	`838`	`subval = str2ident(subval)`
`839`	`839`	`# DT_N_<node-id>_P_<prop-id>_IDX_<i>_EXISTS`