From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.20, mailfrom: chasel.chiu@intel.com) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by groups.io with SMTP; Thu, 11 Jul 2019 02:39:53 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Jul 2019 02:39:52 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.63,478,1557212400"; d="scan'208";a="186343755" Received: from pgsmsx108.gar.corp.intel.com ([10.221.44.103]) by fmsmga001.fm.intel.com with ESMTP; 11 Jul 2019 02:39:50 -0700 Received: from pgsmsx111.gar.corp.intel.com ([169.254.2.22]) by PGSMSX108.gar.corp.intel.com ([169.254.8.190]) with mapi id 14.03.0439.000; Thu, 11 Jul 2019 17:39:49 +0800 From: "Chiu, Chasel" To: "coeur@gmx.fr" , "devel@edk2.groups.io" CC: "Zeng, Star" Subject: Re: [PATCH] Revert "FmpDevicePkg: Fix various typos" Thread-Topic: [PATCH] Revert "FmpDevicePkg: Fix various typos" Thread-Index: AQHVN8xLquD4ORf090GWcS/ZfRUsfabFKX9Q Date: Thu, 11 Jul 2019 09:39:48 +0000 Message-ID: <3C3EFB470A303B4AB093197B6777CCEC50427EC3@PGSMSX111.gar.corp.intel.com> References: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4A6537@SHSMSX104.ccr.corp.intel.com> <20190711093659.45166-1-coeur@gmx.fr> In-Reply-To: <20190711093659.45166-1-coeur@gmx.fr> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNDc2ZWZjNmQtMjhhOC00YmZjLTgzZWYtMTMxOWY0ODBjNTY4IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoicnkxZExnSW1mdGV4cjJjTHN1UlVjYWJcLzl1V3JiQ1FmN3hLMTlTZytHZkJrM2lDaCtLTTQ2dVhKeFNzY3E1OHEifQ== x-ctpclassification: CTP_NT x-originating-ip: [172.30.20.205] MIME-Version: 1.0 Content-Language: en-US Content-Type: text/plain; charset="windows-1256" Content-Transfer-Encoding: quoted-printable Reviewed-by: Chasel Chiu > -----Original Message----- > From: Antoine C=9Cur [mailto:coeur@gmx.fr] > Sent: Thursday, July 11, 2019 5:37 PM > To: devel@edk2.groups.io > Cc: Antoine C=9Cur ; Zeng, Star ; Chiu= , > Chasel > Subject: [PATCH] Revert "FmpDevicePkg: Fix various typos" >=20 > This reverts commit f527942e6bdd9f198db90f2de99a0482e9be5b1b. > Commit message was incorrect. >=20 > Signed-off-by: Coeur > --- > .../FspSecCore/Ia32/FspApiEntryM.nasm | 4 +-- > .../FspSecCore/Ia32/InitializeFpu.nasm | 4 +-- > .../FspSecCore/Ia32/SaveRestoreSseNasm.inc | 4 +-- > IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm | 4 +-- > IntelFsp2Pkg/FspSecCore/SecFsp.c | 4 +-- > IntelFsp2Pkg/FspSecCore/SecMain.c | 2 +- > .../FspSecCore/Vtf0/Ia16/ResetVec.asm16 | 4 +-- > IntelFsp2Pkg/Include/FspEas/FspApi.h | 8 ++--- > .../Include/Library/FspSecPlatformLib.h | 4 +-- > IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c | 12 +++---- > .../BaseFspDebugLibSerialPort/DebugLib.c | 34 +++++++++---------- > .../BaseFspSwitchStackLib/Ia32/Stack.nasm | 4 +-- > .../SecFspSecPlatformLibNull/Ia32/Flat32.nasm | 4 +-- > .../PlatformSecLibNull.c | 4 +-- > IntelFsp2Pkg/Tools/GenCfgOpt.py | 2 +- > IntelFsp2Pkg/Tools/PatchFv.py | 2 +- > .../Tools/UserManuals/GenCfgOptUserManual.md | 2 +- > .../Tools/UserManuals/PatchFvUserManual.md | 2 +- > 18 files changed, 52 insertions(+), 52 deletions(-) >=20 > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > index e7261b41cd..f14c18c7b9 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > @@ -194,9 +194,9 @@ StackSetupDone: >=20 > ; > ; Pass BFV into the PEI Core > - ; It uses relative address to calculate the actual boot FV base > + ; It uses relative address to calucate the actual boot FV base > ; For FSP implementation with single FV, PcdFspBootFirmwareVolumeBase > and > - ; PcdFspAreaBaseAddress are the same. For FSP with multiple FVs, > + ; PcdFspAreaBaseAddress are the same. For FSP with mulitple FVs, > ; they are different. The code below can handle both cases. > ; > call ASM_PFX(AsmGetFspBaseAddress) > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > b/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > index ebc91c41e4..e1886ea11b 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > @@ -1,6 +1,6 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015, Intel Corporation. All rights reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > @@ -46,7 +46,7 @@ ASM_PFX(InitializeFloatingPointUnits): > fldcw [ASM_PFX(mFpuControlWord)] >=20 > ; > - ; Use CpuId instruction (CPUID.01H:EDX.SSE[bit 25] =3D 1) to test > + ; Use CpuId instructuion (CPUID.01H:EDX.SSE[bit 25] =3D 1) to test > ; whether the processor supports SSE instruction. > ; > mov eax, 1 > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > index 4c321cbece..b257deb76c 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > @@ -1,6 +1,6 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015, Intel Corporation. All rights reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > @@ -150,7 +150,7 @@ NextAddress: > fldcw [FpuControlWord] >=20 > ; > - ; Use CpuId instruction (CPUID.01H:EDX.SSE[bit 25] =3D 1) to= test > + ; Use CpuId instructuion (CPUID.01H:EDX.SSE[bit 25] =3D 1) t= o > + test > ; whether the processor supports SSE instruction. > ; > mov eax, 1 > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > b/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > index 5a7e27c240..d72212ed45 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > @@ -1,6 +1,6 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015, Intel Corporation. All rights reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > @@ -58,7 +58,7 @@ ASM_PFX(SecSwitchStack): > mov esp, eax ; From now, esp is pointed to per= manent > memory >=20 > ; > - ; Fixup the ebp point to permanent memory > + ; Fixup the ebp point to permenent memory > ; > mov eax, ebp > sub eax, ebx > diff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.c > b/IntelFsp2Pkg/FspSecCore/SecFsp.c > index 446d1730e9..6497c88ebe 100644 > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.c > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.c > @@ -1,6 +1,6 @@ > /** @file >=20 > - Copyright (c) 2014 - 2019, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2018, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -169,7 +169,7 @@ FspGlobalDataInit ( > SerialPortInitialize (); >=20 > // > - // Ensure the global data pointer is valid > + // Ensure the golbal data pointer is valid > // > ASSERT (GetFspGlobalDataPointer () =3D=3D PeiFspData); >=20 > diff --git a/IntelFsp2Pkg/FspSecCore/SecMain.c > b/IntelFsp2Pkg/FspSecCore/SecMain.c > index a63d1336e4..cd3ab46ce2 100644 > --- a/IntelFsp2Pkg/FspSecCore/SecMain.c > +++ b/IntelFsp2Pkg/FspSecCore/SecMain.c > @@ -110,7 +110,7 @@ SecStartup ( > // |-------------------|----> > // | | > // | | > - // | Heap | PeiTemporaryRamSize > + // | Heap | PeiTemporayRamSize > // | | > // | | > // |-------------------|----> TempRamBase diff --git > a/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > b/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > index c519874809..f25de0206a 100644 > --- a/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > +++ b/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > @@ -2,7 +2,7 @@ > ; Reset Vector Data structure > ; This structure is located at 0xFFFFFFC0 ; -; Copyright (c) 2014 - 20= 19, Intel > Corporation. All rights reserved.
> +; Copyright (c) 2014, Intel Corporation. All rights reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ;; @@ -61,7 +61,7 @@ > ApStartup: > ; > ; Jmp Rel16 instruction > ; Use machine code directly in case of the assembler optimization > - ; SEC entry point relative address will be fixed up by some build to= ol. > + ; SEC entry point relatvie address will be fixed up by some build to= ol. > ; > ; Typically, SEC entry point is the function _ModuleEntryPoint() def= ined in > ; SecEntry.asm > diff --git a/IntelFsp2Pkg/Include/FspEas/FspApi.h > b/IntelFsp2Pkg/Include/FspEas/FspApi.h > index dcf489dbe6..1d38e639e6 100644 > --- a/IntelFsp2Pkg/Include/FspEas/FspApi.h > +++ b/IntelFsp2Pkg/Include/FspEas/FspApi.h > @@ -2,7 +2,7 @@ > Intel FSP API definition from Intel Firmware Support Package External > Architecture Specification v2.0. >=20 > - Copyright (c) 2014 - 2019, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2018, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -211,12 +211,12 @@ EFI_STATUS > each FSP release. > After FspMemInit completes its execution, it passes the pointer to the= HobList > and > returns to the boot loader from where it was called. BootLoader is res= ponsible > to > - migrate its stack and data to Memory. > + migrate it's stack and data to Memory. > FspMemoryInit, TempRamExit and FspSiliconInit APIs provide an alternat= e > method to > complete the silicon initialization and provides bootloader an opportu= nity to > get > control after system memory is available and before the temporary RAM = is > torn down. >=20 > - @param[in] FspmUpdDataPtr Pointer to the FSPM_UPD data > structure. > + @param[in] FspmUpdDataPtr Pointer to the FSPM_UPD data > sructure. > @param[out] HobListPtr Pointer to receive the address of = the HOB > list. >=20 > @retval EFI_SUCCESS FSP execution environment was init= ialized > successfully. > @@ -271,7 +271,7 @@ EFI_STATUS > @retval EFI_INVALID_PARAMETER Input parameters are invalid. > @retval EFI_UNSUPPORTED The FSP calling conditions were no= t > met. > @retval EFI_DEVICE_ERROR FSP initialization failed. > - @retval FSP_STATUS_RESET_REQUIREDx A reset is required. These status > codes will not be returned during S3. > + @retval FSP_STATUS_RESET_REQUIREDx A reset is reuired. These status > codes will not be returned during S3. > **/ > typedef > EFI_STATUS > diff --git a/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > b/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > index 4d01b5f6d9..48b04c5a90 100644 > --- a/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > +++ b/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > @@ -1,6 +1,6 @@ > /** @file >=20 > - Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.
> + Copyright (c) 2015 - 2016, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -66,7 +66,7 @@ SecCarInit ( > ); >=20 > /** > - This function check the signature of UPD. > + This function check the signture of UPD. >=20 > @param[in] ApiIdx Internal index of the FSP API. > @param[in] ApiParam Parameter of the FSP API. > diff --git a/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > b/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > index 17e895c345..927cee13d3 100644 > --- a/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > +++ b/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > @@ -1,6 +1,6 @@ > /** @file >=20 > - Copyright (c) 2014 - 2019, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2015, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -52,8 +52,8 @@ IsDefaultType ( > @param[in] BaseAddress Base address. > @param[in] Size Size. >=20 > - @retval Zero Aligned. > - @retval Non-Zero Not aligned. > + @retval Zero Alligned. > + @retval Non-Zero Not alligned. >=20 > **/ > UINT32 > @@ -217,7 +217,7 @@ Power2MaxMemory ( > } >=20 > // > - // Compute initial power of 2 size to return > + // Compute inital power of 2 size to return > // > Result =3D GetPowerOfTwo64(MemoryLength); >=20 > @@ -247,8 +247,8 @@ Power2MaxMemory ( > @param[in] BaseAddress Base address. > @param[in] Size Size. >=20 > - @retval Zero Aligned. > - @retval Non-Zero Not aligned. > + @retval Zero Alligned. > + @retval Non-Zero Not alligned. >=20 > **/ > UINT32 > diff --git a/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > b/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > index b34905365d..17688c7fcb 100644 > --- a/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > +++ b/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > @@ -186,7 +186,7 @@ DebugBPrint ( > } >=20 > /** > - Convert an UINT32 value into HEX string specified by Buffer. > + Convert an UINT32 value into HEX string sepcified by Buffer. >=20 > @param Value The HEX value to convert to string > @param Buffer The pointer to the target buffer to be filled with HEX= string > @@ -211,8 +211,8 @@ FillHex ( >=20 > Print a message of the form "ASSERT (): > \n" > to the debug output device. If > DEBUG_PROPERTY_ASSERT_BREAKPOINT_ENABLED bit of > - PcdDebugPropertyMask is set then CpuBreakpoint() is called. Otherwise,= if > - DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > PcdDebugPropertyMask is set then > + PcdDebugProperyMask is set then CpuBreakpoint() is called. Otherwise, > + if DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > PcdDebugProperyMask > + is set then > CpuDeadLoop() is called. If neither of these bits are set, then this = function > returns immediately after the message is printed to the debug output d= evice. > DebugAssert() must actively prevent recursion. If DebugAssert() is ca= lled > while @@ -265,8 +265,8 @@ DebugAssertInternal ( >=20 > Print a message of the form "ASSERT (): > \n" > to the debug output device. If > DEBUG_PROPERTY_ASSERT_BREAKPOINT_ENABLED bit of > - PcdDebugPropertyMask is set then CpuBreakpoint() is called. Otherwise,= if > - DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > PcdDebugPropertyMask is set then > + PcdDebugProperyMask is set then CpuBreakpoint() is called. Otherwise, > + if DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > PcdDebugProperyMask > + is set then > CpuDeadLoop() is called. If neither of these bits are set, then this = function > returns immediately after the message is printed to the debug output d= evice. > DebugAssert() must actively prevent recursion. If DebugAssert() is ca= lled > while @@ -322,10 +322,10 @@ DebugClearMemory ( > Returns TRUE if ASSERT() macros are enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > - PcdDebugPropertyMask is set. Otherwise FALSE is returned. > + PcdDebugProperyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugPropertyMask is set. > - @retval FALSE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugPropertyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugProperyMask is set. > + @retval FALSE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugProperyMask is clear. >=20 > **/ > BOOLEAN > @@ -342,10 +342,10 @@ DebugAssertEnabled ( > Returns TRUE if DEBUG() macros are enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > - PcdDebugPropertyMask is set. Otherwise FALSE is returned. > + PcdDebugProperyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugPropertyMask is set. > - @retval FALSE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugPropertyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugProperyMask is set. > + @retval FALSE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugProperyMask is clear. >=20 > **/ > BOOLEAN > @@ -361,10 +361,10 @@ DebugPrintEnabled ( > Returns TRUE if DEBUG_CODE() macros are enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > - PcdDebugPropertyMask is set. Otherwise FALSE is returned. > + PcdDebugProperyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugPropertyMask is set. > - @retval FALSE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugPropertyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugProperyMask is set. > + @retval FALSE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugProperyMask is clear. >=20 > **/ > BOOLEAN > @@ -381,10 +381,10 @@ DebugCodeEnabled ( > Returns TRUE if DEBUG_CLEAR_MEMORY() macro is enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > - PcdDebugPropertyMask is set. Otherwise FALSE is returned. > + PcdDebugProperyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugPropertyMask is set. > - @retval FALSE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugPropertyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugProperyMask is set. > + @retval FALSE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugProperyMask is clear. >=20 > **/ > BOOLEAN > diff --git a/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > b/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > index 6599901906..45cc974788 100644 > --- a/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > +++ b/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > @@ -1,11 +1,11 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2016 - 2019, Intel Corporation. All rights reserved.
> +; Copyright (c) 2016, Intel Corporation. All rights reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > ; > -; Switch the stack from temporary memory to permanent memory. > +; Switch the stack from temporary memory to permenent memory. > ; > ;-----------------------------------------------------------------------= ------- >=20 > diff --git a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.na= sm > b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.nasm > index aef7f96d1d..dc4af7c078 100644 > --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.nasm > +++ b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.nasm > @@ -2,7 +2,7 @@ > ; This is the code that goes from real-mode to protected mode. > ; It consumes the reset vector, configures the stack. > ; > -; Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015 - 2018, Intel Corporation. All rights > +reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ;; >=20 > @@ -54,7 +54,7 @@ ASM_PFX(SecPlatformInit): > ; esp > ; > ; Description: > -; Perform any essential early platform initialisation > +; Perform any essential early platform initilaisation > ; Setup a stack > ; > ;-----------------------------------------------------------------------= ----- > diff --git > a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > index f7945b5240..50cb3142d2 100644 > --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > +++ b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > @@ -1,7 +1,7 @@ > /** @file > Null instance of Platform Sec Lib. >=20 > - Copyright (c) 2014 - 2019, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2016, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -10,7 +10,7 @@ > #include >=20 > /** > - This function check the signature of UPD. > + This function check the signture of UPD. >=20 > @param[in] ApiIdx Internal index of the FSP API. > @param[in] ApiParam Parameter of the FSP API. > diff --git a/IntelFsp2Pkg/Tools/GenCfgOpt.py > b/IntelFsp2Pkg/Tools/GenCfgOpt.py index a42717caae..c4e1e6239d 100644 > --- a/IntelFsp2Pkg/Tools/GenCfgOpt.py > +++ b/IntelFsp2Pkg/Tools/GenCfgOpt.py > @@ -535,7 +535,7 @@ EndList > Offset =3D 0 > else: > if DscLine.startswith('!'): > - print("ERROR: Unrecognized d= irective for > line '%s'" % DscLine) > + print("ERROR: Unrecoginized > + directive for line '%s'" % DscLine) > raise SystemExit > if not Handle: > continue > diff --git a/IntelFsp2Pkg/Tools/PatchFv.py b/IntelFsp2Pkg/Tools/PatchFv.p= y > index edb30c816b..2173984dea 100644 > --- a/IntelFsp2Pkg/Tools/PatchFv.py > +++ b/IntelFsp2Pkg/Tools/PatchFv.py > @@ -160,7 +160,7 @@ class Symbols: > # > def createDicts (self, fvDir, fvNames): > # > - # If the fvDir is not a directory, then raise an exception > + # If the fvDir is not a dirctory, then raise an exception > # > if not os.path.isdir(fvDir): > raise Exception ("'%s' is not a valid directory!" % FvDir) d= iff --git > a/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > b/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > index 0a0f592801..938c18416d 100644 > --- a/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > +++ b/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > @@ -213,7 +213,7 @@ in the third. > ```@Bsf NAME:{Variable 1} TYPE:{Combo}``` >=20 > There is a special **None** type that puts the variable in the **StructD= ef** > -region of the BSF, but doesn't put it in any **Page** section. This make= s the > +region of the BSF, but doesn?t put it in any **Page** section. This > +makes the > variable visible to BCT, but not to the end user. >=20 > ###HELP > diff --git a/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > b/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > index 5f1031e729..becaf96b21 100644 > --- a/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > +++ b/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > @@ -30,7 +30,7 @@ FSP tree. >=20 > The example used contains Windows batch script %VARIABLES%. >=20 > -#FvFileBaseNames (Argument 2: Optional Part 1) > +#FvFileBaseNames (Argument 2: 0ptional Part 1) > The firmware volume file base names (**_FvFileBaseNames_**) are the > independent Fv?s that are to be patched within the FD. (0 or more in the= form > **FVFILEBASENAME:**) The colon **:** is used for delimiting the single > -- > 2.20.1 (Apple Git-117) >=20 > From bee5866c09bd18f9c3d40aeb56c5595be0a07bfe Mon Sep 17 00:00:00 > 2001 > From: =3D?UTF-8?q?Antoine=3D20C=3DC5=3D93ur?=3D > Date: Thu, 11 Jul 2019 17:30:10 +0800 > Subject: [PATCH] IntelFsp2Pkg: Fix various typos >=20 > Fix various typos in IntelFsp2Pkg. >=20 > Signed-off-by: Coeur > Reviewed-by: Star Zeng > Reviewed-by: Chasel Chiu > --- > .../FspSecCore/Ia32/FspApiEntryM.nasm | 4 +-- > .../FspSecCore/Ia32/InitializeFpu.nasm | 4 +-- > .../FspSecCore/Ia32/SaveRestoreSseNasm.inc | 4 +-- > IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm | 4 +-- > IntelFsp2Pkg/FspSecCore/SecFsp.c | 4 +-- > IntelFsp2Pkg/FspSecCore/SecMain.c | 2 +- > .../FspSecCore/Vtf0/Ia16/ResetVec.asm16 | 4 +-- > IntelFsp2Pkg/Include/FspEas/FspApi.h | 8 ++--- > .../Include/Library/FspSecPlatformLib.h | 4 +-- > IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c | 12 +++---- > .../BaseFspDebugLibSerialPort/DebugLib.c | 34 +++++++++---------- > .../BaseFspSwitchStackLib/Ia32/Stack.nasm | 4 +-- > .../SecFspSecPlatformLibNull/Ia32/Flat32.nasm | 4 +-- > .../PlatformSecLibNull.c | 4 +-- > IntelFsp2Pkg/Tools/GenCfgOpt.py | 2 +- > IntelFsp2Pkg/Tools/PatchFv.py | 2 +- > .../Tools/UserManuals/GenCfgOptUserManual.md | 2 +- > .../Tools/UserManuals/PatchFvUserManual.md | 2 +- > 18 files changed, 52 insertions(+), 52 deletions(-) >=20 > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > index f14c18c7b9..e7261b41cd 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryM.nasm > @@ -194,9 +194,9 @@ StackSetupDone: >=20 > ; > ; Pass BFV into the PEI Core > - ; It uses relative address to calucate the actual boot FV base > + ; It uses relative address to calculate the actual boot FV base > ; For FSP implementation with single FV, PcdFspBootFirmwareVolumeBase > and > - ; PcdFspAreaBaseAddress are the same. For FSP with mulitple FVs, > + ; PcdFspAreaBaseAddress are the same. For FSP with multiple FVs, > ; they are different. The code below can handle both cases. > ; > call ASM_PFX(AsmGetFspBaseAddress) > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > b/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > index e1886ea11b..ebc91c41e4 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/InitializeFpu.nasm > @@ -1,6 +1,6 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2015, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015 - 2019, Intel Corporation. All rights > +reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > @@ -46,7 +46,7 @@ ASM_PFX(InitializeFloatingPointUnits): > fldcw [ASM_PFX(mFpuControlWord)] >=20 > ; > - ; Use CpuId instructuion (CPUID.01H:EDX.SSE[bit 25] =3D 1) to test > + ; Use CpuId instruction (CPUID.01H:EDX.SSE[bit 25] =3D 1) to test > ; whether the processor supports SSE instruction. > ; > mov eax, 1 > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > index b257deb76c..4c321cbece 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc > @@ -1,6 +1,6 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2015, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015 - 2019, Intel Corporation. All rights > +reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > @@ -150,7 +150,7 @@ NextAddress: > fldcw [FpuControlWord] >=20 > ; > - ; Use CpuId instructuion (CPUID.01H:EDX.SSE[bit 25] =3D 1) t= o test > + ; Use CpuId instruction (CPUID.01H:EDX.SSE[bit 25] =3D 1) to > + test > ; whether the processor supports SSE instruction. > ; > mov eax, 1 > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > b/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > index d72212ed45..5a7e27c240 100644 > --- a/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/Stack.nasm > @@ -1,6 +1,6 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2015, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015 - 2019, Intel Corporation. All rights > +reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > @@ -58,7 +58,7 @@ ASM_PFX(SecSwitchStack): > mov esp, eax ; From now, esp is pointed to per= manent > memory >=20 > ; > - ; Fixup the ebp point to permenent memory > + ; Fixup the ebp point to permanent memory > ; > mov eax, ebp > sub eax, ebx > diff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.c > b/IntelFsp2Pkg/FspSecCore/SecFsp.c > index 6497c88ebe..446d1730e9 100644 > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.c > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.c > @@ -1,6 +1,6 @@ > /** @file >=20 > - Copyright (c) 2014 - 2018, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2019, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -169,7 +169,7 @@ FspGlobalDataInit ( > SerialPortInitialize (); >=20 > // > - // Ensure the golbal data pointer is valid > + // Ensure the global data pointer is valid > // > ASSERT (GetFspGlobalDataPointer () =3D=3D PeiFspData); >=20 > diff --git a/IntelFsp2Pkg/FspSecCore/SecMain.c > b/IntelFsp2Pkg/FspSecCore/SecMain.c > index cd3ab46ce2..a63d1336e4 100644 > --- a/IntelFsp2Pkg/FspSecCore/SecMain.c > +++ b/IntelFsp2Pkg/FspSecCore/SecMain.c > @@ -110,7 +110,7 @@ SecStartup ( > // |-------------------|----> > // | | > // | | > - // | Heap | PeiTemporayRamSize > + // | Heap | PeiTemporaryRamSize > // | | > // | | > // |-------------------|----> TempRamBase diff --git > a/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > b/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > index f25de0206a..c519874809 100644 > --- a/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > +++ b/IntelFsp2Pkg/FspSecCore/Vtf0/Ia16/ResetVec.asm16 > @@ -2,7 +2,7 @@ > ; Reset Vector Data structure > ; This structure is located at 0xFFFFFFC0 ; -; Copyright (c) 2014, Int= el > Corporation. All rights reserved.
> +; Copyright (c) 2014 - 2019, Intel Corporation. All rights > +reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ;; @@ -61,7 +61,7 @@ > ApStartup: > ; > ; Jmp Rel16 instruction > ; Use machine code directly in case of the assembler optimization > - ; SEC entry point relatvie address will be fixed up by some build to= ol. > + ; SEC entry point relative address will be fixed up by some build to= ol. > ; > ; Typically, SEC entry point is the function _ModuleEntryPoint() def= ined in > ; SecEntry.asm > diff --git a/IntelFsp2Pkg/Include/FspEas/FspApi.h > b/IntelFsp2Pkg/Include/FspEas/FspApi.h > index 1d38e639e6..dcf489dbe6 100644 > --- a/IntelFsp2Pkg/Include/FspEas/FspApi.h > +++ b/IntelFsp2Pkg/Include/FspEas/FspApi.h > @@ -2,7 +2,7 @@ > Intel FSP API definition from Intel Firmware Support Package External > Architecture Specification v2.0. >=20 > - Copyright (c) 2014 - 2018, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2019, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -211,12 +211,12 @@ EFI_STATUS > each FSP release. > After FspMemInit completes its execution, it passes the pointer to the= HobList > and > returns to the boot loader from where it was called. BootLoader is res= ponsible > to > - migrate it's stack and data to Memory. > + migrate its stack and data to Memory. > FspMemoryInit, TempRamExit and FspSiliconInit APIs provide an alternat= e > method to > complete the silicon initialization and provides bootloader an opportu= nity to > get > control after system memory is available and before the temporary RAM = is > torn down. >=20 > - @param[in] FspmUpdDataPtr Pointer to the FSPM_UPD data > sructure. > + @param[in] FspmUpdDataPtr Pointer to the FSPM_UPD data > structure. > @param[out] HobListPtr Pointer to receive the address of = the HOB > list. >=20 > @retval EFI_SUCCESS FSP execution environment was init= ialized > successfully. > @@ -271,7 +271,7 @@ EFI_STATUS > @retval EFI_INVALID_PARAMETER Input parameters are invalid. > @retval EFI_UNSUPPORTED The FSP calling conditions were no= t > met. > @retval EFI_DEVICE_ERROR FSP initialization failed. > - @retval FSP_STATUS_RESET_REQUIREDx A reset is reuired. These status > codes will not be returned during S3. > + @retval FSP_STATUS_RESET_REQUIREDx A reset is required. These status > codes will not be returned during S3. > **/ > typedef > EFI_STATUS > diff --git a/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > b/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > index 48b04c5a90..4d01b5f6d9 100644 > --- a/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > +++ b/IntelFsp2Pkg/Include/Library/FspSecPlatformLib.h > @@ -1,6 +1,6 @@ > /** @file >=20 > - Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.
> + Copyright (c) 2015 - 2019, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -66,7 +66,7 @@ SecCarInit ( > ); >=20 > /** > - This function check the signture of UPD. > + This function check the signature of UPD. >=20 > @param[in] ApiIdx Internal index of the FSP API. > @param[in] ApiParam Parameter of the FSP API. > diff --git a/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > b/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > index 927cee13d3..17e895c345 100644 > --- a/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > +++ b/IntelFsp2Pkg/Library/BaseCacheLib/CacheLib.c > @@ -1,6 +1,6 @@ > /** @file >=20 > - Copyright (c) 2014 - 2015, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2019, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -52,8 +52,8 @@ IsDefaultType ( > @param[in] BaseAddress Base address. > @param[in] Size Size. >=20 > - @retval Zero Alligned. > - @retval Non-Zero Not alligned. > + @retval Zero Aligned. > + @retval Non-Zero Not aligned. >=20 > **/ > UINT32 > @@ -217,7 +217,7 @@ Power2MaxMemory ( > } >=20 > // > - // Compute inital power of 2 size to return > + // Compute initial power of 2 size to return > // > Result =3D GetPowerOfTwo64(MemoryLength); >=20 > @@ -247,8 +247,8 @@ Power2MaxMemory ( > @param[in] BaseAddress Base address. > @param[in] Size Size. >=20 > - @retval Zero Alligned. > - @retval Non-Zero Not alligned. > + @retval Zero Aligned. > + @retval Non-Zero Not aligned. >=20 > **/ > UINT32 > diff --git a/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > b/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > index 17688c7fcb..b34905365d 100644 > --- a/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > +++ b/IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > @@ -186,7 +186,7 @@ DebugBPrint ( > } >=20 > /** > - Convert an UINT32 value into HEX string sepcified by Buffer. > + Convert an UINT32 value into HEX string specified by Buffer. >=20 > @param Value The HEX value to convert to string > @param Buffer The pointer to the target buffer to be filled with HEX= string > @@ -211,8 +211,8 @@ FillHex ( >=20 > Print a message of the form "ASSERT (): > \n" > to the debug output device. If > DEBUG_PROPERTY_ASSERT_BREAKPOINT_ENABLED bit of > - PcdDebugProperyMask is set then CpuBreakpoint() is called. Otherwise, = if > - DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > PcdDebugProperyMask is set then > + PcdDebugPropertyMask is set then CpuBreakpoint() is called. > + Otherwise, if DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > + PcdDebugPropertyMask is set then > CpuDeadLoop() is called. If neither of these bits are set, then this = function > returns immediately after the message is printed to the debug output d= evice. > DebugAssert() must actively prevent recursion. If DebugAssert() is ca= lled > while @@ -265,8 +265,8 @@ DebugAssertInternal ( >=20 > Print a message of the form "ASSERT (): > \n" > to the debug output device. If > DEBUG_PROPERTY_ASSERT_BREAKPOINT_ENABLED bit of > - PcdDebugProperyMask is set then CpuBreakpoint() is called. Otherwise, = if > - DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > PcdDebugProperyMask is set then > + PcdDebugPropertyMask is set then CpuBreakpoint() is called. > + Otherwise, if DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of > + PcdDebugPropertyMask is set then > CpuDeadLoop() is called. If neither of these bits are set, then this = function > returns immediately after the message is printed to the debug output d= evice. > DebugAssert() must actively prevent recursion. If DebugAssert() is ca= lled > while @@ -322,10 +322,10 @@ DebugClearMemory ( > Returns TRUE if ASSERT() macros are enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > - PcdDebugProperyMask is set. Otherwise FALSE is returned. > + PcdDebugPropertyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugProperyMask is set. > - @retval FALSE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugProperyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugPropertyMask is set. > + @retval FALSE The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of > PcdDebugPropertyMask is clear. >=20 > **/ > BOOLEAN > @@ -342,10 +342,10 @@ DebugAssertEnabled ( > Returns TRUE if DEBUG() macros are enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > - PcdDebugProperyMask is set. Otherwise FALSE is returned. > + PcdDebugPropertyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugProperyMask is set. > - @retval FALSE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugProperyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugPropertyMask is set. > + @retval FALSE The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of > PcdDebugPropertyMask is clear. >=20 > **/ > BOOLEAN > @@ -361,10 +361,10 @@ DebugPrintEnabled ( > Returns TRUE if DEBUG_CODE() macros are enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > - PcdDebugProperyMask is set. Otherwise FALSE is returned. > + PcdDebugPropertyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugProperyMask is set. > - @retval FALSE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugProperyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugPropertyMask is set. > + @retval FALSE The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of > PcdDebugPropertyMask is clear. >=20 > **/ > BOOLEAN > @@ -381,10 +381,10 @@ DebugCodeEnabled ( > Returns TRUE if DEBUG_CLEAR_MEMORY() macro is enabled. >=20 > This function returns TRUE if the > DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > - PcdDebugProperyMask is set. Otherwise FALSE is returned. > + PcdDebugPropertyMask is set. Otherwise FALSE is returned. >=20 > - @retval TRUE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugProperyMask is set. > - @retval FALSE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugProperyMask is clear. > + @retval TRUE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugPropertyMask is set. > + @retval FALSE The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of > PcdDebugPropertyMask is clear. >=20 > **/ > BOOLEAN > diff --git a/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > b/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > index 45cc974788..6599901906 100644 > --- a/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > +++ b/IntelFsp2Pkg/Library/BaseFspSwitchStackLib/Ia32/Stack.nasm > @@ -1,11 +1,11 @@ > ;-----------------------------------------------------------------------= ------- > ; > -; Copyright (c) 2016, Intel Corporation. All rights reserved.
> +; Copyright (c) 2016 - 2019, Intel Corporation. All rights > +reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Abstract: > ; > -; Switch the stack from temporary memory to permenent memory. > +; Switch the stack from temporary memory to permanent memory. > ; > ;-----------------------------------------------------------------------= ------- >=20 > diff --git a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.na= sm > b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.nasm > index dc4af7c078..aef7f96d1d 100644 > --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.nasm > +++ b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.nasm > @@ -2,7 +2,7 @@ > ; This is the code that goes from real-mode to protected mode. > ; It consumes the reset vector, configures the stack. > ; > -; Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.
> +; Copyright (c) 2015 - 2019, Intel Corporation. All rights > +reserved.
> ; SPDX-License-Identifier: BSD-2-Clause-Patent ;; >=20 > @@ -54,7 +54,7 @@ ASM_PFX(SecPlatformInit): > ; esp > ; > ; Description: > -; Perform any essential early platform initilaisation > +; Perform any essential early platform initialisation > ; Setup a stack > ; > ;-----------------------------------------------------------------------= ----- > diff --git > a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > index 50cb3142d2..f7945b5240 100644 > --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > +++ b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/PlatformSecLibNull.c > @@ -1,7 +1,7 @@ > /** @file > Null instance of Platform Sec Lib. >=20 > - Copyright (c) 2014 - 2016, Intel Corporation. All rights reserved.
> + Copyright (c) 2014 - 2019, Intel Corporation. All rights > + reserved.
> SPDX-License-Identifier: BSD-2-Clause-Patent >=20 > **/ > @@ -10,7 +10,7 @@ > #include >=20 > /** > - This function check the signture of UPD. > + This function check the signature of UPD. >=20 > @param[in] ApiIdx Internal index of the FSP API. > @param[in] ApiParam Parameter of the FSP API. > diff --git a/IntelFsp2Pkg/Tools/GenCfgOpt.py > b/IntelFsp2Pkg/Tools/GenCfgOpt.py index c4e1e6239d..a42717caae 100644 > --- a/IntelFsp2Pkg/Tools/GenCfgOpt.py > +++ b/IntelFsp2Pkg/Tools/GenCfgOpt.py > @@ -535,7 +535,7 @@ EndList > Offset =3D 0 > else: > if DscLine.startswith('!'): > - print("ERROR: Unrecoginized = directive for > line '%s'" % DscLine) > + print("ERROR: Unrecognized > + directive for line '%s'" % DscLine) > raise SystemExit > if not Handle: > continue > diff --git a/IntelFsp2Pkg/Tools/PatchFv.py b/IntelFsp2Pkg/Tools/PatchFv.p= y > index 2173984dea..edb30c816b 100644 > --- a/IntelFsp2Pkg/Tools/PatchFv.py > +++ b/IntelFsp2Pkg/Tools/PatchFv.py > @@ -160,7 +160,7 @@ class Symbols: > # > def createDicts (self, fvDir, fvNames): > # > - # If the fvDir is not a dirctory, then raise an exception > + # If the fvDir is not a directory, then raise an exception > # > if not os.path.isdir(fvDir): > raise Exception ("'%s' is not a valid directory!" % FvDir) d= iff --git > a/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > b/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > index 938c18416d..0a0f592801 100644 > --- a/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > +++ b/IntelFsp2Pkg/Tools/UserManuals/GenCfgOptUserManual.md > @@ -213,7 +213,7 @@ in the third. > ```@Bsf NAME:{Variable 1} TYPE:{Combo}``` >=20 > There is a special **None** type that puts the variable in the **StructD= ef** > -region of the BSF, but doesn?t put it in any **Page** section. This make= s the > +region of the BSF, but doesn't put it in any **Page** section. This > +makes the > variable visible to BCT, but not to the end user. >=20 > ###HELP > diff --git a/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > b/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > index becaf96b21..5f1031e729 100644 > --- a/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > +++ b/IntelFsp2Pkg/Tools/UserManuals/PatchFvUserManual.md > @@ -30,7 +30,7 @@ FSP tree. >=20 > The example used contains Windows batch script %VARIABLES%. >=20 > -#FvFileBaseNames (Argument 2: 0ptional Part 1) > +#FvFileBaseNames (Argument 2: Optional Part 1) > The firmware volume file base names (**_FvFileBaseNames_**) are the > independent Fv?s that are to be patched within the FD. (0 or more in the= form > **FVFILEBASENAME:**) The colon **:** is used for delimiting the single > -- > 2.20.1 (Apple Git-117)