ACK: [PATCH] misc: fix various spelling mistakes
Alex Hung
alex.hung at canonical.com
Fri Feb 3 03:25:40 UTC 2017
On 2017-01-31 02:08 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> codespell found a bunch of spelling mistakes, fix these.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
> src/acpi/bert/bert.c | 4 ++--
> src/acpi/madt/madt.c | 6 ++++--
> src/acpi/pcc/pcc.c | 2 +-
> src/acpi/wmi/wmi.c | 2 +-
> src/cpu/cpufreq/cpufreq.c | 2 +-
> 5 files changed, 9 insertions(+), 7 deletions(-)
>
> diff --git a/src/acpi/bert/bert.c b/src/acpi/bert/bert.c
> index f51ced6..63c69f7 100644
> --- a/src/acpi/bert/bert.c
> +++ b/src/acpi/bert/bert.c
> @@ -139,14 +139,14 @@ static int bert_test1(fwts_framework *fw)
> } else {
> fwts_warning(fw, "BERT Boot Error Region Raw Data Offset %"
> PRIu32 " is smaller than end of the data"
> - "region. BERT Boot Error Region Data Lenght "
> + "region. BERT Boot Error Region Data Length "
> "is zero.",
> region->raw_data_offset);
> fwts_advice(fw,
> "If there is raw data in the BERT Boot Error Region, "
> "Raw Data Offset must be larger than the end of the "
> "data region if there is raw data. However, since "
> - "BERT Boot Error Region Raw Data Lenght is zero, "
> + "BERT Boot Error Region Raw Data Length is zero, "
> "this may mean that there is no raw data.");
> }
> }
> diff --git a/src/acpi/madt/madt.c b/src/acpi/madt/madt.c
> index e4f2d81..1493dbe 100644
> --- a/src/acpi/madt/madt.c
> +++ b/src/acpi/madt/madt.c
> @@ -1400,8 +1400,10 @@ static int madt_subtables(fwts_framework *fw)
> offset = (int)(mtable->length - length);
> length -= sizeof(fwts_acpi_madt_sub_table_header);
>
> - /* set initial type value, will be overriden for OEM and
> - * reserved entries */
> + /*
> + * set initial type value, will be overridden for OEM and
> + * reserved entries
> + */
> type = hdr->type;
>
> /* check for OEM and reserved entries */
> diff --git a/src/acpi/pcc/pcc.c b/src/acpi/pcc/pcc.c
> index c7642df..fff406a 100644
> --- a/src/acpi/pcc/pcc.c
> +++ b/src/acpi/pcc/pcc.c
> @@ -122,7 +122,7 @@ static void pcc_check_pcc_header(
> fwts_log_info_verbatim(fw, " Major: 0x%" PRIx8, hdr->major);
> fwts_log_info_verbatim(fw, " Minor: 0x%" PRIx8, hdr->minor);
> fwts_log_info_verbatim(fw, " Features: 0x%" PRIx32, hdr->features);
> - fwts_log_info_verbatim(fw, " Commend: 0x%" PRIx16, hdr->command);
> + fwts_log_info_verbatim(fw, " Command: 0x%" PRIx16, hdr->command);
> fwts_log_info_verbatim(fw, " Status: 0x%" PRIx16, hdr->status);
> fwts_log_info_verbatim(fw, " Latency: 0x%" PRIx32, hdr->latency);
> fwts_log_info_verbatim(fw, " Minimum Time: 0x%" PRIx32, hdr->minimum_time);
> diff --git a/src/acpi/wmi/wmi.c b/src/acpi/wmi/wmi.c
> index 56beda2..58a3c06 100644
> --- a/src/acpi/wmi/wmi.c
> +++ b/src/acpi/wmi/wmi.c
> @@ -221,7 +221,7 @@ static void wmi_method_exist_count(
> fwts_failed(fw, LOG_LEVEL_LOW,
> "WMIMultipleMethod",
> "GUID %s has multiple associated methods WM%c%c defined, "
> - "this is a firmware bug that leads to ambigous behaviour.",
> + "this is a firmware bug that leads to ambiguous behaviour.",
> guid_str, info->id.obj_id[0], info->id.obj_id[1]);
> } else
> fwts_passed(fw, "%s has associated method %s", guid_str, objname);
> diff --git a/src/cpu/cpufreq/cpufreq.c b/src/cpu/cpufreq/cpufreq.c
> index d767a51..8175186 100644
> --- a/src/cpu/cpufreq/cpufreq.c
> +++ b/src/cpu/cpufreq/cpufreq.c
> @@ -825,7 +825,7 @@ static int cpufreq_init(fwts_framework *fw)
> int rc = cpu_set_governor(fw, &cpus[i], "userspace");
>
> if (rc != FWTS_OK) {
> - fwts_log_warning(fw, "Failed to intialise cpufreq "
> + fwts_log_warning(fw, "Failed to initialize cpufreq "
> "to set CPU speed");
> cpufreq_settable = false;
> break;
>
Acked-by: Alex Hung <alex.hung at canonical.com>
More information about the fwts-devel
mailing list