ACK: [PATCH] src/*: Fix various spelling mistakes in source code

ivanhu ivan.hu at canonical.com
Mon Jul 17 01:47:50 UTC 2023


Acked-by: Ivan Hu <ivan.hu at canonical.com>

On 7/12/23 05:10, Colin Ian King wrote:
> There are some spelling mistakes found by codespell in literal
> strings and comments. Fix these.
> 
> Signed-off-by: Colin Ian King <colin.i.king at gmail.com>
> ---
>   src/dmi/dmicheck/dmicheck.c              | 2 +-
>   src/lib/src/fwts_acpi_object_eval.c      | 2 +-
>   src/lib/src/fwts_dump_data.c             | 6 +++---
>   src/lib/src/fwts_mmap.c                  | 2 +-
>   src/uefi/uefirtvariable/uefirtvariable.c | 2 +-
>   5 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/src/dmi/dmicheck/dmicheck.c b/src/dmi/dmicheck/dmicheck.c
> index 656b323d..f7262b03 100644
> --- a/src/dmi/dmicheck/dmicheck.c
> +++ b/src/dmi/dmicheck/dmicheck.c
> @@ -1638,7 +1638,7 @@ static void dmicheck_entry(fwts_framework *fw,
>   			fwts_dmi_value_range t16_ranges[] = {{0x1, 0xa}, {0xa0, 0xa4}};
>   			dmi_ranges_uint8_check(fw, table, addr, "Location", hdr, 0x4, t16_ranges);
>   			dmi_min_max_uint8_check(fw, table, addr, "Use", hdr, 0x5, 0x1, 0x7);
> -			dmi_min_max_uint8_check(fw, table, addr, "Error Corrrection Type", hdr, 0x6, 0x1, 0x7);
> +			dmi_min_max_uint8_check(fw, table, addr, "Error Correction Type", hdr, 0x6, 0x1, 0x7);
>   			dmi_min_max_uint32_check(fw, table, addr, "Maximum Capacity", hdr, 0x7, 0, 0x80000000);
>   			if (hdr->length < 0x17)
>   				break;
> diff --git a/src/lib/src/fwts_acpi_object_eval.c b/src/lib/src/fwts_acpi_object_eval.c
> index aaf885e1..ece8fd98 100644
> --- a/src/lib/src/fwts_acpi_object_eval.c
> +++ b/src/lib/src/fwts_acpi_object_eval.c
> @@ -277,7 +277,7 @@ static const acpi_eval_error errors[] = {
>   	{ AE_AML_TARGET_TYPE,		LOG_LEVEL_HIGH,		"AEAMLTargetType",	"Target type",
>   		"A target operand of an incorrect type was encountered."
>   	},
> -	{ AE_AML_PROTOCOL,		LOG_LEVEL_HIGH,		"AEAMLViolateProtocol",	"Violattion protcol",
> +	{ AE_AML_PROTOCOL,		LOG_LEVEL_HIGH,		"AEAMLViolateProtocol",	"Violation protocol",
>   		"Violation of a fixed ACPI protocol."
>   	},
>   	{ AE_AML_BUFFER_LENGTH, 	LOG_LEVEL_HIGH,		"AEAMLBadBufLength",	"Bad buffer length",
> diff --git a/src/lib/src/fwts_dump_data.c b/src/lib/src/fwts_dump_data.c
> index d03a47d3..3cce1005 100644
> --- a/src/lib/src/fwts_dump_data.c
> +++ b/src/lib/src/fwts_dump_data.c
> @@ -31,7 +31,7 @@
>   void fwts_dump_raw_data(
>   	char *buffer,		/* buffer to contained formatted dump */
>   	const size_t len,	/* Length of buffer */
> -	const uint8_t *data,	/* Octects to dump */
> +	const uint8_t *data,	/* Octets to dump */
>   	const int addr,		/* Original buffer data address */
>   	const size_t nbytes)	/* Number of bytes to dump, max 16 */
>   {
> @@ -71,7 +71,7 @@ void fwts_dump_raw_data(
>   void fwts_dump_raw_data_prefix(
>   	char *buffer,		/* buffer to contained formatted dump */
>   	const size_t len,	/* Length of buffer */
> -	const uint8_t *data,	/* Octects to dump */
> +	const uint8_t *data,	/* Octets to dump */
>   	const char *prefix,	/* Prefix string or for alignment */
>   	const size_t nbytes)	/* Number of bytes to dump, max 16 */
>   {
> @@ -95,7 +95,7 @@ void fwts_dump_raw_data_prefix(
>    */
>   void fwts_hexdump_data_prefix_all(
>   	fwts_framework *fw,
> -	const uint8_t *data,	/* Octects to dump */
> +	const uint8_t *data,	/* Octets to dump */
>   	const char *prefix,	/* Prefix string or for alignment for each line */
>   	const size_t nbytes)	/* Number of bytes to dump for all data */
>   {
> diff --git a/src/lib/src/fwts_mmap.c b/src/lib/src/fwts_mmap.c
> index b9cd88f3..1efaf304 100644
> --- a/src/lib/src/fwts_mmap.c
> +++ b/src/lib/src/fwts_mmap.c
> @@ -74,7 +74,7 @@ void *fwts_mmap(const off_t start, const size_t size)
>   
>   /*
>    *  fwts_munmap()
> - *	Unmap memory mapped wuth fwts_mmap. Needs the mmap'd address and size.
> + *	Unmap memory mapped with fwts_mmap. Needs the mmap'd address and size.
>    */
>   int fwts_munmap(void *mem, const size_t size)
>   {
> diff --git a/src/uefi/uefirtvariable/uefirtvariable.c b/src/uefi/uefirtvariable/uefirtvariable.c
> index 41eb1e23..5ca64d8f 100644
> --- a/src/uefi/uefirtvariable/uefirtvariable.c
> +++ b/src/uefi/uefirtvariable/uefirtvariable.c
> @@ -2018,7 +2018,7 @@ static void getvariable_test_invalid(
>   	}
>   	fwts_failed(fw, LOG_LEVEL_HIGH,
>   		"UEFIRuntimeGetVariableInvalid",
> -		"GetVariable wuth %s failed to get an error return status, "
> +		"GetVariable with %s failed to get an error return status, "
>   		"expected EFI_INVALID_PARAMETER.", test);
>   
>   	return;



More information about the fwts-devel mailing list