diff mbox

misc: fix various spelling mistakes

Message ID 20170130180821.32430-1-colin.king@canonical.com
State Accepted
Headers show

Commit Message

Colin Ian King Jan. 30, 2017, 6:08 p.m. UTC 
From: Colin Ian King <colin.king@canonical.com>

codespell found a bunch of spelling mistakes, fix these.

Signed-off-by: Colin Ian King <colin.king@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(-)

Comments

Ivan Hu Feb. 2, 2017, 7:06 a.m. UTC | #1 
On 2017年01月31日 02:08, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> codespell found a bunch of spelling mistakes, fix these.
>
> Signed-off-by: Colin Ian King <colin.king@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: Ivan Hu <ivan.hu@canonical.com>
Alex Hung Feb. 3, 2017, 3:25 a.m. UTC | #2 
On 2017-01-31 02:08 AM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> codespell found a bunch of spelling mistakes, fix these.
>
> Signed-off-by: Colin Ian King <colin.king@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@canonical.com>
diff mbox

Patch

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;