Debug messages that start as natural (English) language phrases (after the debug prefix) should uniformly begin with lower-case or upper-case. In SetMemoryEncDec() we have a mixture now. Stick with lower-case. (Upper-case is better for full sentences that also end with punctuation.)
Cc: Brijesh Singh <brijesh.si...@amd.com> Cc: Jordan Justen <jordan.l.jus...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Laszlo Ersek <ler...@redhat.com> --- OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c b/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c index c37d799bc1f4..96969617e0a3 100644 --- a/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c +++ b/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c @@ -321,7 +321,7 @@ SetMemoryEncDec ( SetOrClearCBit(&PageDirectory1GEntry->Uint64, Mode); DEBUG (( DEBUG_VERBOSE, - "%a:%a: Updated 1GB entry for Physical=0x%Lx\n", + "%a:%a: updated 1GB entry for Physical=0x%Lx\n", gEfiCallerBaseName, __FUNCTION__, PhysicalAddress @@ -334,7 +334,7 @@ SetMemoryEncDec ( // DEBUG (( DEBUG_VERBOSE, - "%a:%a: Spliting 1GB page for Physical=0x%Lx\n", + "%a:%a: spliting 1GB page for Physical=0x%Lx\n", gEfiCallerBaseName, __FUNCTION__, PhysicalAddress @@ -377,7 +377,7 @@ SetMemoryEncDec ( // DEBUG (( DEBUG_VERBOSE, - "%a:%a: Spliting 2MB page for Physical=0x%Lx\n", + "%a:%a: spliting 2MB page for Physical=0x%Lx\n", gEfiCallerBaseName, __FUNCTION__, PhysicalAddress -- 2.14.1.3.gb7cf6e02401b _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel