ACK: [PATCH] treewide: fix spelling mistakes in comments, literals and tests

Alex Hung alex.hung at canonical.com
Mon Apr 12 19:28:32 UTC 2021


On 2021-04-12 6:52 a.m., Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> There are a handful of trivial spelling mistakes. Fix these.
> 
> Signed-off-by: Colin Ian King <colin.king at 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 at canonical.com>



More information about the fwts-devel mailing list