* [PATCH v2 01/17] .pytool: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
@ 2022-09-06 8:28 ` PierreGondois
2022-09-06 8:28 ` [PATCH v2 02/17] BaseTools: " PierreGondois
` (15 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:28 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Sean Brogan, Bret Barkelew
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
.pytool/Plugin/UncrustifyCheck/Readme.md | 2 +-
.pytool/Plugin/UncrustifyCheck/uncrustify.cfg | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/.pytool/Plugin/UncrustifyCheck/Readme.md b/.pytool/Plugin/UncrustifyCheck/Readme.md
index efe7a573e4fa..51f99565f72a 100644
--- a/.pytool/Plugin/UncrustifyCheck/Readme.md
+++ b/.pytool/Plugin/UncrustifyCheck/Readme.md
@@ -94,7 +94,7 @@ to be ignored.
If `True`, output diffs of formatting changes into the test case log. This is helpful to exactly understand what changes
need to be made to the source code in order to fix a coding standard compliance issue.
-Note that calculating the file diffs on a very large set of of results (e.g. >100 files) can significantly slow down
+Note that calculating the file diffs on a very large set of results (e.g. >100 files) can significantly slow down
plugin execution.
### `SkipGitExclusions`
diff --git a/.pytool/Plugin/UncrustifyCheck/uncrustify.cfg b/.pytool/Plugin/UncrustifyCheck/uncrustify.cfg
index 8506c3333715..2b8f2e539e5f 100644
--- a/.pytool/Plugin/UncrustifyCheck/uncrustify.cfg
+++ b/.pytool/Plugin/UncrustifyCheck/uncrustify.cfg
@@ -119,7 +119,7 @@ sp_square_fparen = remove # Add or remove space between ']' an
# 5.2.2.7 Put a space before an open brace if it is not on its own line
sp_do_brace_open = force # Add or remove space between 'do' and '{'.
sp_paren_brace = force # Add or remove space between ')' and '{'.
-sp_sparen_brace = force # Add or remove space between ')' and '{' of of control statements.
+sp_sparen_brace = force # Add or remove space between ')' and '{' of control statements.
# 5.2.2.8 Do not put spaces around structure member and pointer operators
sp_after_byref = remove # Add or remove space after reference sign '&', if followed by a word.
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 02/17] BaseTools: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
2022-09-06 8:28 ` [PATCH v2 01/17] .pytool: " PierreGondois
@ 2022-09-06 8:28 ` PierreGondois
2022-10-01 15:08 ` [edk2-devel] " Bob Feng
[not found] ` <1719FA81E323AB08.30932@groups.io>
2022-09-06 8:28 ` [PATCH v2 03/17] CryptoPkg: " PierreGondois
` (14 subsequent siblings)
16 siblings, 2 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:28 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Bob Feng, Liming Gao
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
BaseTools/Source/C/Common/FirmwareVolumeBuffer.c | 6 +++---
BaseTools/Source/C/DevicePath/DevicePathUtilities.c | 2 +-
BaseTools/Source/C/GenFv/GenFv.c | 2 +-
BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c | 4 ++--
BaseTools/Source/Python/Ecc/EccToolError.py | 2 +-
BaseTools/Source/Python/build/build.py | 2 +-
6 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
index 70741c8afcb5..ace26eb71c6b 100644
--- a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
+++ b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
@@ -91,7 +91,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
@@ -141,7 +141,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
@@ -285,7 +285,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
diff --git a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
index 2ffefa8ceeef..49078aac8caf 100644
--- a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
+++ b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
@@ -37,7 +37,7 @@ CONST EFI_DEVICE_PATH_PROTOCOL mUefiDevicePathLibEndDevicePath = {
@retval TRUE DevicePath is valid.
@retval FALSE DevicePath is NULL.
@retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
- @retval FALSE The length of any node node in the DevicePath is less
+ @retval FALSE The length of any node in the DevicePath is less
than sizeof (EFI_DEVICE_PATH_PROTOCOL).
@retval FALSE If MaxSize is not zero, the size of the DevicePath
exceeds MaxSize.
diff --git a/BaseTools/Source/C/GenFv/GenFv.c b/BaseTools/Source/C/GenFv/GenFv.c
index 43cc5cd3fe82..a742d55e5270 100644
--- a/BaseTools/Source/C/GenFv/GenFv.c
+++ b/BaseTools/Source/C/GenFv/GenFv.c
@@ -299,7 +299,7 @@ Returns:
return STATUS_ERROR;
}
if (TempNumber == 0) {
- Error (NULL, 0, 1003, "Invalid option value", "Fv block size can't be be set to zero");
+ Error (NULL, 0, 1003, "Invalid option value", "Fv block size can't be set to zero");
return STATUS_ERROR;
}
mFvDataInfo.FvBlocks[0].Length = (UINT32) TempNumber;
diff --git a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
index b57f5ded846c..bf3dd18eaf6f 100644
--- a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
+++ b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
@@ -714,7 +714,7 @@ Tree *MR_compute_pred_tree_ctxXX(p)
if (p == NULL) return NULL;
/* this appears strange: why do we OR the context
- of and AND predicate ? It is because of the way
+ of an AND predicate ? It is because of the way
that predicates are evaluated: if the context is
wrong then it's the same as if the predicate was
true. That means that even when one leg of an
@@ -1531,7 +1531,7 @@ Predicate * MR_find_in_aSubBlk(alt)
* 1. if lookahead for alt i is contained in the lookahead for any
* alt j then ignore semantic predicate of alt i
* 2. if lookahead for alt i is not contained in the lookahead for
- * any alt j then add add predicate i to the OR list to be hoisted
+ * any alt j then add predicate i to the OR list to be hoisted
* 3. if lookahead for alt i overlaps the lookahead for some alt j then
* add a dummy semantic predicate for alt j
*
diff --git a/BaseTools/Source/Python/Ecc/EccToolError.py b/BaseTools/Source/Python/Ecc/EccToolError.py
index d97bf7948ce8..2ff36c8329cc 100644
--- a/BaseTools/Source/Python/Ecc/EccToolError.py
+++ b/BaseTools/Source/Python/Ecc/EccToolError.py
@@ -186,7 +186,7 @@ gEccErrorMessage = {
ERROR_META_DATA_FILE_CHECK_BINARY_INF_IN_FDF : "An INF file is specified in the FDF file, but not in the DSC file, therefore the INF file must be for a Binary module only",
ERROR_META_DATA_FILE_CHECK_PCD_DUPLICATE : "Duplicate PCDs found",
ERROR_META_DATA_FILE_CHECK_PCD_FLASH : "PCD settings in the FDF file should only be related to flash",
- ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs declared in INF files that are not specified in in either a DSC or FDF file",
+ ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs declared in INF files that are not specified in either a DSC or FDF file",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_GUID : "Duplicate GUID found",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_PROTOCOL : "Duplicate PROTOCOL found",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_PPI : "Duplicate PPI found",
diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py
index 07187c03618a..51fb1f433eb7 100755
--- a/BaseTools/Source/Python/build/build.py
+++ b/BaseTools/Source/Python/build/build.py
@@ -205,7 +205,7 @@ class MakeSubProc(Popen):
#
# This method will call subprocess.Popen to execute an external program with
# given options in specified directory. Because of the dead-lock issue during
-# redirecting output of the external program, threads are used to to do the
+# redirecting output of the external program, threads are used to do the
# redirection work.
#
# @param Command A list or string containing the call of the program
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
2022-09-06 8:28 ` [PATCH v2 02/17] BaseTools: " PierreGondois
@ 2022-10-01 15:08 ` Bob Feng
2022-10-03 7:59 ` PierreGondois
[not found] ` <1719FA81E323AB08.30932@groups.io>
1 sibling, 1 reply; 27+ messages in thread
From: Bob Feng @ 2022-10-01 15:08 UTC (permalink / raw)
To: devel@edk2.groups.io, pierre.gondois@arm.com; +Cc: Gao, Liming
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of PierreGondois
Sent: Tuesday, September 6, 2022 4:29 PM
To: devel@edk2.groups.io
Cc: Pierre Gondois <pierre.gondois@arm.com>; Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>
Subject: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above package, remove duplicated words.
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
BaseTools/Source/C/Common/FirmwareVolumeBuffer.c | 6 +++---
BaseTools/Source/C/DevicePath/DevicePathUtilities.c | 2 +-
BaseTools/Source/C/GenFv/GenFv.c | 2 +-
BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c | 4 ++--
BaseTools/Source/Python/Ecc/EccToolError.py | 2 +-
BaseTools/Source/Python/build/build.py | 2 +-
6 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
index 70741c8afcb5..ace26eb71c6b 100644
--- a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
+++ b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
@@ -91,7 +91,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
@@ -141,7 +141,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
@@ -285,7 +285,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
diff --git a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
index 2ffefa8ceeef..49078aac8caf 100644
--- a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
+++ b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
@@ -37,7 +37,7 @@ CONST EFI_DEVICE_PATH_PROTOCOL mUefiDevicePathLibEndDevicePath = {
@retval TRUE DevicePath is valid.
@retval FALSE DevicePath is NULL.
@retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
- @retval FALSE The length of any node node in the DevicePath is less
+ @retval FALSE The length of any node in the DevicePath is less
than sizeof (EFI_DEVICE_PATH_PROTOCOL).
@retval FALSE If MaxSize is not zero, the size of the DevicePath
exceeds MaxSize.
diff --git a/BaseTools/Source/C/GenFv/GenFv.c b/BaseTools/Source/C/GenFv/GenFv.c
index 43cc5cd3fe82..a742d55e5270 100644
--- a/BaseTools/Source/C/GenFv/GenFv.c
+++ b/BaseTools/Source/C/GenFv/GenFv.c
@@ -299,7 +299,7 @@ Returns:
return STATUS_ERROR;
}
if (TempNumber == 0) {
- Error (NULL, 0, 1003, "Invalid option value", "Fv block size can't be be set to zero");
+ Error (NULL, 0, 1003, "Invalid option value", "Fv block size
+ can't be set to zero");
return STATUS_ERROR;
}
mFvDataInfo.FvBlocks[0].Length = (UINT32) TempNumber; diff --git a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
index b57f5ded846c..bf3dd18eaf6f 100644
--- a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
+++ b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
@@ -714,7 +714,7 @@ Tree *MR_compute_pred_tree_ctxXX(p)
if (p == NULL) return NULL;
/* this appears strange: why do we OR the context
- of and AND predicate ? It is because of the way
+ of an AND predicate ? It is because of the way
that predicates are evaluated: if the context is
wrong then it's the same as if the predicate was
true. That means that even when one leg of an @@ -1531,7 +1531,7 @@ Predicate * MR_find_in_aSubBlk(alt)
* 1. if lookahead for alt i is contained in the lookahead for any
* alt j then ignore semantic predicate of alt i
* 2. if lookahead for alt i is not contained in the lookahead for
- * any alt j then add add predicate i to the OR list to be hoisted
+ * any alt j then add predicate i to the OR list to be hoisted
* 3. if lookahead for alt i overlaps the lookahead for some alt j then
* add a dummy semantic predicate for alt j
*
diff --git a/BaseTools/Source/Python/Ecc/EccToolError.py b/BaseTools/Source/Python/Ecc/EccToolError.py
index d97bf7948ce8..2ff36c8329cc 100644
--- a/BaseTools/Source/Python/Ecc/EccToolError.py
+++ b/BaseTools/Source/Python/Ecc/EccToolError.py
@@ -186,7 +186,7 @@ gEccErrorMessage = {
ERROR_META_DATA_FILE_CHECK_BINARY_INF_IN_FDF : "An INF file is specified in the FDF file, but not in the DSC file, therefore the INF file must be for a Binary module only",
ERROR_META_DATA_FILE_CHECK_PCD_DUPLICATE : "Duplicate PCDs found",
ERROR_META_DATA_FILE_CHECK_PCD_FLASH : "PCD settings in the FDF file should only be related to flash",
- ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs declared in INF files that are not specified in in either a DSC or FDF file",
+ ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs
+ declared in INF files that are not specified in either a DSC or FDF
+ file",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_GUID : "Duplicate GUID found",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_PROTOCOL : "Duplicate PROTOCOL found",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_PPI : "Duplicate PPI found", diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py
index 07187c03618a..51fb1f433eb7 100755
--- a/BaseTools/Source/Python/build/build.py
+++ b/BaseTools/Source/Python/build/build.py
@@ -205,7 +205,7 @@ class MakeSubProc(Popen):
#
# This method will call subprocess.Popen to execute an external program with # given options in specified directory. Because of the dead-lock issue during -# redirecting output of the external program, threads are used to to do the
+# redirecting output of the external program, threads are used to do
+the
# redirection work.
#
# @param Command A list or string containing the call of the program
--
2.25.1
-=-=-=-=-=-=
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93193): https://edk2.groups.io/g/devel/message/93193
Mute This Topic: https://groups.io/mt/93496599/1768742
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [bob.c.feng@intel.com]
-=-=-=-=-=-=
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
2022-10-01 15:08 ` [edk2-devel] " Bob Feng
@ 2022-10-03 7:59 ` PierreGondois
2022-10-08 2:22 ` 回复: " gaoliming
0 siblings, 1 reply; 27+ messages in thread
From: PierreGondois @ 2022-10-03 7:59 UTC (permalink / raw)
To: Feng, Bob C, devel@edk2.groups.io; +Cc: Gao, Liming
Hello Bob,
Thanks for the review. Other patches in the serie received a Reviewed-by:
- [PATCH v2 04/17] DynamicTablesPkg: Remove duplicated words
- [PATCH v2 09/17] NetworkPkg: Remove duplicated words
- [PATCH v2 11/17] RedfishPkg: Remove duplicated words
- [PATCH v2 13/17] ShellPkg: Remove duplicated words
- [PATCH v2 14/17] SourceLevelDebugPkg: Remove duplicated words
- [PATCH v2 17/17] UnitTestFrameworkPkg: Remove duplicated words
Would it be possible to take them aswell ? I'm not sure who should take them as
they are accross multiple packages,
Regards,
Pierre
On 10/1/22 17:08, Feng, Bob C wrote:
> Reviewed-by: Bob Feng <bob.c.feng@intel.com>
>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of PierreGondois
> Sent: Tuesday, September 6, 2022 4:29 PM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois <pierre.gondois@arm.com>; Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>
> Subject: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
>
> From: Pierre Gondois <pierre.gondois@arm.com>
>
> In an effort to clean the documentation of the above package, remove duplicated words.
>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> ---
> BaseTools/Source/C/Common/FirmwareVolumeBuffer.c | 6 +++---
> BaseTools/Source/C/DevicePath/DevicePathUtilities.c | 2 +-
> BaseTools/Source/C/GenFv/GenFv.c | 2 +-
> BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c | 4 ++--
> BaseTools/Source/Python/Ecc/EccToolError.py | 2 +-
> BaseTools/Source/Python/build/build.py | 2 +-
> 6 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> index 70741c8afcb5..ace26eb71c6b 100644
> --- a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> +++ b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> @@ -91,7 +91,7 @@ Routine Description:
>
> Arguments:
>
> - SourceFv - Address of the Fv in memory, this firmware volume volume will
> + SourceFv - Address of the Fv in memory, this firmware volume will
> be modified, if SourceFfsFile exists
> SourceFfsFile - Input FFS file to replace
>
> @@ -141,7 +141,7 @@ Routine Description:
>
> Arguments:
>
> - SourceFv - Address of the Fv in memory, this firmware volume volume will
> + SourceFv - Address of the Fv in memory, this firmware volume will
> be modified, if SourceFfsFile exists
> SourceFfsFile - Input FFS file to replace
>
> @@ -285,7 +285,7 @@ Routine Description:
>
> Arguments:
>
> - SourceFv - Address of the Fv in memory, this firmware volume volume will
> + SourceFv - Address of the Fv in memory, this firmware volume will
> be modified, if SourceFfsFile exists
> SourceFfsFile - Input FFS file to replace
>
> diff --git a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
> index 2ffefa8ceeef..49078aac8caf 100644
> --- a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
> +++ b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
> @@ -37,7 +37,7 @@ CONST EFI_DEVICE_PATH_PROTOCOL mUefiDevicePathLibEndDevicePath = {
> @retval TRUE DevicePath is valid.
> @retval FALSE DevicePath is NULL.
> @retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
> - @retval FALSE The length of any node node in the DevicePath is less
> + @retval FALSE The length of any node in the DevicePath is less
> than sizeof (EFI_DEVICE_PATH_PROTOCOL).
> @retval FALSE If MaxSize is not zero, the size of the DevicePath
> exceeds MaxSize.
> diff --git a/BaseTools/Source/C/GenFv/GenFv.c b/BaseTools/Source/C/GenFv/GenFv.c
> index 43cc5cd3fe82..a742d55e5270 100644
> --- a/BaseTools/Source/C/GenFv/GenFv.c
> +++ b/BaseTools/Source/C/GenFv/GenFv.c
> @@ -299,7 +299,7 @@ Returns:
> return STATUS_ERROR;
> }
> if (TempNumber == 0) {
> - Error (NULL, 0, 1003, "Invalid option value", "Fv block size can't be be set to zero");
> + Error (NULL, 0, 1003, "Invalid option value", "Fv block size
> + can't be set to zero");
> return STATUS_ERROR;
> }
> mFvDataInfo.FvBlocks[0].Length = (UINT32) TempNumber; diff --git a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
> index b57f5ded846c..bf3dd18eaf6f 100644
> --- a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
> +++ b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
> @@ -714,7 +714,7 @@ Tree *MR_compute_pred_tree_ctxXX(p)
> if (p == NULL) return NULL;
>
> /* this appears strange: why do we OR the context
> - of and AND predicate ? It is because of the way
> + of an AND predicate ? It is because of the way
> that predicates are evaluated: if the context is
> wrong then it's the same as if the predicate was
> true. That means that even when one leg of an @@ -1531,7 +1531,7 @@ Predicate * MR_find_in_aSubBlk(alt)
> * 1. if lookahead for alt i is contained in the lookahead for any
> * alt j then ignore semantic predicate of alt i
> * 2. if lookahead for alt i is not contained in the lookahead for
> - * any alt j then add add predicate i to the OR list to be hoisted
> + * any alt j then add predicate i to the OR list to be hoisted
> * 3. if lookahead for alt i overlaps the lookahead for some alt j then
> * add a dummy semantic predicate for alt j
> *
> diff --git a/BaseTools/Source/Python/Ecc/EccToolError.py b/BaseTools/Source/Python/Ecc/EccToolError.py
> index d97bf7948ce8..2ff36c8329cc 100644
> --- a/BaseTools/Source/Python/Ecc/EccToolError.py
> +++ b/BaseTools/Source/Python/Ecc/EccToolError.py
> @@ -186,7 +186,7 @@ gEccErrorMessage = {
> ERROR_META_DATA_FILE_CHECK_BINARY_INF_IN_FDF : "An INF file is specified in the FDF file, but not in the DSC file, therefore the INF file must be for a Binary module only",
> ERROR_META_DATA_FILE_CHECK_PCD_DUPLICATE : "Duplicate PCDs found",
> ERROR_META_DATA_FILE_CHECK_PCD_FLASH : "PCD settings in the FDF file should only be related to flash",
> - ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs declared in INF files that are not specified in in either a DSC or FDF file",
> + ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs
> + declared in INF files that are not specified in either a DSC or FDF
> + file",
> ERROR_META_DATA_FILE_CHECK_DUPLICATE_GUID : "Duplicate GUID found",
> ERROR_META_DATA_FILE_CHECK_DUPLICATE_PROTOCOL : "Duplicate PROTOCOL found",
> ERROR_META_DATA_FILE_CHECK_DUPLICATE_PPI : "Duplicate PPI found", diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py
> index 07187c03618a..51fb1f433eb7 100755
> --- a/BaseTools/Source/Python/build/build.py
> +++ b/BaseTools/Source/Python/build/build.py
> @@ -205,7 +205,7 @@ class MakeSubProc(Popen):
> #
> # This method will call subprocess.Popen to execute an external program with # given options in specified directory. Because of the dead-lock issue during -# redirecting output of the external program, threads are used to to do the
> +# redirecting output of the external program, threads are used to do
> +the
> # redirection work.
> #
> # @param Command A list or string containing the call of the program
> --
> 2.25.1
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#93193): https://edk2.groups.io/g/devel/message/93193
> Mute This Topic: https://groups.io/mt/93496599/1768742
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [bob.c.feng@intel.com]
> -=-=-=-=-=-=
>
>
^ permalink raw reply [flat|nested] 27+ messages in thread* 回复: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
2022-10-03 7:59 ` PierreGondois
@ 2022-10-08 2:22 ` gaoliming
0 siblings, 0 replies; 27+ messages in thread
From: gaoliming @ 2022-10-08 2:22 UTC (permalink / raw)
To: devel, pierre.gondois, 'Feng, Bob C'
Pierre:
If the whole serials have passed review, one of package maintainer can merge them together.
Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表
> PierreGondois
> 发送时间: 2022年10月3日 16:00
> 收件人: Feng, Bob C <bob.c.feng@intel.com>; devel@edk2.groups.io
> 抄送: Gao, Liming <gaoliming@byosoft.com.cn>
> 主题: Re: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated
> words
>
> Hello Bob,
> Thanks for the review. Other patches in the serie received a Reviewed-by:
> - [PATCH v2 04/17] DynamicTablesPkg: Remove duplicated words
> - [PATCH v2 09/17] NetworkPkg: Remove duplicated words
> - [PATCH v2 11/17] RedfishPkg: Remove duplicated words
> - [PATCH v2 13/17] ShellPkg: Remove duplicated words
> - [PATCH v2 14/17] SourceLevelDebugPkg: Remove duplicated words
> - [PATCH v2 17/17] UnitTestFrameworkPkg: Remove duplicated words
>
> Would it be possible to take them aswell ? I'm not sure who should take them
> as
> they are accross multiple packages,
>
> Regards,
> Pierre
>
>
> On 10/1/22 17:08, Feng, Bob C wrote:
> > Reviewed-by: Bob Feng <bob.c.feng@intel.com>
> >
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> PierreGondois
> > Sent: Tuesday, September 6, 2022 4:29 PM
> > To: devel@edk2.groups.io
> > Cc: Pierre Gondois <pierre.gondois@arm.com>; Feng, Bob C
> <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>
> > Subject: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated
> words
> >
> > From: Pierre Gondois <pierre.gondois@arm.com>
> >
> > In an effort to clean the documentation of the above package, remove
> duplicated words.
> >
> > Cc: Bob Feng <bob.c.feng@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> > ---
> > BaseTools/Source/C/Common/FirmwareVolumeBuffer.c | 6 +++---
> > BaseTools/Source/C/DevicePath/DevicePathUtilities.c | 2 +-
> > BaseTools/Source/C/GenFv/GenFv.c | 2 +-
> > BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c | 4 ++--
> > BaseTools/Source/Python/Ecc/EccToolError.py | 2 +-
> > BaseTools/Source/Python/build/build.py | 2 +-
> > 6 files changed, 9 insertions(+), 9 deletions(-)
> >
> > diff --git a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> > index 70741c8afcb5..ace26eb71c6b 100644
> > --- a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> > +++ b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
> > @@ -91,7 +91,7 @@ Routine Description:
> >
> > Arguments:
> >
> > - SourceFv - Address of the Fv in memory, this firmware volume volume
> will
> > + SourceFv - Address of the Fv in memory, this firmware volume will
> > be modified, if SourceFfsFile exists
> > SourceFfsFile - Input FFS file to replace
> >
> > @@ -141,7 +141,7 @@ Routine Description:
> >
> > Arguments:
> >
> > - SourceFv - Address of the Fv in memory, this firmware volume volume
> will
> > + SourceFv - Address of the Fv in memory, this firmware volume will
> > be modified, if SourceFfsFile exists
> > SourceFfsFile - Input FFS file to replace
> >
> > @@ -285,7 +285,7 @@ Routine Description:
> >
> > Arguments:
> >
> > - SourceFv - Address of the Fv in memory, this firmware volume volume
> will
> > + SourceFv - Address of the Fv in memory, this firmware volume will
> > be modified, if SourceFfsFile exists
> > SourceFfsFile - Input FFS file to replace
> >
> > diff --git a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
> b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
> > index 2ffefa8ceeef..49078aac8caf 100644
> > --- a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
> > +++ b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
> > @@ -37,7 +37,7 @@ CONST EFI_DEVICE_PATH_PROTOCOL
> mUefiDevicePathLibEndDevicePath = {
> > @retval TRUE DevicePath is valid.
> > @retval FALSE DevicePath is NULL.
> > @retval FALSE Maxsize is less than
> sizeof(EFI_DEVICE_PATH_PROTOCOL).
> > - @retval FALSE The length of any node node in the DevicePath is
> less
> > + @retval FALSE The length of any node in the DevicePath is less
> > than sizeof (EFI_DEVICE_PATH_PROTOCOL).
> > @retval FALSE If MaxSize is not zero, the size of the
> DevicePath
> > exceeds MaxSize.
> > diff --git a/BaseTools/Source/C/GenFv/GenFv.c
> b/BaseTools/Source/C/GenFv/GenFv.c
> > index 43cc5cd3fe82..a742d55e5270 100644
> > --- a/BaseTools/Source/C/GenFv/GenFv.c
> > +++ b/BaseTools/Source/C/GenFv/GenFv.c
> > @@ -299,7 +299,7 @@ Returns:
> > return STATUS_ERROR;
> > }
> > if (TempNumber == 0) {
> > - Error (NULL, 0, 1003, "Invalid option value", "Fv block size can't
> be be set to zero");
> > + Error (NULL, 0, 1003, "Invalid option value", "Fv block size
> > + can't be set to zero");
> > return STATUS_ERROR;
> > }
> > mFvDataInfo.FvBlocks[0].Length = (UINT32) TempNumber; diff
> --git a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
> b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
> > index b57f5ded846c..bf3dd18eaf6f 100644
> > --- a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
> > +++ b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
> > @@ -714,7 +714,7 @@ Tree *MR_compute_pred_tree_ctxXX(p)
> > if (p == NULL) return NULL;
> >
> > /* this appears strange: why do we OR the context
> > - of and AND predicate ? It is because of the way
> > + of an AND predicate ? It is because of the way
> > that predicates are evaluated: if the context is
> > wrong then it's the same as if the predicate was
> > true. That means that even when one leg of an @@ -1531,7
> +1531,7 @@ Predicate * MR_find_in_aSubBlk(alt)
> > * 1. if lookahead for alt i is contained in the lookahead for any
> > * alt j then ignore semantic predicate of alt i
> > * 2. if lookahead for alt i is not contained in the lookahead for
> > - * any alt j then add add predicate i to the OR list to be hoisted
> > + * any alt j then add predicate i to the OR list to be hoisted
> > * 3. if lookahead for alt i overlaps the lookahead for some alt j
> then
> > * add a dummy semantic predicate for alt j
> > *
> > diff --git a/BaseTools/Source/Python/Ecc/EccToolError.py
> b/BaseTools/Source/Python/Ecc/EccToolError.py
> > index d97bf7948ce8..2ff36c8329cc 100644
> > --- a/BaseTools/Source/Python/Ecc/EccToolError.py
> > +++ b/BaseTools/Source/Python/Ecc/EccToolError.py
> > @@ -186,7 +186,7 @@ gEccErrorMessage = {
> > ERROR_META_DATA_FILE_CHECK_BINARY_INF_IN_FDF : "An INF
> file is specified in the FDF file, but not in the DSC file, therefore the INF file
> must be for a Binary module only",
> > ERROR_META_DATA_FILE_CHECK_PCD_DUPLICATE : "Duplicate
> PCDs found",
> > ERROR_META_DATA_FILE_CHECK_PCD_FLASH : "PCD settings in
> the FDF file should only be related to flash",
> > - ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be
> no PCDs declared in INF files that are not specified in in either a DSC or FDF
> file",
> > + ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be
> no PCDs
> > + declared in INF files that are not specified in either a DSC or FDF
> > + file",
> > ERROR_META_DATA_FILE_CHECK_DUPLICATE_GUID : "Duplicate
> GUID found",
> > ERROR_META_DATA_FILE_CHECK_DUPLICATE_PROTOCOL :
> "Duplicate PROTOCOL found",
> > ERROR_META_DATA_FILE_CHECK_DUPLICATE_PPI : "Duplicate PPI
> found", diff --git a/BaseTools/Source/Python/build/build.py
> b/BaseTools/Source/Python/build/build.py
> > index 07187c03618a..51fb1f433eb7 100755
> > --- a/BaseTools/Source/Python/build/build.py
> > +++ b/BaseTools/Source/Python/build/build.py
> > @@ -205,7 +205,7 @@ class MakeSubProc(Popen):
> > #
> > # This method will call subprocess.Popen to execute an external program
> with # given options in specified directory. Because of the dead-lock issue
> during -# redirecting output of the external program, threads are used to to
> do the
> > +# redirecting output of the external program, threads are used to do
> > +the
> > # redirection work.
> > #
> > # @param Command A list or string containing the
> call of the program
> > --
> > 2.25.1
> >
> >
> >
> > -=-=-=-=-=-=
> > Groups.io Links: You receive all messages sent to this group.
> > View/Reply Online (#93193):
> https://edk2.groups.io/g/devel/message/93193
> > Mute This Topic: https://groups.io/mt/93496599/1768742
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub [bob.c.feng@intel.com]
> > -=-=-=-=-=-=
> >
> >
>
>
>
>
^ permalink raw reply [flat|nested] 27+ messages in thread
[parent not found: <1719FA81E323AB08.30932@groups.io>]
* Re: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
[not found] ` <1719FA81E323AB08.30932@groups.io>
@ 2022-10-02 3:25 ` Bob Feng
0 siblings, 0 replies; 27+ messages in thread
From: Bob Feng @ 2022-10-02 3:25 UTC (permalink / raw)
To: devel@edk2.groups.io, Feng, Bob C, pierre.gondois@arm.com; +Cc: Gao, Liming
Hi Pierre,
Could you split this patch into 2? One is for basetools C tool, the other is for python tool because the python code patch will be merged to edk2-basetools repo also.
Thanks,
Bob
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Bob Feng
Sent: Saturday, October 1, 2022 11:08 PM
To: devel@edk2.groups.io; pierre.gondois@arm.com
Cc: Gao, Liming <gaoliming@byosoft.com.cn>
Subject: Re: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of PierreGondois
Sent: Tuesday, September 6, 2022 4:29 PM
To: devel@edk2.groups.io
Cc: Pierre Gondois <pierre.gondois@arm.com>; Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>
Subject: [edk2-devel] [PATCH v2 02/17] BaseTools: Remove duplicated words
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above package, remove duplicated words.
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
BaseTools/Source/C/Common/FirmwareVolumeBuffer.c | 6 +++---
BaseTools/Source/C/DevicePath/DevicePathUtilities.c | 2 +-
BaseTools/Source/C/GenFv/GenFv.c | 2 +-
BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c | 4 ++--
BaseTools/Source/Python/Ecc/EccToolError.py | 2 +-
BaseTools/Source/Python/build/build.py | 2 +-
6 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
index 70741c8afcb5..ace26eb71c6b 100644
--- a/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
+++ b/BaseTools/Source/C/Common/FirmwareVolumeBuffer.c
@@ -91,7 +91,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
@@ -141,7 +141,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
@@ -285,7 +285,7 @@ Routine Description:
Arguments:
- SourceFv - Address of the Fv in memory, this firmware volume volume will
+ SourceFv - Address of the Fv in memory, this firmware volume will
be modified, if SourceFfsFile exists
SourceFfsFile - Input FFS file to replace
diff --git a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
index 2ffefa8ceeef..49078aac8caf 100644
--- a/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
+++ b/BaseTools/Source/C/DevicePath/DevicePathUtilities.c
@@ -37,7 +37,7 @@ CONST EFI_DEVICE_PATH_PROTOCOL mUefiDevicePathLibEndDevicePath = {
@retval TRUE DevicePath is valid.
@retval FALSE DevicePath is NULL.
@retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
- @retval FALSE The length of any node node in the DevicePath is less
+ @retval FALSE The length of any node in the DevicePath is less
than sizeof (EFI_DEVICE_PATH_PROTOCOL).
@retval FALSE If MaxSize is not zero, the size of the DevicePath
exceeds MaxSize.
diff --git a/BaseTools/Source/C/GenFv/GenFv.c b/BaseTools/Source/C/GenFv/GenFv.c
index 43cc5cd3fe82..a742d55e5270 100644
--- a/BaseTools/Source/C/GenFv/GenFv.c
+++ b/BaseTools/Source/C/GenFv/GenFv.c
@@ -299,7 +299,7 @@ Returns:
return STATUS_ERROR;
}
if (TempNumber == 0) {
- Error (NULL, 0, 1003, "Invalid option value", "Fv block size can't be be set to zero");
+ Error (NULL, 0, 1003, "Invalid option value", "Fv block size
+ can't be set to zero");
return STATUS_ERROR;
}
mFvDataInfo.FvBlocks[0].Length = (UINT32) TempNumber; diff --git a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
index b57f5ded846c..bf3dd18eaf6f 100644
--- a/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
+++ b/BaseTools/Source/C/VfrCompile/Pccts/antlr/mrhoist.c
@@ -714,7 +714,7 @@ Tree *MR_compute_pred_tree_ctxXX(p)
if (p == NULL) return NULL;
/* this appears strange: why do we OR the context
- of and AND predicate ? It is because of the way
+ of an AND predicate ? It is because of the way
that predicates are evaluated: if the context is
wrong then it's the same as if the predicate was
true. That means that even when one leg of an @@ -1531,7 +1531,7 @@ Predicate * MR_find_in_aSubBlk(alt)
* 1. if lookahead for alt i is contained in the lookahead for any
* alt j then ignore semantic predicate of alt i
* 2. if lookahead for alt i is not contained in the lookahead for
- * any alt j then add add predicate i to the OR list to be hoisted
+ * any alt j then add predicate i to the OR list to be hoisted
* 3. if lookahead for alt i overlaps the lookahead for some alt j then
* add a dummy semantic predicate for alt j
*
diff --git a/BaseTools/Source/Python/Ecc/EccToolError.py b/BaseTools/Source/Python/Ecc/EccToolError.py
index d97bf7948ce8..2ff36c8329cc 100644
--- a/BaseTools/Source/Python/Ecc/EccToolError.py
+++ b/BaseTools/Source/Python/Ecc/EccToolError.py
@@ -186,7 +186,7 @@ gEccErrorMessage = {
ERROR_META_DATA_FILE_CHECK_BINARY_INF_IN_FDF : "An INF file is specified in the FDF file, but not in the DSC file, therefore the INF file must be for a Binary module only",
ERROR_META_DATA_FILE_CHECK_PCD_DUPLICATE : "Duplicate PCDs found",
ERROR_META_DATA_FILE_CHECK_PCD_FLASH : "PCD settings in the FDF file should only be related to flash",
- ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs declared in INF files that are not specified in in either a DSC or FDF file",
+ ERROR_META_DATA_FILE_CHECK_PCD_NO_USE : "There should be no PCDs
+ declared in INF files that are not specified in either a DSC or FDF
+ file",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_GUID : "Duplicate GUID found",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_PROTOCOL : "Duplicate PROTOCOL found",
ERROR_META_DATA_FILE_CHECK_DUPLICATE_PPI : "Duplicate PPI found", diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Python/build/build.py
index 07187c03618a..51fb1f433eb7 100755
--- a/BaseTools/Source/Python/build/build.py
+++ b/BaseTools/Source/Python/build/build.py
@@ -205,7 +205,7 @@ class MakeSubProc(Popen):
#
# This method will call subprocess.Popen to execute an external program with # given options in specified directory. Because of the dead-lock issue during -# redirecting output of the external program, threads are used to to do the
+# redirecting output of the external program, threads are used to do
+the
# redirection work.
#
# @param Command A list or string containing the call of the program
--
2.25.1
-=-=-=-=-=-=
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93193): https://edk2.groups.io/g/devel/message/93193
Mute This Topic: https://groups.io/mt/93496599/1768742
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [bob.c.feng@intel.com] -=-=-=-=-=-=
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PATCH v2 03/17] CryptoPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
2022-09-06 8:28 ` [PATCH v2 01/17] .pytool: " PierreGondois
2022-09-06 8:28 ` [PATCH v2 02/17] BaseTools: " PierreGondois
@ 2022-09-06 8:28 ` PierreGondois
2022-09-06 8:28 ` [PATCH v2 04/17] DynamicTablesPkg: " PierreGondois
` (13 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:28 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Jiewen Yao, Jian J Wang
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
CryptoPkg/Library/BaseCryptLib/Hash/CryptCShake256.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/CryptoPkg/Library/BaseCryptLib/Hash/CryptCShake256.c b/CryptoPkg/Library/BaseCryptLib/Hash/CryptCShake256.c
index 2a9eaf9eec40..76ed211083c2 100644
--- a/CryptoPkg/Library/BaseCryptLib/Hash/CryptCShake256.c
+++ b/CryptoPkg/Library/BaseCryptLib/Hash/CryptCShake256.c
@@ -83,7 +83,7 @@ CShake256Init (
AbsorbLen = 0;
//
- // Absorb Absorb bytepad(.., rate).
+ // Absorb bytepad(.., rate).
//
EncLen = LeftEncode (EncBuf, CSHAKE256_RATE_IN_BYTES);
Status = (BOOLEAN)Sha3Update ((Keccak1600_Ctx *)CShake256Context, EncBuf, EncLen);
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 04/17] DynamicTablesPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (2 preceding siblings ...)
2022-09-06 8:28 ` [PATCH v2 03/17] CryptoPkg: " PierreGondois
@ 2022-09-06 8:28 ` PierreGondois
2022-12-13 14:37 ` Sami Mujawar
2022-09-06 8:29 ` [PATCH v2 05/17] EmulatorPkg: " PierreGondois
` (12 subsequent siblings)
16 siblings, 1 reply; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:28 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Sami Mujawar, Alexei Fedorov, Sami Mujawar
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Sami Mujawar <Sami.Mujawar@arm.com>
Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
---
DynamicTablesPkg/Include/Library/HwInfoParserLib.h | 2 +-
.../Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c | 2 +-
.../Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c | 2 +-
DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h | 2 +-
DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c | 2 +-
DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h | 2 +-
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c | 2 +-
DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c | 2 +-
.../Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c | 2 +-
.../Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c | 2 +-
10 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/DynamicTablesPkg/Include/Library/HwInfoParserLib.h b/DynamicTablesPkg/Include/Library/HwInfoParserLib.h
index 05c21aeac00a..9710f7466170 100644
--- a/DynamicTablesPkg/Include/Library/HwInfoParserLib.h
+++ b/DynamicTablesPkg/Include/Library/HwInfoParserLib.h
@@ -83,7 +83,7 @@ HwInfoParse (
);
/** Cleanup any internal state and resources that were allocated
- by the the HwInfoParser.
+ by the HwInfoParser.
@param [in] ParserHandle A handle to the parser instance.
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c
index 18c8b8d24525..b990686d40a3 100644
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c
@@ -587,7 +587,7 @@ BuildSsdtCmn600TableEx (
return Status;
}
- // Setup the table list early so that that appropriate cleanup
+ // Setup the table list early so that appropriate cleanup
// can be done in case of failure.
*Table = TableList;
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c
index 39cfbad95926..b850320eede7 100644
--- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c
+++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c
@@ -239,7 +239,7 @@ BuildSsdtSerialPortTableEx (
return Status;
}
- // Setup the table list early so that that appropriate cleanup
+ // Setup the table list early so that appropriate cleanup
// can be done in case of failure.
*Table = TableList;
diff --git a/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h b/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h
index 8ceb20ff415e..d199f06cc54d 100644
--- a/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h
+++ b/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h
@@ -34,7 +34,7 @@ typedef AML_NODE_HEADER *AML_NODE_HANDLE;
/** AML root node.
- The root node is unique and at the head of of tree. It is a fake node used
+ The root node is unique and at the head of tree. It is a fake node used
to maintain the list of AML statements (stored as object nodes) which are
at the first scope level.
*/
diff --git a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c
index 391c72bd322e..8f460a972841 100644
--- a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c
+++ b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c
@@ -27,7 +27,7 @@
of NameSegs that have less that 4 chars, like "DEV". "DEV" will be expanded
as "DEV_".
- An AML NameString is not NULL terminated and is is only composed of
+ An AML NameString is not NULL terminated and is only composed of
4 chars long NameSegs.
@param [in] NameOpNode NameOp object node defining a variable.
diff --git a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h
index 8bbf62bbcd24..2b4c60d3ee14 100644
--- a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h
+++ b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h
@@ -19,7 +19,7 @@
of NameSegs that have less that 4 chars, like "DEV". "DEV" will be expanded
as "DEV_".
- An AML NameString is not NULL terminated and is is only composed of
+ An AML NameString is not NULL terminated and is only composed of
4 chars long NameSegs.
@param [in] NameOpNode NameOp object node defining a variable.
diff --git a/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c b/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
index 5fb39d077b7a..d4768193cb1f 100644
--- a/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
+++ b/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
@@ -1071,7 +1071,7 @@ AmlAddPrtEntry (
goto error_handler;
}
- // Append to the the list of _PRT entries.
+ // Append to the list of _PRT entries.
Status = AmlVarListAddTail (
(AML_NODE_HANDLE)PrtEntryList,
(AML_NODE_HANDLE)PackageNode
diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c
index 35e34e765f71..376de078bcb1 100644
--- a/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c
+++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c
@@ -168,7 +168,7 @@ HwInfoParse (
}
/** Cleanup any internal state and resources that were allocated
- by the the HwInfoParser.
+ by the HwInfoParser.
@param [in] ParserHandle A handle to the parser instance.
diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c
index 831f0d5b7365..1f3af1f968a9 100644
--- a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c
+++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c
@@ -130,7 +130,7 @@ ArmGicDispatcher (
Fdt = FdtParserHandle->Fdt;
- // The "cpus" node resides at the the root of the DT. Fetch it.
+ // The "cpus" node resides at the root of the DT. Fetch it.
CpusNode = fdt_path_offset (Fdt, "/cpus");
if (CpusNode < 0) {
return EFI_NOT_FOUND;
diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c
index 0557e416b44c..cfd032df4d74 100644
--- a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c
+++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c
@@ -278,7 +278,7 @@ GetSerialConsoleNode (
return EFI_INVALID_PARAMETER;
}
- // The "chosen" node resides at the the root of the DT. Fetch it.
+ // The "chosen" node resides at the root of the DT. Fetch it.
ChosenNode = fdt_path_offset (Fdt, "/chosen");
if (ChosenNode < 0) {
return EFI_NOT_FOUND;
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 04/17] DynamicTablesPkg: Remove duplicated words
2022-09-06 8:28 ` [PATCH v2 04/17] DynamicTablesPkg: " PierreGondois
@ 2022-12-13 14:37 ` Sami Mujawar
0 siblings, 0 replies; 27+ messages in thread
From: Sami Mujawar @ 2022-12-13 14:37 UTC (permalink / raw)
To: Pierre.Gondois, devel; +Cc: Alexei Fedorov, nd@arm.com
Merged as 5890a18409a0..11ce7a2de754
Thanks.
Regards,
Sami Mujawar
On 06/09/2022 09:28 am, Pierre.Gondois@arm.com wrote:
> From: Pierre Gondois <pierre.gondois@arm.com>
>
> In an effort to clean the documentation of the above
> package, remove duplicated words.
>
> Cc: Sami Mujawar <Sami.Mujawar@arm.com>
> Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
> Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
> ---
> DynamicTablesPkg/Include/Library/HwInfoParserLib.h | 2 +-
> .../Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c | 2 +-
> .../Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c | 2 +-
> DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h | 2 +-
> DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c | 2 +-
> DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h | 2 +-
> DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c | 2 +-
> DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c | 2 +-
> .../Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c | 2 +-
> .../Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c | 2 +-
> 10 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/DynamicTablesPkg/Include/Library/HwInfoParserLib.h b/DynamicTablesPkg/Include/Library/HwInfoParserLib.h
> index 05c21aeac00a..9710f7466170 100644
> --- a/DynamicTablesPkg/Include/Library/HwInfoParserLib.h
> +++ b/DynamicTablesPkg/Include/Library/HwInfoParserLib.h
> @@ -83,7 +83,7 @@ HwInfoParse (
> );
>
> /** Cleanup any internal state and resources that were allocated
> - by the the HwInfoParser.
> + by the HwInfoParser.
>
> @param [in] ParserHandle A handle to the parser instance.
>
> diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c
> index 18c8b8d24525..b990686d40a3 100644
> --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c
> +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtCmn600LibArm/SsdtCmn600Generator.c
> @@ -587,7 +587,7 @@ BuildSsdtCmn600TableEx (
> return Status;
> }
>
> - // Setup the table list early so that that appropriate cleanup
> + // Setup the table list early so that appropriate cleanup
> // can be done in case of failure.
> *Table = TableList;
>
> diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c
> index 39cfbad95926..b850320eede7 100644
> --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c
> +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c
> @@ -239,7 +239,7 @@ BuildSsdtSerialPortTableEx (
> return Status;
> }
>
> - // Setup the table list early so that that appropriate cleanup
> + // Setup the table list early so that appropriate cleanup
> // can be done in case of failure.
> *Table = TableList;
>
> diff --git a/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h b/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h
> index 8ceb20ff415e..d199f06cc54d 100644
> --- a/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h
> +++ b/DynamicTablesPkg/Library/Common/AmlLib/AmlNodeDefines.h
> @@ -34,7 +34,7 @@ typedef AML_NODE_HEADER *AML_NODE_HANDLE;
>
> /** AML root node.
>
> - The root node is unique and at the head of of tree. It is a fake node used
> + The root node is unique and at the head of tree. It is a fake node used
> to maintain the list of AML statements (stored as object nodes) which are
> at the first scope level.
> */
> diff --git a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c
> index 391c72bd322e..8f460a972841 100644
> --- a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c
> +++ b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.c
> @@ -27,7 +27,7 @@
> of NameSegs that have less that 4 chars, like "DEV". "DEV" will be expanded
> as "DEV_".
>
> - An AML NameString is not NULL terminated and is is only composed of
> + An AML NameString is not NULL terminated and is only composed of
> 4 chars long NameSegs.
>
> @param [in] NameOpNode NameOp object node defining a variable.
> diff --git a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h
> index 8bbf62bbcd24..2b4c60d3ee14 100644
> --- a/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h
> +++ b/DynamicTablesPkg/Library/Common/AmlLib/Api/AmlApiHelper.h
> @@ -19,7 +19,7 @@
> of NameSegs that have less that 4 chars, like "DEV". "DEV" will be expanded
> as "DEV_".
>
> - An AML NameString is not NULL terminated and is is only composed of
> + An AML NameString is not NULL terminated and is only composed of
> 4 chars long NameSegs.
>
> @param [in] NameOpNode NameOp object node defining a variable.
> diff --git a/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c b/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
> index 5fb39d077b7a..d4768193cb1f 100644
> --- a/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
> +++ b/DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
> @@ -1071,7 +1071,7 @@ AmlAddPrtEntry (
> goto error_handler;
> }
>
> - // Append to the the list of _PRT entries.
> + // Append to the list of _PRT entries.
> Status = AmlVarListAddTail (
> (AML_NODE_HANDLE)PrtEntryList,
> (AML_NODE_HANDLE)PackageNode
> diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c
> index 35e34e765f71..376de078bcb1 100644
> --- a/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c
> +++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/FdtHwInfoParser.c
> @@ -168,7 +168,7 @@ HwInfoParse (
> }
>
> /** Cleanup any internal state and resources that were allocated
> - by the the HwInfoParser.
> + by the HwInfoParser.
>
> @param [in] ParserHandle A handle to the parser instance.
>
> diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c
> index 831f0d5b7365..1f3af1f968a9 100644
> --- a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c
> +++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Gic/ArmGicDispatcher.c
> @@ -130,7 +130,7 @@ ArmGicDispatcher (
>
> Fdt = FdtParserHandle->Fdt;
>
> - // The "cpus" node resides at the the root of the DT. Fetch it.
> + // The "cpus" node resides at the root of the DT. Fetch it.
> CpusNode = fdt_path_offset (Fdt, "/cpus");
> if (CpusNode < 0) {
> return EFI_NOT_FOUND;
> diff --git a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c
> index 0557e416b44c..cfd032df4d74 100644
> --- a/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c
> +++ b/DynamicTablesPkg/Library/FdtHwInfoParserLib/Serial/ArmSerialPortParser.c
> @@ -278,7 +278,7 @@ GetSerialConsoleNode (
> return EFI_INVALID_PARAMETER;
> }
>
> - // The "chosen" node resides at the the root of the DT. Fetch it.
> + // The "chosen" node resides at the root of the DT. Fetch it.
> ChosenNode = fdt_path_offset (Fdt, "/chosen");
> if (ChosenNode < 0) {
> return EFI_NOT_FOUND;
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v2 05/17] EmulatorPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (3 preceding siblings ...)
2022-09-06 8:28 ` [PATCH v2 04/17] DynamicTablesPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 06/17] FatPkg: " PierreGondois
` (11 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Andrew Fish, Ray Ni
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Andrew Fish <afish@apple.com>
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
EmulatorPkg/FvbServicesRuntimeDxe/FWBlockService.c | 2 +-
EmulatorPkg/Win/Host/WinThunk.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/EmulatorPkg/FvbServicesRuntimeDxe/FWBlockService.c b/EmulatorPkg/FvbServicesRuntimeDxe/FWBlockService.c
index 1c4d8120c1c9..29fdaeafc223 100644
--- a/EmulatorPkg/FvbServicesRuntimeDxe/FWBlockService.c
+++ b/EmulatorPkg/FvbServicesRuntimeDxe/FWBlockService.c
@@ -1212,7 +1212,7 @@ Returns:
//
// Only need to allocate once. There is only one copy of physical memory for
// the private data of each FV instance. But in virtual mode or in physical
- // mode, the address of the the physical memory may be different.
+ // mode, the address of the physical memory may be different.
//
Status = gBS->AllocatePool (
EfiRuntimeServicesData,
diff --git a/EmulatorPkg/Win/Host/WinThunk.c b/EmulatorPkg/Win/Host/WinThunk.c
index 008e5755dbde..5d5799deefef 100644
--- a/EmulatorPkg/Win/Host/WinThunk.c
+++ b/EmulatorPkg/Win/Host/WinThunk.c
@@ -23,7 +23,7 @@ Abstract:
API calls, so please reference Microsoft documentation.
- gWinNt is a a public exported global that contains the initialized
+ gWinNt is a public exported global that contains the initialized
data.
**/
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 06/17] FatPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (4 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 05/17] EmulatorPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 07/17] MdeModulePkg: " PierreGondois
` (10 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Ray Ni
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
FatPkg/EnhancedFatDxe/UnicodeCollation.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/FatPkg/EnhancedFatDxe/UnicodeCollation.c b/FatPkg/EnhancedFatDxe/UnicodeCollation.c
index 813f153617ec..aac81bc09611 100644
--- a/FatPkg/EnhancedFatDxe/UnicodeCollation.c
+++ b/FatPkg/EnhancedFatDxe/UnicodeCollation.c
@@ -76,7 +76,7 @@ InitializeUnicodeCollationSupportWorker (
}
//
- // Find the best matching matching language from the supported languages
+ // Find the best matching language from the supported languages
// of Unicode Collation (2) protocol.
//
BestLanguage = GetBestLanguage (
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 07/17] MdeModulePkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (5 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 06/17] FatPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 08/17] MdePkg: " PierreGondois
` (9 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Jian J Wang, Liming Gao
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c | 2 +-
MdeModulePkg/Bus/Pci/EhciPei/EhcPeim.c | 2 +-
.../Pci/NonDiscoverablePciDeviceDxe/ComponentName.c | 4 ++--
.../Bus/Pci/PciBusDxe/PciEnumeratorSupport.c | 2 +-
MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c | 12 ++++++------
MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h | 12 ++++++------
.../Bus/Pci/SataControllerDxe/ComponentName.c | 4 ++--
.../Bus/Pci/SataControllerDxe/SataController.h | 4 ++--
MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.c | 2 +-
MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.h | 2 +-
MdeModulePkg/Bus/Pci/XhciDxe/Xhci.c | 4 ++--
MdeModulePkg/Bus/Pci/XhciDxe/Xhci.h | 2 +-
MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c | 8 ++++----
MdeModulePkg/Bus/Pci/XhciPei/XhcPeim.c | 2 +-
MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c | 6 +++---
MdeModulePkg/Core/Dxe/DxeMain.h | 2 +-
MdeModulePkg/Core/Dxe/Mem/Page.c | 4 ++--
MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c | 2 +-
MdeModulePkg/Core/DxeIplPeim/Ia32/DxeLoadFunc.c | 2 +-
MdeModulePkg/Core/DxeIplPeim/X64/VirtualMemory.c | 6 +++---
MdeModulePkg/Include/Guid/MdeModuleHii.h | 2 +-
.../Include/Library/DisplayUpdateProgressLib.h | 2 +-
MdeModulePkg/Include/Library/UefiBootManagerLib.h | 2 +-
.../DisplayUpdateProgressLibGraphics.c | 4 ++--
.../DisplayUpdateProgressLibText.c | 2 +-
.../Library/PiDxeS3BootScriptLib/BootScriptSave.c | 4 ++--
MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 2 +-
MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c | 2 +-
MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c | 2 +-
MdeModulePkg/Library/UefiHiiLib/HiiLib.c | 2 +-
MdeModulePkg/Library/UefiHiiLib/HiiString.c | 2 +-
MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 2 +-
MdeModulePkg/Universal/BdsDxe/Language.c | 2 +-
MdeModulePkg/Universal/CapsulePei/UefiCapsule.c | 4 ++--
.../Universal/Console/ConSplitterDxe/ComponentName.c | 2 +-
.../Universal/Console/ConSplitterDxe/ConSplitter.h | 2 +-
.../Universal/DebugSupportDxe/Ia32/AsmFuncs.nasm | 2 +-
.../Universal/DebugSupportDxe/X64/AsmFuncs.nasm | 2 +-
MdeModulePkg/Universal/Disk/DiskIoDxe/DiskIo.c | 2 +-
.../Universal/DisplayEngineDxe/FormDisplay.c | 2 +-
MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c | 2 +-
.../FvSimpleFileSystemDxe/FvSimpleFileSystem.c | 2 +-
.../FvSimpleFileSystemDxe/FvSimpleFileSystemDxe.inf | 2 +-
.../FvSimpleFileSystemEntryPoint.c | 4 ++--
.../Universal/HiiDatabaseDxe/ConfigRouting.c | 2 +-
.../PlatformDriOverrideDxe/PlatDriOverrideLib.c | 2 +-
MdeModulePkg/Universal/SerialDxe/SerialIo.c | 4 ++--
.../Universal/SetupBrowserDxe/Presentation.c | 2 +-
48 files changed, 75 insertions(+), 75 deletions(-)
diff --git a/MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c b/MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c
index 0b7270f4e930..a872dd833fd1 100644
--- a/MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c
+++ b/MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c
@@ -812,7 +812,7 @@ ON_EXIT:
from or receive into.
@param DataLength The lenght of the data buffer.
@param DataToggle On input, the initial data toggle for the transfer;
- On output, it is updated to to next data toggle to
+ On output, it is updated to next data toggle to
use of the subsequent bulk transfer.
@param TimeOut Indicates the maximum time, in millisecond, which
the transfer is allowed to complete.
diff --git a/MdeModulePkg/Bus/Pci/EhciPei/EhcPeim.c b/MdeModulePkg/Bus/Pci/EhciPei/EhcPeim.c
index 37dd9012e277..3ead647ed1a1 100644
--- a/MdeModulePkg/Bus/Pci/EhciPei/EhcPeim.c
+++ b/MdeModulePkg/Bus/Pci/EhciPei/EhcPeim.c
@@ -529,7 +529,7 @@ EhcInitHC (
from or receive into.
@param DataLength The lenght of the data buffer.
@param DataToggle On input, the initial data toggle for the transfer;
- On output, it is updated to to next data toggle to use of
+ On output, it is updated to next data toggle to use of
the subsequent bulk transfer.
@param TimeOut Indicates the maximum time, in millisecond, which the
transfer is allowed to complete.
diff --git a/MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe/ComponentName.c b/MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe/ComponentName.c
index af1b2e5526bc..1cb5789052ad 100644
--- a/MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe/ComponentName.c
+++ b/MdeModulePkg/Bus/Pci/NonDiscoverablePciDeviceDxe/ComponentName.c
@@ -29,7 +29,7 @@ EFI_COMPONENT_NAME_PROTOCOL gComponentName;
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -79,7 +79,7 @@ NonDiscoverablePciGetDriverName (
that wishes to retrieve the name of a child controller.
@param Language A pointer to a three character ISO 639-2 language
identifier. This is the language of the controller name
- that that the caller is requesting, and it must match one
+ that the caller is requesting, and it must match one
of the languages specified in SupportedLanguages. The
number of languages supported by a driver is up to the
driver writer.
diff --git a/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c b/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c
index 509f828b621d..bc901f1f0a4e 100644
--- a/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c
+++ b/MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c
@@ -1085,7 +1085,7 @@ PciSetDeviceAttribute (
// When this attribute is clear, the RomImage and RomSize fields in the PCI IO were
// initialized based on the PCI option ROM found through the ROM BAR of the PCI controller.
// When this attribute is set, the PCI option ROM described by the RomImage and RomSize
- // fields is not from the the ROM BAR of the PCI controller.
+ // fields is not from the ROM BAR of the PCI controller.
//
if (!PciIoDevice->EmbeddedRom) {
Attributes |= EFI_PCI_IO_ATTRIBUTE_EMBEDDED_ROM;
diff --git a/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c b/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c
index 843815d0cb18..cd464b44f3b0 100644
--- a/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c
+++ b/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c
@@ -73,12 +73,12 @@ InitializePciIoInstance (
**/
EFI_STATUS
PciIoVerifyBarAccess (
- IN PCI_IO_DEVICE *PciIoDevice,
- IN UINT8 BarIndex,
- IN PCI_BAR_TYPE Type,
- IN IN EFI_PCI_IO_PROTOCOL_WIDTH Width,
- IN IN UINTN Count,
- IN UINT64 *Offset
+ IN PCI_IO_DEVICE *PciIoDevice,
+ IN UINT8 BarIndex,
+ IN PCI_BAR_TYPE Type,
+ IN EFI_PCI_IO_PROTOCOL_WIDTH Width,
+ IN UINTN Count,
+ IN UINT64 *Offset
)
{
if ((UINT32)Width >= EfiPciIoWidthMaximum) {
diff --git a/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h b/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h
index a73bd06bcf0a..3d4a37fb2d99 100644
--- a/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h
+++ b/MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h
@@ -37,12 +37,12 @@ InitializePciIoInstance (
**/
EFI_STATUS
PciIoVerifyBarAccess (
- IN PCI_IO_DEVICE *PciIoDevice,
- IN UINT8 BarIndex,
- IN PCI_BAR_TYPE Type,
- IN IN EFI_PCI_IO_PROTOCOL_WIDTH Width,
- IN IN UINTN Count,
- IN UINT64 *Offset
+ IN PCI_IO_DEVICE *PciIoDevice,
+ IN UINT8 BarIndex,
+ IN PCI_BAR_TYPE Type,
+ IN EFI_PCI_IO_PROTOCOL_WIDTH Width,
+ IN UINTN Count,
+ IN UINT64 *Offset
);
/**
diff --git a/MdeModulePkg/Bus/Pci/SataControllerDxe/ComponentName.c b/MdeModulePkg/Bus/Pci/SataControllerDxe/ComponentName.c
index 9cc416fc1f90..9e127fc65847 100644
--- a/MdeModulePkg/Bus/Pci/SataControllerDxe/ComponentName.c
+++ b/MdeModulePkg/Bus/Pci/SataControllerDxe/ComponentName.c
@@ -59,7 +59,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED EFI_UNICODE_STRING_TABLE mSataControllerControlle
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -108,7 +108,7 @@ SataControllerComponentNameGetDriverName (
that wishes to retrieve the name of a child controller.
@param Language A pointer to a three character ISO 639-2 language
identifier. This is the language of the controller name
- that that the caller is requesting, and it must match one
+ that the caller is requesting, and it must match one
of the languages specified in SupportedLanguages. The
number of languages supported by a driver is up to the
driver writer.
diff --git a/MdeModulePkg/Bus/Pci/SataControllerDxe/SataController.h b/MdeModulePkg/Bus/Pci/SataControllerDxe/SataController.h
index 4d545fb1f958..466670599a7e 100644
--- a/MdeModulePkg/Bus/Pci/SataControllerDxe/SataController.h
+++ b/MdeModulePkg/Bus/Pci/SataControllerDxe/SataController.h
@@ -468,7 +468,7 @@ IdeInitSetTiming (
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -508,7 +508,7 @@ SataControllerComponentNameGetDriverName (
that wishes to retrieve the name of a child controller.
@param Language A pointer to a three character ISO 639-2 language
identifier. This is the language of the controller name
- that that the caller is requesting, and it must match one
+ that the caller is requesting, and it must match one
of the languages specified in SupportedLanguages. The
number of languages supported by a driver is up to the
driver writer.
diff --git a/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.c b/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.c
index 96abf3ab13bd..f325547f990d 100644
--- a/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.c
+++ b/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.c
@@ -468,7 +468,7 @@ UhcControlTransfer (
from or receive into.
@param DataLength The lenght of the data buffer.
@param DataToggle On input, the initial data toggle for the transfer;
- On output, it is updated to to next data toggle to use of
+ On output, it is updated to next data toggle to use of
the subsequent bulk transfer.
@param TimeOut Indicates the maximum time, in millisecond, which the
transfer is allowed to complete.
diff --git a/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.h b/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.h
index 5b135f2558fd..29e5c8af7a1d 100644
--- a/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.h
+++ b/MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.h
@@ -243,7 +243,7 @@ UhcControlTransfer (
from or receive into.
@param DataLength The lenght of the data buffer.
@param DataToggle On input, the initial data toggle for the transfer;
- On output, it is updated to to next data toggle to use of
+ On output, it is updated to next data toggle to use of
the subsequent bulk transfer.
@param TimeOut Indicates the maximum time, in millisecond, which the
transfer is allowed to complete.
diff --git a/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.c b/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.c
index c05431ff30ec..d110399ec1e7 100644
--- a/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.c
+++ b/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.c
@@ -965,7 +965,7 @@ XhcControlTransfer (
//
// The actual device address has been assigned by XHCI during initializing the device slot.
// So we just need establish the mapping relationship between the device address requested from UsbBus
- // and the actual device address assigned by XHCI. The the following invocations through EFI_USB2_HC_PROTOCOL interface
+ // and the actual device address assigned by XHCI. the following invocations through EFI_USB2_HC_PROTOCOL interface
// can find out the actual device address by it.
//
Xhc->UsbDevContext[SlotId].BusDevAddr = (UINT8)Request->Value;
@@ -1206,7 +1206,7 @@ ON_EXIT:
from or receive into.
@param DataLength The lenght of the data buffer.
@param DataToggle On input, the initial data toggle for the transfer;
- On output, it is updated to to next data toggle to
+ On output, it is updated to next data toggle to
use of the subsequent bulk transfer.
@param Timeout Indicates the maximum time, in millisecond, which
the transfer is allowed to complete.
diff --git a/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.h b/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.h
index ca223bd20cbd..c0c57d48811f 100644
--- a/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.h
+++ b/MdeModulePkg/Bus/Pci/XhciDxe/Xhci.h
@@ -524,7 +524,7 @@ XhcControlTransfer (
from or receive into.
@param DataLength The lenght of the data buffer.
@param DataToggle On input, the initial data toggle for the transfer;
- On output, it is updated to to next data toggle to
+ On output, it is updated to next data toggle to
use of the subsequent bulk transfer.
@param Timeout Indicates the maximum time, in millisecond, which
the transfer is allowed to complete.
diff --git a/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c b/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c
index c2906e06fd84..d42b7c8d09ec 100644
--- a/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c
+++ b/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c
@@ -2875,7 +2875,7 @@ XhcInitializeEndpointContext (
}
//
- // Get the bInterval from descriptor and init the the interval field of endpoint context.
+ // Get the bInterval from descriptor and init the interval field of endpoint context.
// Refer to XHCI 1.1 spec section 6.2.3.6.
//
if (DeviceSpeed == EFI_USB_SPEED_FULL) {
@@ -2906,7 +2906,7 @@ XhcInitializeEndpointContext (
InputContext->EP[Dci-1].AverageTRBLength = 0x1000;
InputContext->EP[Dci-1].MaxESITPayload = EpDesc->MaxPacketSize;
//
- // Get the bInterval from descriptor and init the the interval field of endpoint context
+ // Get the bInterval from descriptor and init the interval field of endpoint context
//
if ((DeviceSpeed == EFI_USB_SPEED_FULL) || (DeviceSpeed == EFI_USB_SPEED_LOW)) {
Interval = EpDesc->Interval;
@@ -3074,7 +3074,7 @@ XhcInitializeEndpointContext64 (
}
//
- // Get the bInterval from descriptor and init the the interval field of endpoint context.
+ // Get the bInterval from descriptor and init the interval field of endpoint context.
// Refer to XHCI 1.1 spec section 6.2.3.6.
//
if (DeviceSpeed == EFI_USB_SPEED_FULL) {
@@ -3105,7 +3105,7 @@ XhcInitializeEndpointContext64 (
InputContext->EP[Dci-1].AverageTRBLength = 0x1000;
InputContext->EP[Dci-1].MaxESITPayload = EpDesc->MaxPacketSize;
//
- // Get the bInterval from descriptor and init the the interval field of endpoint context
+ // Get the bInterval from descriptor and init the interval field of endpoint context
//
if ((DeviceSpeed == EFI_USB_SPEED_FULL) || (DeviceSpeed == EFI_USB_SPEED_LOW)) {
Interval = EpDesc->Interval;
diff --git a/MdeModulePkg/Bus/Pci/XhciPei/XhcPeim.c b/MdeModulePkg/Bus/Pci/XhciPei/XhcPeim.c
index 301f376b040d..292eb2b8f2a4 100644
--- a/MdeModulePkg/Bus/Pci/XhciPei/XhcPeim.c
+++ b/MdeModulePkg/Bus/Pci/XhciPei/XhcPeim.c
@@ -862,7 +862,7 @@ ON_EXIT:
from or receive into.
@param DataLength The lenght of the data buffer.
@param DataToggle On input, the initial data toggle for the transfer;
- On output, it is updated to to next data toggle to use of
+ On output, it is updated to next data toggle to use of
the subsequent bulk transfer.
@param TimeOut Indicates the maximum time, in millisecond, which the
transfer is allowed to complete.
diff --git a/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c b/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c
index cc597a437174..835df8fa317d 100644
--- a/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c
+++ b/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c
@@ -1807,7 +1807,7 @@ XhcPeiSetConfigCmd (
}
//
- // Get the bInterval from descriptor and init the the interval field of endpoint context.
+ // Get the bInterval from descriptor and init the interval field of endpoint context.
// Refer to XHCI 1.1 spec section 6.2.3.6.
//
if (DeviceSpeed == EFI_USB_SPEED_FULL) {
@@ -2030,7 +2030,7 @@ XhcPeiSetConfigCmd64 (
}
//
- // Get the bInterval from descriptor and init the the interval field of endpoint context.
+ // Get the bInterval from descriptor and init the interval field of endpoint context.
// Refer to XHCI 1.1 spec section 6.2.3.6.
//
if (DeviceSpeed == EFI_USB_SPEED_FULL) {
@@ -2061,7 +2061,7 @@ XhcPeiSetConfigCmd64 (
InputContext->EP[Dci-1].AverageTRBLength = 0x1000;
InputContext->EP[Dci-1].MaxESITPayload = EpDesc->MaxPacketSize;
//
- // Get the bInterval from descriptor and init the the interval field of endpoint context
+ // Get the bInterval from descriptor and init the interval field of endpoint context
//
if ((DeviceSpeed == EFI_USB_SPEED_FULL) || (DeviceSpeed == EFI_USB_SPEED_LOW)) {
Interval = EpDesc->Interval;
diff --git a/MdeModulePkg/Core/Dxe/DxeMain.h b/MdeModulePkg/Core/Dxe/DxeMain.h
index 815a6b4bd844..f4004e3ace24 100644
--- a/MdeModulePkg/Core/Dxe/DxeMain.h
+++ b/MdeModulePkg/Core/Dxe/DxeMain.h
@@ -2769,7 +2769,7 @@ MemoryProtectionExitBootServicesCallback (
@param Memory The base address of the range
@param Length The size of the range (in bytes)
- @return EFI_SUCCESS If the the CPU arch protocol is not installed yet
+ @return EFI_SUCCESS If the CPU arch protocol is not installed yet
@return EFI_SUCCESS If no DXE memory protection policy has been configured
@return EFI_SUCCESS If OldType and NewType use the same permission attributes
@return other Return value of gCpu->SetMemoryAttributes()
diff --git a/MdeModulePkg/Core/Dxe/Mem/Page.c b/MdeModulePkg/Core/Dxe/Mem/Page.c
index cc0b90ac0df5..db4cbe054c5d 100644
--- a/MdeModulePkg/Core/Dxe/Mem/Page.c
+++ b/MdeModulePkg/Core/Dxe/Mem/Page.c
@@ -192,7 +192,7 @@ CoreAddRange (
// UEFI 2.0 added an event group for notificaiton on memory map changes.
// So we need to signal this Event Group every time the memory map changes.
// If we are in EFI 1.10 compatability mode no event groups will be
- // found and nothing will happen we we call this function. These events
+ // found and nothing will happen we call this function. These events
// will get signaled but since a lock is held around the call to this
// function the notificaiton events will only be called after this function
// returns and the lock is released.
@@ -659,7 +659,7 @@ CoreAddMemoryDescriptor (
}
//
- // There was enough system memory for all the the memory types were allocated. So,
+ // There was enough system memory for all the memory types were allocated. So,
// those memory areas can be freed for future allocations, and all future memory
// allocations can occur within their respective bins
//
diff --git a/MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c b/MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c
index b89ab046fa73..c794e6e89a66 100644
--- a/MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c
+++ b/MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c
@@ -1231,7 +1231,7 @@ IsInSmm (
@return EFI_SUCCESS If we are executing in SMM mode. No permission attributes
are updated in this case
- @return EFI_SUCCESS If the the CPU arch protocol is not installed yet
+ @return EFI_SUCCESS If the CPU arch protocol is not installed yet
@return EFI_SUCCESS If no DXE memory protection policy has been configured
@return EFI_SUCCESS If OldType and NewType use the same permission attributes
@return other Return value of gCpu->SetMemoryAttributes()
diff --git a/MdeModulePkg/Core/DxeIplPeim/Ia32/DxeLoadFunc.c b/MdeModulePkg/Core/DxeIplPeim/Ia32/DxeLoadFunc.c
index fdeaaa39d88e..499535891c20 100644
--- a/MdeModulePkg/Core/DxeIplPeim/Ia32/DxeLoadFunc.c
+++ b/MdeModulePkg/Core/DxeIplPeim/Ia32/DxeLoadFunc.c
@@ -123,7 +123,7 @@ Create4GPageTablesIa32Pae (
for (IndexOfPdpEntries = 0; IndexOfPdpEntries < NumberOfPdpEntriesNeeded; IndexOfPdpEntries++, PageDirectoryPointerEntry++) {
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (VOID *)PageAddress;
PageAddress += SIZE_4KB;
diff --git a/MdeModulePkg/Core/DxeIplPeim/X64/VirtualMemory.c b/MdeModulePkg/Core/DxeIplPeim/X64/VirtualMemory.c
index 18b121d7688c..892941c87d23 100644
--- a/MdeModulePkg/Core/DxeIplPeim/X64/VirtualMemory.c
+++ b/MdeModulePkg/Core/DxeIplPeim/X64/VirtualMemory.c
@@ -833,7 +833,7 @@ CreateIdentityMappingPageTables (
{
//
// Each PML5 entry points to a page of PML4 entires.
- // So lets allocate space for them and fill them in in the IndexOfPml4Entries loop.
+ // So lets allocate space for them and fill them in the IndexOfPml4Entries loop.
// When 5-Level Paging is disabled, below allocation happens only once.
//
PageMapLevel4Entry = (VOID *)BigPageAddress;
@@ -855,7 +855,7 @@ CreateIdentityMappingPageTables (
{
//
// Each PML4 entry points to a page of Page Directory Pointer entires.
- // So lets allocate space for them and fill them in in the IndexOfPdpEntries loop.
+ // So lets allocate space for them and fill them in the IndexOfPdpEntries loop.
//
PageDirectoryPointerEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
@@ -890,7 +890,7 @@ CreateIdentityMappingPageTables (
{
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
diff --git a/MdeModulePkg/Include/Guid/MdeModuleHii.h b/MdeModulePkg/Include/Guid/MdeModuleHii.h
index 798bcbaaae93..ff9dc1600cb2 100644
--- a/MdeModulePkg/Include/Guid/MdeModuleHii.h
+++ b/MdeModulePkg/Include/Guid/MdeModuleHii.h
@@ -200,7 +200,7 @@ typedef struct _EFI_IFR_GUID_VAREQNAME {
/// For vareqval (0x100), NameId is 0x100.
/// This value will convert to a Unicode String following this rule;
/// sprintf(StringBuffer, "%d", NameId) .
- /// The the Unicode String will be used as a EFI Variable Name.
+ /// the Unicode String will be used as a EFI Variable Name.
///
UINT16 NameId;
} EFI_IFR_GUID_VAREQNAME;
diff --git a/MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h b/MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
index bb262413806c..ab2c644c6af7 100644
--- a/MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
+++ b/MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
@@ -22,7 +22,7 @@
@param[in] Completion A value between 0 and 100 indicating the current
completion progress of a firmware update. This
- value must the the same or higher than previous
+ value must the same or higher than previous
calls to this service. The first call of 0 or a
value of 0 after reaching a value of 100 resets
the progress indicator to 0.
diff --git a/MdeModulePkg/Include/Library/UefiBootManagerLib.h b/MdeModulePkg/Include/Library/UefiBootManagerLib.h
index 34e217707e71..b9f3ae4cf211 100644
--- a/MdeModulePkg/Include/Library/UefiBootManagerLib.h
+++ b/MdeModulePkg/Include/Library/UefiBootManagerLib.h
@@ -683,7 +683,7 @@ EfiBootManagerGetGopDevicePath (
);
/**
- Connect the platform active active video controller.
+ Connect the platform active video controller.
@param VideoController PCI handle of video controller.
diff --git a/MdeModulePkg/Library/DisplayUpdateProgressLibGraphics/DisplayUpdateProgressLibGraphics.c b/MdeModulePkg/Library/DisplayUpdateProgressLibGraphics/DisplayUpdateProgressLibGraphics.c
index 83053464e06e..b002bae0c3f6 100644
--- a/MdeModulePkg/Library/DisplayUpdateProgressLibGraphics/DisplayUpdateProgressLibGraphics.c
+++ b/MdeModulePkg/Library/DisplayUpdateProgressLibGraphics/DisplayUpdateProgressLibGraphics.c
@@ -21,7 +21,7 @@
#include <Protocol/BootLogo2.h>
//
-// Values in percent of of logo height.
+// Values in percent of logo height.
//
#define LOGO_BOTTOM_PADDING 20
#define PROGRESS_BLOCK_HEIGHT 10
@@ -316,7 +316,7 @@ FindDim (
@param[in] Completion A value between 0 and 100 indicating the current
completion progress of a firmware update. This
- value must the the same or higher than previous
+ value must the same or higher than previous
calls to this service. The first call of 0 or a
value of 0 after reaching a value of 100 resets
the progress indicator to 0.
diff --git a/MdeModulePkg/Library/DisplayUpdateProgressLibText/DisplayUpdateProgressLibText.c b/MdeModulePkg/Library/DisplayUpdateProgressLibText/DisplayUpdateProgressLibText.c
index 4823a24d0da3..da1c5651d0d6 100644
--- a/MdeModulePkg/Library/DisplayUpdateProgressLibText/DisplayUpdateProgressLibText.c
+++ b/MdeModulePkg/Library/DisplayUpdateProgressLibText/DisplayUpdateProgressLibText.c
@@ -30,7 +30,7 @@ UINTN mProgressBarForegroundColor;
@param[in] Completion A value between 0 and 100 indicating the current
completion progress of a firmware update. This
- value must the the same or higher than previous
+ value must the same or higher than previous
calls to this service. The first call of 0 or a
value of 0 after reaching a value of 100 resets
the progress indicator to 0.
diff --git a/MdeModulePkg/Library/PiDxeS3BootScriptLib/BootScriptSave.c b/MdeModulePkg/Library/PiDxeS3BootScriptLib/BootScriptSave.c
index f8d4983d81e6..69a50e32fbe1 100644
--- a/MdeModulePkg/Library/PiDxeS3BootScriptLib/BootScriptSave.c
+++ b/MdeModulePkg/Library/PiDxeS3BootScriptLib/BootScriptSave.c
@@ -730,7 +730,7 @@ S3BootScriptGetBootTimeEntryAddAddress (
}
//
- // calculate the the start address for the new entry.
+ // calculate the start address for the new entry.
//
NewEntryPtr = mS3BootScriptTablePtr->TableBase + TableLength;
@@ -2159,7 +2159,7 @@ S3BootScriptCalculateInsertAddress (
//
CopyMem (S3TableBase+PositionOffset+EntryLength, S3TableBase+PositionOffset, TableLength - PositionOffset);
//
- // calculate the the start address for the new entry.
+ // calculate the start address for the new entry.
//
*Script = S3TableBase + PositionOffset;
} else {
diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
index 962892d38f14..d4e93b50ae1a 100644
--- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
+++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
@@ -1116,7 +1116,7 @@ BmExpandMediaDevicePath (
}
//
- // Detect the the default boot file from removable Media
+ // Detect the default boot file from removable Media
//
NextFullPath = NULL;
Size = GetDevicePathSize (DevicePath) - END_DEVICE_PATH_LENGTH;
diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c b/MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c
index 60e7b1099afe..dae91d8e35bc 100644
--- a/MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c
+++ b/MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c
@@ -220,7 +220,7 @@ EfiBootManagerGetGopDevicePath (
}
/**
- Connect the platform active active video controller.
+ Connect the platform active video controller.
@param VideoController PCI handle of video controller.
diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c b/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c
index a5e32ebdbab6..c6b9618d4d5c 100644
--- a/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c
+++ b/MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c
@@ -156,7 +156,7 @@ BmSetMemoryTypeInformationVariable (
}
//
- // Only check the the Memory Type Information variable in the boot mode
+ // Only check the Memory Type Information variable in the boot mode
// other than BOOT_WITH_DEFAULT_SETTINGS because the Memory Type
// Information is not valid in this boot mode.
//
diff --git a/MdeModulePkg/Library/UefiHiiLib/HiiLib.c b/MdeModulePkg/Library/UefiHiiLib/HiiLib.c
index 63a37ab59ae4..c5278fda79b3 100644
--- a/MdeModulePkg/Library/UefiHiiLib/HiiLib.c
+++ b/MdeModulePkg/Library/UefiHiiLib/HiiLib.c
@@ -635,7 +635,7 @@ InternalHiiBrowserCallback (
if (SetResultsData != NULL) {
//
- // Request to to set data in the uncommitted browser state information
+ // Request to set data in the uncommitted browser state information
//
ResultsData = SetResultsData;
} else {
diff --git a/MdeModulePkg/Library/UefiHiiLib/HiiString.c b/MdeModulePkg/Library/UefiHiiLib/HiiString.c
index 2c586ef64d5e..1e8a8c510776 100644
--- a/MdeModulePkg/Library/UefiHiiLib/HiiString.c
+++ b/MdeModulePkg/Library/UefiHiiLib/HiiString.c
@@ -36,7 +36,7 @@
String is added or updated in the String Package
associated with HiiHandle for all the languages
that the String Package supports. If this
- parameter is not NULL, then then String is added
+ parameter is not NULL, then String is added
or updated in the String Package associated with
HiiHandle for the set oflanguages specified by
SupportedLanguages. The format of
diff --git a/MdeModulePkg/Universal/BdsDxe/BdsEntry.c b/MdeModulePkg/Universal/BdsDxe/BdsEntry.c
index 766dde3aaeeb..8e4b24302922 100644
--- a/MdeModulePkg/Universal/BdsDxe/BdsEntry.c
+++ b/MdeModulePkg/Universal/BdsDxe/BdsEntry.c
@@ -832,7 +832,7 @@ BdsEntry (
for (Index = 0; Index < LoadOptionCount; Index++) {
//
// The PlatformRecovery#### options are sorted by OptionNumber.
- // Find the the smallest unused number as the new OptionNumber.
+ // Find the smallest unused number as the new OptionNumber.
//
if (LoadOptions[Index].OptionNumber != Index) {
break;
diff --git a/MdeModulePkg/Universal/BdsDxe/Language.c b/MdeModulePkg/Universal/BdsDxe/Language.c
index 8724b13f4ca1..6b95b1622b46 100644
--- a/MdeModulePkg/Universal/BdsDxe/Language.c
+++ b/MdeModulePkg/Universal/BdsDxe/Language.c
@@ -12,7 +12,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
/**
Check if lang is in supported language codes according to language string.
- This code is used to check if lang is in in supported language codes. It can handle
+ This code is used to check if lang is in supported language codes. It can handle
RFC4646 and ISO639 language tags.
In ISO639 language tags, take 3-characters as a delimitation to find matched string.
In RFC4646 language tags, take semicolon as a delimitation to find matched string.
diff --git a/MdeModulePkg/Universal/CapsulePei/UefiCapsule.c b/MdeModulePkg/Universal/CapsulePei/UefiCapsule.c
index ef60d4e1b7b7..f2b306b8129e 100644
--- a/MdeModulePkg/Universal/CapsulePei/UefiCapsule.c
+++ b/MdeModulePkg/Universal/CapsulePei/UefiCapsule.c
@@ -200,7 +200,7 @@ Create4GPageTables (
for (IndexOfPml4Entries = 0; IndexOfPml4Entries < NumberOfPml4EntriesNeeded; IndexOfPml4Entries++, PageMapLevel4Entry++) {
//
// Each PML4 entry points to a page of Page Directory Pointer entires.
- // So lets allocate space for them and fill them in in the IndexOfPdpEntries loop.
+ // So lets allocate space for them and fill them in the IndexOfPdpEntries loop.
//
PageDirectoryPointerEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
@@ -228,7 +228,7 @@ Create4GPageTables (
for (IndexOfPdpEntries = 0; IndexOfPdpEntries < NumberOfPdpEntriesNeeded; IndexOfPdpEntries++, PageDirectoryPointerEntry++) {
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
diff --git a/MdeModulePkg/Universal/Console/ConSplitterDxe/ComponentName.c b/MdeModulePkg/Universal/Console/ConSplitterDxe/ComponentName.c
index fd9c00c0e12d..ef8a45e944fd 100644
--- a/MdeModulePkg/Universal/Console/ConSplitterDxe/ComponentName.c
+++ b/MdeModulePkg/Universal/Console/ConSplitterDxe/ComponentName.c
@@ -507,7 +507,7 @@ ConSplitterSimplePointerComponentNameGetControllerName (
not be NULL for a bus driver that wishes to
retrieve the name of a child controller.
@param Language A pointer to RFC4646 language identifier. This is
- the language of the controller name that that the
+ the language of the controller name that the
caller is requesting, and it must match one of the
languages specified in SupportedLanguages. The
number of languages supported by a driver is up to
diff --git a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.h b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.h
index a1fe74726058..9e32772ae528 100644
--- a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.h
+++ b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.h
@@ -928,7 +928,7 @@ ConSplitterSimplePointerComponentNameGetControllerName (
not be NULL for a bus driver that wishes to
retrieve the name of a child controller.
@param Language A pointer to RFC4646 language identifier. This is
- the language of the controller name that that the
+ the language of the controller name that the
caller is requesting, and it must match one of the
languages specified in SupportedLanguages. The
number of languages supported by a driver is up to
diff --git a/MdeModulePkg/Universal/DebugSupportDxe/Ia32/AsmFuncs.nasm b/MdeModulePkg/Universal/DebugSupportDxe/Ia32/AsmFuncs.nasm
index 07fc912fe82d..5efa03db1e47 100644
--- a/MdeModulePkg/Universal/DebugSupportDxe/Ia32/AsmFuncs.nasm
+++ b/MdeModulePkg/Universal/DebugSupportDxe/Ia32/AsmFuncs.nasm
@@ -78,7 +78,7 @@ align 16
DebugStackEnd: db "DbgStkEnd >>>>>>" ;; 16 byte long string - must be 16 bytes to preserve alignment
times 0x1ffc dd 0x0 ;; 32K should be enough stack
;; This allocation is coocked to insure
- ;; that the the buffer for the FXSTORE instruction
+ ;; that the buffer for the FXSTORE instruction
;; will be 16 byte aligned also.
;;
ExceptionNumber: dd 0 ;; first entry will be the vector number pushed by the stub
diff --git a/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.nasm b/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.nasm
index c6c5e4918931..7af0e8dfd9ef 100644
--- a/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.nasm
+++ b/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.nasm
@@ -79,7 +79,7 @@ align 16
DebugStackEnd: db "DbgStkEnd >>>>>>" ;; 16 byte long string - must be 16 bytes to preserve alignment
times 0x1ffc dd 0x0 ;; 32K should be enough stack
;; This allocation is coocked to insure
- ;; that the the buffer for the FXSTORE instruction
+ ;; that the buffer for the FXSTORE instruction
;; will be 16 byte aligned also.
;;
ExceptionNumber: dq 0 ;; first entry will be the vector number pushed by the stub
diff --git a/MdeModulePkg/Universal/Disk/DiskIoDxe/DiskIo.c b/MdeModulePkg/Universal/Disk/DiskIoDxe/DiskIo.c
index 38af39f41ec2..c999fb5e7af7 100644
--- a/MdeModulePkg/Universal/Disk/DiskIoDxe/DiskIo.c
+++ b/MdeModulePkg/Universal/Disk/DiskIoDxe/DiskIo.c
@@ -1002,7 +1002,7 @@ DiskIo2ReadWriteDisk (
if (!EFI_ERROR (Status) && (Task->Token != NULL)) {
//
// Task->Token should be set to NULL by the DiskIo2OnReadWriteComplete
- // It it's not, that means the non-blocking request was downgraded to blocking request.
+ // If it's not, that means the non-blocking request was downgraded to blocking request.
//
DEBUG ((DEBUG_VERBOSE, "DiskIo: Non-blocking request was downgraded to blocking request, signal event directly.\n"));
Task->Token->TransactionStatus = Status;
diff --git a/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c b/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c
index a7b7dc743aae..8c6ed5d713af 100644
--- a/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c
+++ b/MdeModulePkg/Universal/DisplayEngineDxe/FormDisplay.c
@@ -1793,7 +1793,7 @@ GetDistanceBetweenMenus (
@param HighLightMenu The link_entry pointer to highlight menu.
- @retval Return the the link_entry pointer top of screen menu.
+ @retval Return the link_entry pointer top of screen menu.
**/
LIST_ENTRY *
diff --git a/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c b/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c
index 4f47c55cce99..4a96801a3007 100644
--- a/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c
+++ b/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c
@@ -127,7 +127,7 @@ IsSystemFmp (
in Table.
@retval EFI_SUCCESS The ImageTypeId GUID in FmpImageInfoBuf matches an
existing ESRT entry in Table, and the information
- from FmpImageInfoBuf was merged into the the existing
+ from FmpImageInfoBuf was merged into the existing
ESRT entry.
@retval EFI_UNSPOORTED The GUID/HardareInstance in FmpImageInfoBuf has is a
duplicate.
diff --git a/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystem.c b/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystem.c
index 0252db19ade9..ac43825386d9 100644
--- a/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystem.c
+++ b/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystem.c
@@ -1,6 +1,6 @@
/** @file
This driver uses the EFI_FIRMWARE_VOLUME2_PROTOCOL to expose files in firmware
- volumes via the the EFI_SIMPLE_FILESYSTEM_PROTOCOL and EFI_FILE_PROTOCOL.
+ volumes via the EFI_SIMPLE_FILESYSTEM_PROTOCOL and EFI_FILE_PROTOCOL.
It will expose a single directory, containing one file for each file in the firmware
volume. If a file has a UI section, its contents will be used as a filename.
diff --git a/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemDxe.inf b/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemDxe.inf
index 03797847d936..dbca5c5dc4d0 100644
--- a/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemDxe.inf
+++ b/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemDxe.inf
@@ -2,7 +2,7 @@
# Support for Simple File System over Firmware Volume.
#
# This driver uses the EFI_FIRMWARE_VOLUME2_PROTOCOL to expose files in firmware
-# volumes via the the EFI_SIMPLE_FILESYSTEM_PROTOCOL and EFI_FILE_PROTOCOL.
+# volumes via the EFI_SIMPLE_FILESYSTEM_PROTOCOL and EFI_FILE_PROTOCOL.
#
# It will expose a single directory, containing one file for each file in the firmware
# volume. If a file has a UI section, its contents will be used as a filename.
diff --git a/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemEntryPoint.c b/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemEntryPoint.c
index 3f96407dc755..4f4446a9df51 100644
--- a/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemEntryPoint.c
+++ b/MdeModulePkg/Universal/FvSimpleFileSystemDxe/FvSimpleFileSystemEntryPoint.c
@@ -1,6 +1,6 @@
/** @file
This driver uses the EFI_FIRMWARE_VOLUME2_PROTOCOL to expose files in firmware
- volumes via the the EFI_SIMPLE_FILESYSTEM_PROTOCOL and EFI_FILE_PROTOCOL.
+ volumes via the EFI_SIMPLE_FILESYSTEM_PROTOCOL and EFI_FILE_PROTOCOL.
It will expose a single directory, containing one file for each file in the firmware
volume. If a file has a UI section, its contents will be used as a filename.
@@ -292,7 +292,7 @@ InitializeUnicodeCollationSupportWorker (
}
//
- // Find the best matching matching language from the supported languages
+ // Find the best matching language from the supported languages
// of Unicode Collation (2) protocol.
//
BestLanguage = GetBestLanguage (
diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
index 5ae6189a2843..442530b19561 100644
--- a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
+++ b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
@@ -2027,7 +2027,7 @@ Done:
@param IfrOpHdr Ifr opcode header for this opcode.
@param VarWidth The buffer width for this opcode.
@param ReturnData The data block added for this opcode.
- @param IsBitVar Whether the the opcode refers to bit storage.
+ @param IsBitVar Whether the opcode refers to bit storage.
@retval EFI_SUCCESS This opcode is required.
@retval EFI_NOT_FOUND This opcode is not required.
diff --git a/MdeModulePkg/Universal/PlatformDriOverrideDxe/PlatDriOverrideLib.c b/MdeModulePkg/Universal/PlatformDriOverrideDxe/PlatDriOverrideLib.c
index 9a6050d32298..8883d321eff9 100644
--- a/MdeModulePkg/Universal/PlatformDriOverrideDxe/PlatDriOverrideLib.c
+++ b/MdeModulePkg/Universal/PlatformDriOverrideDxe/PlatDriOverrideLib.c
@@ -983,7 +983,7 @@ SaveOverridesMapping (
}
//
- // Get the the maximum size of an individual EFI variable in current system
+ // Get the maximum size of an individual EFI variable in current system
//
gRT->QueryVariableInfo (
EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE,
diff --git a/MdeModulePkg/Universal/SerialDxe/SerialIo.c b/MdeModulePkg/Universal/SerialDxe/SerialIo.c
index 25be8634abcd..b9309f1c98c7 100644
--- a/MdeModulePkg/Universal/SerialDxe/SerialIo.c
+++ b/MdeModulePkg/Universal/SerialDxe/SerialIo.c
@@ -44,7 +44,7 @@ SerialReset (
data bits, and stop bits on a serial device.
@param This Protocol instance pointer.
- @param BaudRate The requested baud rate. A BaudRate value of 0 will use the the
+ @param BaudRate The requested baud rate. A BaudRate value of 0 will use the
device's default interface speed.
@param ReceiveFifoDepth The requested depth of the FIFO on the receive side of the
serial interface. A ReceiveFifoDepth value of 0 will use
@@ -255,7 +255,7 @@ SerialReset (
data bits, and stop bits on a serial device.
@param This Protocol instance pointer.
- @param BaudRate The requested baud rate. A BaudRate value of 0 will use the the
+ @param BaudRate The requested baud rate. A BaudRate value of 0 will use the
device's default interface speed.
@param ReceiveFifoDepth The requested depth of the FIFO on the receive side of the
serial interface. A ReceiveFifoDepth value of 0 will use
diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
index babef214974e..f88de881e38f 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
@@ -918,7 +918,7 @@ UpdateStatementStatusForForm (
Link = GetNextNode (&Form->StatementListHead, Link);
//
- // For password opcode, not set the the value changed flag.
+ // For password opcode, not set the value changed flag.
//
if (Question->Operand == EFI_IFR_PASSWORD_OP) {
continue;
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 08/17] MdePkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (6 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 07/17] MdeModulePkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 18:50 ` Michael D Kinney
2022-09-06 8:29 ` [PATCH v2 09/17] NetworkPkg: " PierreGondois
` (8 subsequent siblings)
16 siblings, 1 reply; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Michael D Kinney, Liming Gao
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
MdePkg/Include/Guid/GraphicsInfoHob.h | 2 +-
MdePkg/Include/Library/DevicePathLib.h | 2 +-
MdePkg/Include/Library/DxeServicesLib.h | 6 +++---
| 6 +++---
MdePkg/Include/Library/FileHandleLib.h | 4 ++--
MdePkg/Include/Library/PeCoffLib.h | 2 +-
MdePkg/Include/Library/UefiLib.h | 2 +-
MdePkg/Include/Library/UnitTestLib.h | 2 +-
MdePkg/Include/Pi/PiMmCis.h | 2 +-
MdePkg/Include/Pi/PiSmmCis.h | 2 +-
MdePkg/Include/Protocol/ComponentName.h | 2 +-
MdePkg/Include/Protocol/Smbios.h | 2 +-
MdePkg/Include/Uefi/UefiMultiPhase.h | 2 +-
MdePkg/Include/Uefi/UefiSpec.h | 2 +-
MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c | 2 +-
| 6 +++---
MdePkg/Library/BaseS3StallLib/S3StallLib.c | 2 +-
MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c | 2 +-
MdePkg/Library/DxeServicesLib/DxeServicesLib.c | 6 +++---
| 6 +++---
MdePkg/Library/SmmIoLib/SmmIoLib.c | 2 +-
MdePkg/Library/SmmIoLib/SmmIoLib.inf | 2 +-
MdePkg/Library/SmmIoLib/SmmIoLib.uni | 4 ++--
MdePkg/Library/SmmMemLib/SmmMemLib.c | 2 +-
MdePkg/Library/SmmMemLib/SmmMemLib.inf | 2 +-
MdePkg/Library/SmmMemLib/SmmMemLib.uni | 4 ++--
MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c | 4 ++--
.../StandaloneMmDriverEntryPoint.c | 4 ++--
MdePkg/Library/UefiDebugLibConOut/DebugLib.c | 2 +-
MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c | 2 +-
MdePkg/Library/UefiDebugLibStdErr/DebugLib.c | 2 +-
MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf | 2 +-
MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni | 2 +-
MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c | 2 +-
.../UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c | 2 +-
MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c | 4 ++--
MdePkg/Library/UefiLib/Console.c | 2 +-
MdePkg/Library/UefiLib/UefiLib.c | 2 +-
MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c | 4 ++--
39 files changed, 56 insertions(+), 56 deletions(-)
diff --git a/MdePkg/Include/Guid/GraphicsInfoHob.h b/MdePkg/Include/Guid/GraphicsInfoHob.h
index 237911e63aa0..ecfa283f924e 100644
--- a/MdePkg/Include/Guid/GraphicsInfoHob.h
+++ b/MdePkg/Include/Guid/GraphicsInfoHob.h
@@ -5,7 +5,7 @@
SPDX-License-Identifier: BSD-2-Clause-Patent
@par Revision Reference:
- This HOB is introduced in in PI Version 1.4.
+ This HOB is introduced in PI Version 1.4.
**/
diff --git a/MdePkg/Include/Library/DevicePathLib.h b/MdePkg/Include/Library/DevicePathLib.h
index 7a077e4217d2..039427d5b648 100644
--- a/MdePkg/Include/Library/DevicePathLib.h
+++ b/MdePkg/Include/Library/DevicePathLib.h
@@ -23,7 +23,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
@retval TRUE DevicePath is valid.
@retval FALSE DevicePath is NULL.
@retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
- @retval FALSE The length of any node node in the DevicePath is less
+ @retval FALSE The length of any node in the DevicePath is less
than sizeof (EFI_DEVICE_PATH_PROTOCOL).
@retval FALSE If MaxSize is not zero, the size of the DevicePath
exceeds MaxSize.
diff --git a/MdePkg/Include/Library/DxeServicesLib.h b/MdePkg/Include/Library/DxeServicesLib.h
index b5c5c4dbdf27..d3304f65a5fd 100644
--- a/MdePkg/Include/Library/DxeServicesLib.h
+++ b/MdePkg/Include/Library/DxeServicesLib.h
@@ -82,7 +82,7 @@ GetSectionFromAnyFvByFileType (
If Size is NULL, then ASSERT().
- @param NameGuid A pointer to to the FFS filename GUID to search for
+ @param NameGuid A pointer to the FFS filename GUID to search for
within any of the firmware volumes in the platform.
@param SectionType Indicates the FFS section type to search for within
the FFS file specified by NameGuid.
@@ -135,7 +135,7 @@ GetSectionFromAnyFv (
If Buffer is NULL, then ASSERT().
If Size is NULL, then ASSERT().
- @param NameGuid A pointer to to the FFS filename GUID to search for
+ @param NameGuid A pointer to the FFS filename GUID to search for
within the firmware volumes that the currently
executing module was loaded from.
@param SectionType Indicates the FFS section type to search for within
@@ -269,7 +269,7 @@ GetFileBufferByFilePath (
If NameGuid is NULL, then ASSERT().
- @param NameGuid A pointer to to the FFS filename GUID to search for
+ @param NameGuid A pointer to the FFS filename GUID to search for
within any of the firmware volumes in the platform.
@param SectionType Indicates the FFS section type to search for within
the FFS file specified by NameGuid.
--git a/MdePkg/Include/Library/ExtractGuidedSectionLib.h b/MdePkg/Include/Library/ExtractGuidedSectionLib.h
index 15058ae636ff..6cd5288a9456 100644
--- a/MdePkg/Include/Library/ExtractGuidedSectionLib.h
+++ b/MdePkg/Include/Library/ExtractGuidedSectionLib.h
@@ -114,7 +114,7 @@ RETURN_STATUS
If GetInfoHandler is NULL, then ASSERT().
If DecodeHandler is NULL, then ASSERT().
- @param[in] SectionGuid A pointer to the GUID associated with the the handlers
+ @param[in] SectionGuid A pointer to the GUID associated with the handlers
of the GUIDed section type being registered.
@param[in] GetInfoHandler Pointer to a function that examines a GUIDed section and returns the
size of the decoded buffer and the size of an optional scratch buffer
@@ -163,7 +163,7 @@ ExtractGuidedSectionGetGuidList (
Examines a GUIDed section specified by InputSection.
If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
then RETURN_UNSUPPORTED is returned.
- If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
+ If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
of type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
is used to retrieve the OututBufferSize, ScratchSize, and Attributes values. The return status from the handler of
type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER is returned.
@@ -206,7 +206,7 @@ ExtractGuidedSectionGetInfo (
Decodes the GUIDed section specified by InputSection.
If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
then RETURN_UNSUPPORTED is returned.
- If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
+ If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
of type EXTRACT_GUIDED_SECTION_DECODE_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
is used to decode InputSection into the buffer specified by OutputBuffer and the authentication status of this
decode operation is returned in AuthenticationStatus. If the decoded buffer is identical to the data in InputSection,
diff --git a/MdePkg/Include/Library/FileHandleLib.h b/MdePkg/Include/Library/FileHandleLib.h
index 1a238edee38c..4eb2b9a35101 100644
--- a/MdePkg/Include/Library/FileHandleLib.h
+++ b/MdePkg/Include/Library/FileHandleLib.h
@@ -384,9 +384,9 @@ FileHandleGetFileName (
@param[in, out] Buffer The pointer to buffer to read into.
@param[in, out] Size The pointer to number of bytes in Buffer.
@param[in] Truncate If the buffer is large enough, this has no effect.
- If the buffer is is too small and Truncate is TRUE,
+ If the buffer is too small and Truncate is TRUE,
the line will be truncated.
- If the buffer is is too small and Truncate is FALSE,
+ If the buffer is too small and Truncate is FALSE,
then no read will occur.
@param[in, out] Ascii Boolean value for indicating whether the file is
diff --git a/MdePkg/Include/Library/PeCoffLib.h b/MdePkg/Include/Library/PeCoffLib.h
index b45879453785..022dc24bccbc 100644
--- a/MdePkg/Include/Library/PeCoffLib.h
+++ b/MdePkg/Include/Library/PeCoffLib.h
@@ -90,7 +90,7 @@ typedef struct {
///
PHYSICAL_ADDRESS DestinationAddress;
///
- /// PeCoffLoaderLoadImage() sets EntryPoint to to the entry point of the PE/COFF image.
+ /// PeCoffLoaderLoadImage() sets EntryPoint to the entry point of the PE/COFF image.
///
PHYSICAL_ADDRESS EntryPoint;
///
diff --git a/MdePkg/Include/Library/UefiLib.h b/MdePkg/Include/Library/UefiLib.h
index be7da7fdf7da..54af95459cee 100644
--- a/MdePkg/Include/Library/UefiLib.h
+++ b/MdePkg/Include/Library/UefiLib.h
@@ -406,7 +406,7 @@ EfiReleaseLock (
This function tests whether the driver specified by DriverBindingHandle is
currently managing the controller specified by ControllerHandle. This test
- is performed by evaluating if the the protocol specified by ProtocolGuid is
+ is performed by evaluating if the protocol specified by ProtocolGuid is
present on ControllerHandle and is was opened by DriverBindingHandle with an
attribute of EFI_OPEN_PROTOCOL_BY_DRIVER.
If ProtocolGuid is NULL, then ASSERT().
diff --git a/MdePkg/Include/Library/UnitTestLib.h b/MdePkg/Include/Library/UnitTestLib.h
index 71c205b1b143..e9c4e23a7d43 100644
--- a/MdePkg/Include/Library/UnitTestLib.h
+++ b/MdePkg/Include/Library/UnitTestLib.h
@@ -755,7 +755,7 @@ UnitTestAssertNotNull (
is either pass, skipped, or failed.
@param[in] FunctionName Null-terminated ASCII string of the function
executing the UT_EXPECT_ASSERT_FAILURE() macro.
- @param[in] LineNumber The source file line number of the the function
+ @param[in] LineNumber The source file line number of the function
executing the UT_EXPECT_ASSERT_FAILURE() macro.
@param[in] FileName Null-terminated ASCII string of the filename
executing the UT_EXPECT_ASSERT_FAILURE() macro.
diff --git a/MdePkg/Include/Pi/PiMmCis.h b/MdePkg/Include/Pi/PiMmCis.h
index 01340952d89b..53269d1bf68e 100644
--- a/MdePkg/Include/Pi/PiMmCis.h
+++ b/MdePkg/Include/Pi/PiMmCis.h
@@ -289,7 +289,7 @@ struct _EFI_MM_SYSTEM_TABLE {
///
///
- /// A number between zero and and the NumberOfCpus field. This field designates
+ /// A number between zero and the NumberOfCpus field. This field designates
/// which processor is executing the MM infrastructure.
///
UINTN CurrentlyExecutingCpu;
diff --git a/MdePkg/Include/Pi/PiSmmCis.h b/MdePkg/Include/Pi/PiSmmCis.h
index bf5b580c3245..540603343ae2 100644
--- a/MdePkg/Include/Pi/PiSmmCis.h
+++ b/MdePkg/Include/Pi/PiSmmCis.h
@@ -144,7 +144,7 @@ struct _EFI_SMM_SYSTEM_TABLE2 {
///
///
- /// A number between zero and and the NumberOfCpus field. This field designates
+ /// A number between zero and the NumberOfCpus field. This field designates
/// which processor is executing the SMM infrastructure.
///
UINTN CurrentlyExecutingCpu;
diff --git a/MdePkg/Include/Protocol/ComponentName.h b/MdePkg/Include/Protocol/ComponentName.h
index 46fa5e7ad8e5..da4bbddb8fb1 100644
--- a/MdePkg/Include/Protocol/ComponentName.h
+++ b/MdePkg/Include/Protocol/ComponentName.h
@@ -26,7 +26,7 @@ typedef struct _EFI_COMPONENT_NAME_PROTOCOL EFI_COMPONENT_NAME_PROTOCOL;
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/MdePkg/Include/Protocol/Smbios.h b/MdePkg/Include/Protocol/Smbios.h
index f9346aac7224..a10db6c43f22 100644
--- a/MdePkg/Include/Protocol/Smbios.h
+++ b/MdePkg/Include/Protocol/Smbios.h
@@ -174,7 +174,7 @@ EFI_STATUS
@param[in] Type On entry, it points to the type of the next SMBIOS record to return. If NULL, it
indicates that the next record of any type will be returned. Type is not
modified by the this function.
- @param[out] Record On exit, points to a pointer to the the SMBIOS Record consisting of the formatted area
+ @param[out] Record On exit, points to a pointer to the SMBIOS Record consisting of the formatted area
followed by the unformatted area. The unformatted area optionally contains text strings.
@param[out] ProducerHandle On exit, points to the ProducerHandle registered by Add(). If no
ProducerHandle was passed into Add() NULL is returned. If a NULL pointer is
diff --git a/MdePkg/Include/Uefi/UefiMultiPhase.h b/MdePkg/Include/Uefi/UefiMultiPhase.h
index 22bae43e36e8..df9fcd880f70 100644
--- a/MdePkg/Include/Uefi/UefiMultiPhase.h
+++ b/MdePkg/Include/Uefi/UefiMultiPhase.h
@@ -177,7 +177,7 @@ typedef struct {
/// EFI_CERT_TYPE_RSA2048_SHA256_GUID. If the attribute specifies
/// authenticated access, then the Data buffer should begin with an
/// authentication descriptor prior to the data payload and DataSize
-/// should reflect the the data.and descriptor size. The caller
+/// should reflect the data.and descriptor size. The caller
/// shall digest the Monotonic Count value and the associated data
/// for the variable update using the SHA-256 1-way hash algorithm.
/// The ensuing the 32-byte digest will be signed using the private
diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSpec.h
index 2b38b100f6a5..acfd07eeaa5b 100644
--- a/MdePkg/Include/Uefi/UefiSpec.h
+++ b/MdePkg/Include/Uefi/UefiSpec.h
@@ -809,7 +809,7 @@ EFI_STATUS
@retval EFI_SUCCESS The operation completed successfully.
@retval EFI_INVALID_PARAMETER A time field is out of range.
- @retval EFI_DEVICE_ERROR The time could not be set due due to hardware error.
+ @retval EFI_DEVICE_ERROR The time could not be set due to hardware error.
**/
typedef
diff --git a/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c b/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c
index bd5686947712..2a8881637d45 100644
--- a/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c
+++ b/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c
@@ -264,7 +264,7 @@ DebugClearMemory (
ASSERT (Buffer != NULL);
//
- // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
+ // SetMem() checks for the ASSERT() condition on Length and returns Buffer
//
return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
}
--git a/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c b/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c
index e6f479b829f6..d364a406af71 100644
--- a/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c
+++ b/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c
@@ -144,7 +144,7 @@ ExtractGuidedSectionGetGuidList (
If GetInfoHandler is NULL, then ASSERT().
If DecodeHandler is NULL, then ASSERT().
- @param[in] SectionGuid A pointer to the GUID associated with the the handlers
+ @param[in] SectionGuid A pointer to the GUID associated with the handlers
of the GUIDed section type being registered.
@param[in] GetInfoHandler The pointer to a function that examines a GUIDed section and returns the
size of the decoded buffer and the size of an optional scratch buffer
@@ -224,7 +224,7 @@ ExtractGuidedSectionRegisterHandlers (
Examines a GUIDed section specified by InputSection.
If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
then RETURN_UNSUPPORTED is returned.
- If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
+ If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
of type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
is used to retrieve the OututBufferSize, ScratchSize, and Attributes values. The return status from the handler of
type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER is returned.
@@ -318,7 +318,7 @@ ExtractGuidedSectionGetInfo (
Decodes the GUIDed section specified by InputSection.
If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
then RETURN_UNSUPPORTED is returned.
- If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
+ If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
of type EXTRACT_GUIDED_SECTION_DECODE_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
is used to decode InputSection into the buffer specified by OutputBuffer and the authentication status of this
decode operation is returned in AuthenticationStatus. If the decoded buffer is identical to the data in InputSection,
diff --git a/MdePkg/Library/BaseS3StallLib/S3StallLib.c b/MdePkg/Library/BaseS3StallLib/S3StallLib.c
index ea886ca61cd3..6be227c5fb9a 100644
--- a/MdePkg/Library/BaseS3StallLib/S3StallLib.c
+++ b/MdePkg/Library/BaseS3StallLib/S3StallLib.c
@@ -17,7 +17,7 @@
#include <Library/S3StallLib.h>
/**
- Stalls the CPU for at least the given number of microseconds and and saves
+ Stalls the CPU for at least the given number of microseconds and saves
the value in the S3 script to be replayed on S3 resume.
Stalls the CPU for the number of microseconds specified by MicroSeconds.
diff --git a/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c b/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c
index 11d188d07ff6..8c201daf3914 100644
--- a/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c
+++ b/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c
@@ -337,7 +337,7 @@ DebugClearMemory (
ASSERT (Buffer != NULL);
//
- // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
+ // SetMem() checks for the ASSERT() condition on Length and returns Buffer
//
return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
}
diff --git a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c
index 8f5a15dedc9f..7a5c031a7daa 100644
--- a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c
+++ b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c
@@ -347,7 +347,7 @@ Done:
If Size is NULL, then ASSERT().
- @param NameGuid A pointer to to the FFS filename GUID to search for
+ @param NameGuid A pointer to the FFS filename GUID to search for
within any of the firmware volumes in the platform.
@param SectionType Indicates the FFS section type to search for within
the FFS file specified by NameGuid.
@@ -470,7 +470,7 @@ Done:
If Buffer is NULL, then ASSERT().
If Size is NULL, then ASSERT().
- @param NameGuid A pointer to to the FFS filename GUID to search for
+ @param NameGuid A pointer to the FFS filename GUID to search for
within the firmware volumes that the currently
executing module was loaded from.
@param SectionType Indicates the FFS section type to search for within
@@ -957,7 +957,7 @@ Finish:
If NameGuid is NULL, then ASSERT().
- @param NameGuid A pointer to to the FFS filename GUID to search for
+ @param NameGuid A pointer to the FFS filename GUID to search for
within any of the firmware volumes in the platform.
@param SectionType Indicates the FFS section type to search for within
the FFS file specified by NameGuid.
--git a/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c b/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c
index c538f157a242..21c8b470ccf5 100644
--- a/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c
+++ b/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c
@@ -170,7 +170,7 @@ ExtractGuidedSectionGetGuidList (
If GetInfoHandler is NULL, then ASSERT().
If DecodeHandler is NULL, then ASSERT().
- @param[in] SectionGuid A pointer to the GUID associated with the the handlers
+ @param[in] SectionGuid A pointer to the GUID associated with the handlers
of the GUIDed section type being registered.
@param[in] GetInfoHandler The pointer to a function that examines a GUIDed section and returns the
size of the decoded buffer and the size of an optional scratch buffer
@@ -260,7 +260,7 @@ ExtractGuidedSectionRegisterHandlers (
Examines a GUIDed section specified by InputSection.
If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
then RETURN_UNSUPPORTED is returned.
- If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
+ If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
of type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
is used to retrieve the OututBufferSize, ScratchSize, and Attributes values. The return status from the handler of
type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER is returned.
@@ -354,7 +354,7 @@ ExtractGuidedSectionGetInfo (
Decodes the GUIDed section specified by InputSection.
If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
then RETURN_UNSUPPORTED is returned.
- If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
+ If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
of type EXTRACT_GUIDED_SECTION_DECODE_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
is used to decode InputSection into the buffer specified by OutputBuffer and the authentication status of this
decode operation is returned in AuthenticationStatus. If the decoded buffer is identical to the data in InputSection,
diff --git a/MdePkg/Library/SmmIoLib/SmmIoLib.c b/MdePkg/Library/SmmIoLib/SmmIoLib.c
index ad4bfff68c8a..90fbdde96f98 100644
--- a/MdePkg/Library/SmmIoLib/SmmIoLib.c
+++ b/MdePkg/Library/SmmIoLib/SmmIoLib.c
@@ -1,7 +1,7 @@
/** @file
Instance of SMM IO check library.
- SMM IO check library library implementation. This library consumes GCD to collect all valid
+ SMM IO check library implementation. This library consumes GCD to collect all valid
IO space defined by a platform.
A platform may have its own SmmIoLib instance to exclude more IO space.
diff --git a/MdePkg/Library/SmmIoLib/SmmIoLib.inf b/MdePkg/Library/SmmIoLib/SmmIoLib.inf
index 5f7356de5d7a..d195afe0a0e1 100644
--- a/MdePkg/Library/SmmIoLib/SmmIoLib.inf
+++ b/MdePkg/Library/SmmIoLib/SmmIoLib.inf
@@ -1,7 +1,7 @@
## @file
# Instance of SMM IO check library.
#
-# SMM IO check library library implementation. This library consumes GCD to collect all valid
+# SMM IO check library implementation. This library consumes GCD to collect all valid
# IO space defined by a platform.
# A platform may have its own SmmIoLib instance to exclude more IO space.
#
diff --git a/MdePkg/Library/SmmIoLib/SmmIoLib.uni b/MdePkg/Library/SmmIoLib/SmmIoLib.uni
index d46bc0a7f9d8..fcce039b4aec 100644
--- a/MdePkg/Library/SmmIoLib/SmmIoLib.uni
+++ b/MdePkg/Library/SmmIoLib/SmmIoLib.uni
@@ -1,7 +1,7 @@
// /** @file
// Instance of SMM IO check library.
//
-// SMM IO check library library implementation. This library consumes GCD to collect all valid
+// SMM IO check library implementation. This library consumes GCD to collect all valid
// IO space defined by a platform.
// A platform may have its own SmmIoLib instance to exclude more IO space.
//
@@ -14,5 +14,5 @@
#string STR_MODULE_ABSTRACT #language en-US "Instance of SMM IO check library."
-#string STR_MODULE_DESCRIPTION #language en-US "SMM IO check library library implementation. This library consumes GCD to collect all valid IO space defined by a platform. A platform may have its own SmmIoLib instance to exclude more IO space."
+#string STR_MODULE_DESCRIPTION #language en-US "SMM IO check library implementation. This library consumes GCD to collect all valid IO space defined by a platform. A platform may have its own SmmIoLib instance to exclude more IO space."
diff --git a/MdePkg/Library/SmmMemLib/SmmMemLib.c b/MdePkg/Library/SmmMemLib/SmmMemLib.c
index ef087d0f050b..0eae5d2ed5e0 100644
--- a/MdePkg/Library/SmmMemLib/SmmMemLib.c
+++ b/MdePkg/Library/SmmMemLib/SmmMemLib.c
@@ -1,7 +1,7 @@
/** @file
Instance of SMM memory check library.
- SMM memory check library library implementation. This library consumes SMM_ACCESS2_PROTOCOL
+ SMM memory check library implementation. This library consumes SMM_ACCESS2_PROTOCOL
to get SMRAM information. In order to use this library instance, the platform should produce
all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core
and SMM driver) and/or specific dedicated hardware.
diff --git a/MdePkg/Library/SmmMemLib/SmmMemLib.inf b/MdePkg/Library/SmmMemLib/SmmMemLib.inf
index bc30057d772f..f53e81acb9c2 100644
--- a/MdePkg/Library/SmmMemLib/SmmMemLib.inf
+++ b/MdePkg/Library/SmmMemLib/SmmMemLib.inf
@@ -1,7 +1,7 @@
## @file
# Instance of SMM memory check library.
#
-# SMM memory check library library implementation. This library consumes SMM_ACCESS2_PROTOCOL
+# SMM memory check library implementation. This library consumes SMM_ACCESS2_PROTOCOL
# to get SMRAM information. In order to use this library instance, the platform should produce
# all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core
# and SMM driver) and/or specific dedicated hardware.
diff --git a/MdePkg/Library/SmmMemLib/SmmMemLib.uni b/MdePkg/Library/SmmMemLib/SmmMemLib.uni
index f1d678344055..3d553884376e 100644
--- a/MdePkg/Library/SmmMemLib/SmmMemLib.uni
+++ b/MdePkg/Library/SmmMemLib/SmmMemLib.uni
@@ -1,7 +1,7 @@
// /** @file
// SMM memory check library.
//
-// SMM memory check library library implementation. This library consumes SMM_ACCESS2_PROTOCOL
+// SMM memory check library implementation. This library consumes SMM_ACCESS2_PROTOCOL
// to get SMRAM information. In order to use this library instance, the platform should produce
// all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core
// and SMM driver) and/or specific dedicated hardware.
@@ -15,5 +15,5 @@
#string STR_MODULE_ABSTRACT #language en-US "SMM memory check library"
-#string STR_MODULE_DESCRIPTION #language en-US "SMM memory check library library implementation. This library consumes SMM_ACCESS2_PROTOCOL to get SMRAM information. In order to use this library instance, the platform should produce all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core and SMM driver) and/or specific dedicated hardware."
+#string STR_MODULE_DESCRIPTION #language en-US "SMM memory check library implementation. This library consumes SMM_ACCESS2_PROTOCOL to get SMRAM information. In order to use this library instance, the platform should produce all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core and SMM driver) and/or specific dedicated hardware."
diff --git a/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c b/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c
index ff143f00ab9a..1b293565b0a9 100644
--- a/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c
+++ b/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c
@@ -240,7 +240,7 @@ LookupPeriodicSmiLibraryHandler (
}
//
- // Search the periodic SMI handler entries for a a matching DispatchHandle
+ // Search the periodic SMI handler entries for a matching DispatchHandle
//
for ( Link = GetFirstNode (&gPeriodicSmiLibraryHandlers)
; !IsNull (&gPeriodicSmiLibraryHandlers, Link)
@@ -353,7 +353,7 @@ EnlargeFreePeriodicSmiLibraryHandlerList (
SMI handler. If no free entries are available, then additional
entries are allocated.
- @retval NULL There are not enough resources available to to allocate a free entry.
+ @retval NULL There are not enough resources available to allocate a free entry.
@retval other Pointer to a free PERIODIC_SMI_LIBRARY_HANDLER_CONTEXT structure.
**/
diff --git a/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c b/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c
index ff1c53b185ac..11ba5d5a36ae 100644
--- a/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c
+++ b/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c
@@ -81,8 +81,8 @@ _DriverUnloadHandler (
EFI_STATUS
EFIAPI
_ModuleEntryPoint (
- IN EFI_HANDLE ImageHandle,
- IN IN EFI_MM_SYSTEM_TABLE *MmSystemTable
+ IN EFI_HANDLE ImageHandle,
+ IN EFI_MM_SYSTEM_TABLE *MmSystemTable
)
{
EFI_STATUS Status;
diff --git a/MdePkg/Library/UefiDebugLibConOut/DebugLib.c b/MdePkg/Library/UefiDebugLibConOut/DebugLib.c
index 65c8dc2b4654..ac8b46d670d7 100644
--- a/MdePkg/Library/UefiDebugLibConOut/DebugLib.c
+++ b/MdePkg/Library/UefiDebugLibConOut/DebugLib.c
@@ -264,7 +264,7 @@ DebugClearMemory (
ASSERT (Buffer != NULL);
//
- // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
+ // SetMem() checks for the ASSERT() condition on Length and returns Buffer
//
return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
}
diff --git a/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c b/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c
index c25199b53fcf..f61113c739f8 100644
--- a/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c
+++ b/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c
@@ -318,7 +318,7 @@ DebugClearMemory (
ASSERT (Buffer != NULL);
//
- // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
+ // SetMem() checks for the ASSERT() condition on Length and returns Buffer
//
return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
}
diff --git a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
index 5b28cd10ae21..9bbac56ea5b0 100644
--- a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
+++ b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
@@ -264,7 +264,7 @@ DebugClearMemory (
ASSERT (Buffer != NULL);
//
- // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
+ // SetMem() checks for the ASSERT() condition on Length and returns Buffer
//
return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
}
diff --git a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
index 5ecb971a0ac5..3a4fc15525a5 100644
--- a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
+++ b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
@@ -1,7 +1,7 @@
## @file
# Instance of Debug Library using the Standard Error Device.
#
-# Debug Lib that sends messages to the the Standard Error Device in the EFI System Table.
+# Debug Lib that sends messages to the Standard Error Device in the EFI System Table.
#
# Copyright (c) 2018, Microsoft Corporation
#
diff --git a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni
index 1748fb3ff0b0..e6d0cca70224 100644
--- a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni
+++ b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni
@@ -1,7 +1,7 @@
// /** @file
// Instance of Debug Library using the Standard Error Device.
//
-// Debug Lib that sends messages to the the Standard Error Device in the EFI System Table.
+// Debug Lib that sends messages to the Standard Error Device in the EFI System Table.
//
// Copyright (c) 2007 - 2014, Intel Corporation. All rights reserved.<BR>
//
diff --git a/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c b/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c
index 5ee3e9a31f48..ce76bc8ff97b 100644
--- a/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c
+++ b/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c
@@ -36,7 +36,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED CONST EFI_DEVICE_PATH_PROTOCOL mUefiDevicePathLib
@retval TRUE DevicePath is valid.
@retval FALSE DevicePath is NULL.
@retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
- @retval FALSE The length of any node node in the DevicePath is less
+ @retval FALSE The length of any node in the DevicePath is less
than sizeof (EFI_DEVICE_PATH_PROTOCOL).
@retval FALSE If MaxSize is not zero, the size of the DevicePath
exceeds MaxSize.
diff --git a/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c b/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c
index 4ff9308a68fe..5cc30397a3a1 100644
--- a/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c
+++ b/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c
@@ -76,7 +76,7 @@ DevicePathLibConstructor (
@param MaxSize The maximum size of the device path data structure.
@retval TRUE DevicePath is valid.
- @retval FALSE The length of any node node in the DevicePath is less
+ @retval FALSE The length of any node in the DevicePath is less
than sizeof (EFI_DEVICE_PATH_PROTOCOL).
@retval FALSE If MaxSize is not zero, the size of the DevicePath
exceeds MaxSize.
diff --git a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
index 86678e965d99..f1a07c4fbde3 100644
--- a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
+++ b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
@@ -954,9 +954,9 @@ FileHandleReturnLine (
@param[in, out] Buffer The pointer to buffer to read into.
@param[in, out] Size The pointer to number of bytes in Buffer.
@param[in] Truncate If the buffer is large enough, this has no effect.
- If the buffer is is too small and Truncate is TRUE,
+ If the buffer is too small and Truncate is TRUE,
the line will be truncated.
- If the buffer is is too small and Truncate is FALSE,
+ If the buffer is too small and Truncate is FALSE,
then no read will occur.
@param[in, out] Ascii Boolean value for indicating whether the file is
diff --git a/MdePkg/Library/UefiLib/Console.c b/MdePkg/Library/UefiLib/Console.c
index c37e3d0f5810..a5425740a42c 100644
--- a/MdePkg/Library/UefiLib/Console.c
+++ b/MdePkg/Library/UefiLib/Console.c
@@ -413,7 +413,7 @@ CreatePopUp (
CHAR16 *TmpString;
//
- // Determine the length of the longest line in the popup and the the total
+ // Determine the length of the longest line in the popup and the total
// number of lines in the popup
//
VA_START (Args, Key);
diff --git a/MdePkg/Library/UefiLib/UefiLib.c b/MdePkg/Library/UefiLib/UefiLib.c
index 95d0319d3a33..c26cb0b6eb08 100644
--- a/MdePkg/Library/UefiLib/UefiLib.c
+++ b/MdePkg/Library/UefiLib/UefiLib.c
@@ -517,7 +517,7 @@ EfiReleaseLock (
This function tests whether the driver specified by DriverBindingHandle is
currently managing the controller specified by ControllerHandle. This test
- is performed by evaluating if the the protocol specified by ProtocolGuid is
+ is performed by evaluating if the protocol specified by ProtocolGuid is
present on ControllerHandle and is was opened by DriverBindingHandle with an
attribute of EFI_OPEN_PROTOCOL_BY_DRIVER.
If ProtocolGuid is NULL, then ASSERT().
diff --git a/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c b/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c
index 9f2d2bd9e6ff..cdcca4df8074 100644
--- a/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c
+++ b/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c
@@ -164,7 +164,7 @@ RfcEncodeTest (
b64String = Btc->TestOutput;
//
- // Only testing the the translate functionality, so preallocate the proper
+ // Only testing the translate functionality, so preallocate the proper
// string buffer.
//
@@ -248,7 +248,7 @@ RfcDecodeTest (
binString = Btc->TestOutput;
//
- // Only testing the the translate functionality
+ // Only testing the translate functionality
//
b64StringLen = AsciiStrnLenS (b64String, MAX_TEST_STRING_SIZE);
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 08/17] MdePkg: Remove duplicated words
2022-09-06 8:29 ` [PATCH v2 08/17] MdePkg: " PierreGondois
@ 2022-09-06 18:50 ` Michael D Kinney
0 siblings, 0 replies; 27+ messages in thread
From: Michael D Kinney @ 2022-09-06 18:50 UTC (permalink / raw)
To: Pierre.Gondois@arm.com, devel@edk2.groups.io, Kinney, Michael D
Cc: Gao, Liming
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> -----Original Message-----
> From: Pierre.Gondois@arm.com <Pierre.Gondois@arm.com>
> Sent: Tuesday, September 6, 2022 1:29 AM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois <pierre.gondois@arm.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>
> Subject: [PATCH v2 08/17] MdePkg: Remove duplicated words
>
> From: Pierre Gondois <pierre.gondois@arm.com>
>
> In an effort to clean the documentation of the above
> package, remove duplicated words.
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> ---
> MdePkg/Include/Guid/GraphicsInfoHob.h | 2 +-
> MdePkg/Include/Library/DevicePathLib.h | 2 +-
> MdePkg/Include/Library/DxeServicesLib.h | 6 +++---
> MdePkg/Include/Library/ExtractGuidedSectionLib.h | 6 +++---
> MdePkg/Include/Library/FileHandleLib.h | 4 ++--
> MdePkg/Include/Library/PeCoffLib.h | 2 +-
> MdePkg/Include/Library/UefiLib.h | 2 +-
> MdePkg/Include/Library/UnitTestLib.h | 2 +-
> MdePkg/Include/Pi/PiMmCis.h | 2 +-
> MdePkg/Include/Pi/PiSmmCis.h | 2 +-
> MdePkg/Include/Protocol/ComponentName.h | 2 +-
> MdePkg/Include/Protocol/Smbios.h | 2 +-
> MdePkg/Include/Uefi/UefiMultiPhase.h | 2 +-
> MdePkg/Include/Uefi/UefiSpec.h | 2 +-
> MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c | 2 +-
> .../BaseExtractGuidedSectionLib.c | 6 +++---
> MdePkg/Library/BaseS3StallLib/S3StallLib.c | 2 +-
> MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c | 2 +-
> MdePkg/Library/DxeServicesLib/DxeServicesLib.c | 6 +++---
> .../PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c | 6 +++---
> MdePkg/Library/SmmIoLib/SmmIoLib.c | 2 +-
> MdePkg/Library/SmmIoLib/SmmIoLib.inf | 2 +-
> MdePkg/Library/SmmIoLib/SmmIoLib.uni | 4 ++--
> MdePkg/Library/SmmMemLib/SmmMemLib.c | 2 +-
> MdePkg/Library/SmmMemLib/SmmMemLib.inf | 2 +-
> MdePkg/Library/SmmMemLib/SmmMemLib.uni | 4 ++--
> MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c | 4 ++--
> .../StandaloneMmDriverEntryPoint.c | 4 ++--
> MdePkg/Library/UefiDebugLibConOut/DebugLib.c | 2 +-
> MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c | 2 +-
> MdePkg/Library/UefiDebugLibStdErr/DebugLib.c | 2 +-
> MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf | 2 +-
> MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni | 2 +-
> MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c | 2 +-
> .../UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c | 2 +-
> MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c | 4 ++--
> MdePkg/Library/UefiLib/Console.c | 2 +-
> MdePkg/Library/UefiLib/UefiLib.c | 2 +-
> MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c | 4 ++--
> 39 files changed, 56 insertions(+), 56 deletions(-)
>
> diff --git a/MdePkg/Include/Guid/GraphicsInfoHob.h b/MdePkg/Include/Guid/GraphicsInfoHob.h
> index 237911e63aa0..ecfa283f924e 100644
> --- a/MdePkg/Include/Guid/GraphicsInfoHob.h
> +++ b/MdePkg/Include/Guid/GraphicsInfoHob.h
> @@ -5,7 +5,7 @@
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @par Revision Reference:
> - This HOB is introduced in in PI Version 1.4.
> + This HOB is introduced in PI Version 1.4.
>
> **/
>
> diff --git a/MdePkg/Include/Library/DevicePathLib.h b/MdePkg/Include/Library/DevicePathLib.h
> index 7a077e4217d2..039427d5b648 100644
> --- a/MdePkg/Include/Library/DevicePathLib.h
> +++ b/MdePkg/Include/Library/DevicePathLib.h
> @@ -23,7 +23,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> @retval TRUE DevicePath is valid.
> @retval FALSE DevicePath is NULL.
> @retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
> - @retval FALSE The length of any node node in the DevicePath is less
> + @retval FALSE The length of any node in the DevicePath is less
> than sizeof (EFI_DEVICE_PATH_PROTOCOL).
> @retval FALSE If MaxSize is not zero, the size of the DevicePath
> exceeds MaxSize.
> diff --git a/MdePkg/Include/Library/DxeServicesLib.h b/MdePkg/Include/Library/DxeServicesLib.h
> index b5c5c4dbdf27..d3304f65a5fd 100644
> --- a/MdePkg/Include/Library/DxeServicesLib.h
> +++ b/MdePkg/Include/Library/DxeServicesLib.h
> @@ -82,7 +82,7 @@ GetSectionFromAnyFvByFileType (
> If Size is NULL, then ASSERT().
>
>
> - @param NameGuid A pointer to to the FFS filename GUID to search for
> + @param NameGuid A pointer to the FFS filename GUID to search for
> within any of the firmware volumes in the platform.
> @param SectionType Indicates the FFS section type to search for within
> the FFS file specified by NameGuid.
> @@ -135,7 +135,7 @@ GetSectionFromAnyFv (
> If Buffer is NULL, then ASSERT().
> If Size is NULL, then ASSERT().
>
> - @param NameGuid A pointer to to the FFS filename GUID to search for
> + @param NameGuid A pointer to the FFS filename GUID to search for
> within the firmware volumes that the currently
> executing module was loaded from.
> @param SectionType Indicates the FFS section type to search for within
> @@ -269,7 +269,7 @@ GetFileBufferByFilePath (
>
> If NameGuid is NULL, then ASSERT().
>
> - @param NameGuid A pointer to to the FFS filename GUID to search for
> + @param NameGuid A pointer to the FFS filename GUID to search for
> within any of the firmware volumes in the platform.
> @param SectionType Indicates the FFS section type to search for within
> the FFS file specified by NameGuid.
> diff --git a/MdePkg/Include/Library/ExtractGuidedSectionLib.h b/MdePkg/Include/Library/ExtractGuidedSectionLib.h
> index 15058ae636ff..6cd5288a9456 100644
> --- a/MdePkg/Include/Library/ExtractGuidedSectionLib.h
> +++ b/MdePkg/Include/Library/ExtractGuidedSectionLib.h
> @@ -114,7 +114,7 @@ RETURN_STATUS
> If GetInfoHandler is NULL, then ASSERT().
> If DecodeHandler is NULL, then ASSERT().
>
> - @param[in] SectionGuid A pointer to the GUID associated with the the handlers
> + @param[in] SectionGuid A pointer to the GUID associated with the handlers
> of the GUIDed section type being registered.
> @param[in] GetInfoHandler Pointer to a function that examines a GUIDed section and returns the
> size of the decoded buffer and the size of an optional scratch buffer
> @@ -163,7 +163,7 @@ ExtractGuidedSectionGetGuidList (
> Examines a GUIDed section specified by InputSection.
> If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
> then RETURN_UNSUPPORTED is returned.
> - If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
> + If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
> of type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
> is used to retrieve the OututBufferSize, ScratchSize, and Attributes values. The return status from the handler of
> type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER is returned.
> @@ -206,7 +206,7 @@ ExtractGuidedSectionGetInfo (
> Decodes the GUIDed section specified by InputSection.
> If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
> then RETURN_UNSUPPORTED is returned.
> - If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
> + If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
> of type EXTRACT_GUIDED_SECTION_DECODE_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
> is used to decode InputSection into the buffer specified by OutputBuffer and the authentication status of this
> decode operation is returned in AuthenticationStatus. If the decoded buffer is identical to the data in InputSection,
> diff --git a/MdePkg/Include/Library/FileHandleLib.h b/MdePkg/Include/Library/FileHandleLib.h
> index 1a238edee38c..4eb2b9a35101 100644
> --- a/MdePkg/Include/Library/FileHandleLib.h
> +++ b/MdePkg/Include/Library/FileHandleLib.h
> @@ -384,9 +384,9 @@ FileHandleGetFileName (
> @param[in, out] Buffer The pointer to buffer to read into.
> @param[in, out] Size The pointer to number of bytes in Buffer.
> @param[in] Truncate If the buffer is large enough, this has no effect.
> - If the buffer is is too small and Truncate is TRUE,
> + If the buffer is too small and Truncate is TRUE,
> the line will be truncated.
> - If the buffer is is too small and Truncate is FALSE,
> + If the buffer is too small and Truncate is FALSE,
> then no read will occur.
>
> @param[in, out] Ascii Boolean value for indicating whether the file is
> diff --git a/MdePkg/Include/Library/PeCoffLib.h b/MdePkg/Include/Library/PeCoffLib.h
> index b45879453785..022dc24bccbc 100644
> --- a/MdePkg/Include/Library/PeCoffLib.h
> +++ b/MdePkg/Include/Library/PeCoffLib.h
> @@ -90,7 +90,7 @@ typedef struct {
> ///
> PHYSICAL_ADDRESS DestinationAddress;
> ///
> - /// PeCoffLoaderLoadImage() sets EntryPoint to to the entry point of the PE/COFF image.
> + /// PeCoffLoaderLoadImage() sets EntryPoint to the entry point of the PE/COFF image.
> ///
> PHYSICAL_ADDRESS EntryPoint;
> ///
> diff --git a/MdePkg/Include/Library/UefiLib.h b/MdePkg/Include/Library/UefiLib.h
> index be7da7fdf7da..54af95459cee 100644
> --- a/MdePkg/Include/Library/UefiLib.h
> +++ b/MdePkg/Include/Library/UefiLib.h
> @@ -406,7 +406,7 @@ EfiReleaseLock (
>
> This function tests whether the driver specified by DriverBindingHandle is
> currently managing the controller specified by ControllerHandle. This test
> - is performed by evaluating if the the protocol specified by ProtocolGuid is
> + is performed by evaluating if the protocol specified by ProtocolGuid is
> present on ControllerHandle and is was opened by DriverBindingHandle with an
> attribute of EFI_OPEN_PROTOCOL_BY_DRIVER.
> If ProtocolGuid is NULL, then ASSERT().
> diff --git a/MdePkg/Include/Library/UnitTestLib.h b/MdePkg/Include/Library/UnitTestLib.h
> index 71c205b1b143..e9c4e23a7d43 100644
> --- a/MdePkg/Include/Library/UnitTestLib.h
> +++ b/MdePkg/Include/Library/UnitTestLib.h
> @@ -755,7 +755,7 @@ UnitTestAssertNotNull (
> is either pass, skipped, or failed.
> @param[in] FunctionName Null-terminated ASCII string of the function
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> - @param[in] LineNumber The source file line number of the the function
> + @param[in] LineNumber The source file line number of the function
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> @param[in] FileName Null-terminated ASCII string of the filename
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> diff --git a/MdePkg/Include/Pi/PiMmCis.h b/MdePkg/Include/Pi/PiMmCis.h
> index 01340952d89b..53269d1bf68e 100644
> --- a/MdePkg/Include/Pi/PiMmCis.h
> +++ b/MdePkg/Include/Pi/PiMmCis.h
> @@ -289,7 +289,7 @@ struct _EFI_MM_SYSTEM_TABLE {
> ///
>
> ///
> - /// A number between zero and and the NumberOfCpus field. This field designates
> + /// A number between zero and the NumberOfCpus field. This field designates
> /// which processor is executing the MM infrastructure.
> ///
> UINTN CurrentlyExecutingCpu;
> diff --git a/MdePkg/Include/Pi/PiSmmCis.h b/MdePkg/Include/Pi/PiSmmCis.h
> index bf5b580c3245..540603343ae2 100644
> --- a/MdePkg/Include/Pi/PiSmmCis.h
> +++ b/MdePkg/Include/Pi/PiSmmCis.h
> @@ -144,7 +144,7 @@ struct _EFI_SMM_SYSTEM_TABLE2 {
> ///
>
> ///
> - /// A number between zero and and the NumberOfCpus field. This field designates
> + /// A number between zero and the NumberOfCpus field. This field designates
> /// which processor is executing the SMM infrastructure.
> ///
> UINTN CurrentlyExecutingCpu;
> diff --git a/MdePkg/Include/Protocol/ComponentName.h b/MdePkg/Include/Protocol/ComponentName.h
> index 46fa5e7ad8e5..da4bbddb8fb1 100644
> --- a/MdePkg/Include/Protocol/ComponentName.h
> +++ b/MdePkg/Include/Protocol/ComponentName.h
> @@ -26,7 +26,7 @@ typedef struct _EFI_COMPONENT_NAME_PROTOCOL EFI_COMPONENT_NAME_PROTOCOL;
>
> @param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
> @param Language A pointer to a three-character ISO 639-2 language identifier.
> - This is the language of the driver name that that the caller
> + This is the language of the driver name that the caller
> is requesting, and it must match one of the languages specified
> in SupportedLanguages. The number of languages supported by a
> driver is up to the driver writer.
> diff --git a/MdePkg/Include/Protocol/Smbios.h b/MdePkg/Include/Protocol/Smbios.h
> index f9346aac7224..a10db6c43f22 100644
> --- a/MdePkg/Include/Protocol/Smbios.h
> +++ b/MdePkg/Include/Protocol/Smbios.h
> @@ -174,7 +174,7 @@ EFI_STATUS
> @param[in] Type On entry, it points to the type of the next SMBIOS record to return. If NULL, it
> indicates that the next record of any type will be returned. Type is not
> modified by the this function.
> - @param[out] Record On exit, points to a pointer to the the SMBIOS Record consisting of the formatted area
> + @param[out] Record On exit, points to a pointer to the SMBIOS Record consisting of the formatted area
> followed by the unformatted area. The unformatted area optionally contains text strings.
> @param[out] ProducerHandle On exit, points to the ProducerHandle registered by Add(). If no
> ProducerHandle was passed into Add() NULL is returned. If a NULL pointer is
> diff --git a/MdePkg/Include/Uefi/UefiMultiPhase.h b/MdePkg/Include/Uefi/UefiMultiPhase.h
> index 22bae43e36e8..df9fcd880f70 100644
> --- a/MdePkg/Include/Uefi/UefiMultiPhase.h
> +++ b/MdePkg/Include/Uefi/UefiMultiPhase.h
> @@ -177,7 +177,7 @@ typedef struct {
> /// EFI_CERT_TYPE_RSA2048_SHA256_GUID. If the attribute specifies
> /// authenticated access, then the Data buffer should begin with an
> /// authentication descriptor prior to the data payload and DataSize
> -/// should reflect the the data.and descriptor size. The caller
> +/// should reflect the data.and descriptor size. The caller
> /// shall digest the Monotonic Count value and the associated data
> /// for the variable update using the SHA-256 1-way hash algorithm.
> /// The ensuing the 32-byte digest will be signed using the private
> diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSpec.h
> index 2b38b100f6a5..acfd07eeaa5b 100644
> --- a/MdePkg/Include/Uefi/UefiSpec.h
> +++ b/MdePkg/Include/Uefi/UefiSpec.h
> @@ -809,7 +809,7 @@ EFI_STATUS
>
> @retval EFI_SUCCESS The operation completed successfully.
> @retval EFI_INVALID_PARAMETER A time field is out of range.
> - @retval EFI_DEVICE_ERROR The time could not be set due due to hardware error.
> + @retval EFI_DEVICE_ERROR The time could not be set due to hardware error.
>
> **/
> typedef
> diff --git a/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c b/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c
> index bd5686947712..2a8881637d45 100644
> --- a/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c
> +++ b/MdePkg/Library/BaseDebugLibSerialPort/DebugLib.c
> @@ -264,7 +264,7 @@ DebugClearMemory (
> ASSERT (Buffer != NULL);
>
> //
> - // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
> + // SetMem() checks for the ASSERT() condition on Length and returns Buffer
> //
> return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
> }
> diff --git a/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c
> b/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c
> index e6f479b829f6..d364a406af71 100644
> --- a/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c
> +++ b/MdePkg/Library/BaseExtractGuidedSectionLib/BaseExtractGuidedSectionLib.c
> @@ -144,7 +144,7 @@ ExtractGuidedSectionGetGuidList (
> If GetInfoHandler is NULL, then ASSERT().
> If DecodeHandler is NULL, then ASSERT().
>
> - @param[in] SectionGuid A pointer to the GUID associated with the the handlers
> + @param[in] SectionGuid A pointer to the GUID associated with the handlers
> of the GUIDed section type being registered.
> @param[in] GetInfoHandler The pointer to a function that examines a GUIDed section and returns the
> size of the decoded buffer and the size of an optional scratch buffer
> @@ -224,7 +224,7 @@ ExtractGuidedSectionRegisterHandlers (
> Examines a GUIDed section specified by InputSection.
> If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
> then RETURN_UNSUPPORTED is returned.
> - If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
> + If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
> of type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
> is used to retrieve the OututBufferSize, ScratchSize, and Attributes values. The return status from the handler of
> type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER is returned.
> @@ -318,7 +318,7 @@ ExtractGuidedSectionGetInfo (
> Decodes the GUIDed section specified by InputSection.
> If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
> then RETURN_UNSUPPORTED is returned.
> - If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
> + If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
> of type EXTRACT_GUIDED_SECTION_DECODE_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
> is used to decode InputSection into the buffer specified by OutputBuffer and the authentication status of this
> decode operation is returned in AuthenticationStatus. If the decoded buffer is identical to the data in InputSection,
> diff --git a/MdePkg/Library/BaseS3StallLib/S3StallLib.c b/MdePkg/Library/BaseS3StallLib/S3StallLib.c
> index ea886ca61cd3..6be227c5fb9a 100644
> --- a/MdePkg/Library/BaseS3StallLib/S3StallLib.c
> +++ b/MdePkg/Library/BaseS3StallLib/S3StallLib.c
> @@ -17,7 +17,7 @@
> #include <Library/S3StallLib.h>
>
> /**
> - Stalls the CPU for at least the given number of microseconds and and saves
> + Stalls the CPU for at least the given number of microseconds and saves
> the value in the S3 script to be replayed on S3 resume.
>
> Stalls the CPU for the number of microseconds specified by MicroSeconds.
> diff --git a/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c b/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c
> index 11d188d07ff6..8c201daf3914 100644
> --- a/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c
> +++ b/MdePkg/Library/DxeRuntimeDebugLibSerialPort/DebugLib.c
> @@ -337,7 +337,7 @@ DebugClearMemory (
> ASSERT (Buffer != NULL);
>
> //
> - // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
> + // SetMem() checks for the ASSERT() condition on Length and returns Buffer
> //
> return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
> }
> diff --git a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c
> index 8f5a15dedc9f..7a5c031a7daa 100644
> --- a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c
> +++ b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c
> @@ -347,7 +347,7 @@ Done:
> If Size is NULL, then ASSERT().
>
>
> - @param NameGuid A pointer to to the FFS filename GUID to search for
> + @param NameGuid A pointer to the FFS filename GUID to search for
> within any of the firmware volumes in the platform.
> @param SectionType Indicates the FFS section type to search for within
> the FFS file specified by NameGuid.
> @@ -470,7 +470,7 @@ Done:
> If Buffer is NULL, then ASSERT().
> If Size is NULL, then ASSERT().
>
> - @param NameGuid A pointer to to the FFS filename GUID to search for
> + @param NameGuid A pointer to the FFS filename GUID to search for
> within the firmware volumes that the currently
> executing module was loaded from.
> @param SectionType Indicates the FFS section type to search for within
> @@ -957,7 +957,7 @@ Finish:
>
> If NameGuid is NULL, then ASSERT().
>
> - @param NameGuid A pointer to to the FFS filename GUID to search for
> + @param NameGuid A pointer to the FFS filename GUID to search for
> within any of the firmware volumes in the platform.
> @param SectionType Indicates the FFS section type to search for within
> the FFS file specified by NameGuid.
> diff --git a/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c
> b/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c
> index c538f157a242..21c8b470ccf5 100644
> --- a/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c
> +++ b/MdePkg/Library/PeiExtractGuidedSectionLib/PeiExtractGuidedSectionLib.c
> @@ -170,7 +170,7 @@ ExtractGuidedSectionGetGuidList (
> If GetInfoHandler is NULL, then ASSERT().
> If DecodeHandler is NULL, then ASSERT().
>
> - @param[in] SectionGuid A pointer to the GUID associated with the the handlers
> + @param[in] SectionGuid A pointer to the GUID associated with the handlers
> of the GUIDed section type being registered.
> @param[in] GetInfoHandler The pointer to a function that examines a GUIDed section and returns the
> size of the decoded buffer and the size of an optional scratch buffer
> @@ -260,7 +260,7 @@ ExtractGuidedSectionRegisterHandlers (
> Examines a GUIDed section specified by InputSection.
> If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
> then RETURN_UNSUPPORTED is returned.
> - If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
> + If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
> of type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
> is used to retrieve the OututBufferSize, ScratchSize, and Attributes values. The return status from the handler of
> type EXTRACT_GUIDED_SECTION_GET_INFO_HANDLER is returned.
> @@ -354,7 +354,7 @@ ExtractGuidedSectionGetInfo (
> Decodes the GUIDed section specified by InputSection.
> If GUID for InputSection does not match any of the GUIDs registered through ExtractGuidedSectionRegisterHandlers(),
> then RETURN_UNSUPPORTED is returned.
> - If the GUID of InputSection does match the GUID that this handler supports, then the the associated handler
> + If the GUID of InputSection does match the GUID that this handler supports, then the associated handler
> of type EXTRACT_GUIDED_SECTION_DECODE_HANDLER that was registered with ExtractGuidedSectionRegisterHandlers()
> is used to decode InputSection into the buffer specified by OutputBuffer and the authentication status of this
> decode operation is returned in AuthenticationStatus. If the decoded buffer is identical to the data in InputSection,
> diff --git a/MdePkg/Library/SmmIoLib/SmmIoLib.c b/MdePkg/Library/SmmIoLib/SmmIoLib.c
> index ad4bfff68c8a..90fbdde96f98 100644
> --- a/MdePkg/Library/SmmIoLib/SmmIoLib.c
> +++ b/MdePkg/Library/SmmIoLib/SmmIoLib.c
> @@ -1,7 +1,7 @@
> /** @file
> Instance of SMM IO check library.
>
> - SMM IO check library library implementation. This library consumes GCD to collect all valid
> + SMM IO check library implementation. This library consumes GCD to collect all valid
> IO space defined by a platform.
> A platform may have its own SmmIoLib instance to exclude more IO space.
>
> diff --git a/MdePkg/Library/SmmIoLib/SmmIoLib.inf b/MdePkg/Library/SmmIoLib/SmmIoLib.inf
> index 5f7356de5d7a..d195afe0a0e1 100644
> --- a/MdePkg/Library/SmmIoLib/SmmIoLib.inf
> +++ b/MdePkg/Library/SmmIoLib/SmmIoLib.inf
> @@ -1,7 +1,7 @@
> ## @file
> # Instance of SMM IO check library.
> #
> -# SMM IO check library library implementation. This library consumes GCD to collect all valid
> +# SMM IO check library implementation. This library consumes GCD to collect all valid
> # IO space defined by a platform.
> # A platform may have its own SmmIoLib instance to exclude more IO space.
> #
> diff --git a/MdePkg/Library/SmmIoLib/SmmIoLib.uni b/MdePkg/Library/SmmIoLib/SmmIoLib.uni
> index d46bc0a7f9d8..fcce039b4aec 100644
> --- a/MdePkg/Library/SmmIoLib/SmmIoLib.uni
> +++ b/MdePkg/Library/SmmIoLib/SmmIoLib.uni
> @@ -1,7 +1,7 @@
> // /** @file
> // Instance of SMM IO check library.
> //
> -// SMM IO check library library implementation. This library consumes GCD to collect all valid
> +// SMM IO check library implementation. This library consumes GCD to collect all valid
> // IO space defined by a platform.
> // A platform may have its own SmmIoLib instance to exclude more IO space.
> //
> @@ -14,5 +14,5 @@
>
> #string STR_MODULE_ABSTRACT #language en-US "Instance of SMM IO check library."
>
> -#string STR_MODULE_DESCRIPTION #language en-US "SMM IO check library library implementation. This library consumes
> GCD to collect all valid IO space defined by a platform. A platform may have its own SmmIoLib instance to exclude more IO
> space."
> +#string STR_MODULE_DESCRIPTION #language en-US "SMM IO check library implementation. This library consumes GCD to
> collect all valid IO space defined by a platform. A platform may have its own SmmIoLib instance to exclude more IO space."
>
> diff --git a/MdePkg/Library/SmmMemLib/SmmMemLib.c b/MdePkg/Library/SmmMemLib/SmmMemLib.c
> index ef087d0f050b..0eae5d2ed5e0 100644
> --- a/MdePkg/Library/SmmMemLib/SmmMemLib.c
> +++ b/MdePkg/Library/SmmMemLib/SmmMemLib.c
> @@ -1,7 +1,7 @@
> /** @file
> Instance of SMM memory check library.
>
> - SMM memory check library library implementation. This library consumes SMM_ACCESS2_PROTOCOL
> + SMM memory check library implementation. This library consumes SMM_ACCESS2_PROTOCOL
> to get SMRAM information. In order to use this library instance, the platform should produce
> all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core
> and SMM driver) and/or specific dedicated hardware.
> diff --git a/MdePkg/Library/SmmMemLib/SmmMemLib.inf b/MdePkg/Library/SmmMemLib/SmmMemLib.inf
> index bc30057d772f..f53e81acb9c2 100644
> --- a/MdePkg/Library/SmmMemLib/SmmMemLib.inf
> +++ b/MdePkg/Library/SmmMemLib/SmmMemLib.inf
> @@ -1,7 +1,7 @@
> ## @file
> # Instance of SMM memory check library.
> #
> -# SMM memory check library library implementation. This library consumes SMM_ACCESS2_PROTOCOL
> +# SMM memory check library implementation. This library consumes SMM_ACCESS2_PROTOCOL
> # to get SMRAM information. In order to use this library instance, the platform should produce
> # all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core
> # and SMM driver) and/or specific dedicated hardware.
> diff --git a/MdePkg/Library/SmmMemLib/SmmMemLib.uni b/MdePkg/Library/SmmMemLib/SmmMemLib.uni
> index f1d678344055..3d553884376e 100644
> --- a/MdePkg/Library/SmmMemLib/SmmMemLib.uni
> +++ b/MdePkg/Library/SmmMemLib/SmmMemLib.uni
> @@ -1,7 +1,7 @@
> // /** @file
> // SMM memory check library.
> //
> -// SMM memory check library library implementation. This library consumes SMM_ACCESS2_PROTOCOL
> +// SMM memory check library implementation. This library consumes SMM_ACCESS2_PROTOCOL
> // to get SMRAM information. In order to use this library instance, the platform should produce
> // all SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core
> // and SMM driver) and/or specific dedicated hardware.
> @@ -15,5 +15,5 @@
>
> #string STR_MODULE_ABSTRACT #language en-US "SMM memory check library"
>
> -#string STR_MODULE_DESCRIPTION #language en-US "SMM memory check library library implementation. This library
> consumes SMM_ACCESS2_PROTOCOL to get SMRAM information. In order to use this library instance, the platform should produce all
> SMRAM range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core and SMM driver) and/or specific
> dedicated hardware."
> +#string STR_MODULE_DESCRIPTION #language en-US "SMM memory check library implementation. This library consumes
> SMM_ACCESS2_PROTOCOL to get SMRAM information. In order to use this library instance, the platform should produce all SMRAM
> range via SMM_ACCESS2_PROTOCOL, including the range for firmware (like SMM Core and SMM driver) and/or specific dedicated
> hardware."
>
> diff --git a/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c b/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c
> index ff143f00ab9a..1b293565b0a9 100644
> --- a/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c
> +++ b/MdePkg/Library/SmmPeriodicSmiLib/SmmPeriodicSmiLib.c
> @@ -240,7 +240,7 @@ LookupPeriodicSmiLibraryHandler (
> }
>
> //
> - // Search the periodic SMI handler entries for a a matching DispatchHandle
> + // Search the periodic SMI handler entries for a matching DispatchHandle
> //
> for ( Link = GetFirstNode (&gPeriodicSmiLibraryHandlers)
> ; !IsNull (&gPeriodicSmiLibraryHandlers, Link)
> @@ -353,7 +353,7 @@ EnlargeFreePeriodicSmiLibraryHandlerList (
> SMI handler. If no free entries are available, then additional
> entries are allocated.
>
> - @retval NULL There are not enough resources available to to allocate a free entry.
> + @retval NULL There are not enough resources available to allocate a free entry.
> @retval other Pointer to a free PERIODIC_SMI_LIBRARY_HANDLER_CONTEXT structure.
>
> **/
> diff --git a/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c
> b/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c
> index ff1c53b185ac..11ba5d5a36ae 100644
> --- a/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c
> +++ b/MdePkg/Library/StandaloneMmDriverEntryPoint/StandaloneMmDriverEntryPoint.c
> @@ -81,8 +81,8 @@ _DriverUnloadHandler (
> EFI_STATUS
> EFIAPI
> _ModuleEntryPoint (
> - IN EFI_HANDLE ImageHandle,
> - IN IN EFI_MM_SYSTEM_TABLE *MmSystemTable
> + IN EFI_HANDLE ImageHandle,
> + IN EFI_MM_SYSTEM_TABLE *MmSystemTable
> )
> {
> EFI_STATUS Status;
> diff --git a/MdePkg/Library/UefiDebugLibConOut/DebugLib.c b/MdePkg/Library/UefiDebugLibConOut/DebugLib.c
> index 65c8dc2b4654..ac8b46d670d7 100644
> --- a/MdePkg/Library/UefiDebugLibConOut/DebugLib.c
> +++ b/MdePkg/Library/UefiDebugLibConOut/DebugLib.c
> @@ -264,7 +264,7 @@ DebugClearMemory (
> ASSERT (Buffer != NULL);
>
> //
> - // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
> + // SetMem() checks for the ASSERT() condition on Length and returns Buffer
> //
> return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
> }
> diff --git a/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c b/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c
> index c25199b53fcf..f61113c739f8 100644
> --- a/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c
> +++ b/MdePkg/Library/UefiDebugLibDebugPortProtocol/DebugLib.c
> @@ -318,7 +318,7 @@ DebugClearMemory (
> ASSERT (Buffer != NULL);
>
> //
> - // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
> + // SetMem() checks for the ASSERT() condition on Length and returns Buffer
> //
> return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
> }
> diff --git a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
> index 5b28cd10ae21..9bbac56ea5b0 100644
> --- a/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
> +++ b/MdePkg/Library/UefiDebugLibStdErr/DebugLib.c
> @@ -264,7 +264,7 @@ DebugClearMemory (
> ASSERT (Buffer != NULL);
>
> //
> - // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
> + // SetMem() checks for the ASSERT() condition on Length and returns Buffer
> //
> return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
> }
> diff --git a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
> b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
> index 5ecb971a0ac5..3a4fc15525a5 100644
> --- a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
> +++ b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
> @@ -1,7 +1,7 @@
> ## @file
> # Instance of Debug Library using the Standard Error Device.
> #
> -# Debug Lib that sends messages to the the Standard Error Device in the EFI System Table.
> +# Debug Lib that sends messages to the Standard Error Device in the EFI System Table.
> #
> # Copyright (c) 2018, Microsoft Corporation
> #
> diff --git a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni
> b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni
> index 1748fb3ff0b0..e6d0cca70224 100644
> --- a/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni
> +++ b/MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.uni
> @@ -1,7 +1,7 @@
> // /** @file
> // Instance of Debug Library using the Standard Error Device.
> //
> -// Debug Lib that sends messages to the the Standard Error Device in the EFI System Table.
> +// Debug Lib that sends messages to the Standard Error Device in the EFI System Table.
> //
> // Copyright (c) 2007 - 2014, Intel Corporation. All rights reserved.<BR>
> //
> diff --git a/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c b/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c
> index 5ee3e9a31f48..ce76bc8ff97b 100644
> --- a/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c
> +++ b/MdePkg/Library/UefiDevicePathLib/DevicePathUtilities.c
> @@ -36,7 +36,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED CONST EFI_DEVICE_PATH_PROTOCOL mUefiDevicePathLib
> @retval TRUE DevicePath is valid.
> @retval FALSE DevicePath is NULL.
> @retval FALSE Maxsize is less than sizeof(EFI_DEVICE_PATH_PROTOCOL).
> - @retval FALSE The length of any node node in the DevicePath is less
> + @retval FALSE The length of any node in the DevicePath is less
> than sizeof (EFI_DEVICE_PATH_PROTOCOL).
> @retval FALSE If MaxSize is not zero, the size of the DevicePath
> exceeds MaxSize.
> diff --git a/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c
> b/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c
> index 4ff9308a68fe..5cc30397a3a1 100644
> --- a/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c
> +++ b/MdePkg/Library/UefiDevicePathLibDevicePathProtocol/UefiDevicePathLib.c
> @@ -76,7 +76,7 @@ DevicePathLibConstructor (
> @param MaxSize The maximum size of the device path data structure.
>
> @retval TRUE DevicePath is valid.
> - @retval FALSE The length of any node node in the DevicePath is less
> + @retval FALSE The length of any node in the DevicePath is less
> than sizeof (EFI_DEVICE_PATH_PROTOCOL).
> @retval FALSE If MaxSize is not zero, the size of the DevicePath
> exceeds MaxSize.
> diff --git a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
> index 86678e965d99..f1a07c4fbde3 100644
> --- a/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
> +++ b/MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.c
> @@ -954,9 +954,9 @@ FileHandleReturnLine (
> @param[in, out] Buffer The pointer to buffer to read into.
> @param[in, out] Size The pointer to number of bytes in Buffer.
> @param[in] Truncate If the buffer is large enough, this has no effect.
> - If the buffer is is too small and Truncate is TRUE,
> + If the buffer is too small and Truncate is TRUE,
> the line will be truncated.
> - If the buffer is is too small and Truncate is FALSE,
> + If the buffer is too small and Truncate is FALSE,
> then no read will occur.
>
> @param[in, out] Ascii Boolean value for indicating whether the file is
> diff --git a/MdePkg/Library/UefiLib/Console.c b/MdePkg/Library/UefiLib/Console.c
> index c37e3d0f5810..a5425740a42c 100644
> --- a/MdePkg/Library/UefiLib/Console.c
> +++ b/MdePkg/Library/UefiLib/Console.c
> @@ -413,7 +413,7 @@ CreatePopUp (
> CHAR16 *TmpString;
>
> //
> - // Determine the length of the longest line in the popup and the the total
> + // Determine the length of the longest line in the popup and the total
> // number of lines in the popup
> //
> VA_START (Args, Key);
> diff --git a/MdePkg/Library/UefiLib/UefiLib.c b/MdePkg/Library/UefiLib/UefiLib.c
> index 95d0319d3a33..c26cb0b6eb08 100644
> --- a/MdePkg/Library/UefiLib/UefiLib.c
> +++ b/MdePkg/Library/UefiLib/UefiLib.c
> @@ -517,7 +517,7 @@ EfiReleaseLock (
>
> This function tests whether the driver specified by DriverBindingHandle is
> currently managing the controller specified by ControllerHandle. This test
> - is performed by evaluating if the the protocol specified by ProtocolGuid is
> + is performed by evaluating if the protocol specified by ProtocolGuid is
> present on ControllerHandle and is was opened by DriverBindingHandle with an
> attribute of EFI_OPEN_PROTOCOL_BY_DRIVER.
> If ProtocolGuid is NULL, then ASSERT().
> diff --git a/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c b/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c
> index 9f2d2bd9e6ff..cdcca4df8074 100644
> --- a/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c
> +++ b/MdePkg/Test/UnitTest/Library/BaseLib/Base64UnitTest.c
> @@ -164,7 +164,7 @@ RfcEncodeTest (
> b64String = Btc->TestOutput;
>
> //
> - // Only testing the the translate functionality, so preallocate the proper
> + // Only testing the translate functionality, so preallocate the proper
> // string buffer.
> //
>
> @@ -248,7 +248,7 @@ RfcDecodeTest (
> binString = Btc->TestOutput;
>
> //
> - // Only testing the the translate functionality
> + // Only testing the translate functionality
> //
>
> b64StringLen = AsciiStrnLenS (b64String, MAX_TEST_STRING_SIZE);
> --
> 2.25.1
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v2 09/17] NetworkPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (7 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 08/17] MdePkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 10/17] PcAtChipsetPkg: " PierreGondois
` (7 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Maciej Rabeda
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
Reviewed-by: Maciej Rabeda <maciej.rabeda@linux.intel.com>
---
NetworkPkg/DnsDxe/ComponentName.c | 4 ++--
NetworkPkg/HttpBootDxe/HttpBootComponentName.c | 2 +-
NetworkPkg/HttpBootDxe/HttpBootComponentName.h | 2 +-
NetworkPkg/HttpDxe/ComponentName.c | 2 +-
NetworkPkg/HttpDxe/ComponentName.h | 2 +-
NetworkPkg/Library/DxeNetLib/DxeNetLib.c | 2 +-
.../WifiConnectionManagerDxe/WifiConnectionMgrComponentName.c | 2 +-
.../WifiConnectionManagerDxe/WifiConnectionMgrComponentName.h | 2 +-
8 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/NetworkPkg/DnsDxe/ComponentName.c b/NetworkPkg/DnsDxe/ComponentName.c
index 32adb93f4130..cf17779aa550 100644
--- a/NetworkPkg/DnsDxe/ComponentName.c
+++ b/NetworkPkg/DnsDxe/ComponentName.c
@@ -17,7 +17,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -128,7 +128,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED EFI_UNICODE_STRING_TABLE *gDnsControllerNameTable
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/NetworkPkg/HttpBootDxe/HttpBootComponentName.c b/NetworkPkg/HttpBootDxe/HttpBootComponentName.c
index bc1993df431b..3fdc22b551fd 100644
--- a/NetworkPkg/HttpBootDxe/HttpBootComponentName.c
+++ b/NetworkPkg/HttpBootDxe/HttpBootComponentName.c
@@ -51,7 +51,7 @@ EFI_UNICODE_STRING_TABLE mHttpBootDxeControllerNameTable[] = {
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/NetworkPkg/HttpBootDxe/HttpBootComponentName.h b/NetworkPkg/HttpBootDxe/HttpBootComponentName.h
index f53af8762c0c..59415989c507 100644
--- a/NetworkPkg/HttpBootDxe/HttpBootComponentName.h
+++ b/NetworkPkg/HttpBootDxe/HttpBootComponentName.h
@@ -15,7 +15,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/NetworkPkg/HttpDxe/ComponentName.c b/NetworkPkg/HttpDxe/ComponentName.c
index db41d582c3b2..29cbae6b57b2 100644
--- a/NetworkPkg/HttpDxe/ComponentName.c
+++ b/NetworkPkg/HttpDxe/ComponentName.c
@@ -44,7 +44,7 @@ EFI_UNICODE_STRING_TABLE mHttpDxeDriverNameTable[] = {
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/NetworkPkg/HttpDxe/ComponentName.h b/NetworkPkg/HttpDxe/ComponentName.h
index 3dc7c4f66fdc..9d39db3d6a9f 100644
--- a/NetworkPkg/HttpDxe/ComponentName.h
+++ b/NetworkPkg/HttpDxe/ComponentName.h
@@ -14,7 +14,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/NetworkPkg/Library/DxeNetLib/DxeNetLib.c b/NetworkPkg/Library/DxeNetLib/DxeNetLib.c
index fd4a9e15a892..41f8a7c364da 100644
--- a/NetworkPkg/Library/DxeNetLib/DxeNetLib.c
+++ b/NetworkPkg/Library/DxeNetLib/DxeNetLib.c
@@ -1011,7 +1011,7 @@ NetListRemoveHead (
}
/**
- Remove the last node entry on the list and and return the removed node entry.
+ Remove the last node entry on the list and return the removed node entry.
Removes the last node entry from a doubly linked list. It is up to the caller of
this function to release the memory used by the first node if that is required. On
diff --git a/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.c b/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.c
index c72f9f7095c2..fabdc1e5d881 100644
--- a/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.c
+++ b/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.c
@@ -66,7 +66,7 @@ EFI_UNICODE_STRING_TABLE mWifiMgrDxeControllerNameTable[] = {
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.h b/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.h
index c1865f8dcc77..6edb796091ad 100644
--- a/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.h
+++ b/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrComponentName.h
@@ -15,7 +15,7 @@
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 10/17] PcAtChipsetPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (8 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 09/17] NetworkPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 11/17] RedfishPkg: " PierreGondois
` (6 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Ray Ni
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
.../Bus/Pci/IdeControllerDxe/ComponentName.c | 4 ++--
.../Bus/Pci/IdeControllerDxe/IdeController.h | 4 ++--
PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c | 10 +++++-----
PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.h | 2 +-
.../PcatRealTimeClockRuntimeDxe/PcRtcEntry.c | 2 +-
5 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/ComponentName.c b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/ComponentName.c
index e5231a59146d..b2d98d641f35 100644
--- a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/ComponentName.c
+++ b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/ComponentName.c
@@ -60,7 +60,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED EFI_UNICODE_STRING_TABLE mIdeControllerController
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -109,7 +109,7 @@ IdeControllerComponentNameGetDriverName (
that wishes to retrieve the name of a child controller.
@param Language A pointer to a three character ISO 639-2 language
identifier. This is the language of the controller name
- that that the caller is requesting, and it must match one
+ that the caller is requesting, and it must match one
of the languages specified in SupportedLanguages. The
number of languages supported by a driver is up to the
driver writer.
diff --git a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h
index 978a8f709c7b..48b45e0e67c9 100644
--- a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h
+++ b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h
@@ -396,7 +396,7 @@ IdeInitSetTiming (
@param This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param Language A pointer to a three character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -437,7 +437,7 @@ IdeControllerComponentNameGetDriverName (
that wishes to retrieve the name of a child controller.
@param Language A pointer to a three character ISO 639-2 language
identifier. This is the language of the controller name
- that that the caller is requesting, and it must match one
+ that the caller is requesting, and it must match one
of the languages specified in SupportedLanguages. The
number of languages supported by a driver is up to the
driver writer.
diff --git a/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c b/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c
index 9242a2e82600..e248ff0b1432 100644
--- a/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c
+++ b/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c
@@ -48,8 +48,8 @@ CompareHMS (
@param From the first date
@param To the second date
- @retval TRUE From is previous to To within 24 hours.
- @retval FALSE From is later, or it is previous to To more than 24 hours.
+ @retval TRUE From is previous to within 24 hours.
+ @retval FALSE From is later, or it is previous to more than 24 hours.
**/
BOOLEAN
IsWithinOneDay (
@@ -546,7 +546,7 @@ PcRtcGetTime (
@retval EFI_SUCCESS The operation completed successfully.
@retval EFI_INVALID_PARAMETER A time field is out of range.
- @retval EFI_DEVICE_ERROR The time could not be set due due to hardware error.
+ @retval EFI_DEVICE_ERROR The time could not be set due to hardware error.
**/
EFI_STATUS
@@ -1277,8 +1277,8 @@ CompareHMS (
@param From the first date
@param To the second date
- @retval TRUE From is previous to To within 24 hours.
- @retval FALSE From is later, or it is previous to To more than 24 hours.
+ @retval TRUE From is previous to within 24 hours.
+ @retval FALSE From is later, or it is previous to more than 24 hours.
**/
BOOLEAN
IsWithinOneDay (
diff --git a/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.h b/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.h
index 7e0e98fbcc18..672dc5dadf4e 100644
--- a/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.h
+++ b/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.h
@@ -153,7 +153,7 @@ PcRtcInit (
@retval EFI_SUCCESS The operation completed successfully.
@retval EFI_INVALID_PARAMETER A time field is out of range.
- @retval EFI_DEVICE_ERROR The time could not be set due due to hardware error.
+ @retval EFI_DEVICE_ERROR The time could not be set due to hardware error.
**/
EFI_STATUS
diff --git a/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtcEntry.c b/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtcEntry.c
index e6e9458d7544..9e1201c81449 100644
--- a/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtcEntry.c
+++ b/PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtcEntry.c
@@ -49,7 +49,7 @@ PcRtcEfiGetTime (
@retval EFI_SUCCESS The operation completed successfully.
@retval EFI_INVALID_PARAMETER A time field is out of range.
- @retval EFI_DEVICE_ERROR The time could not be set due due to hardware error.
+ @retval EFI_DEVICE_ERROR The time could not be set due to hardware error.
**/
EFI_STATUS
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 11/17] RedfishPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (9 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 10/17] PcAtChipsetPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 12/17] SecurityPkg: " PierreGondois
` (5 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Abner Chang
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Abner Chang <abner.chang@amd.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
Reviewed-by: Abner Chang <abner.chang@amd.com>
---
RedfishPkg/RedfishDiscoverDxe/ComponentName.c | 4 ++--
RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 2 +-
RedfishPkg/RedfishRestExDxe/ComponentName.c | 4 ++--
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/RedfishPkg/RedfishDiscoverDxe/ComponentName.c b/RedfishPkg/RedfishDiscoverDxe/ComponentName.c
index f34d7ab5d953..886f0b97b0de 100644
--- a/RedfishPkg/RedfishDiscoverDxe/ComponentName.c
+++ b/RedfishPkg/RedfishDiscoverDxe/ComponentName.c
@@ -19,7 +19,7 @@
@param[in] This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param[in] Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -130,7 +130,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED EFI_UNICODE_STRING_TABLE *gRedfishDiscoverControl
@param[in] This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param[in] Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c
index bf50c78c9280..a542d97fe09e 100644
--- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c
+++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c
@@ -1325,7 +1325,7 @@ EFI_REDFISH_DISCOVER_PROTOCOL mRedfishDiscover = {
@param[in] ControllerHandle MAC address of this network interface.
@param[in] NetworkProtocolType Network protocol type.
@param[out] IsNewInstance BOOLEAN means new instance or not.
- @param[out] NetworkInterface Pointer to to EFI_REDFISH_DISCOVER_NETWORK_INTERFACE_INTERNAL.
+ @param[out] NetworkInterface Pointer to EFI_REDFISH_DISCOVER_NETWORK_INTERFACE_INTERNAL.
@retval EFI_STATUS
**/
diff --git a/RedfishPkg/RedfishRestExDxe/ComponentName.c b/RedfishPkg/RedfishRestExDxe/ComponentName.c
index de4828369083..a1b61162a3f5 100644
--- a/RedfishPkg/RedfishRestExDxe/ComponentName.c
+++ b/RedfishPkg/RedfishRestExDxe/ComponentName.c
@@ -24,7 +24,7 @@
@param[in] This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param[in] Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
@@ -135,7 +135,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED EFI_UNICODE_STRING_TABLE *gRedfishRestExControlle
@param[in] This A pointer to the EFI_COMPONENT_NAME_PROTOCOL instance.
@param[in] Language A pointer to a three-character ISO 639-2 language identifier.
- This is the language of the driver name that that the caller
+ This is the language of the driver name that the caller
is requesting, and it must match one of the languages specified
in SupportedLanguages. The number of languages supported by a
driver is up to the driver writer.
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 12/17] SecurityPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (10 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 11/17] RedfishPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:37 ` Yao, Jiewen
2022-09-06 8:29 ` [PATCH v2 13/17] ShellPkg: " PierreGondois
` (4 subsequent siblings)
16 siblings, 1 reply; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Jiewen Yao, Jian J Wang
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
SecurityPkg/Include/Library/Tpm2CommandLib.h | 2 +-
.../Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.uni | 4 ++--
.../Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.uni | 4 ++--
.../SecureBootVariableProvisionLib.c | 2 +-
SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c | 2 +-
SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c | 2 +-
SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c | 2 +-
SecurityPkg/Tcg/TcgDxe/TcgDxe.c | 4 ++--
8 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/SecurityPkg/Include/Library/Tpm2CommandLib.h b/SecurityPkg/Include/Library/Tpm2CommandLib.h
index a2fb97f18dfe..1912f54c8416 100644
--- a/SecurityPkg/Include/Library/Tpm2CommandLib.h
+++ b/SecurityPkg/Include/Library/Tpm2CommandLib.h
@@ -136,7 +136,7 @@ Tpm2SelfTest (
/**
This command allows setting of the authorization policy for the platform hierarchy (platformPolicy), the
- storage hierarchy (ownerPolicy), and and the endorsement hierarchy (endorsementPolicy).
+ storage hierarchy (ownerPolicy), and the endorsement hierarchy (endorsementPolicy).
@param[in] AuthHandle TPM_RH_ENDORSEMENT, TPM_RH_OWNER or TPM_RH_PLATFORM+{PP} parameters to be validated
@param[in] AuthSession Auth Session context
diff --git a/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.uni b/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.uni
index a1139810b1cb..ab043e8f5a1c 100644
--- a/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.uni
+++ b/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.uni
@@ -1,7 +1,7 @@
// /** @file
// Provides TPM measurement functions for TPM1.2 and TPM 2.0
//
-// This library provides TpmMeasureAndLogData() to to measure and log data, and
+// This library provides TpmMeasureAndLogData() to measure and log data, and
// extend the measurement result into a specific PCR.
//
// Copyright (c) 2012 - 2014, Intel Corporation. All rights reserved.<BR>
@@ -13,5 +13,5 @@
#string STR_MODULE_ABSTRACT #language en-US "Provides TPM measurement functions for TPM1.2 and TPM 2.0"
-#string STR_MODULE_DESCRIPTION #language en-US "This library provides TpmMeasureAndLogData() to to measure and log data, and extend the measurement result into a specific PCR."
+#string STR_MODULE_DESCRIPTION #language en-US "This library provides TpmMeasureAndLogData() to measure and log data, and extend the measurement result into a specific PCR."
diff --git a/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.uni b/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.uni
index 7b4341b449f4..4d0159d18ce3 100644
--- a/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.uni
+++ b/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.uni
@@ -1,7 +1,7 @@
// /** @file
// Provides TPM measurement functions for TPM1.2 and TPM 2.0
//
-// This library provides TpmMeasureAndLogData() to to measure and log data, and
+// This library provides TpmMeasureAndLogData() to measure and log data, and
// extend the measurement result into a specific PCR.
//
// Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
@@ -13,5 +13,5 @@
#string STR_MODULE_ABSTRACT #language en-US "Provides TPM measurement functions for TPM1.2 and TPM 2.0"
-#string STR_MODULE_DESCRIPTION #language en-US "This library provides TpmMeasureAndLogData() to to measure and log data, and extend the measurement result into a specific PCR."
+#string STR_MODULE_DESCRIPTION #language en-US "This library provides TpmMeasureAndLogData() to measure and log data, and extend the measurement result into a specific PCR."
diff --git a/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariableProvisionLib.c b/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariableProvisionLib.c
index bed1fe86205d..9e7b8946274d 100644
--- a/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariableProvisionLib.c
+++ b/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariableProvisionLib.c
@@ -28,7 +28,7 @@
Create a EFI Signature List with data fetched from section specified as a argument.
Found keys are verified using RsaGetPublicKeyFromX509().
- @param[in] KeyFileGuid A pointer to to the FFS filename GUID
+ @param[in] KeyFileGuid A pointer to the FFS filename GUID
@param[out] SigListsSize A pointer to size of signature list
@param[out] SigListOut a pointer to a callee-allocated buffer with signature lists
diff --git a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
index 7144955be1a5..8e518317aa19 100644
--- a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
+++ b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
@@ -116,7 +116,7 @@ typedef struct {
/**
This command allows setting of the authorization policy for the platform hierarchy (platformPolicy), the
- storage hierarchy (ownerPolicy), and and the endorsement hierarchy (endorsementPolicy).
+ storage hierarchy (ownerPolicy), and the endorsement hierarchy (endorsementPolicy).
@param[in] AuthHandle TPM_RH_ENDORSEMENT, TPM_RH_OWNER or TPM_RH_PLATFORM+{PP} parameters to be validated
@param[in] AuthSession Auth Session context
diff --git a/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c b/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c
index e8822cbeb015..36243321a6cc 100644
--- a/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c
+++ b/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c
@@ -97,7 +97,7 @@ EFI_TPM2_ACPI_TABLE_V4 mTpm2AcpiTemplate = {
EFI_TPM2_ACPI_TABLE_REVISION,
//
// Compiler initializes the remaining bytes to 0
- // These fields should be filled in in production
+ // These fields should be filled in production
//
},
0, // BIT0~15: PlatformClass
diff --git a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c
index 6eb04c014448..e7b74e8d399d 100644
--- a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c
+++ b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c
@@ -278,7 +278,7 @@ SaveTcg2PpRequestParameter (
}
/**
- Save Tcg2 PCR Banks request request to variable space.
+ Save Tcg2 PCR Banks request to variable space.
@param[in] PCRBankIndex PCR Bank Index.
@param[in] Enable Enable or disable this PCR Bank.
diff --git a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
index ee6c6273033b..8b596a99262e 100644
--- a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
+++ b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
@@ -67,7 +67,7 @@ EFI_TCG_CLIENT_ACPI_TABLE mTcgClientAcpiTemplate = {
0x02 // Revision
//
// Compiler initializes the remaining bytes to 0
- // These fields should be filled in in production
+ // These fields should be filled in production
//
},
0, // 0 for PC Client Platform Class
@@ -88,7 +88,7 @@ EFI_TCG_SERVER_ACPI_TABLE mTcgServerAcpiTemplate = {
0x02 // Revision
//
// Compiler initializes the remaining bytes to 0
- // These fields should be filled in in production
+ // These fields should be filled in production
//
},
1, // 1 for Server Platform Class
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 12/17] SecurityPkg: Remove duplicated words
2022-09-06 8:29 ` [PATCH v2 12/17] SecurityPkg: " PierreGondois
@ 2022-09-06 8:37 ` Yao, Jiewen
0 siblings, 0 replies; 27+ messages in thread
From: Yao, Jiewen @ 2022-09-06 8:37 UTC (permalink / raw)
To: Pierre.Gondois@arm.com, devel@edk2.groups.io; +Cc: Wang, Jian J
Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
> -----Original Message-----
> From: Pierre.Gondois@arm.com <Pierre.Gondois@arm.com>
> Sent: Tuesday, September 6, 2022 4:29 PM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois <pierre.gondois@arm.com>; Yao, Jiewen
> <jiewen.yao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>
> Subject: [PATCH v2 12/17] SecurityPkg: Remove duplicated words
>
> From: Pierre Gondois <pierre.gondois@arm.com>
>
> In an effort to clean the documentation of the above
> package, remove duplicated words.
>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> ---
> SecurityPkg/Include/Library/Tpm2CommandLib.h | 2 +-
> .../Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.uni | 4 ++-
> -
> .../Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.uni | 4 ++--
> .../SecureBootVariableProvisionLib.c | 2 +-
> SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c | 2 +-
> SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c | 2 +-
> SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c | 2 +-
> SecurityPkg/Tcg/TcgDxe/TcgDxe.c | 4 ++--
> 8 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/SecurityPkg/Include/Library/Tpm2CommandLib.h
> b/SecurityPkg/Include/Library/Tpm2CommandLib.h
> index a2fb97f18dfe..1912f54c8416 100644
> --- a/SecurityPkg/Include/Library/Tpm2CommandLib.h
> +++ b/SecurityPkg/Include/Library/Tpm2CommandLib.h
> @@ -136,7 +136,7 @@ Tpm2SelfTest (
>
> /**
> This command allows setting of the authorization policy for the platform
> hierarchy (platformPolicy), the
> - storage hierarchy (ownerPolicy), and and the endorsement hierarchy
> (endorsementPolicy).
> + storage hierarchy (ownerPolicy), and the endorsement hierarchy
> (endorsementPolicy).
>
> @param[in] AuthHandle TPM_RH_ENDORSEMENT,
> TPM_RH_OWNER or TPM_RH_PLATFORM+{PP} parameters to be validated
> @param[in] AuthSession Auth Session context
> diff --git
> a/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.u
> ni
> b/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.u
> ni
> index a1139810b1cb..ab043e8f5a1c 100644
> ---
> a/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.u
> ni
> +++
> b/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.u
> ni
> @@ -1,7 +1,7 @@
> // /** @file
> // Provides TPM measurement functions for TPM1.2 and TPM 2.0
> //
> -// This library provides TpmMeasureAndLogData() to to measure and log
> data, and
> +// This library provides TpmMeasureAndLogData() to measure and log
> data, and
> // extend the measurement result into a specific PCR.
> //
> // Copyright (c) 2012 - 2014, Intel Corporation. All rights reserved.<BR>
> @@ -13,5 +13,5 @@
>
> #string STR_MODULE_ABSTRACT #language en-US "Provides TPM
> measurement functions for TPM1.2 and TPM 2.0"
>
> -#string STR_MODULE_DESCRIPTION #language en-US "This library
> provides TpmMeasureAndLogData() to to measure and log data, and extend
> the measurement result into a specific PCR."
> +#string STR_MODULE_DESCRIPTION #language en-US "This library
> provides TpmMeasureAndLogData() to measure and log data, and extend
> the measurement result into a specific PCR."
>
> diff --git
> a/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.un
> i
> b/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.un
> i
> index 7b4341b449f4..4d0159d18ce3 100644
> ---
> a/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.un
> i
> +++
> b/SecurityPkg/Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.un
> i
> @@ -1,7 +1,7 @@
> // /** @file
> // Provides TPM measurement functions for TPM1.2 and TPM 2.0
> //
> -// This library provides TpmMeasureAndLogData() to to measure and log
> data, and
> +// This library provides TpmMeasureAndLogData() to measure and log
> data, and
> // extend the measurement result into a specific PCR.
> //
> // Copyright (c) 2020, Intel Corporation. All rights reserved.<BR>
> @@ -13,5 +13,5 @@
>
> #string STR_MODULE_ABSTRACT #language en-US "Provides TPM
> measurement functions for TPM1.2 and TPM 2.0"
>
> -#string STR_MODULE_DESCRIPTION #language en-US "This library
> provides TpmMeasureAndLogData() to to measure and log data, and extend
> the measurement result into a specific PCR."
> +#string STR_MODULE_DESCRIPTION #language en-US "This library
> provides TpmMeasureAndLogData() to measure and log data, and extend
> the measurement result into a specific PCR."
>
> diff --git
> a/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariable
> ProvisionLib.c
> b/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariable
> ProvisionLib.c
> index bed1fe86205d..9e7b8946274d 100644
> ---
> a/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariable
> ProvisionLib.c
> +++
> b/SecurityPkg/Library/SecureBootVariableProvisionLib/SecureBootVariable
> ProvisionLib.c
> @@ -28,7 +28,7 @@
> Create a EFI Signature List with data fetched from section specified as a
> argument.
> Found keys are verified using RsaGetPublicKeyFromX509().
>
> - @param[in] KeyFileGuid A pointer to to the FFS filename GUID
> + @param[in] KeyFileGuid A pointer to the FFS filename GUID
> @param[out] SigListsSize A pointer to size of signature list
> @param[out] SigListOut a pointer to a callee-allocated buffer with
> signature lists
>
> diff --git a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> index 7144955be1a5..8e518317aa19 100644
> --- a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> +++ b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> @@ -116,7 +116,7 @@ typedef struct {
>
> /**
> This command allows setting of the authorization policy for the platform
> hierarchy (platformPolicy), the
> - storage hierarchy (ownerPolicy), and and the endorsement hierarchy
> (endorsementPolicy).
> + storage hierarchy (ownerPolicy), and the endorsement hierarchy
> (endorsementPolicy).
>
> @param[in] AuthHandle TPM_RH_ENDORSEMENT,
> TPM_RH_OWNER or TPM_RH_PLATFORM+{PP} parameters to be validated
> @param[in] AuthSession Auth Session context
> diff --git a/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c
> b/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c
> index e8822cbeb015..36243321a6cc 100644
> --- a/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c
> +++ b/SecurityPkg/Tcg/Tcg2Acpi/Tcg2Acpi.c
> @@ -97,7 +97,7 @@ EFI_TPM2_ACPI_TABLE_V4 mTpm2AcpiTemplate = {
> EFI_TPM2_ACPI_TABLE_REVISION,
> //
> // Compiler initializes the remaining bytes to 0
> - // These fields should be filled in in production
> + // These fields should be filled in production
> //
> },
> 0, // BIT0~15: PlatformClass
> diff --git a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c
> b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c
> index 6eb04c014448..e7b74e8d399d 100644
> --- a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c
> +++ b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.c
> @@ -278,7 +278,7 @@ SaveTcg2PpRequestParameter (
> }
>
> /**
> - Save Tcg2 PCR Banks request request to variable space.
> + Save Tcg2 PCR Banks request to variable space.
>
> @param[in] PCRBankIndex PCR Bank Index.
> @param[in] Enable Enable or disable this PCR Bank.
> diff --git a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> index ee6c6273033b..8b596a99262e 100644
> --- a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> +++ b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> @@ -67,7 +67,7 @@ EFI_TCG_CLIENT_ACPI_TABLE
> mTcgClientAcpiTemplate = {
> 0x02 // Revision
> //
> // Compiler initializes the remaining bytes to 0
> - // These fields should be filled in in production
> + // These fields should be filled in production
> //
> },
> 0, // 0 for PC Client Platform Class
> @@ -88,7 +88,7 @@ EFI_TCG_SERVER_ACPI_TABLE
> mTcgServerAcpiTemplate = {
> 0x02 // Revision
> //
> // Compiler initializes the remaining bytes to 0
> - // These fields should be filled in in production
> + // These fields should be filled in production
> //
> },
> 1, // 1 for Server Platform Class
> --
> 2.25.1
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v2 13/17] ShellPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (11 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 12/17] SecurityPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 14/17] SourceLevelDebugPkg: " PierreGondois
` (3 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Ray Ni, Zhichao Gao
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Ray Ni <ray.ni@intel.com>
Cc: Zhichao Gao <zhichao.gao@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
Reviewed-by: Zhichao Gao <zhichao.gao@intel.com>
---
ShellPkg/Include/Library/ShellCommandLib.h | 2 +-
ShellPkg/Include/Library/ShellLib.h | 4 ++--
ShellPkg/Include/Protocol/EfiShellEnvironment2.h | 2 +-
.../Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c | 2 +-
ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c | 2 +-
.../UefiShellDriver1CommandsLib.uni | 2 +-
ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c | 2 +-
ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c | 2 +-
ShellPkg/Library/UefiShellLib/UefiShellLib.c | 4 ++--
9 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/ShellPkg/Include/Library/ShellCommandLib.h b/ShellPkg/Include/Library/ShellCommandLib.h
index 7a97b203f1ff..de7db40a1525 100644
--- a/ShellPkg/Include/Library/ShellCommandLib.h
+++ b/ShellPkg/Include/Library/ShellCommandLib.h
@@ -87,7 +87,7 @@ SHELL_STATUS
If the ShellSupportLevel is greater than the value of
PcdShellSupportLevel, then return RETURN_UNSUPPORTED.
- Registers the the handlers specified by GetHelpInfoHandler and CommandHandler
+ Registers the handlers specified by GetHelpInfoHandler and CommandHandler
with the command specified by CommandString. If the command named by
CommandString has already been registered, then return
RETURN_ALREADY_STARTED.
diff --git a/ShellPkg/Include/Library/ShellLib.h b/ShellPkg/Include/Library/ShellLib.h
index 10cf7f4fefe0..cd0c3bccc89f 100644
--- a/ShellPkg/Include/Library/ShellLib.h
+++ b/ShellPkg/Include/Library/ShellLib.h
@@ -1354,9 +1354,9 @@ ShellFileHandleReturnLine (
@param[in, out] Buffer The pointer to buffer to read into.
@param[in, out] Size The pointer to number of bytes in Buffer.
@param[in] Truncate If the buffer is large enough, this has no effect.
- If the buffer is is too small and Truncate is TRUE,
+ If the buffer is too small and Truncate is TRUE,
the line will be truncated.
- If the buffer is is too small and Truncate is FALSE,
+ If the buffer is too small and Truncate is FALSE,
then no read will occur.
@param[in, out] Ascii Boolean value for indicating whether the file is
diff --git a/ShellPkg/Include/Protocol/EfiShellEnvironment2.h b/ShellPkg/Include/Protocol/EfiShellEnvironment2.h
index 8b7c86dc24fc..59714c1b589b 100644
--- a/ShellPkg/Include/Protocol/EfiShellEnvironment2.h
+++ b/ShellPkg/Include/Protocol/EfiShellEnvironment2.h
@@ -541,7 +541,7 @@ EFI_STATUS
/**
This is an internal shell function to enumerate the handle database.
- This function resets the the handle database so that NEXT_HANDLE and SKIP_HANDLE
+ This function resets the handle database so that NEXT_HANDLE and SKIP_HANDLE
will start from EnumIndex on the next call.
This must be called after INIT_HANDLE_ENUMERATOR and before CLOSE_HANDLE_ENUMERATOR.
diff --git a/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c b/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c
index 4a19794b9fe0..0f33d52a67cc 100644
--- a/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c
+++ b/ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.c
@@ -1252,7 +1252,7 @@ BcfgAddOpt (
}
//
- // Shouldn't be possible to have have both. Neither is ok though.
+ // Shouldn't be possible to have both. Neither is ok though.
//
ASSERT (FileName == NULL || Data == NULL);
diff --git a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c
index 36cf46fb2c38..75fa81f7e448 100644
--- a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c
+++ b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c
@@ -119,7 +119,7 @@ CommandInit (
}
//
- // Find the best matching matching language from the supported languages
+ // Find the best matching language from the supported languages
// of Unicode Collation2 protocol.
//
BestLanguage = GetBestLanguage (
diff --git a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.uni b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.uni
index fc4986c8c62f..ba4dbc4e8b48 100644
--- a/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.uni
+++ b/ShellPkg/Library/UefiShellDriver1CommandsLib/UefiShellDriver1CommandsLib.uni
@@ -171,7 +171,7 @@
" 1 - Manufacturing Defaults.\r\n"
" 2 - Safe Defaults.\r\n"
" 4000-FFFF - Custom Defaults.\r\n"
-" DriverHandle - Specifies the the handle of the driver to configure.\r\n"
+" DriverHandle - Specifies the handle of the driver to configure.\r\n"
" DeviceHandle - Specifies the handle of a device that the DriverHandle is managing.\r\n"
" ChildHandle - Specifies the handle of a device that is a child of the DeviceHandle. \r\n"
".SH DESCRIPTION\r\n"
diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c b/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c
index f92cfa502df1..f88c9e547943 100644
--- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c
+++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Load.c
@@ -9,7 +9,7 @@
#include "UefiShellLevel2CommandsLib.h"
-// This function was from from the BdsLib implementation in
+// This function was from the BdsLib implementation in
// IntelFrameworkModulePkg\Library\GenericBdsLib\BdsConnect.c
// function name: BdsLibConnectAllEfi
diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c b/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c
index f3c888edd48c..3b06246d3e95 100644
--- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c
+++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Map.c
@@ -122,7 +122,7 @@ SearchList (
/**
Determine what type of device is represented and return it's string. The
- string is in allocated memory and must be callee freed. The HII is is listed below.
+ string is in allocated memory and must be callee freed. The HII is listed below.
The actual string cannot be determined.
@param[in] DevicePath The device to analyze.
diff --git a/ShellPkg/Library/UefiShellLib/UefiShellLib.c b/ShellPkg/Library/UefiShellLib/UefiShellLib.c
index a72767bd869f..b2819f964a77 100644
--- a/ShellPkg/Library/UefiShellLib/UefiShellLib.c
+++ b/ShellPkg/Library/UefiShellLib/UefiShellLib.c
@@ -4338,9 +4338,9 @@ ShellFileHandleReturnLine (
to hold the next line of the file (as a UCS2
string, even if it is an ASCII file).
@param[in] Truncate If the buffer is large enough, this has no effect.
- If the buffer is is too small and Truncate is TRUE,
+ If the buffer is too small and Truncate is TRUE,
the line will be truncated.
- If the buffer is is too small and Truncate is FALSE,
+ If the buffer is too small and Truncate is FALSE,
then no read will occur.
@param[in, out] Ascii Boolean value for indicating whether the file is
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 14/17] SourceLevelDebugPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (12 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 13/17] ShellPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 15/17] UefiCpuPkg: " PierreGondois
` (2 subsequent siblings)
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Hao A Wu
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Also fix small typos.
Cc: Hao A Wu <hao.a.wu@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
Reviewed-by: Hao A Wu <hao.a.wu@intel.com>
---
Notes:
v2:
- Fix other typos in the modified line. [Ray Ni]
.../Library/DebugAgent/SmmDebugAgent/SmmDebugAgentLib.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/SourceLevelDebugPkg/Library/DebugAgent/SmmDebugAgent/SmmDebugAgentLib.c b/SourceLevelDebugPkg/Library/DebugAgent/SmmDebugAgent/SmmDebugAgentLib.c
index f49a592d2762..e3e76ffb84ad 100644
--- a/SourceLevelDebugPkg/Library/DebugAgent/SmmDebugAgent/SmmDebugAgentLib.c
+++ b/SourceLevelDebugPkg/Library/DebugAgent/SmmDebugAgent/SmmDebugAgentLib.c
@@ -158,7 +158,7 @@ RestoreDebugRegister (
If InitFlag is DEBUG_AGENT_INIT_SMM, it will override IDT table entries
and initialize debug port. It will get debug agent Mailbox from GUIDed HOB,
- it it exists, debug agent wiil copied it into the local Mailbox in SMM space.
+ if it exists, debug agent will copy it into the local Mailbox in SMM space.
it will override IDT table entries and initialize debug port. Context will be
NULL.
If InitFlag is DEBUG_AGENT_INIT_ENTER_SMI, debug agent will save Debug
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 15/17] UefiCpuPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (13 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 14/17] SourceLevelDebugPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 8:29 ` [PATCH v2 16/17] UefiPayloadPkg: " PierreGondois
2022-09-06 8:29 ` [PATCH v2 17/17] UnitTestFrameworkPkg: " PierreGondois
16 siblings, 0 replies; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Eric Dong, Ray Ni
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Eric Dong <eric.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
.../Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h | 2 +-
.../Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c | 4 ++--
UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmStm.c | 2 +-
UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c | 4 ++--
UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c | 4 ++--
5 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h
index 4a84954cac9d..7024533ea430 100644
--- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h
+++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeatures.h
@@ -176,7 +176,7 @@ SwitchNewBsp (
);
/**
- Function that uses DEBUG() macros to display the contents of a a CPU feature bit mask.
+ Function that uses DEBUG() macros to display the contents of a CPU feature bit mask.
@param[in] FeatureMask A pointer to the CPU feature bit mask.
@param[in] BitMaskSize CPU feature bits mask buffer size.
diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c
index 0285aaf5c91a..a43d8b0aa0e8 100644
--- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c
+++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c
@@ -9,7 +9,7 @@
#include "RegisterCpuFeatures.h"
/**
- Function that uses DEBUG() macros to display the contents of a a CPU feature bit mask.
+ Function that uses DEBUG() macros to display the contents of a CPU feature bit mask.
@param[in] FeatureMask A pointer to the CPU feature bit mask.
@param[in] BitMaskSize CPU feature bits mask buffer size.
@@ -370,7 +370,7 @@ AdjustEntry (
// than core type dependence. So driver will adjust the feature order to B -> A -> C. and driver
// will remove the feature dependence in feature B.
// Driver just needs to make sure before feature C been executed, feature A has finished its task
- // in all all thread. Feature A finished in all threads also means feature B have finshed in all
+ // in all thread. Feature A finished in all threads also means feature B have finshed in all
// threads.
//
if (Before) {
diff --git a/UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmStm.c b/UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmStm.c
index 4e8f897f5e9c..17c221407215 100644
--- a/UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmStm.c
+++ b/UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmStm.c
@@ -1074,7 +1074,7 @@ GetVmcsSize (
MSR_IA32_VMX_BASIC_REGISTER VmxBasic;
//
- // Read VMCS size and and align to 4KB
+ // Read VMCS size and align to 4KB
//
VmxBasic.Uint64 = AsmReadMsr64 (MSR_IA32_VMX_BASIC);
return ALIGN_VALUE (VmxBasic.Bits.VmcsSize, SIZE_4KB);
diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c b/UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c
index 6587212f4e37..0456d96a0c48 100644
--- a/UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c
+++ b/UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c
@@ -260,7 +260,7 @@ SetStaticPageTable (
{
//
// Each PML5 entry points to a page of PML4 entires.
- // So lets allocate space for them and fill them in in the IndexOfPml4Entries loop.
+ // So lets allocate space for them and fill them in the IndexOfPml4Entries loop.
// When 5-Level Paging is disabled, below allocation happens only once.
//
if (m5LevelPagingNeeded) {
@@ -314,7 +314,7 @@ SetStaticPageTable (
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (UINT64 *)((*PageDirectoryPointerEntry) & ~mAddressEncMask & gPhyMask);
if (PageDirectoryEntry == NULL) {
diff --git a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c b/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c
index 8419a4e32acb..3ff739339c79 100644
--- a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c
+++ b/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c
@@ -688,7 +688,7 @@ RestoreS3PageTables (
for (IndexOfPml4Entries = 0; IndexOfPml4Entries < NumberOfPml4EntriesNeeded; IndexOfPml4Entries++, PageMapLevel4Entry++) {
//
// Each PML4 entry points to a page of Page Directory Pointer entires.
- // So lets allocate space for them and fill them in in the IndexOfPdpEntries loop.
+ // So lets allocate space for them and fill them in the IndexOfPdpEntries loop.
//
PageDirectoryPointerEntry = (PAGE_MAP_AND_DIRECTORY_POINTER *)S3NvsPageTableAddress;
S3NvsPageTableAddress += SIZE_4KB;
@@ -716,7 +716,7 @@ RestoreS3PageTables (
for (IndexOfPdpEntries = 0; IndexOfPdpEntries < NumberOfPdpEntriesNeeded; IndexOfPdpEntries++, PageDirectoryPointerEntry++) {
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (PAGE_TABLE_ENTRY *)S3NvsPageTableAddress;
S3NvsPageTableAddress += SIZE_4KB;
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH v2 16/17] UefiPayloadPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (14 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 15/17] UefiCpuPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-12 4:26 ` [edk2-devel] " Guo, Gua
2022-09-06 8:29 ` [PATCH v2 17/17] UnitTestFrameworkPkg: " PierreGondois
16 siblings, 1 reply; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Guo Dong, Ray Ni, Sean Rhodes
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Cc: Guo Dong <guo.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Sean Rhodes <sean@starlabs.systems>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
UefiPayloadPkg/Include/Library/SpiFlashLib.h | 2 +-
UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c | 2 +-
UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c | 2 +-
UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c | 2 +-
UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c | 2 +-
UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c | 6 +++---
6 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/UefiPayloadPkg/Include/Library/SpiFlashLib.h b/UefiPayloadPkg/Include/Library/SpiFlashLib.h
index 07f40502f251..8906dad043b0 100644
--- a/UefiPayloadPkg/Include/Library/SpiFlashLib.h
+++ b/UefiPayloadPkg/Include/Library/SpiFlashLib.h
@@ -194,7 +194,7 @@ SpiConstructor (
/**
Get the SPI region base and size, based on the enum type
- @param[in] FlashRegionType The Flash Region type for for the base address which is listed in the Descriptor.
+ @param[in] FlashRegionType The Flash Region type for the base address which is listed in the Descriptor.
@param[out] BaseAddress The Flash Linear Address for the Region 'n' Base
@param[out] RegionSize The size for the Region 'n'
diff --git a/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c b/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c
index 22639a748c04..58c86ade8997 100644
--- a/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c
+++ b/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c
@@ -807,7 +807,7 @@ WaitForSpiCycleComplete (
/**
Get the SPI region base and size, based on the enum type
- @param[in] FlashRegionType The Flash Region type for for the base address which is listed in the Descriptor.
+ @param[in] FlashRegionType The Flash Region type for the base address which is listed in the Descriptor.
@param[out] BaseAddress The Flash Linear Address for the Region 'n' Base
@param[out] RegionSize The size for the Region 'n'
diff --git a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c
index cb915e7142eb..6df421bcf99e 100644
--- a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c
+++ b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c
@@ -467,7 +467,7 @@ LoadElf32Image (
}
//
- // Relocate when new new image base is not the preferred image base.
+ // Relocate when new image base is not the preferred image base.
//
if (ElfCt->ImageAddress != ElfCt->PreferredImageAddress) {
RelocateElf32Sections (ElfCt);
diff --git a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c
index c8dbb887340b..e95b31d82671 100644
--- a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c
+++ b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c
@@ -476,7 +476,7 @@ LoadElf64Image (
}
//
- // Relocate when new new image base is not the preferred image base.
+ // Relocate when new image base is not the preferred image base.
//
if (ElfCt->ImageAddress != ElfCt->PreferredImageAddress) {
RelocateElf64Sections (ElfCt);
diff --git a/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c b/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c
index c66e56aee15a..43f53d4cfae9 100644
--- a/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c
+++ b/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c
@@ -131,7 +131,7 @@ Create4GPageTablesIa32Pae (
for (IndexOfPdpEntries = 0; IndexOfPdpEntries < NumberOfPdpEntriesNeeded; IndexOfPdpEntries++, PageDirectoryPointerEntry++) {
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (VOID *)PageAddress;
PageAddress += SIZE_4KB;
diff --git a/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c b/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c
index 1899404b244c..ec942ead3445 100644
--- a/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c
+++ b/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c
@@ -822,7 +822,7 @@ CreateIdentityMappingPageTables (
{
//
// Each PML5 entry points to a page of PML4 entires.
- // So lets allocate space for them and fill them in in the IndexOfPml4Entries loop.
+ // So lets allocate space for them and fill them in the IndexOfPml4Entries loop.
// When 5-Level Paging is disabled, below allocation happens only once.
//
PageMapLevel4Entry = (VOID *)BigPageAddress;
@@ -844,7 +844,7 @@ CreateIdentityMappingPageTables (
{
//
// Each PML4 entry points to a page of Page Directory Pointer entires.
- // So lets allocate space for them and fill them in in the IndexOfPdpEntries loop.
+ // So lets allocate space for them and fill them in the IndexOfPdpEntries loop.
//
PageDirectoryPointerEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
@@ -879,7 +879,7 @@ CreateIdentityMappingPageTables (
{
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [edk2-devel] [PATCH v2 16/17] UefiPayloadPkg: Remove duplicated words
2022-09-06 8:29 ` [PATCH v2 16/17] UefiPayloadPkg: " PierreGondois
@ 2022-09-12 4:26 ` Guo, Gua
0 siblings, 0 replies; 27+ messages in thread
From: Guo, Gua @ 2022-09-12 4:26 UTC (permalink / raw)
To: devel@edk2.groups.io, pierre.gondois@arm.com
Cc: Dong, Guo, Ni, Ray, Rhodes, Sean
Reviewed-by: Gua Guo <gua.guo@intel.com>
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of PierreGondois
Sent: Tuesday, September 6, 2022 4:29 PM
To: devel@edk2.groups.io
Cc: Pierre Gondois <pierre.gondois@arm.com>; Dong, Guo <guo.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Rhodes, Sean <sean@starlabs.systems>
Subject: [edk2-devel] [PATCH v2 16/17] UefiPayloadPkg: Remove duplicated words
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above package, remove duplicated words.
Cc: Guo Dong <guo.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Sean Rhodes <sean@starlabs.systems>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
UefiPayloadPkg/Include/Library/SpiFlashLib.h | 2 +-
UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c | 2 +-
UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c | 2 +- UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c | 2 +- UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c | 2 +- UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c | 6 +++---
6 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/UefiPayloadPkg/Include/Library/SpiFlashLib.h b/UefiPayloadPkg/Include/Library/SpiFlashLib.h
index 07f40502f251..8906dad043b0 100644
--- a/UefiPayloadPkg/Include/Library/SpiFlashLib.h
+++ b/UefiPayloadPkg/Include/Library/SpiFlashLib.h
@@ -194,7 +194,7 @@ SpiConstructor (
/**
Get the SPI region base and size, based on the enum type
- @param[in] FlashRegionType The Flash Region type for for the base address which is listed in the Descriptor.
+ @param[in] FlashRegionType The Flash Region type for the base address which is listed in the Descriptor.
@param[out] BaseAddress The Flash Linear Address for the Region 'n' Base
@param[out] RegionSize The size for the Region 'n'
diff --git a/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c b/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c
index 22639a748c04..58c86ade8997 100644
--- a/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c
+++ b/UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.c
@@ -807,7 +807,7 @@ WaitForSpiCycleComplete (
/**
Get the SPI region base and size, based on the enum type
- @param[in] FlashRegionType The Flash Region type for for the base address which is listed in the Descriptor.
+ @param[in] FlashRegionType The Flash Region type for the base address which is listed in the Descriptor.
@param[out] BaseAddress The Flash Linear Address for the Region 'n' Base
@param[out] RegionSize The size for the Region 'n'
diff --git a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c
index cb915e7142eb..6df421bcf99e 100644
--- a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c
+++ b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf32Lib.c
@@ -467,7 +467,7 @@ LoadElf32Image (
}
//
- // Relocate when new new image base is not the preferred image base.
+ // Relocate when new image base is not the preferred image base.
//
if (ElfCt->ImageAddress != ElfCt->PreferredImageAddress) {
RelocateElf32Sections (ElfCt);
diff --git a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c
index c8dbb887340b..e95b31d82671 100644
--- a/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c
+++ b/UefiPayloadPkg/PayloadLoaderPeim/ElfLib/Elf64Lib.c
@@ -476,7 +476,7 @@ LoadElf64Image (
}
//
- // Relocate when new new image base is not the preferred image base.
+ // Relocate when new image base is not the preferred image base.
//
if (ElfCt->ImageAddress != ElfCt->PreferredImageAddress) {
RelocateElf64Sections (ElfCt);
diff --git a/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c b/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c
index c66e56aee15a..43f53d4cfae9 100644
--- a/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c
+++ b/UefiPayloadPkg/UefiPayloadEntry/Ia32/DxeLoadFunc.c
@@ -131,7 +131,7 @@ Create4GPageTablesIa32Pae (
for (IndexOfPdpEntries = 0; IndexOfPdpEntries < NumberOfPdpEntriesNeeded; IndexOfPdpEntries++, PageDirectoryPointerEntry++) {
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (VOID *)PageAddress;
PageAddress += SIZE_4KB;
diff --git a/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c b/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c
index 1899404b244c..ec942ead3445 100644
--- a/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c
+++ b/UefiPayloadPkg/UefiPayloadEntry/X64/VirtualMemory.c
@@ -822,7 +822,7 @@ CreateIdentityMappingPageTables (
{
//
// Each PML5 entry points to a page of PML4 entires.
- // So lets allocate space for them and fill them in in the IndexOfPml4Entries loop.
+ // So lets allocate space for them and fill them in the IndexOfPml4Entries loop.
// When 5-Level Paging is disabled, below allocation happens only once.
//
PageMapLevel4Entry = (VOID *)BigPageAddress; @@ -844,7 +844,7 @@ CreateIdentityMappingPageTables (
{
//
// Each PML4 entry points to a page of Page Directory Pointer entires.
- // So lets allocate space for them and fill them in in the IndexOfPdpEntries loop.
+ // So lets allocate space for them and fill them in the IndexOfPdpEntries loop.
//
PageDirectoryPointerEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
@@ -879,7 +879,7 @@ CreateIdentityMappingPageTables (
{
//
// Each Directory Pointer entries points to a page of Page Directory entires.
- // So allocate space for them and fill them in in the IndexOfPageDirectoryEntries loop.
+ // So allocate space for them and fill them in the IndexOfPageDirectoryEntries loop.
//
PageDirectoryEntry = (VOID *)BigPageAddress;
BigPageAddress += SIZE_4KB;
--
2.25.1
-=-=-=-=-=-=
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93207): https://edk2.groups.io/g/devel/message/93207
Mute This Topic: https://groups.io/mt/93496614/6998960
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [gua.guo@intel.com]
-=-=-=-=-=-=
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PATCH v2 17/17] UnitTestFrameworkPkg: Remove duplicated words
2022-09-06 8:28 [PATCH v2 00/17] Remove duplicated words PierreGondois
` (15 preceding siblings ...)
2022-09-06 8:29 ` [PATCH v2 16/17] UefiPayloadPkg: " PierreGondois
@ 2022-09-06 8:29 ` PierreGondois
2022-09-06 18:47 ` Michael D Kinney
16 siblings, 1 reply; 27+ messages in thread
From: PierreGondois @ 2022-09-06 8:29 UTC (permalink / raw)
To: devel; +Cc: Pierre Gondois, Michael D Kinney, Michael Kubacki,
Michael Kubacki
From: Pierre Gondois <pierre.gondois@arm.com>
In an effort to clean the documentation of the above
package, remove duplicated words.
Also change in/out tag to pass the EccCheck.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
Reviewed-by: Michael Kubacki <michael.kubacki@microsoft.com>
---
Notes:
v2:
- Correctly align comments. [Michael Kubacki]
.../Library/Posix/DebugLibPosix/DebugLibPosix.c | 2 +-
UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c | 2 +-
UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c | 2 +-
.../UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c | 2 +-
.../UnitTestPersistenceLibSimpleFileSystem.c | 2 +-
.../PrivateInclude/Library/UnitTestPersistenceLib.h | 2 +-
UnitTestFrameworkPkg/ReadMe.md | 6 +++---
7 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c b/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c
index 694f4beb8bf5..cfee36d2ac7f 100644
--- a/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c
+++ b/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c
@@ -179,7 +179,7 @@ DebugClearMemory (
ASSERT (Buffer != NULL);
//
- // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
+ // SetMem() checks for the ASSERT() condition on Length and returns Buffer
//
return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
}
diff --git a/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c b/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c
index 35636565b783..9bb0798dd801 100644
--- a/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c
+++ b/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c
@@ -516,7 +516,7 @@ UnitTestAssertNotNull (
is either pass, skipped, or failed.
@param[in] FunctionName Null-terminated ASCII string of the function
executing the UT_EXPECT_ASSERT_FAILURE() macro.
- @param[in] LineNumber The source file line number of the the function
+ @param[in] LineNumber The source file line number of the function
executing the UT_EXPECT_ASSERT_FAILURE() macro.
@param[in] FileName Null-terminated ASCII string of the filename
executing the UT_EXPECT_ASSERT_FAILURE() macro.
diff --git a/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c b/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c
index dc05bbd4384c..0b1f6b197a35 100644
--- a/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c
+++ b/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c
@@ -349,7 +349,7 @@ UnitTestAssertNotNull (
is either pass, skipped, or failed.
@param[in] FunctionName Null-terminated ASCII string of the function
executing the UT_EXPECT_ASSERT_FAILURE() macro.
- @param[in] LineNumber The source file line number of the the function
+ @param[in] LineNumber The source file line number of the function
executing the UT_EXPECT_ASSERT_FAILURE() macro.
@param[in] FileName Null-terminated ASCII string of the filename
executing the UT_EXPECT_ASSERT_FAILURE() macro.
diff --git a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c
index e28327652eaf..064b967b13b3 100644
--- a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c
+++ b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c
@@ -55,7 +55,7 @@ SaveUnitTestCache (
Will allocate a buffer to hold the loaded data.
@param[in] FrameworkHandle A pointer to the framework that is being persisted.
- @param[in] SaveData A pointer pointer that will be updated with the address
+ @param[out] SaveData A pointer that will be updated with the address
of the loaded data buffer.
@retval EFI_SUCCESS Data has been loaded successfully and SaveData is updated
diff --git a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c
index ed4a7d161524..940c7e42d43c 100644
--- a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c
+++ b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c
@@ -312,7 +312,7 @@ Exit:
Will allocate a buffer to hold the loaded data.
@param[in] FrameworkHandle A pointer to the framework that is being persisted.
- @param[in] SaveData A pointer pointer that will be updated with the address
+ @param[out] SaveData A pointer that will be updated with the address
of the loaded data buffer.
@retval EFI_SUCCESS Data has been loaded successfully and SaveData is updated
diff --git a/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h b/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h
index be29e079ec3e..e473bebec7e2 100644
--- a/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h
+++ b/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h
@@ -57,7 +57,7 @@ SaveUnitTestCache (
Will allocate a buffer to hold the loaded data.
@param[in] FrameworkHandle A pointer to the framework that is being persisted.
- @param[in] SaveData A pointer pointer that will be updated with the address
+ @param[out] SaveData A pointer that will be updated with the address
of the loaded data buffer.
@retval EFI_SUCCESS Data has been loaded successfully and SaveData is updated
diff --git a/UnitTestFrameworkPkg/ReadMe.md b/UnitTestFrameworkPkg/ReadMe.md
index e696412cb3cf..a7b64bd65392 100644
--- a/UnitTestFrameworkPkg/ReadMe.md
+++ b/UnitTestFrameworkPkg/ReadMe.md
@@ -144,7 +144,7 @@ will be used when adding test cases.
Great! Now we've finished some of the cruft, red tape, and busy work. We're ready to add some tests. Adding a test
to a test suite is accomplished with the -- you guessed it -- `AddTestCase` function. It takes in the suite handle;
a `CHAR8` string for the description and class name; a function pointer for the test case itself; additional, optional
-function pointers for prerequisite check and cleanup routines; and and optional pointer to a context structure.
+function pointers for prerequisite check and cleanup routines; and optional pointer to a context structure.
Okay, that's a lot. Let's take it one piece at a time. The description and class name strings are very similar in
usage to the suite title and package name strings in the test suites. The former is for user presentation and the
@@ -268,8 +268,8 @@ stuart_ci_build -c .pytool/CISettings.py TOOL_CHAIN_TAG=VS2019 -p MdePkg -t NOOP
## Building and Running Host-Based Tests
-The EDK2 CI infrastructure provides a convenient way to run all host-based tests -- in the the entire tree or just
-selected packages -- and aggregate all the the reports, including highlighting any failures. This functionality is
+The EDK2 CI infrastructure provides a convenient way to run all host-based tests -- in the entire tree or just
+selected packages -- and aggregate all the reports, including highlighting any failures. This functionality is
provided through the Stuart build system (published by EDK2-PyTools) and the `NOOPT` build target.
### Building Locally
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v2 17/17] UnitTestFrameworkPkg: Remove duplicated words
2022-09-06 8:29 ` [PATCH v2 17/17] UnitTestFrameworkPkg: " PierreGondois
@ 2022-09-06 18:47 ` Michael D Kinney
0 siblings, 0 replies; 27+ messages in thread
From: Michael D Kinney @ 2022-09-06 18:47 UTC (permalink / raw)
To: Pierre.Gondois@arm.com, devel@edk2.groups.io, Kinney, Michael D
Cc: Michael Kubacki, Kubacki, Michael
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> -----Original Message-----
> From: Pierre.Gondois@arm.com <Pierre.Gondois@arm.com>
> Sent: Tuesday, September 6, 2022 1:29 AM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois <pierre.gondois@arm.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Michael Kubacki
> <mikuback@linux.microsoft.com>; Kubacki, Michael <michael.kubacki@microsoft.com>
> Subject: [PATCH v2 17/17] UnitTestFrameworkPkg: Remove duplicated words
>
> From: Pierre Gondois <pierre.gondois@arm.com>
>
> In an effort to clean the documentation of the above
> package, remove duplicated words.
>
> Also change in/out tag to pass the EccCheck.
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Michael Kubacki <mikuback@linux.microsoft.com>
> Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> Reviewed-by: Michael Kubacki <michael.kubacki@microsoft.com>
> ---
>
> Notes:
> v2:
> - Correctly align comments. [Michael Kubacki]
>
> .../Library/Posix/DebugLibPosix/DebugLibPosix.c | 2 +-
> UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c | 2 +-
> UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c | 2 +-
> .../UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c | 2 +-
> .../UnitTestPersistenceLibSimpleFileSystem.c | 2 +-
> .../PrivateInclude/Library/UnitTestPersistenceLib.h | 2 +-
> UnitTestFrameworkPkg/ReadMe.md | 6 +++---
> 7 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c
> b/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c
> index 694f4beb8bf5..cfee36d2ac7f 100644
> --- a/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c
> +++ b/UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.c
> @@ -179,7 +179,7 @@ DebugClearMemory (
> ASSERT (Buffer != NULL);
>
> //
> - // SetMem() checks for the the ASSERT() condition on Length and returns Buffer
> + // SetMem() checks for the ASSERT() condition on Length and returns Buffer
> //
> return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
> }
> diff --git a/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c b/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c
> index 35636565b783..9bb0798dd801 100644
> --- a/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c
> +++ b/UnitTestFrameworkPkg/Library/UnitTestLib/Assert.c
> @@ -516,7 +516,7 @@ UnitTestAssertNotNull (
> is either pass, skipped, or failed.
> @param[in] FunctionName Null-terminated ASCII string of the function
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> - @param[in] LineNumber The source file line number of the the function
> + @param[in] LineNumber The source file line number of the function
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> @param[in] FileName Null-terminated ASCII string of the filename
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> diff --git a/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c b/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c
> index dc05bbd4384c..0b1f6b197a35 100644
> --- a/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c
> +++ b/UnitTestFrameworkPkg/Library/UnitTestLib/AssertCmocka.c
> @@ -349,7 +349,7 @@ UnitTestAssertNotNull (
> is either pass, skipped, or failed.
> @param[in] FunctionName Null-terminated ASCII string of the function
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> - @param[in] LineNumber The source file line number of the the function
> + @param[in] LineNumber The source file line number of the function
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> @param[in] FileName Null-terminated ASCII string of the filename
> executing the UT_EXPECT_ASSERT_FAILURE() macro.
> diff --git a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c
> b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c
> index e28327652eaf..064b967b13b3 100644
> --- a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c
> +++ b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibNull/UnitTestPersistenceLibNull.c
> @@ -55,7 +55,7 @@ SaveUnitTestCache (
> Will allocate a buffer to hold the loaded data.
>
> @param[in] FrameworkHandle A pointer to the framework that is being persisted.
> - @param[in] SaveData A pointer pointer that will be updated with the address
> + @param[out] SaveData A pointer that will be updated with the address
> of the loaded data buffer.
>
> @retval EFI_SUCCESS Data has been loaded successfully and SaveData is updated
> diff --git a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c
> b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c
> index ed4a7d161524..940c7e42d43c 100644
> --- a/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c
> +++ b/UnitTestFrameworkPkg/Library/UnitTestPersistenceLibSimpleFileSystem/UnitTestPersistenceLibSimpleFileSystem.c
> @@ -312,7 +312,7 @@ Exit:
> Will allocate a buffer to hold the loaded data.
>
> @param[in] FrameworkHandle A pointer to the framework that is being persisted.
> - @param[in] SaveData A pointer pointer that will be updated with the address
> + @param[out] SaveData A pointer that will be updated with the address
> of the loaded data buffer.
>
> @retval EFI_SUCCESS Data has been loaded successfully and SaveData is updated
> diff --git a/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h
> b/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h
> index be29e079ec3e..e473bebec7e2 100644
> --- a/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h
> +++ b/UnitTestFrameworkPkg/PrivateInclude/Library/UnitTestPersistenceLib.h
> @@ -57,7 +57,7 @@ SaveUnitTestCache (
> Will allocate a buffer to hold the loaded data.
>
> @param[in] FrameworkHandle A pointer to the framework that is being persisted.
> - @param[in] SaveData A pointer pointer that will be updated with the address
> + @param[out] SaveData A pointer that will be updated with the address
> of the loaded data buffer.
>
> @retval EFI_SUCCESS Data has been loaded successfully and SaveData is updated
> diff --git a/UnitTestFrameworkPkg/ReadMe.md b/UnitTestFrameworkPkg/ReadMe.md
> index e696412cb3cf..a7b64bd65392 100644
> --- a/UnitTestFrameworkPkg/ReadMe.md
> +++ b/UnitTestFrameworkPkg/ReadMe.md
> @@ -144,7 +144,7 @@ will be used when adding test cases.
> Great! Now we've finished some of the cruft, red tape, and busy work. We're ready to add some tests. Adding a test
> to a test suite is accomplished with the -- you guessed it -- `AddTestCase` function. It takes in the suite handle;
> a `CHAR8` string for the description and class name; a function pointer for the test case itself; additional, optional
> -function pointers for prerequisite check and cleanup routines; and and optional pointer to a context structure.
> +function pointers for prerequisite check and cleanup routines; and optional pointer to a context structure.
>
> Okay, that's a lot. Let's take it one piece at a time. The description and class name strings are very similar in
> usage to the suite title and package name strings in the test suites. The former is for user presentation and the
> @@ -268,8 +268,8 @@ stuart_ci_build -c .pytool/CISettings.py TOOL_CHAIN_TAG=VS2019 -p MdePkg -t NOOP
>
> ## Building and Running Host-Based Tests
>
> -The EDK2 CI infrastructure provides a convenient way to run all host-based tests -- in the the entire tree or just
> -selected packages -- and aggregate all the the reports, including highlighting any failures. This functionality is
> +The EDK2 CI infrastructure provides a convenient way to run all host-based tests -- in the entire tree or just
> +selected packages -- and aggregate all the reports, including highlighting any failures. This functionality is
> provided through the Stuart build system (published by EDK2-PyTools) and the `NOOPT` build target.
>
> ### Building Locally
> --
> 2.25.1
^ permalink raw reply [flat|nested] 27+ messages in thread