| Message ID | 20210412125227.1384924-1-colin.king@canonical.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | treewide: fix spelling mistakes in comments, literals and tests | expand |
On 2021-04-12 6:52 a.m., 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: Alex Hung <alex.hung@canonical.com>
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>
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." },