Hi Michael,
I just noted that 'interfacer' is no longer required in the
'ExtendWords' list. Please find my response inline marked [SAMI].
With the comment below fixed,
Reviewed-by: Sami Mujawar <sami.muja...@arm.com>
Regards,
Sami Mujawar
On 18/05/2022 01:13 am, mikub...@linux.microsoft.com wrote:
From: Michael Kubacki <michael.kuba...@microsoft.com>
The SpellCheck plugin began reporting new typos that were previously
missed. This change fixes those typos.
Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>
Cc: Sami Mujawar <sami.muja...@arm.com>
Cc: Jiewen Yao <jiewen....@intel.com>
Cc: Supreeth Venkatesh <supreeth.venkat...@arm.com>
Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com>
Reviewed-by: Sami Mujawar <sami.muja...@arm.com>
---
StandaloneMmPkg/Core/Dependency.c | 2 +-
StandaloneMmPkg/Core/Dispatcher.c | 18 +++++++++---------
StandaloneMmPkg/Core/Handle.c | 2 +-
StandaloneMmPkg/Core/StandaloneMmCore.h | 2 +-
StandaloneMmPkg/Include/Guid/MmCoreData.h | 2 +-
StandaloneMmPkg/StandaloneMmPkg.ci.yaml | 9 +++++++++
6 files changed, 22 insertions(+), 13 deletions(-)
diff --git a/StandaloneMmPkg/Core/Dependency.c
b/StandaloneMmPkg/Core/Dependency.c
index 440fe3e45238..0e9c0e96a276 100644
--- a/StandaloneMmPkg/Core/Dependency.c
+++ b/StandaloneMmPkg/Core/Dependency.c
@@ -38,7 +38,7 @@ BOOLEAN *mDepexEvaluationStackPointer = NULL;
/**
Grow size of the Depex stack
- @retval EFI_SUCCESS Stack successfully growed.
+ @retval EFI_SUCCESS Stack successfully grew.
@retval EFI_OUT_OF_RESOURCES There is not enough system memory to grow the
stack.
**/
diff --git a/StandaloneMmPkg/Core/Dispatcher.c
b/StandaloneMmPkg/Core/Dispatcher.c
index b1ccba15b060..01da395be3c4 100644
--- a/StandaloneMmPkg/Core/Dispatcher.c
+++ b/StandaloneMmPkg/Core/Dispatcher.c
@@ -17,7 +17,7 @@
and After dependencies. This is done recursively as the call to
add
to the mScheduledQueue checks for Before Depexes and recursively
adds all Before Depexes. It then adds the item that was passed in
- and then processess the After dependencies by recursively calling
+ and then processes the After dependencies by recursively calling
the routine.
Dispatcher Rules:
@@ -93,29 +93,29 @@ LIST_ENTRY mScheduledQueue = INITIALIZE_LIST_HEAD_VARIABLE
(mScheduledQueue);
LIST_ENTRY mFwVolList = INITIALIZE_LIST_HEAD_VARIABLE (mFwVolList);
//
-// Flag for the MM Dispacher. TRUE if dispatcher is executing.
+// Flag for the MM Dispatcher. TRUE if dispatcher is executing.
//
BOOLEAN gDispatcherRunning = FALSE;
//
-// Flag for the MM Dispacher. TRUE if there is one or more MM drivers ready
to be dispatched
+// Flag for the MM Dispatcher. TRUE if there is one or more MM drivers ready
to be dispatched
//
BOOLEAN gRequestDispatch = FALSE;
//
// The global variable is defined for Loading modules at fixed address
feature to track the MM code
-// memory range usage. It is a bit mapped array in which every bit indicates
the correspoding
+// memory range usage. It is a bit mapped array in which every bit indicates
the corresponding
// memory page available or not.
//
GLOBAL_REMOVE_IF_UNREFERENCED UINT64 *mMmCodeMemoryRangeUsageBitMap =
NULL;
/**
To check memory usage bit map array to figure out if the memory range in
which the image will be loaded
- is available or not. If memory range is avaliable, the function will mark
the corresponding bits to 1
+ is available or not. If memory range is available, the function will mark
the corresponding bits to 1
which indicates the memory range is used. The function is only invoked when
load modules at fixed address
feature is enabled.
- @param ImageBase The base addres the image will be loaded at.
+ @param ImageBase The base address the image will be loaded
at.
@param ImageSize The size of the image
@retval EFI_SUCCESS The memory range the image will be loaded
in is available
@@ -194,7 +194,7 @@ CheckAndMarkFixLoadingMemoryUsageBitMap (
@param ImageContext Pointer to the image context structure
that describes the PE/COFF
image that needs to be examined by this
function.
@retval EFI_SUCCESS An fixed loading address is assigned to
this image by build tools .
- @retval EFI_NOT_FOUND The image has no assigned fixed loadding
address.
+ @retval EFI_NOT_FOUND The image has no assigned fixed loading
address.
**/
EFI_STATUS
@@ -513,7 +513,7 @@ MmPreProcessDepex (
@param DriverEntry Driver to work on.
- @retval EFI_SUCCESS Depex read and preprossesed
+ @retval EFI_SUCCESS Depex read and pre-processed
@retval EFI_PROTOCOL_ERROR The section extraction protocol returned an
error
and Depex reading needs to be retried.
@retval Error DEPEX not found.
@@ -851,7 +851,7 @@ FvIsBeingProcessed (
The Discovered list is never freed and contains booleans that represent the
other possible MM driver states.
- @param [in] FwVolHeader Pointer to the formware volume header.
+ @param [in] FwVolHeader Pointer to the firmware volume header.
@param [in] Pe32Data Pointer to the PE data.
@param [in] Pe32DataSize Size of the PE data.
@param [in] Depex Pointer to the Depex info.
diff --git a/StandaloneMmPkg/Core/Handle.c b/StandaloneMmPkg/Core/Handle.c
index 7df4629cf291..bd0005fcb84d 100644
--- a/StandaloneMmPkg/Core/Handle.c
+++ b/StandaloneMmPkg/Core/Handle.c
@@ -347,7 +347,7 @@ MmInstallProtocolInterfaceNotify (
}
/**
- Uninstalls all instances of a protocol:interfacer from a handle.
+ Uninstalls all instances of a protocol interface from a handle.
If the last protocol interface is remove from the handle, the
handle is freed.
diff --git a/StandaloneMmPkg/Core/StandaloneMmCore.h
b/StandaloneMmPkg/Core/StandaloneMmCore.h
index 822d95358c39..b5e2e4b9083b 100644
--- a/StandaloneMmPkg/Core/StandaloneMmCore.h
+++ b/StandaloneMmPkg/Core/StandaloneMmCore.h
@@ -426,7 +426,7 @@ MmInstallProtocolInterfaceNotify (
);
/**
- Uninstalls all instances of a protocol:interfacer from a handle.
+ Uninstalls all instances of a protocol interface from a handle.
If the last protocol interface is remove from the handle, the
handle is freed.
diff --git a/StandaloneMmPkg/Include/Guid/MmCoreData.h
b/StandaloneMmPkg/Include/Guid/MmCoreData.h
index b8be92c6abfd..0f58f41d8060 100644
--- a/StandaloneMmPkg/Include/Guid/MmCoreData.h
+++ b/StandaloneMmPkg/Include/Guid/MmCoreData.h
@@ -70,7 +70,7 @@ typedef struct {
///
/// The MM Foundation Entry Point. The MM Core fills in this field when the
- /// MM Core is initialized. The MM IPL is responsbile for registering this
entry
+ /// MM Core is initialized. The MM IPL is responsible for registering this
entry
/// point with the MM Configuration Protocol. The MM Configuration
Protocol may
/// not be available at the time the MM IPL and MM Core are started, so the
MM IPL
/// sets up a protocol notification on the MM Configuration Protocol and
registers
diff --git a/StandaloneMmPkg/StandaloneMmPkg.ci.yaml
b/StandaloneMmPkg/StandaloneMmPkg.ci.yaml
index 4777532a7ede..535e269e7891 100644
--- a/StandaloneMmPkg/StandaloneMmPkg.ci.yaml
+++ b/StandaloneMmPkg/StandaloneMmPkg.ci.yaml
@@ -81,12 +81,21 @@
"IgnoreFiles": [], # use gitignore syntax to ignore errors
# in matching files
"ExtendWords": [
+ "bootinformation",
"Bsymbolic",
+ "bugbug",
+ "combineable",
+ "depexes",
"FwVol",
+ "hoblist",
+ "imagehandle",
+ "interfacer",
[SAMI] I think the above word is fixed in StandaloneMmPkg/Core/Handle.c
and StandaloneMmPkg/Core/StandaloneMmCore and 'interfacer' could be
removed. Can you check, please?
[/SAMI]
"mpidr",
"mstrict",
"schedulable",
"StandaloneMMCore",
+ "systemtable",
+ "uncacheable"
], # words to extend to the dictionary for this package
"IgnoreStandardPaths": [], # Standard Plugin defined paths that
# should be ignore
IMPORTANT NOTICE: The contents of this email and any attachments are
confidential and may also be privileged. If you are not the intended recipient,
please notify the sender immediately and do not disclose the contents to any
other person, use it for any purpose, or store or copy the information in any
medium. Thank you.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97302): https://edk2.groups.io/g/devel/message/97302
Mute This Topic: https://groups.io/mt/91176788/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
