[prev in list] [next in list] [prev in thread] [next in thread]
List: fwts-devel
Subject: ACK: [PATCH] treewide: fix spelling mistakes in comments, literals and tests
From: ivanhu <ivan.hu () canonical ! com>
Date: 2021-04-21 7:37:31
Message-ID: 6bfd873b-4a1c-3f6a-cd25-7eda63658edc () canonical ! com
[Download RAW message or body]
On 4/12/21 8:52 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> There are a handful of trivial spelling mistakes. Fix these.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
> auto-packager/mkpackage.README | 4 ++--
> fwts-test/method-0001/method-0001.log | 2 +-
> fwts-test/sbst-0001/sbst-0001.log | 2 +-
> fwts-test/sbst-0001/sbst-0002.log | 2 +-
> src/acpi/brightness/autobrightness.c | 4 ++--
> src/acpi/brightness/brightness-helper.c | 2 +-
> src/acpi/crsdump/crsdump.c | 2 +-
> src/acpi/gtdt/gtdt.c | 2 +-
> src/acpi/hpet/hpet.c | 4 ++--
> src/acpi/madt/madt.c | 4 ++--
> src/acpi/mcfg/mcfg.c | 2 +-
> src/acpi/method/method.c | 6 +++---
> src/acpi/plddump/plddump.c | 2 +-
> src/acpi/s4/s4.c | 2 +-
> src/acpi/sbst/sbst.c | 2 +-
> src/acpi/slit/slit.c | 2 +-
> src/acpi/uefi/uefi.c | 2 +-
> src/acpi/wpbt/wpbt.c | 2 +-
> src/bios/bios32/bios32.c | 2 +-
> src/bios/mtrr/mtrr.c | 2 +-
> src/cmos/cmosdump/cmosdump.c | 2 +-
> src/cpu/maxfreq/maxfreq.c | 2 +-
> src/dmi/dmicheck/dmicheck.c | 4 ++--
> src/lib/include/fwts_acpi.h | 2 +-
> src/lib/src/fwts_acpi_object_eval.c | 14 +++++++-------
> src/lib/src/fwts_button.c | 2 +-
> src/lib/src/fwts_clog.c | 2 +-
> src/lib/src/fwts_cpu.c | 2 +-
> src/lib/src/fwts_uefi.c | 2 +-
> src/libfwtsacpica/fwts_acpica.c | 2 +-
> src/tpm/tpmevlog/tpmevlog.c | 2 +-
> src/uefi/uefirttime/uefirttime.c | 6 +++---
> 32 files changed, 47 insertions(+), 47 deletions(-)
>
> diff --git a/auto-packager/mkpackage.README b/auto-packager/mkpackage.README
> index f2a2fdc5..2aff0fe6 100644
> --- a/auto-packager/mkpackage.README
> +++ b/auto-packager/mkpackage.README
> @@ -60,7 +60,7 @@ git push
> git push --tags
>
> Next, create a directory for creating the tarball and checksum and run the mktar.sh \
> script.
> -Upload the generated tarball and SHA256SUMS to the desired localtion for the \
> releasing. +Upload the generated tarball and SHA256SUMS to the desired location for \
> the releasing.
> Next, create an auto-packager directory somewhere and run the mkpackage.sh script \
> from within it. E.g
> @@ -82,7 +82,7 @@ one that gets periodically uploaded to universe in the current \
> release until we
> And dput these into the ppa:firmware-testing-team/ppa-fwts-unstable-crack PPA
>
> -Let it shake out - build + test etc. Get users to test against this. Onece you \
> are OK with it, +Let it shake out - build + test etc. Get users to test against \
> this. Once you are OK with it, copy the packages into \
> ppa:firmware-testing-team/ppa-fwts-devel
> Every so often during the distro release cycle copy the stable current package into \
> universe. You can either do this on every version number bump or when you deem that \
> a new version is really stable.
> diff --git a/fwts-test/method-0001/method-0001.log \
> b/fwts-test/method-0001/method-0001.log index 9e6e8ab5..295e8404 100644
> --- a/fwts-test/method-0001/method-0001.log
> +++ b/fwts-test/method-0001/method-0001.log
> @@ -900,7 +900,7 @@ method Test 150 of 207: Test _GAI (Get Averaging \
> Level). method SKIPPED: Test 150, Skipping test for non-existent object
> method _GAI.
> method
> -method Test 151 of 207: Test _GHL (Get Harware Limit).
> +method Test 151 of 207: Test _GHL (Get Hardware Limit).
> method SKIPPED: Test 151, Skipping test for non-existent object
> method _GHL.
> method
> diff --git a/fwts-test/sbst-0001/sbst-0001.log b/fwts-test/sbst-0001/sbst-0001.log
> index ba230460..1df5365f 100644
> --- a/fwts-test/sbst-0001/sbst-0001.log
> +++ b/fwts-test/sbst-0001/sbst-0001.log
> @@ -1,6 +1,6 @@
> sbst sbst: SBST Smart Battery Specification Table test.
> sbst ----------------------------------------------------------
> -sbst Test 1 of 1: SBST Smart Battery Specificiation Table test.
> +sbst Test 1 of 1: SBST Smart Battery Specification Table test.
> sbst PASSED: Test 1, No issues found in SBST table.
> sbst
> sbst ==========================================================
> diff --git a/fwts-test/sbst-0001/sbst-0002.log b/fwts-test/sbst-0001/sbst-0002.log
> index 91a3bef9..ddff1af2 100644
> --- a/fwts-test/sbst-0001/sbst-0002.log
> +++ b/fwts-test/sbst-0001/sbst-0002.log
> @@ -1,6 +1,6 @@
> sbst sbst: SBST Smart Battery Specification Table test.
> sbst ----------------------------------------------------------
> -sbst Test 1 of 1: SBST Smart Battery Specificiation Table test.
> +sbst Test 1 of 1: SBST Smart Battery Specification Table test.
> sbst FAILED [MEDIUM] SBSTEnergeyLevel2: Test 1, SBST Low Energy
> sbst Energy Level (1) is greater than the Warning Energy Level
> sbst (0).
> diff --git a/src/acpi/brightness/autobrightness.c \
> b/src/acpi/brightness/autobrightness.c index a224467a..b346106e 100644
> --- a/src/acpi/brightness/autobrightness.c
> +++ b/src/acpi/brightness/autobrightness.c
> @@ -181,13 +181,13 @@ static int auto_brightness_test2(fwts_framework *fw)
> }
> fwts_failed(fw, LOG_LEVEL_MEDIUM,
> "BrightnessMismatch",
> - "%d brightness levels did not match the brightnesss level "
> + "%d brightness levels did not match the brightness level "
> "just set for backlight %s.",
> failures, entry->d_name);
> fwts_log_info(fw, "The failed brightness levels were:%s.", msg);
> free(msg);
> } else
> - fwts_passed(fw, "Actual brightness matches the brightnesss level for "
> + fwts_passed(fw, "Actual brightness matches the brightness level for "
> "all %d levels for backlight %s.", max_brightness, entry->d_name);
>
> free(brightness_fail);
> diff --git a/src/acpi/brightness/brightness-helper.c \
> b/src/acpi/brightness/brightness-helper.c index b1a4c3a0..ec7c6502 100644
> --- a/src/acpi/brightness/brightness-helper.c
> +++ b/src/acpi/brightness/brightness-helper.c
> @@ -55,7 +55,7 @@ const char *brightness_get_path(void)
> /*
> * brightness_init()
> * generic brightness test init, if successful
> - * it opens a diretory for the /sys interface
> + * it opens a directory for the /sys interface
> */
> int brightness_init(fwts_framework *fw)
> {
> diff --git a/src/acpi/crsdump/crsdump.c b/src/acpi/crsdump/crsdump.c
> index 30cf9dd0..bef2cf35 100644
> --- a/src/acpi/crsdump/crsdump.c
> +++ b/src/acpi/crsdump/crsdump.c
> @@ -242,7 +242,7 @@ static const char *crs_address_space_id(const uint64_t val)
> case 0x7f:
> return "Functional Fixed Hardware";
> default:
> - return "Uknown";
> + return "Unknown";
> }
> }
>
> diff --git a/src/acpi/gtdt/gtdt.c b/src/acpi/gtdt/gtdt.c
> index f5d78946..1642c1e8 100644
> --- a/src/acpi/gtdt/gtdt.c
> +++ b/src/acpi/gtdt/gtdt.c
> @@ -250,7 +250,7 @@ static int gtdt_test1(fwts_framework *fw)
> passed = false;
> fwts_failed(fw, LOG_LEVEL_HIGH,
> "GTDTInvalidType",
> - "GTDT platform timer strucuture %" PRIu32
> + "GTDT platform timer structure %" PRIu32
> " has an invalid type: 0x%" PRIx8, i, *ptr);
> /* Can't determine field length, so end of parsing */
> goto done;
> diff --git a/src/acpi/hpet/hpet.c b/src/acpi/hpet/hpet.c
> index f8240388..9e024757 100644
> --- a/src/acpi/hpet/hpet.c
> +++ b/src/acpi/hpet/hpet.c
> @@ -297,7 +297,7 @@ static int hpet_check_test2(fwts_framework *fw)
> fwts_advice(fw,
> "Bogus HPET base address can be worked around "
> "by using the kernel parameter 'hpet=force' if "
> - "the base addess is 0xfed0000000000000. "
> + "the base address is 0xfed0000000000000. "
> "This will make the kernel shift the address "
> "down 32 bits to 0xfed00000.");
> passed = false;
> @@ -332,7 +332,7 @@ static int hpet_check_test2(fwts_framework *fw)
> (hpet->event_timer_block_id >> 15) & 1);
> fwts_log_info_verbatim(fw, " COUNT_SIZE_CAP counter size: %" PRIu32,
> (hpet->event_timer_block_id >> 13) & 1);
> - fwts_log_info_verbatim(fw, " Number of comparitors : %" PRIu32,
> + fwts_log_info_verbatim(fw, " Number of comparators : %" PRIu32,
> (hpet->event_timer_block_id >> 8) & 0x1f);
> fwts_log_info_verbatim(fw, " Hardwre Revision ID : 0x%" PRIx8,
> hpet->event_timer_block_id & 0xff);
> diff --git a/src/acpi/madt/madt.c b/src/acpi/madt/madt.c
> index e164e5e1..ec99c524 100644
> --- a/src/acpi/madt/madt.c
> +++ b/src/acpi/madt/madt.c
> @@ -49,7 +49,7 @@
> * and made a pact to keep their own counter. Every time the MADT found a
> * new subtable, or a subtable grew up, the wizards promised they would
> * increment their counter. Well, wizards being the forgetful sort, they
> - * didn't alway do that. And, since there quite a lot of them, they
> + * didn't always do that. And, since there quite a lot of them, they
> * couldn't always remember who was supposed to keep track of the MADT,
> * especially if dinner was coming up soon. Their counter was called the
> * spec version.
> @@ -1417,7 +1417,7 @@ static int madt_mp_wakup(fwts_framework *fw,
> if (mp_wakeup->mail_box_address % 0x1000)
> fwts_failed(fw, LOG_LEVEL_HIGH,
> "SPECMADTMPWAKEUPAddress",
> - "MADT %s mailbox adress should be 4K bytes aligned, "
> + "MADT %s mailbox address should be 4K bytes aligned, "
> "but instead have 0x%" PRIx64 ".",
> madt_sub_names[hdr->type], mp_wakeup->mail_box_address);
> else
> diff --git a/src/acpi/mcfg/mcfg.c b/src/acpi/mcfg/mcfg.c
> index a4636f0b..35f07fb3 100644
> --- a/src/acpi/mcfg/mcfg.c
> +++ b/src/acpi/mcfg/mcfg.c
> @@ -201,7 +201,7 @@ static int mcfg_test1(fwts_framework *fw)
> "optionally defined in the %s if ACPI MCFG is "
> "present. Linux checks if the region is reserved "
> "in the memory map table and will reject the "
> - "MMCONFIG if there is a discrepency between MCFG "
> + "MMCONFIG if there is a discrepancy between MCFG "
> "and the memory map table for the PCI Express region. "
> "[See arch/x86/pci/mmconfig-shared.c pci_mmcfg_reject_broken()]. "
> "It is recommended that this is defined in the "
> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
> index 763c3252..1c9203dd 100644
> --- a/src/acpi/method/method.c
> +++ b/src/acpi/method/method.c
> @@ -2207,7 +2207,7 @@ static void method_test_PSS_return(
> "power dissipation setting than the previous "
> "sub-package.", name, i);
> fwts_advice(fw,
> - "_PSS P-States must be ordered in decending "
> + "_PSS P-States must be ordered in descending "
> "order of power dissipation, so that the "
> "zero'th entry has the highest power "
> "dissipation level and the Nth has the "
> @@ -3701,7 +3701,7 @@ static void method_test_THERM_return(
> "hard-coded thermal value which is "
> "out of range because fwts did not "
> "detect any ACPI region handler "
> - "accesses of I/O or system memeory "
> + "accesses of I/O or system memory "
> "to evaluate the thermal value. "
> "It is worth sanity checking these "
> "values in "
> @@ -4934,7 +4934,7 @@ static fwts_framework_minor_test method_tests[] = {
>
> /* Section 10.4 Power Meters */
> { method_test_GAI, "Test _GAI (Get Averaging Level)." },
> - { method_test_GHL, "Test _GHL (Get Harware Limit)." },
> + { method_test_GHL, "Test _GHL (Get Hardware Limit)." },
> /* { method_test_PAI, "Test _PAI (Power Averaging Interval)." }, */
> { method_test_PMC, "Test _PMC (Power Meter Capabilities)." },
> { method_test_PMD, "Test _PMD (Power Meter Devices)." },
> diff --git a/src/acpi/plddump/plddump.c b/src/acpi/plddump/plddump.c
> index cda71bdd..73ec4efe 100644
> --- a/src/acpi/plddump/plddump.c
> +++ b/src/acpi/plddump/plddump.c
> @@ -107,7 +107,7 @@ static const char *shape[] = {
> "Square",
> "Vertical Rectangle",
> "Horizontal Rectangle",
> - "Veritical Trapezoid",
> + "Vertical Trapezoid",
> "Horizontal Trapezoid",
> "Unknown",
> "Chamfered",
> diff --git a/src/acpi/s4/s4.c b/src/acpi/s4/s4.c
> index b622b05a..f92ecc2d 100644
> --- a/src/acpi/s4/s4.c
> +++ b/src/acpi/s4/s4.c
> @@ -280,7 +280,7 @@ static int s4_hibernate(fwts_framework *fw,
>
> if (differences > 0) {
> fwts_failed(fw, LOG_LEVEL_HIGH, "DevConfigDiffAfterS4",
> - "Found %d differences in device configuation during S4 cycle.", differences);
> + "Found %d differences in device configuration during S4 cycle.", differences);
> (*hw_errors)++;
> }
> }
> diff --git a/src/acpi/sbst/sbst.c b/src/acpi/sbst/sbst.c
> index e96857a1..81c8179b 100644
> --- a/src/acpi/sbst/sbst.c
> +++ b/src/acpi/sbst/sbst.c
> @@ -70,7 +70,7 @@ static int sbst_test1(fwts_framework *fw)
> }
>
> static fwts_framework_minor_test sbst_tests[] = {
> - { sbst_test1, "SBST Smart Battery Specificiation Table test." },
> + { sbst_test1, "SBST Smart Battery Specification Table test." },
> { NULL, NULL }
> };
>
> diff --git a/src/acpi/slit/slit.c b/src/acpi/slit/slit.c
> index 7028eaf3..011c718c 100644
> --- a/src/acpi/slit/slit.c
> +++ b/src/acpi/slit/slit.c
> @@ -55,7 +55,7 @@ static int slit_test1(fwts_framework *fw)
> /*
> * ACPI table length is 32 bits, so maximum matrix of entries size is
> * is 2^32 - sizeof(fwts_acpi_table_slit) = 2^32 - 44 = 4294967252
> - * and table is a N x N matrix, so maxium number of localities is
> + * and table is a N x N matrix, so maximum number of localities is
> * limited to int(sqrt(4294967252)) = 65535.
> */
> if (n > 0xffff) {
> diff --git a/src/acpi/uefi/uefi.c b/src/acpi/uefi/uefi.c
> index e25429f1..ce54aefe 100644
> --- a/src/acpi/uefi/uefi.c
> +++ b/src/acpi/uefi/uefi.c
> @@ -74,7 +74,7 @@ static int uefi_test1(fwts_framework *fw)
> if (memcmp(uefi->uuid, guid_smm, 16) == 0) {
> fwts_acpi_table_uefi_smmcomm *uefi_smmcomm = (fwts_acpi_table_uefi_smmcomm \
> *)table->data;
> - /* chekc the dataoffset for SMM Comm table */
> + /* check the dataoffset for SMM Comm table */
> if (uefi_smmcomm->boot.dataoffset != 54) {
> passed = false;
> fwts_failed(fw, LOG_LEVEL_HIGH,
> diff --git a/src/acpi/wpbt/wpbt.c b/src/acpi/wpbt/wpbt.c
> index f12882e8..55bf1c17 100644
> --- a/src/acpi/wpbt/wpbt.c
> +++ b/src/acpi/wpbt/wpbt.c
> @@ -56,7 +56,7 @@ static int wpbt_test1(fwts_framework *fw)
> passed = false;
> fwts_failed(fw, LOG_LEVEL_HIGH,
> "WPBTBadArgumentLength",
> - "WPBT arugments length must be multiple of 2, got "
> + "WPBT arguments length must be multiple of 2, got "
> "0x%4.4" PRIx16 " instead", type->arguments_length);
> }
> }
> diff --git a/src/bios/bios32/bios32.c b/src/bios/bios32/bios32.c
> index 77ef8314..5ab75ac2 100644
> --- a/src/bios/bios32/bios32.c
> +++ b/src/bios/bios32/bios32.c
> @@ -84,7 +84,7 @@ static int bios32_test1(fwts_framework *fw)
> bios32->signature);
> fwts_log_info_verbatim(fw, " Entry Point: 0x%8.8" PRIx32,
> bios32->entry_point);
> - fwts_log_info_verbatim(fw, " Revsion : 0x%2.2" PRIx8,
> + fwts_log_info_verbatim(fw, " Revision : 0x%2.2" PRIx8,
> bios32->revision_level);
> fwts_log_info_verbatim(fw, " Length : 0x%2.2" PRIx8,
> bios32->length);
> diff --git a/src/bios/mtrr/mtrr.c b/src/bios/mtrr/mtrr.c
> index 46de585b..bb45f428 100644
> --- a/src/bios/mtrr/mtrr.c
> +++ b/src/bios/mtrr/mtrr.c
> @@ -390,7 +390,7 @@ static void multi_types_check(fwts_framework *fw, int *type)
> {
> int n_types = 0, i;
>
> - /* checking number fo types set, UNCACHED, WRITE_BACK, WRITE_COMBINING, \
> WRITE_THROUGH, WRITE_PROTECT */ + /* checking number of types set, UNCACHED, \
> WRITE_BACK, WRITE_COMBINING, WRITE_THROUGH, WRITE_PROTECT */ for (i = 0; i < 5; \
> i++) { if (*type & (1 << i))
> n_types++;
> diff --git a/src/cmos/cmosdump/cmosdump.c b/src/cmos/cmosdump/cmosdump.c
> index 603ec21e..176da096 100644
> --- a/src/cmos/cmosdump/cmosdump.c
> +++ b/src/cmos/cmosdump/cmosdump.c
> @@ -99,7 +99,7 @@ static int cmosdump_test1(fwts_framework *fw)
>
> static const char *const rate_selection[16] = {
> "none",
> - "3.90625 millseconds",
> + "3.90625 milliseconds",
> "7.8215 milliseconds",
> "122.070 microseconds",
> "244.141 microseconds",
> diff --git a/src/cpu/maxfreq/maxfreq.c b/src/cpu/maxfreq/maxfreq.c
> index a177e957..9bd16447 100644
> --- a/src/cpu/maxfreq/maxfreq.c
> +++ b/src/cpu/maxfreq/maxfreq.c
> @@ -62,7 +62,7 @@ static int maxfreq_test1(fwts_framework *fw)
>
> fwts_log_info(fw,
> "This test checks the maximum CPU frequency as detected by "
> - "the kernel for each CPU against maxiumum frequency as "
> + "the kernel for each CPU against maximum frequency as "
> "specified by the BIOS frequency scaling settings.");
>
> cpuinfo = fwts_file_open_and_read(CPU_INFO_PATH);
> diff --git a/src/dmi/dmicheck/dmicheck.c b/src/dmi/dmicheck/dmicheck.c
> index 8fbbfe90..a2403653 100644
> --- a/src/dmi/dmicheck/dmicheck.c
> +++ b/src/dmi/dmicheck/dmicheck.c
> @@ -2029,7 +2029,7 @@ static void dmi_scan_tables(fwts_framework *fw,
> /* Sanity check */
> if (hdr.length < 4) {
> fwts_failed(fw, LOG_LEVEL_HIGH, DMI_INVALID_ENTRY_LENGTH,
> - "Invald header length of entry #%d, "
> + "Invalid header length of entry #%d, "
> "length was 0x%2.2" PRIx8 ".",
> i, hdr.length);
> fwts_advice(fw,
> @@ -2148,7 +2148,7 @@ static void dmi_scan_smbios30_table(fwts_framework *fw,
> /* Sanity check */
> if (hdr.length < 4) {
> fwts_failed(fw, LOG_LEVEL_HIGH, DMI_INVALID_ENTRY_LENGTH,
> - "Invald header length of entry #%d, "
> + "Invalid header length of entry #%d, "
> "length was 0x%2.2" PRIx8 ".",
> i, hdr.length);
> fwts_advice(fw,
> diff --git a/src/lib/include/fwts_acpi.h b/src/lib/include/fwts_acpi.h
> index d223d934..e48554eb 100644
> --- a/src/lib/include/fwts_acpi.h
> +++ b/src/lib/include/fwts_acpi.h
> @@ -886,7 +886,7 @@ typedef struct {
> uint8_t device_scope[0];
> } __attribute__ ((packed)) fwts_acpi_table_dmar_hardware_unit;
>
> -/* Reserved Memory Defininition */
> +/* Reserved Memory Definition */
> typedef struct {
> fwts_acpi_table_dmar_header header;
> uint16_t reserved;
> diff --git a/src/lib/src/fwts_acpi_object_eval.c \
> b/src/lib/src/fwts_acpi_object_eval.c index df80b635..d5a31db4 100644
> --- a/src/lib/src/fwts_acpi_object_eval.c
> +++ b/src/lib/src/fwts_acpi_object_eval.c
> @@ -194,7 +194,7 @@ static const acpi_eval_error errors[] = {
> "A method attempted to use an empty element in an ACPI package."
> },
> { AE_AML_NUMERIC_OVERFLOW, LOG_LEVEL_HIGH, "AEAMLNumericOverflow", "Numeric \
> overflow",
> - "A numeric overflow occurred, for example when coverting a BCD value."
> + "A numeric overflow occurred, for example when converting a BCD value."
> },
> { AE_AML_REGION_LIMIT, LOG_LEVEL_CRITICAL, "AEAMLRegionLimit", "Region limit",
> "An attempt to access beyond the end of an ACPI Operation Region occurred."
> @@ -206,7 +206,7 @@ static const acpi_eval_error errors[] = {
> "An attempt to access beyond the endof an ACPI package occurred."
> },
> { AE_AML_DIVIDE_BY_ZERO, LOG_LEVEL_CRITICAL, "AEAMLDivideByZero", "Division by \
> zero",
> - "A division by zero error occurred while excuting the AML Divide op-code"
> + "A division by zero error occurred while executing the AML Divide op-code"
> },
> { AE_AML_NAME_NOT_FOUND, LOG_LEVEL_HIGH, "AEAMLNameNotFound", "Name not found",
> "An ACPI named reference could not be resolved."
> @@ -234,7 +234,7 @@ static const acpi_eval_error errors[] = {
> { AE_AML_MUTEX_ORDER, LOG_LEVEL_HIGH, "AEAMLMutexOrder", "Mutex order",
> "Deadlock prevention has detected that the current mutex sync level is too \
> large." },
> - { AE_AML_MUTEX_NOT_ACQUIRED, LOG_LEVEL_HIGH, "AEAMLMutexNotAcq", "Mutux not \
> acquired", + { AE_AML_MUTEX_NOT_ACQUIRED, \
> LOG_LEVEL_HIGH, "AEAMLMutexNotAcq", "Mutex not acquired", "An attempt was made to \
> release a mutex that had not been acquired." },
> { AE_AML_INVALID_RESOURCE_TYPE, LOG_LEVEL_HIGH, "AEAMLInvResourceType", "Invalid \
> resource type", @@ -245,7 +245,7 @@ static const acpi_eval_error errors[] = {
> "ArgN or LocalN were N was out of range."
> },
> { AE_AML_REGISTER_LIMIT, LOG_LEVEL_HIGH, "AEAMLRegisterLimit", "Register limit",
> - "An attempt to use a bank value that is beyong the capacity of a register \
> occurred." + "An attempt to use a bank value that is beyond the capacity of a \
> register occurred." },
> { AE_AML_NO_WHILE, LOG_LEVEL_CRITICAL, "AEAMLNoWhile", "No while",
> "A Break or Continue op-code was reached without a matching While op-code."
> @@ -905,7 +905,7 @@ void fwts_method_test_reference_return(
>
> /*
> * fwts_method_test_NULL_return
> - * check if no object was retuned
> + * check if no object was returned
> */
> void fwts_method_test_NULL_return(
> fwts_framework *fw,
> @@ -1096,7 +1096,7 @@ void fwts_method_test_passed_failed_return(
> fwts_advice(fw,
> "Method %s should be returning the correct "
> "0 or 1 return values. "
> - "Unexpected behaviour may occur becauses of "
> + "Unexpected behaviour may occur because of "
> "this error, the AML code does not conform to "
> "the ACPI specification and should be fixed.",
> method);
> @@ -1760,7 +1760,7 @@ void fwts_method_test_CRS_large_resource_items(
> "WORD Address Space Descriptor",
> "Extended IRQ Descriptor",
> "QWORD Address Space Descriptor",
> - "Extended Addresss Space Descriptor",
> + "Extended Address Space Descriptor",
> "GPIO Connection Descriptor",
> "Reserved",
> "Generic Serial Bus Connection Descriptor",
> diff --git a/src/lib/src/fwts_button.c b/src/lib/src/fwts_button.c
> index 82a05606..d4e07cd5 100644
> --- a/src/lib/src/fwts_button.c
> +++ b/src/lib/src/fwts_button.c
> @@ -47,7 +47,7 @@ static inline void fwts_button_path(
> /*
> * fwts_button_match_state_proc()
> * find matching button state and keep count of matching
> - * any non-matching states found, via proc iterface
> + * any non-matching states found, via proc interface
> */
> static int fwts_button_match_state_proc(
> const uint32_t button,
> diff --git a/src/lib/src/fwts_clog.c b/src/lib/src/fwts_clog.c
> index e556ab86..d988cc4d 100644
> --- a/src/lib/src/fwts_clog.c
> +++ b/src/lib/src/fwts_clog.c
> @@ -22,7 +22,7 @@
>
> /*
> * coreboot logfile exposed by Google firmware module
> - * Kernel option nessesary (GOOGLE_MEMCONSOLE_COREBOOT=m/y)
> + * Kernel option necessary (GOOGLE_MEMCONSOLE_COREBOOT=m/y)
> */
> #define GOOGLE_MEMCONSOLE_COREBOOT_PATH "/sys/firmware/log"
>
> diff --git a/src/lib/src/fwts_cpu.c b/src/lib/src/fwts_cpu.c
> index e70b4fff..a5d9c01f 100644
> --- a/src/lib/src/fwts_cpu.c
> +++ b/src/lib/src/fwts_cpu.c
> @@ -51,7 +51,7 @@ static pid_t *fwts_cpu_pids;
>
> /*
> * fwts_cpu_readmsr()
> - * Read a given msr on a specificied CPU
> + * Read a given msr on a specified CPU
> */
> int fwts_cpu_readmsr(
> fwts_framework *fw,
> diff --git a/src/lib/src/fwts_uefi.c b/src/lib/src/fwts_uefi.c
> index 138f1164..03ff8356 100644
> --- a/src/lib/src/fwts_uefi.c
> +++ b/src/lib/src/fwts_uefi.c
> @@ -147,7 +147,7 @@ static int fwts_uefi_get_interface(char **path)
>
> /*
> * fwts_uefi_str_to_str16()
> - * convert 8 bit C string to 16 bit sring.
> + * convert 8 bit C string to 16 bit string.
> */
> void fwts_uefi_str_to_str16(uint16_t *dst, const size_t len, const char *src)
> {
> diff --git a/src/libfwtsacpica/fwts_acpica.c b/src/libfwtsacpica/fwts_acpica.c
> index 7ddc62a3..6fb3e984 100644
> --- a/src/libfwtsacpica/fwts_acpica.c
> +++ b/src/libfwtsacpica/fwts_acpica.c
> @@ -219,7 +219,7 @@ static ACPI_STATUS fwts_exception_handler(
> if (AcpiGbl_IgnoreErrors) {
> if (AmlStatus != AE_OK) {
> fwts_log_info(fwts_acpica_fw,
> - "ACPICA Exception override, forcing AE_OK for execption %s",
> + "ACPICA Exception override, forcing AE_OK for exception %s",
> exception);
> AmlStatus = AE_OK;
> }
> diff --git a/src/tpm/tpmevlog/tpmevlog.c b/src/tpm/tpmevlog/tpmevlog.c
> index df0ef6aa..85c4b907 100644
> --- a/src/tpm/tpmevlog/tpmevlog.c
> +++ b/src/tpm/tpmevlog/tpmevlog.c
> @@ -309,7 +309,7 @@ static int tpmevlog_v2_check(
> hash_size = fwts_tpm_get_hash_size(alg_id);
> if (!hash_size) {
> fwts_failed(fw, LOG_LEVEL_MEDIUM, "EventV2HashSize",
> - "The hash sie of the event2 is %zd bytes "
> + "The hash size of the event2 is %zd bytes "
> "is smaller than the tcg pcr event2 %zd bytes.",
> len_remain,
> sizeof(fwts_tcg_pcr_event2));
> diff --git a/src/uefi/uefirttime/uefirttime.c b/src/uefi/uefirttime/uefirttime.c
> index 61d691d2..fa157736 100644
> --- a/src/uefi/uefirttime/uefirttime.c
> +++ b/src/uefi/uefirttime/uefirttime.c
> @@ -847,7 +847,7 @@ static int uefirttime_test23(fwts_framework *fw)
>
> if (*getwakeuptime.Pending != false) {
> fwts_failed(fw, LOG_LEVEL_HIGH, "UEFIRuntimeSetWakeupTimePending",
> - "Get error alarm signle status.");
> + "Get error alarm signal status.");
> return FWTS_ERROR;
> }
>
> @@ -1342,7 +1342,7 @@ static int uefirttime_test38(fwts_framework *fw)
> }
> } else {
> if (status != EFI_SUCCESS ){
> - fwts_skipped(fw, "Unknow error occurred, skip test.");
> + fwts_skipped(fw, "Unknown error occurred, skip test.");
> return FWTS_SKIP;
> }
> if (runtimeservicessupported & EFI_RT_SUPPORTED_GET_WAKEUP_TIME) {
> @@ -1362,7 +1362,7 @@ static int uefirttime_test38(fwts_framework *fw)
> static fwts_framework_minor_test uefirttime_tests[] = {
> { uefirttime_test1, "Test UEFI RT service get time interface." },
> { uefirttime_test2, "Test UEFI RT service get time interface, NULL time \
> parameter." },
> - { uefirttime_test3, "Test UEFI RT service get time interface, NULL time and NULL \
> capabilties parameters." }, + { uefirttime_test3, "Test UEFI RT service get time \
> interface, NULL time and NULL capabilities parameters." },
> { uefirttime_test4, "Test UEFI RT service set time interface." },
> { uefirttime_test5, "Test UEFI RT service set time interface, invalid year 1899." \
> },
Acked-by: Ivan Hu <ivan.hu@canonical.com>
--
fwts-devel mailing list
fwts-devel@lists.ubuntu.com
Modify settings or unsubscribe at: \
https://lists.ubuntu.com/mailman/listinfo/fwts-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic