From: "Sunil V L" <sunilvl@ventanamicro.com>
To: Abner Chang <abner.chang@hpe.com>
Cc: devel@edk2.groups.io, Daniel Schaefer <daniel.schaefer@hpe.com>
Subject: Re: [PATCH 68/79] RiscVPlatformVPkg: Address Core CI Spelling errors.
Date: Sun, 9 Jan 2022 20:23:58 +0530 [thread overview]
Message-ID: <20220109145358.GD4879@sunil-ThinkPad-T490> (raw)
In-Reply-To: <20220108041420.16064-7-abner.chang@hpe.com>
On Sat, Jan 08, 2022 at 12:14:09PM +0800, Abner Chang wrote:
> From: changab <abner.chang@hpe.com>
>
> Signed-off-by: Abner Chang <abner.chang@hpe.com>
> Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
> Cc: Sunil V L <sunilvl@ventanamicro.com>
> ---
> RiscVPlatformPkg/RiscVPlatformPkg.dec | 4 ++--
> .../FirmwareContextProcessorSpecificLib.inf | 4 ++--
> RiscVPlatformPkg/Universal/FdtPeim/FdtPeim.inf | 2 +-
> RiscVPkg/Include/ProcessorSpecificHobData.h | 8 ++++----
> .../Library/FirmwareContextProcessorSpecificLib.h | 4 ++--
> RiscVPlatformPkg/Universal/Sec/SecMain.h | 4 ++--
> RiscVPkg/Universal/SmbiosDxe/RiscVSmbiosDxe.c | 12 ++++++------
> .../Edk2OpensbiPlatformWrapperLib.c | 10 +++++-----
> .../FirmwareContextProcessorSpecificLib.c | 8 ++++----
> .../PlatformBootManagerLib/PlatformBootManager.c | 4 ++--
> .../Universal/Pei/PlatformPei/Platform.c | 4 ++--
> RiscVPlatformPkg/Universal/Sec/SecMain.c | 12 ++++++------
> RiscVPlatformPkg/Readme.md | 14 +++++++-------
> RiscVPlatformPkg/Universal/Sec/Riscv64/SecEntry.S | 10 +++++-----
> 14 files changed, 50 insertions(+), 50 deletions(-)
>
> diff --git a/RiscVPlatformPkg/RiscVPlatformPkg.dec b/RiscVPlatformPkg/RiscVPlatformPkg.dec
> index 53d424c901..f3217e4a05 100644
> --- a/RiscVPlatformPkg/RiscVPlatformPkg.dec
> +++ b/RiscVPlatformPkg/RiscVPlatformPkg.dec
> @@ -1,7 +1,7 @@
> ## @file RiscVPlatformPkg.dec
> # This Package provides UEFI RISC-V platform modules and libraries.
> #
> -# Copyright (c) 2021, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> +# Copyright (c) 2021-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> #
> # SPDX-License-Identifier: BSD-2-Clause-Patent
> #
> @@ -20,7 +20,7 @@
> [LibraryClasses]
> FirmwareContextProcessorSpecificLib|Include/Library/FirmwareContextProcessorSpecificLib.h
> RiscVPlatformTempMemoryInitLib|Include/Library/RiscVPlatformTempMemoryInitLib.h
> - Edk2OpensbiPlatformiLib|Include/Library/Edk2OpensbiPlatformiWrapperLib.h
> + Edk2OpensbiPlatformWrapperLib|Include/Library/Edk2OpensbiPlatformWrapperLib.h
>
> [Guids]
> gUefiRiscVPlatformPkgTokenSpaceGuid = {0x6A67AF99, 0x4592, 0x40F8, { 0xB6, 0xBE, 0x62, 0xBC, 0xA1, 0x0D, 0xA1, 0xEC}}
> diff --git a/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.inf b/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.inf
> index 4da78178a9..0d8aa9828f 100644
> --- a/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.inf
> +++ b/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.inf
> @@ -1,8 +1,8 @@
> ## @file
> # This is the library module of RISC-V EDK2 OpenSBI Firmware Context
> -# Processor Specific hwardware information.
> +# Processor Specific hardware information.
> #
> -# Copyright (c) 2019, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> +# Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> #
> # SPDX-License-Identifier: BSD-2-Clause-Patent
> #
> diff --git a/RiscVPlatformPkg/Universal/FdtPeim/FdtPeim.inf b/RiscVPlatformPkg/Universal/FdtPeim/FdtPeim.inf
> index cc870b8a91..45898c9dcd 100644
> --- a/RiscVPlatformPkg/Universal/FdtPeim/FdtPeim.inf
> +++ b/RiscVPlatformPkg/Universal/FdtPeim/FdtPeim.inf
> @@ -1,7 +1,7 @@
> ## @file
> # The FDT Peim driver is used to pass the device tree to DXE phase.
> #
> -# Copyright (c) 2021, Hewlett Packard Enterprise Developmente LP. All rights reserved.<BR>
> +# Copyright (c) 2021-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> #
> # SPDX-License-Identifier: BSD-2-Clause-Patent
> #
> diff --git a/RiscVPkg/Include/ProcessorSpecificHobData.h b/RiscVPkg/Include/ProcessorSpecificHobData.h
> index 2f5847e53e..97285289f7 100644
> --- a/RiscVPkg/Include/ProcessorSpecificHobData.h
> +++ b/RiscVPkg/Include/ProcessorSpecificHobData.h
> @@ -1,7 +1,7 @@
> /** @file
> Definition of Processor Specific Data HOB.
>
> - Copyright (c) 2019, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -24,7 +24,7 @@
> /// RISC-V processor specific data HOB
> ///
> typedef struct {
> - EFI_GUID ParentPrcessorGuid;
> + EFI_GUID ParentProcessorGuid;
> UINTN ParentProcessorUid;
> EFI_GUID CoreGuid;
> VOID *Context; // The additional information of this core which
> @@ -37,7 +37,7 @@ typedef struct {
> /// RISC-V SMBIOS type 4 (Processor) GUID data HOB
> ///
> typedef struct {
> - EFI_GUID PrcessorGuid;
> + EFI_GUID ProcessorGuid;
> UINTN ProcessorUid;
> SMBIOS_TABLE_TYPE4 SmbiosType4Processor;
> UINT16 EndingZero;
> @@ -75,7 +75,7 @@ typedef struct {
> /// RISC-V SMBIOS type 7 (Cache) GUID data HOB
> ///
> typedef struct {
> - EFI_GUID PrcessorGuid;
> + EFI_GUID ProcessorGuid;
> UINTN ProcessorUid;
> SMBIOS_TABLE_TYPE7 SmbiosType7Cache;
> UINT16 EndingZero;
> diff --git a/RiscVPlatformPkg/Include/Library/FirmwareContextProcessorSpecificLib.h b/RiscVPlatformPkg/Include/Library/FirmwareContextProcessorSpecificLib.h
> index 3920c61155..0eec62033b 100644
> --- a/RiscVPlatformPkg/Include/Library/FirmwareContextProcessorSpecificLib.h
> +++ b/RiscVPlatformPkg/Include/Library/FirmwareContextProcessorSpecificLib.h
> @@ -1,7 +1,7 @@
> /** @file
> Firmware Context Processor-specific common library
>
> - Copyright (c) 2019, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -18,7 +18,7 @@
>
> @param FirmwareContextHartSpecific Pointer to EFI_RISCV_FIRMWARE_CONTEXT_HART_SPECIFIC
> @param ParentProcessorGuid Pointer to GUID of Processor which contains this core
> - @param ParentProcessorUid Unique ID of pysical processor which owns this core.
> + @param ParentProcessorUid Unique ID of physical processor which owns this core.
> @param CoreGuid Pointer to GUID of core
> @param HartId Hart ID of this core.
> @param IsBootHart This is boot hart or not
> diff --git a/RiscVPlatformPkg/Universal/Sec/SecMain.h b/RiscVPlatformPkg/Universal/Sec/SecMain.h
> index 63a610fbd0..4098bd7d92 100644
> --- a/RiscVPlatformPkg/Universal/Sec/SecMain.h
> +++ b/RiscVPlatformPkg/Universal/Sec/SecMain.h
> @@ -1,7 +1,7 @@
> /** @file
> RISC-V SEC phase module definitions..
>
> - Copyright (c) 2019, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -35,7 +35,7 @@
>
> **/
> INT32
> -SecPostOpenSbiPlatformEarlylInit(
> +SecPostOpenSbiPlatformEarlyInit(
> IN BOOLEAN ColdBoot
> );
>
> diff --git a/RiscVPkg/Universal/SmbiosDxe/RiscVSmbiosDxe.c b/RiscVPkg/Universal/SmbiosDxe/RiscVSmbiosDxe.c
> index b30f9d7f6a..14f62c4036 100644
> --- a/RiscVPkg/Universal/SmbiosDxe/RiscVSmbiosDxe.c
> +++ b/RiscVPkg/Universal/SmbiosDxe/RiscVSmbiosDxe.c
> @@ -1,7 +1,7 @@
> /** @file
> RISC-V generic SMBIOS DXE driver to build up SMBIOS type 4, type 7 and type 44 records.
>
> - Copyright (c) 2019, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -33,7 +33,7 @@ BuildSmbiosType7 (
> EFI_STATUS Status;
> SMBIOS_HANDLE Handle;
>
> - if (!CompareGuid (&Type4HobData->PrcessorGuid, &Type7DataHob->PrcessorGuid) ||
> + if (!CompareGuid (&Type4HobData->ProcessorGuid, &Type7DataHob->ProcessorGuid) ||
> Type4HobData->ProcessorUid != Type7DataHob->ProcessorUid) {
> return EFI_INVALID_PARAMETER;
> }
> @@ -48,7 +48,7 @@ BuildSmbiosType7 (
> return Status;
> }
> DEBUG ((DEBUG_INFO, "SMBIOS Type 7 was added. SMBIOS Handle: 0x%x\n", Handle));
> - DEBUG ((DEBUG_VERBOSE, " Cache belone to processor GUID: %g\n", &Type7DataHob->PrcessorGuid));
> + DEBUG ((DEBUG_VERBOSE, " Cache belone to processor GUID: %g\n", &Type7DataHob->ProcessorGuid));
belone -> belongs?
> DEBUG ((DEBUG_VERBOSE, " Cache belone processor UID: %d\n", Type7DataHob->ProcessorUid));
> DEBUG ((DEBUG_VERBOSE, " ==============================\n"));
> DEBUG ((DEBUG_VERBOSE, " Socket Designation: %d\n", Type7DataHob->SmbiosType7Cache.SocketDesignation));
> @@ -90,7 +90,7 @@ BuildSmbiosType4 (
> EFI_STATUS Status;
>
> DEBUG ((DEBUG_INFO, "Building Type 4.\n"));
> - DEBUG ((DEBUG_INFO, " Processor GUID: %g\n", &Type4HobData->PrcessorGuid));
> + DEBUG ((DEBUG_INFO, " Processor GUID: %g\n", &Type4HobData->ProcessorGuid));
> DEBUG ((DEBUG_INFO, " Processor UUID: %d\n", Type4HobData->ProcessorUid));
>
> Type4HobData->SmbiosType4Processor.L1CacheHandle = RISC_V_CACHE_INFO_NOT_PROVIDED;
> @@ -193,7 +193,7 @@ BuildSmbiosType44 (
> EFI_STATUS Status;
>
> DEBUG ((DEBUG_INFO, "Building Type 44 for...\n"));
> - DEBUG ((DEBUG_VERBOSE, " Processor GUID: %g\n", &Type4HobData->PrcessorGuid));
> + DEBUG ((DEBUG_VERBOSE, " Processor GUID: %g\n", &Type4HobData->ProcessorGuid));
> DEBUG ((DEBUG_VERBOSE, " Processor UUID: %d\n", Type4HobData->ProcessorUid));
>
> GuidHob = (EFI_HOB_GUID_TYPE *)GetFirstGuidHob ((EFI_GUID *)PcdGetPtr(PcdProcessorSpecificDataGuidHobGuid));
> @@ -206,7 +206,7 @@ BuildSmbiosType44 (
> //
> do {
> ProcessorSpecificData = (RISC_V_PROCESSOR_SPECIFIC_HOB_DATA *)GET_GUID_HOB_DATA (GuidHob);
> - if (!CompareGuid (&ProcessorSpecificData->ParentPrcessorGuid, &Type4HobData->PrcessorGuid) ||
> + if (!CompareGuid (&ProcessorSpecificData->ParentProcessorGuid, &Type4HobData->ProcessorGuid) ||
> ProcessorSpecificData->ParentProcessorUid != Type4HobData->ProcessorUid) {
> GuidHob = GetNextGuidHob((EFI_GUID *)PcdGetPtr(PcdProcessorSpecificDataGuidHobGuid), GET_NEXT_HOB(GuidHob));
> if (GuidHob == NULL) {
> diff --git a/RiscVPlatformPkg/Library/Edk2OpensbiPlatformWrapperLib/Edk2OpensbiPlatformWrapperLib.c b/RiscVPlatformPkg/Library/Edk2OpensbiPlatformWrapperLib/Edk2OpensbiPlatformWrapperLib.c
> index 2137c6c619..0bd1b44241 100644
> --- a/RiscVPlatformPkg/Library/Edk2OpensbiPlatformWrapperLib/Edk2OpensbiPlatformWrapperLib.c
> +++ b/RiscVPlatformPkg/Library/Edk2OpensbiPlatformWrapperLib/Edk2OpensbiPlatformWrapperLib.c
> @@ -1,7 +1,7 @@
> /** @file
> EDK2 OpenSBI generic platform wrapper library
>
> - Copyright (c) 2021, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -43,7 +43,7 @@ SecSetEdk2FwMemoryRegions (
> fw_memregs.flags = SBI_DOMAIN_MEMREGION_EXECUTABLE | SBI_DOMAIN_MEMREGION_READABLE;
> Ret = sbi_domain_root_add_memregion ((CONST struct sbi_domain_memregion *)&fw_memregs);
> if (Ret != 0) {
> - DEBUG ((DEBUG_ERROR, "%a: Add firmware regiosn of FW Domain fail\n", __FUNCTION__));
> + DEBUG ((DEBUG_ERROR, "%a: Add firmware regions of FW Domain fail\n", __FUNCTION__));
> }
>
> //
> @@ -54,7 +54,7 @@ SecSetEdk2FwMemoryRegions (
> fw_memregs.flags = SBI_DOMAIN_MEMREGION_READABLE | SBI_DOMAIN_MEMREGION_WRITEABLE;
> Ret = sbi_domain_root_add_memregion ((CONST struct sbi_domain_memregion *)&fw_memregs);
> if (Ret != 0) {
> - DEBUG ((DEBUG_ERROR, "%a: Add firmware regiosn of variable FW Domain fail\n", __FUNCTION__));
> + DEBUG ((DEBUG_ERROR, "%a: Add firmware regions of variable FW Domain fail\n", __FUNCTION__));
> }
> return Ret;
> }
> @@ -66,7 +66,7 @@ SecSetEdk2FwMemoryRegions (
>
> **/
> INT32
> -SecPostOpenSbiPlatformEarlylInit(
> +SecPostOpenSbiPlatformEarlyInit(
> IN BOOLEAN ColdBoot
> )
> {
> @@ -190,7 +190,7 @@ Edk2OpensbiPlatformEarlyInit (
> }
> }
> if (ColdBoot) {
> - return SecPostOpenSbiPlatformEarlylInit(ColdBoot);
> + return SecPostOpenSbiPlatformEarlyInit(ColdBoot);
> }
> return 0;
> }
> diff --git a/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.c b/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.c
> index 143c18d62c..c94f7881c2 100644
> --- a/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.c
> +++ b/RiscVPlatformPkg/Library/FirmwareContextProcessorSpecificLib/FirmwareContextProcessorSpecificLib.c
> @@ -1,7 +1,7 @@
> /** @file
> - Common library to build upfirmware context processor-specific information
> + Common library to build up firmware context processor-specific information
>
> - Copyright (c) 2019, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -28,7 +28,7 @@
>
> @param FirmwareContextHartSpecific Pointer to EFI_RISCV_FIRMWARE_CONTEXT_HART_SPECIFIC
> @param ParentProcessorGuid Pointer to GUID of Processor which contains this core
> - @param ParentProcessorUid Unique ID of pysical processor which owns this core.
> + @param ParentProcessorUid Unique ID of physical processor which owns this core.
> @param CoreGuid Pointer to GUID of core
> @param HartId Hart ID of this core.
> @param IsBootHart This is boot hart or not
> @@ -52,7 +52,7 @@ CommonFirmwareContextHartSpecificInfo (
> //
> // Build up RISC_V_PROCESSOR_SPECIFIC_DATA_HOB.
> //
> - CopyGuid (&ProcessorSpecificDataHob->ParentPrcessorGuid, ParentProcessorGuid);
> + CopyGuid (&ProcessorSpecificDataHob->ParentProcessorGuid, ParentProcessorGuid);
> ProcessorSpecificDataHob->ParentProcessorUid = ParentProcessorUid;
> CopyGuid (&ProcessorSpecificDataHob->CoreGuid, CoreGuid);
> ProcessorSpecificDataHob->Context = NULL;
> diff --git a/RiscVPlatformPkg/Library/PlatformBootManagerLib/PlatformBootManager.c b/RiscVPlatformPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> index deaad7d5a1..9ad4ef17db 100644
> --- a/RiscVPlatformPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> +++ b/RiscVPlatformPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> @@ -1,7 +1,7 @@
> /** @file
> This file include all platform actions
>
> -Copyright (c) 2021, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> +Copyright (c) 2021-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
> @@ -215,7 +215,7 @@ PlatformBootManagerBeforeConsole (
> Signal console ready platform customized event;
> Run diagnostics like memory testing;
> Connect certain devices;
> - Dispatch aditional option roms.
> + Dispatch additional option roms.
> **/
> VOID
> EFIAPI
> diff --git a/RiscVPlatformPkg/Universal/Pei/PlatformPei/Platform.c b/RiscVPlatformPkg/Universal/Pei/PlatformPei/Platform.c
> index 8a248f3630..c28b2ed373 100644
> --- a/RiscVPlatformPkg/Universal/Pei/PlatformPei/Platform.c
> +++ b/RiscVPlatformPkg/Universal/Pei/PlatformPei/Platform.c
> @@ -1,7 +1,7 @@
> /** @file
> Platform PEI driver
>
> - Copyright (c) 2019-2021, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2019-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
> Copyright (c) 2011, Andrei Warkentin <andreiw@motorola.com>
>
> @@ -357,7 +357,7 @@ InitializePlatform (
> MiscInitialization ();
> Status = BuildCoreInformationHob ();
> if (EFI_ERROR (Status)) {
> - DEBUG ((DEBUG_ERROR, "Fail to build processor informstion HOB.\n"));
> + DEBUG ((DEBUG_ERROR, "Fail to build processor information HOB.\n"));
> ASSERT(FALSE);
> }
> return EFI_SUCCESS;
> diff --git a/RiscVPlatformPkg/Universal/Sec/SecMain.c b/RiscVPlatformPkg/Universal/Sec/SecMain.c
> index 1fafed2799..f67dbdf059 100644
> --- a/RiscVPlatformPkg/Universal/Sec/SecMain.c
> +++ b/RiscVPlatformPkg/Universal/Sec/SecMain.c
> @@ -1,7 +1,7 @@
> /** @file
> RISC-V SEC phase module.
>
> - Copyright (c) 2021, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
> + Copyright (c) 2021-2022, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -291,7 +291,7 @@ FindPeiCoreImageBase (
> /**
> Find and return Pei Core entry point.
>
> - It also find SEC and PEI Core file debug inforamtion. It will report them if
> + It also find SEC and PEI Core file debug information. It will report them if
find -> finds
Regards
Sunil
> remote debug is enabled.
>
> @param[in] BootFirmwareVolumePtr The firmware volume pointer to search
> @@ -516,7 +516,7 @@ LaunchPeiCore (
> @param[in] FuncArg1 Arg1 to pass to next phase entry point address.
> @param[in] NextAddr Entry point of next phase.
> @param[in] NextMode Privilege mode of next phase.
> - @param[in] NextVirt Next phase is in virtualiztion.
> + @param[in] NextVirt Next phase is in virtualization.
>
> **/
> VOID
> @@ -600,7 +600,7 @@ Edk2PlatformHartIndex2Id (
> }
>
> /**
> - This function initilizes hart specific information and SBI.
> + This function initializes hart specific information and SBI.
> For the boot hart, it boots system through PEI core and initial SBI in the DXE IPL.
> For others, it goes to initial SBI and halt.
>
> @@ -658,9 +658,9 @@ SecCoreStartUpWithStack(
> HartFirmwareContext->HartSwitchMode = RiscVOpenSbiHartSwitchMode;
>
> //
> - // Hook platorm_ops with EDK2 one. Thus we can have interface
> + // Hook platform_ops with EDK2 one. Thus we can have interface
> // call out to OEM EDK2 platform code in M-mode before switching
> - // to S-mode in opensbo init.
> + // to S-mode in opensbi init.
> //
> ThisSbiPlatform = (struct sbi_platform *)sbi_platform_ptr(Scratch);
> ThisSbiPlatform->platform_ops_addr = (unsigned long)&Edk2OpensbiPlatformOps;
> diff --git a/RiscVPlatformPkg/Readme.md b/RiscVPlatformPkg/Readme.md
> index 66fba15544..5a344a864a 100644
> --- a/RiscVPlatformPkg/Readme.md
> +++ b/RiscVPlatformPkg/Readme.md
> @@ -26,16 +26,16 @@ differently from the default settings according to the OEM platform design.
> to align with OpenSBI project. As mentioned earlier, ***RiscVOpensbiLib*** provides the RISC-V SBI
> implementation and initialize the OpenSBI boot flow. SEC module is also linked with below libraries,
> - edk2 [OpenSbiPlatformLib](#OpenSbiPlatformLib-library) library that provides the generic RISC-V platform initialization code.
> -- edk2 [RiscVSpecifialPlatformLib](#RiscVSpecifialPlatformLib-library) library which is provided by the RISC-V
> +- edk2 [RiscVSpecialPlatformLib](#RiscVSpecialPlatformLib-library) library which is provided by the RISC-V
> platform vendor for the platform-specific initialization. The underlying implementation of above two edk2 libraries
> are from OpenSBI project. edk2 libraries are introduced as the wrapper libraries that separates and organizes OpenSBI core and platform code based on edk2 framework and the the build mechanism for edk2 RISC-V platforms. ***RiscVOpensbiLib*** library is located under [RISC-V ProcessorPkg](https://github.com/tianocore/edk2-platforms/tree/master/Silicon/RISC-V/ProcessorPkg) while the platform code (e.g. OpenSbiPlatformLib) is located under [RISC-V PlatformPkg](https://github.com/tianocore/edk2-platforms/tree/master/Platform/RISC-V/PlatformPkg).
> -- edk2 [RiscVSpecifialPlatformLib](#riscvspecifialplatformlib) library is provided by the platform vendor and located under edk2 RISC-V platform-specific folder.
> +- edk2 [RiscVSpecialPlatformLib](#riscvspecialplatformlib) library is provided by the platform vendor and located under edk2 RISC-V platform-specific folder.
>
> ##### OpenSbiPlatformLib Library
> [Indicated as #2 in the figure](#risc-v-edk2-port-design-diagrams)
> > ***OpenSbiPlatformLib*** provides the generic RISC-V platform initialization code. Platform vendor can just utilize this library if they don't have additional requirements on the platform initialization.
>
> -##### RiscVSpecifialPlatformLib Library
> +##### RiscVSpecialPlatformLib Library
> [Indicated as #3 in the figure](#risc-v-edk2-port-design-diagrams)
> > The major use case of this library is to facilitate the interfaces for platform vendors to provide the special
> platform initialization based on the generic platform initialization library.
> @@ -57,7 +57,7 @@ privilege according to the PCD.
>
> #### PEI Phase
> SEC module hands off the boot process to PEI core in the privilege configured by ***PcdPeiCorePrivilegeMode*** PCD *(TODO, currently the privilege is forced to S-mode)*. PEI and later phases are allowed to executed in M-mode
> -if the platform doesn't require Hypervisor-extended Supervisor mode (HS-mode) for the virtualization. RISC-V edk2 port provides its own instance ***PeiCoreEntryPoint*** library [(indicated as #7 in the figure)](#risc-v-edk2-port-design-diagrams) and linked with [PlatformSecPpiLib](#platformsecppilib-library) in order to support the S-mode PEI phase. PEI core requires [RiscVFirmwareContextLib](#riscVfirmwarecontextlib-library) library to retrieve the information of RISC-V HARTs and platform (e.g. FDT) configurations that built up in SEC phase. ***PeiServicePointer*** is also maintained in the ***RISC-V OpenSBI FirmwareContext*** structure and the pointer is retrieved by [PeiServiceTablePointerOpensbi](#peiservicetablepointeropensbi-library) library.
> +if the platform doesn't require Hypervisor-extended Supervisor mode (HS-mode) for the virtualization. RISC-V edk2 port provides its own instance ***PeiCoreEntryPoint*** library [(indicated as #7 in the figure)](#risc-v-edk2-port-design-diagrams) and linked with [PlatformSecPpiLib](#platformsecppilib-library) in order to support the S-mode PEI phase. PEI core requires [RiscVFirmwareContextLib](#riscvfirmwarecontextlib-library) library to retrieve the information of RISC-V HARTs and platform (e.g. FDT) configurations that built up in SEC phase. ***PeiServicePointer*** is also maintained in the ***RISC-V OpenSBI FirmwareContext*** structure and the pointer is retrieved by [PeiServiceTablePointerOpensbi](#peiservicetablepointeropensbi-library) library.
>
> ##### PlatformSecPpiLib Library
> [Indicated as #8 in the figure](#risc-v-edk2-port-design-diagrams)
> @@ -178,7 +178,7 @@ The PCD settings regard to EFI Variable
> |PcdVariableFdSize| The EFI variable firmware device size|
> |PcdVariableFdBlockSize| The block size of EFI variable firmware device|
> |PcdPlatformFlashNvStorageVariableBase| EFI variable base address within firmware device|
> -|PcdPlatformFlashNvStorageFtwWorkingBase| The base address of EFI variable fault tolerance worksapce (FTW) within firmware device|
> +|PcdPlatformFlashNvStorageFtwWorkingBase| The base address of EFI variable fault tolerance workspace (FTW) within firmware device|
> |PcdPlatformFlashNvStorageFtwSpareBase| The base address of EFI variable spare FTW within firmware device|
>
> ### RISC-V Physical Memory Protection (PMP) Region Settings
> @@ -190,7 +190,7 @@ Below PCDs could be set in platform FDF file.
> |PcdRootFirmwareDomainSize| The size of root firmware domain|-|-|
> |PcdFirmwareDomainBaseAddress| The starting address of firmware domain that can be accessed and executed in S-mode|Full access|Readable and Executable|
> |PcdFirmwareDomainSize| The size of firmware domain|-|-|
> -|PcdVariableFirmwareRegionBaseAddress| The starting address of EFI variable region that can be accessed in S-mode|Full access|Readale and Writable|
> +|PcdVariableFirmwareRegionBaseAddress| The starting address of EFI variable region that can be accessed in S-mode|Full access|Readable and Writable|
> |PcdVariableFirmwareRegionSize| The size of EFI variable firmware region|-|-|
>
> ### RISC-V Processor HART Settings
> @@ -198,7 +198,7 @@ Below PCDs could be set in platform FDF file.
> | **PCD name** |**Usage**|
> |--------------|---------|
> |PcdHartCount| Number of RISC-V HARTs, the value is processor-implementation specific|
> -|PcdBootHartId| The ID of RISC-V HART to execute main fimrware code and boot system to OS|
> +|PcdBootHartId| The ID of RISC-V HART to execute main firmware code and boot system to OS|
> |PcdBootableHartNumber|The bootable HART number, which is incorporate with RISC-V OpenSBI platform hart_index2id value|
> |PcdBootableHartIndexToId| if PcdBootableHartNumber == 0, hart_index2id is built from Device Tree, otherwise this is an array of HART index to HART ID|
>
> diff --git a/RiscVPlatformPkg/Universal/Sec/Riscv64/SecEntry.S b/RiscVPlatformPkg/Universal/Sec/Riscv64/SecEntry.S
> index 0fc7817665..b7b41e4c11 100644
> --- a/RiscVPlatformPkg/Universal/Sec/Riscv64/SecEntry.S
> +++ b/RiscVPlatformPkg/Universal/Sec/Riscv64/SecEntry.S
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2021 , Hewlett Packard Enterprise Development LP. All rights reserved.
> + * Copyright (c) 2021-2022 , Hewlett Packard Enterprise Development LP. All rights reserved.
> *
> * SPDX-License-Identifier: BSD-2-Clause
> *
> @@ -67,7 +67,7 @@ skip_fw_init:
> * DTB for this processor. We allocate the
> * scratch buffer according to this number.
> */
> - la a4, _pysical_hart_count
> + la a4, _physical_hart_count
> sd s7, (a4)
>
> li s8, FixedPcdGet32 (PcdOpenSbiStackSize)
> @@ -227,7 +227,7 @@ _start_warm:
> csrr a0, CSR_MHARTID
> j _uninitialized_hart_wait
> 4:
> - la a5, _pysical_hart_count
> + la a5, _physical_hart_count
> ld s7, (a5)
> /* Find the scratch space for this hart
> *
> @@ -294,7 +294,7 @@ _start_warm:
> .section .data, "aw"
> _boot_hart_done:
> RISCV_PTR 0
> -_pysical_hart_count:
> +_physical_hart_count:
> RISCV_PTR 0
>
> .align 3
> @@ -323,7 +323,7 @@ _hartid_to_scratch:
> lw s0, SBI_PLATFORM_HART_STACK_SIZE_OFFSET(s2)
> #endif
>
> - la s1, _pysical_hart_count /* total HART count */
> + la s1, _physical_hart_count /* total HART count */
> ld s2, (s1)
> mul s2, s2, s0
> li s1, FixedPcdGet32 (PcdScratchRamBase)
> --
> 2.31.1
>
next prev parent reply other threads:[~2022-01-09 14:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20220108041420.16064-1-abner.chang@hpe.com>
[not found] ` <20220108041420.16064-2-abner.chang@hpe.com>
2022-01-09 14:27 ` [PATCH 63/79] RiscVPkg: Rename ProcessorPkg to RiscVPkg Sunil V L
[not found] ` <20220108041420.16064-4-abner.chang@hpe.com>
2022-01-09 14:28 ` [PATCH 65/79] RiscVPkg/PlatformPei: Fix the build error Sunil V L
[not found] ` <20220108041420.16064-6-abner.chang@hpe.com>
2022-01-09 14:42 ` [PATCH 67/79] RiscVPlatformPkg: Address Core CI ECC errors Sunil V L
[not found] ` <20220108041420.16064-7-abner.chang@hpe.com>
2022-01-09 14:53 ` Sunil V L [this message]
[not found] ` <20220108041420.16064-12-abner.chang@hpe.com>
2022-01-09 15:32 ` [PATCH 73/79] RiscVPkg: " Sunil V L
[not found] ` <20220108041420.16064-14-abner.chang@hpe.com>
2022-01-09 15:39 ` [PATCH 75/79] RiscVPkg: Address Core CI Spelling errors Sunil V L
2022-01-08 7:27 [PATCH 62/79] RiscVPlatformPkg: Rename PlatformPkg to RiscVPlatformPkg Abner Chang
2022-01-08 7:27 ` [PATCH 68/79] RiscVPlatformVPkg: Address Core CI Spelling errors Abner Chang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220109145358.GD4879@sunil-ThinkPad-T490 \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox