public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Leif Lindholm" <leif@nuviainc.com>
To: Nhi Pham <nhi@os.amperecomputing.com>
Cc: devel@edk2.groups.io, Vu Nguyen <vunguyen@os.amperecomputing.com>,
	Thang Nguyen <thang@os.amperecomputing.com>,
	Chuong Tran <chuong@os.amperecomputing.com>,
	Phong Vo <phong@os.amperecomputing.com>,
	Michael D Kinney <michael.d.kinney@intel.com>,
	Ard Biesheuvel <ardb+tianocore@kernel.org>,
	Nate DeSimone <nathaniel.l.desimone@intel.com>
Subject: Re: [edk2-platforms][PATCH v2 01/32] Ampere: Initial support for Ampere Altra processor and Mt. Jade platform
Date: Sat, 5 Jun 2021 00:04:10 +0100	[thread overview]
Message-ID: <20210604230410.s7vuarxir2fqkqtw@leviathan> (raw)
In-Reply-To: <20210526100724.5359-2-nhi@os.amperecomputing.com>

On Wed, May 26, 2021 at 17:06:52 +0700, Nhi Pham wrote:
> From: Vu Nguyen <vunguyen@os.amperecomputing.com>
> 
> This commit adds the support for Ampere’s Altra processor-based Mt. Jade
> platform that provides up to 160 processor cores in a dual socket
> configuration. The essential modules are wired up enough to boot system
> to EDK2 UiApp.
> 
> Cc: Thang Nguyen <thang@os.amperecomputing.com>
> Cc: Chuong Tran <chuong@os.amperecomputing.com>
> Cc: Phong Vo <phong@os.amperecomputing.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> 
> Signed-off-by: Vu Nguyen <vunguyen@os.amperecomputing.com>
> ---
>  Platform/Ampere/AmperePlatformPkg/AmperePlatformPkg.dec                                         |  28 +
>  Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dec                                                |  42 ++
>  Silicon/Ampere/AmpereSiliconPkg/AmpereSiliconPkg.dec                                            |  46 ++
>  Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc                                            | 674 +++++++++++++++++++
>  Platform/Ampere/JadePkg/Jade.dsc                                                                | 100 +++
>  Platform/Ampere/JadePkg/Jade.fdf                                                                | 225 +++++++
>  Silicon/Ampere/AmpereAltraPkg/Drivers/ATFHobPei/ATFHobPeim.inf                                  |  41 ++
>  Silicon/Ampere/AmpereAltraPkg/Drivers/MemoryInitPeim/MemoryInitPeim.inf                         |  64 ++
>  Silicon/Ampere/AmpereAltraPkg/Library/AmpereCpuLib/AmpereCpuLib.inf                             |  44 ++
>  Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLib.inf                         |  57 ++
>  Silicon/Ampere/AmpereAltraPkg/Library/MailboxInterfaceLib/MailboxInterfaceLib.inf               |  37 +
>  Silicon/Ampere/AmpereAltraPkg/Library/MemoryInitPeiLib/MemoryInitPeiLib.inf                     |  63 ++
>  Silicon/Ampere/AmpereAltraPkg/Library/MmCommunicationLib/MmCommunicationLib.inf                 |  35 +
>  Silicon/Ampere/AmpereAltraPkg/Library/NVParamLib/NVParamLib.inf                                 |  32 +
>  Silicon/Ampere/AmpereAltraPkg/Library/PlatformPeiLib/PlatformPeiLib.inf                         |  42 ++
>  Silicon/Ampere/AmpereAltraPkg/Library/RngLib/RngLib.inf                                         |  29 +
>  Silicon/Ampere/AmpereAltraPkg/Library/SystemFirmwareInterfaceLib/SystemFirmwareInterfaceLib.inf |  30 +
>  Silicon/Ampere/AmpereAltraPkg/Library/TrngLib/TrngLib.inf                                       |  29 +
>  Silicon/Ampere/AmpereAltraPkg/Include/Guid/PlatformInfoHobGuid.h                                |  17 +
>  Silicon/Ampere/AmpereAltraPkg/Include/Library/AmpereCpuLib.h                                    | 282 ++++++++
>  Silicon/Ampere/AmpereAltraPkg/Include/Library/MailboxInterfaceLib.h                             | 172 +++++
>  Silicon/Ampere/AmpereAltraPkg/Include/Library/MmCommunicationLib.h                              |  19 +
>  Silicon/Ampere/AmpereAltraPkg/Include/Library/NVParamLib.h                                      | 133 ++++
>  Silicon/Ampere/AmpereAltraPkg/Include/Library/SystemFirmwareInterfaceLib.h                      | 282 ++++++++
>  Silicon/Ampere/AmpereAltraPkg/Include/Library/TrngLib.h                                         |  31 +
>  Silicon/Ampere/AmpereAltraPkg/Include/MmLib.h                                                   |  79 +++
>  Silicon/Ampere/AmpereAltraPkg/Include/NVParamDef.h                                              | 515 ++++++++++++++
>  Silicon/Ampere/AmpereAltraPkg/Include/Platform/Ac01.h                                           | 146 ++++
>  Silicon/Ampere/AmpereAltraPkg/Include/PlatformInfoHob.h                                         | 182 +++++
>  Silicon/Ampere/AmpereAltraPkg/Drivers/ATFHobPei/ATFHobPeim.c                                    |  52 ++
>  Silicon/Ampere/AmpereAltraPkg/Drivers/MemoryInitPeim/MemoryInitPeim.c                           | 151 +++++
>  Silicon/Ampere/AmpereAltraPkg/Library/AmpereCpuLib/AmpereCpuLib.c                               | 706 ++++++++++++++++++++
>  Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLib.c                           | 169 +++++
>  Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLibMemory.c                     | 399 +++++++++++
>  Silicon/Ampere/AmpereAltraPkg/Library/MailboxInterfaceLib/MailboxInterfaceLib.c                 | 282 ++++++++
>  Silicon/Ampere/AmpereAltraPkg/Library/MemoryInitPeiLib/MemoryInitPeiLib.c                       |  93 +++
>  Silicon/Ampere/AmpereAltraPkg/Library/MmCommunicationLib/MmCommunicationLib.c                   | 184 +++++
>  Silicon/Ampere/AmpereAltraPkg/Library/NVParamLib/NVParamLib.c                                   | 202 ++++++
>  Silicon/Ampere/AmpereAltraPkg/Library/PlatformPeiLib/PlatformPeiLib.c                           |  40 ++
>  Silicon/Ampere/AmpereAltraPkg/Library/RngLib/RngLib.c                                           | 141 ++++
>  Silicon/Ampere/AmpereAltraPkg/Library/SystemFirmwareInterfaceLib/SystemFirmwareInterfaceLib.c   | 328 +++++++++
>  Silicon/Ampere/AmpereAltraPkg/Library/TrngLib/TrngLib.c                                         |  63 ++
>  Platform/Ampere/AmperePlatformPkg/FvRules.fdf.inc                                               | 176 +++++
>  Platform/Ampere/JadePkg/JadeBoardSetting.cfg                                                    | 209 ++++++
>  Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformHelper.S                        |  45 ++
>  Silicon/Ampere/AmpereAltraPkg/Library/RngLib/RngLib.uni                                         |  13 +
>  46 files changed, 6729 insertions(+)
> 

> diff --git a/Platform/Ampere/JadePkg/Jade.dsc b/Platform/Ampere/JadePkg/Jade.dsc
> new file mode 100755
> index 000000000000..f68af24a0d78
> --- /dev/null
> +++ b/Platform/Ampere/JadePkg/Jade.dsc
> @@ -0,0 +1,100 @@
> +## @file
> +#
> +# Copyright (c) 2020 - 2021, Ampere Computing LLC. All rights reserved.<BR>
> +#
> +# SPDX-License-Identifier: BSD-2-Clause-Patent
> +#
> +##
> +
> +################################################################################
> +#
> +# Defines Section - statements that will be processed to create a Makefile.
> +#
> +################################################################################
> +[Defines]
> +  PLATFORM_NAME                  = Jade
> +  PLATFORM_GUID                  = 7BDD00C0-68F3-4CC1-8775-F0F00572019F
> +  PLATFORM_VERSION               = 0.1
> +  DSC_SPECIFICATION              = 0x0001001B
> +  OUTPUT_DIRECTORY               = Build/Jade
> +  SUPPORTED_ARCHITECTURES        = AARCH64
> +  BUILD_TARGETS                  = DEBUG|RELEASE|NOOPT
> +  SKUID_IDENTIFIER               = DEFAULT
> +  FLASH_DEFINITION               = Platform/Ampere/JadePkg/Jade.fdf
> +
> +  #
> +  # Defines for default states. These can be changed on the command line.
> +  # -D FLAG=VALUE
> +  #
> +
> +  #  DEBUG_INIT      0x00000001  // Initialization
> +  #  DEBUG_WARN      0x00000002  // Warnings
> +  #  DEBUG_LOAD      0x00000004  // Load events
> +  #  DEBUG_FS        0x00000008  // EFI File system
> +  #  DEBUG_POOL      0x00000010  // Alloc & Free (pool)
> +  #  DEBUG_PAGE      0x00000020  // Alloc & Free (page)
> +  #  DEBUG_INFO      0x00000040  // Informational debug messages
> +  #  DEBUG_DISPATCH  0x00000080  // PEI/DXE/SMM Dispatchers
> +  #  DEBUG_VARIABLE  0x00000100  // Variable
> +  #  DEBUG_BM        0x00000400  // Boot Manager
> +  #  DEBUG_BLKIO     0x00001000  // BlkIo Driver
> +  #  DEBUG_NET       0x00004000  // SNP Driver
> +  #  DEBUG_UNDI      0x00010000  // UNDI Driver
> +  #  DEBUG_LOADFILE  0x00020000  // LoadFile
> +  #  DEBUG_EVENT     0x00080000  // Event messages
> +  #  DEBUG_GCD       0x00100000  // Global Coherency Database changes
> +  #  DEBUG_CACHE     0x00200000  // Memory range cachability changes
> +  #  DEBUG_VERBOSE   0x00400000  // Detailed debug messages that may
> +  #                              // significantly impact boot performance
> +  #  DEBUG_ERROR     0x80000000  // Error
> +  DEFINE DEBUG_PRINT_ERROR_LEVEL = 0x8000000F
> +  DEFINE FIRMWARE_VER            = 0.01.001
> +  DEFINE EDK2_SKIP_PEICORE       = TRUE
> +  DEFINE SECURE_BOOT_ENABLE      = FALSE
> +  DEFINE INCLUDE_TFTP_COMMAND    = TRUE
> +
> +  #
> +  # Network definition
> +  #
> +  DEFINE NETWORK_IP6_ENABLE                  = FALSE
> +  DEFINE NETWORK_HTTP_BOOT_ENABLE            = TRUE
> +  DEFINE NETWORK_ALLOW_HTTP_CONNECTIONS      = TRUE
> +  DEFINE NETWORK_TLS_ENABLE                  = FALSE
> +

You'll want to include that !include MdePkg/MdeLibs.dsc.inc bit
somewhere here.

> +# Include default Ampere Platform DSC file
> +!include Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc
> +
> +################################################################################
> +#
> +# Specific Platform Library
> +#
> +################################################################################
> +[LibraryClasses]
> +  #
> +  # RTC Library: Common RTC
> +  #
> +  RealTimeClockLib|EmbeddedPkg/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.inf
> +
> +################################################################################
> +#
> +# Specific Platform Pcds
> +#
> +################################################################################
> +[PcdsFeatureFlag.common]
> +[PcdsFixedAtBuild.common]
> +
> +!if $(SECURE_BOOT_ENABLE) == TRUE
> +  # Override the default values from SecurityPkg to ensure images
> +  # from all sources are verified in secure boot
> +  gEfiSecurityPkgTokenSpaceGuid.PcdOptionRomImageVerificationPolicy|0x04
> +  gEfiSecurityPkgTokenSpaceGuid.PcdFixedMediaImageVerificationPolicy|0x04
> +  gEfiSecurityPkgTokenSpaceGuid.PcdRemovableMediaImageVerificationPolicy|0x04
> +!endif
> +
> +
> +################################################################################
> +#
> +# Specific Platform Component
> +#
> +################################################################################
> +[Components.common]



> diff --git a/Silicon/Ampere/AmpereAltraPkg/Include/Library/AmpereCpuLib.h b/Silicon/Ampere/AmpereAltraPkg/Include/Library/AmpereCpuLib.h
> new file mode 100644
> index 000000000000..de576474fb48
> --- /dev/null
> +++ b/Silicon/Ampere/AmpereAltraPkg/Include/Library/AmpereCpuLib.h
> @@ -0,0 +1,282 @@
> +/** @file
> +
> +  Copyright (c) 2020 - 2021, Ampere Computing LLC. All rights reserved.<BR>
> +
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#ifndef AMPERE_CPU_LIB_H_
> +#define AMPERE_CPU_LIB_H_
> +
> +/* Ctypen, bits[3(n - 1) + 2 : 3(n - 1)], for n = 1 to 7 */
> +#define CLIDR_CTYPE_SHIFT(Level)    (3 * (Level - 1))
> +#define CLIDR_CTYPE_MASK(Level)     (7 << CLIDR_CTYPE_SHIFT(Level))
> +#define CLIDR_CTYPE(Clidr, Level) \
> +  (((Clidr) & CLIDR_CTYPE_MASK(Level)) >> CLIDR_CTYPE_SHIFT(Level))
> +
> +#define CCSIDR_NUMSETS_SHIFT        13
> +#define CCSIDR_NUMSETS_MASK         0xFFFE000
> +#define CCSIDR_NUMSETS(Ccsidr) \
> +  (((Ccsidr) & CCSIDR_NUMSETS_MASK) >> CCSIDR_NUMSETS_SHIFT)
> +#define CCSIDR_ASSOCIATIVITY_SHIFT  3
> +#define CCSIDR_ASSOCIATIVITY_MASK   0x1FF8
> +#define CCSIDR_ASSOCIATIVITY(Ccsidr) \
> +  (((Ccsidr) & CCSIDR_ASSOCIATIVITY_MASK) >> CCSIDR_ASSOCIATIVITY_SHIFT)
> +#define CCSIDR_LINE_SIZE_SHIFT      0
> +#define CCSIDR_LINE_SIZE_MASK       0x7
> +#define CCSIDR_LINE_SIZE(Ccsidr) \
> +  (((Ccsidr) & CCSIDR_LINE_SIZE_MASK) >> CCSIDR_LINE_SIZE_SHIFT)

All of the above (CLIDR/CCSIDR) are architectural.
We've also developed some new accessors and stuff for these, as part
of ArmPkg/Universal/SmbiosDxe work that's gone on since v1 of this set.
We may need to clean some interfaces up, but ideally I'd prefer if you
could use some accessors from ArmLib or such.

> +
> +#define SUBNUMA_MODE_MONOLITHIC        0
> +#define SUBNUMA_MODE_HEMISPHERE        1
> +#define SUBNUMA_MODE_QUADRANT          2
> +
> +#define MONOLITIC_NUM_OF_REGION        1
> +#define HEMISPHERE_NUM_OF_REGION       2
> +#define QUADRANT_NUM_OF_REGION         4
> +#define SUBNUMA_CPM_REGION_SIZE        4
> +#define NUM_OF_CPM_PER_MESH_ROW        8
> +
> +#define CPM_PER_ROW_OFFSET(CpmId)      ((CpmId) % NUM_OF_CPM_PER_MESH_ROW)
> +#define CPM_ROW_NUMBER(CpmId)          ((CpmId) / NUM_OF_CPM_PER_MESH_ROW)
> +
> +#define SOCKET_ID(CpuId)               ((CpuId) / (PLATFORM_CPU_MAX_CPM * PLATFORM_CPU_NUM_CORES_PER_CPM))
> +#define CLUSTER_ID(CpuId)              (((CpuId) / PLATFORM_CPU_NUM_CORES_PER_CPM) % PLATFORM_CPU_MAX_CPM)
> +
> +
> +/**
> +  Get the SubNUMA mode.
> +
> +  @return   UINT8      The SubNUMA mode.
> +
> +**/
> +UINT8
> +EFIAPI
> +CpuGetSubNumaMode (
> +  VOID
> +  );
> +
> +/**
> +  Get the number of SubNUMA region.
> +
> +  @return   UINT8      The number of SubNUMA region.
> +
> +**/
> +UINT8
> +EFIAPI
> +CpuGetNumberOfSubNumaRegion (
> +  VOID
> +  );
> +
> +/**
> +  Get the SubNUMA node of a CPM.
> +
> +  @param    SocketId    Socket index.
> +  @param    Cpm         CPM index.
> +  @return   UINT8       The SubNUMA node of a CPM.
> +
> +**/
> +UINT8
> +EFIAPI
> +CpuGetSubNumNode (
> +  UINT8  Socket,
> +  UINT16 Cpm
> +  );
> +
> +/**
> +  Get the associativity of cache.
> +
> +  @param    Level       Cache level.
> +  @return   UINT32      Associativity of cache.
> +
> +**/
> +UINT32
> +EFIAPI
> +CpuGetAssociativity (
> +  UINT32 Level
> +  );
> +
> +/**
> +  Get the cache size.
> +
> +  @param    Level       Cache level.
> +  @return   UINT32      Cache size.
> +
> +**/
> +UINT32
> +EFIAPI
> +CpuGetCacheSize (
> +  UINT32 Level
> +  );
> +
> +/**
> +  Get the number of supported socket.
> +
> +  @return   UINT8      Number of supported socket.
> +
> +**/
> +UINT8
> +EFIAPI
> +GetNumberOfSupportedSockets (
> +  VOID
> +  );
> +
> +/**
> +  Get the number of active socket.
> +
> +  @return   UINT8      Number of active socket.
> +
> +**/
> +UINT8
> +EFIAPI
> +GetNumberOfActiveSockets (
> +  VOID
> +  );
> +
> +/**
> +  Get the number of active CPM per socket.
> +
> +  @param    SocketId    Socket index.
> +  @return   UINT16      Number of CPM.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfActiveCPMsPerSocket (
> +  UINT8 SocketId
> +  );
> +
> +/**
> +  Get the number of configured CPM per socket.
> +
> +  @param    SocketId    Socket index.
> +  @return   UINT16      Number of configured CPM.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfConfiguredCPMs (
> +  UINT8 SocketId
> +  );
> +
> +/**
> +  Set the number of configured CPM per socket.
> +
> +  @param    SocketId        Socket index.
> +  @param    NumberOfCPMs    Number of CPM to be configured.
> +  @return   EFI_SUCCESS     Operation succeeded.
> +  @return   Others          An error has occurred.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +SetNumberOfConfiguredCPMs (
> +  UINT8  SocketId,
> +  UINT16 NumberOfCPMs
> +  );
> +
> +/**
> +  Get the maximum number of core per socket. This number
> +  should be the same for all sockets.
> +
> +  @return   UINT16      Maximum number of core.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetMaximumNumberOfCores (
> +  VOID
> +  );
> +
> +/**
> +  Get the maximum number of CPM per socket. This number
> +  should be the same for all sockets.
> +
> +  @return   UINT32      Maximum number of CPM.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetMaximumNumberOfCPMs (
> +  VOID
> +  );
> +
> +/**
> +  Get the number of active cores of a sockets.
> +
> +  @return   UINT16      Number of active core.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfActiveCoresPerSocket (
> +  UINT8 SocketId
> +  );
> +
> +/**
> +  Get the number of active cores of all socket.
> +
> +  @return   UINT16      Number of active core.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfActiveCores (
> +  VOID
> +  );
> +
> +/**
> +  Check if the logical CPU is enabled or not.
> +
> +  @param    CpuId       The logical Cpu ID. Started from 0.
> +  @return   BOOLEAN     TRUE if the Cpu enabled
> +                        FALSE if the Cpu disabled.
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsCpuEnabled (
> +  UINT16 CpuId
> +  );
> +
> +
> +/**
> +  Check if the slave socket is present
> +
> +  @return   BOOLEAN     TRUE if the Slave Cpu is present
> +                        FALSE if the Slave Cpu is not present
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsSlaveSocketPresent (
> +  VOID
> +  );
> +
> +/**
> +  Check if the slave socket is active
> +
> +  @return   BOOLEAN     TRUE if the Slave CPU Socket is active.
> +                        FALSE if the Slave CPU Socket is not active.
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsSlaveSocketActive (
> +  VOID
> +  );
> +
> +/**
> +  Check if the CPU product ID is Ac01
> +  @return   BOOLEAN     TRUE if the Product ID is Ac01
> +                        FALSE otherwise.
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsAc01Processor (
> +  VOID
> +  );
> +
> +#endif /* AMPERE_CPU_LIB_H_ */


> diff --git a/Silicon/Ampere/AmpereAltraPkg/Library/AmpereCpuLib/AmpereCpuLib.c b/Silicon/Ampere/AmpereAltraPkg/Library/AmpereCpuLib/AmpereCpuLib.c
> new file mode 100644
> index 000000000000..8da698e0b855
> --- /dev/null
> +++ b/Silicon/Ampere/AmpereAltraPkg/Library/AmpereCpuLib/AmpereCpuLib.c
> @@ -0,0 +1,706 @@
> +/** @file
> +
> +  Copyright (c) 2020 - 2021, Ampere Computing LLC. All rights reserved.<BR>
> +
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <PiPei.h>
> +#include <Uefi.h>
> +
> +#include <Guid/PlatformInfoHobGuid.h>
> +#include <Library/AmpereCpuLib.h>
> +#include <Library/ArmLib/ArmLibPrivate.h>
> +#include <Library/BaseMemoryLib.h>
> +#include <Library/DebugLib.h>
> +#include <Library/HobLib.h>
> +#include <Library/IoLib.h>
> +#include <Library/NVParamLib.h>
> +#include <NVParamDef.h>
> +#include <Platform/Ac01.h>
> +#include <PlatformInfoHob.h>
> +
> +STATIC PLATFORM_INFO_HOB *mPlatformInfoHob = NULL;
> +
> +PLATFORM_INFO_HOB *
> +GetPlatformHob (
> +  VOID
> +  )
> +{
> +  VOID *Hob;
> +
> +  if (mPlatformInfoHob == NULL) {
> +    Hob = GetNextGuidHob (
> +            &gPlatformHobGuid,
> +            (CONST VOID *)FixedPcdGet64 (PcdSystemMemoryBase)
> +            );
> +    ASSERT (Hob != NULL);
> +    if (Hob == NULL) {
> +      DEBUG ((DEBUG_ERROR, "%a: Failed to get gPlatformHobGuid!\n", __FUNCTION__));
> +      return NULL;
> +    }
> +
> +    mPlatformInfoHob = (PLATFORM_INFO_HOB *)GET_GUID_HOB_DATA (Hob);
> +  }
> +
> +  return mPlatformInfoHob;
> +}
> +
> +/**
> +  Get the SubNUMA mode.
> +
> +  @return   UINT8      The SubNUMA mode.
> +
> +**/
> +UINT8
> +EFIAPI
> +CpuGetSubNumaMode (
> +  VOID
> +  )
> +{
> +  PLATFORM_INFO_HOB  *PlatformHob;
> +
> +  PlatformHob = GetPlatformHob ();
> +  if (PlatformHob == NULL) {
> +    return SUBNUMA_MODE_MONOLITHIC;
> +  }
> +
> +  return PlatformHob->SubNumaMode[0];
> +}
> +
> +/**
> +  Get the number of SubNUMA region.
> +
> +  @return   UINT8      The number of SubNUMA region.
> +
> +**/
> +UINT8
> +EFIAPI
> +CpuGetNumberOfSubNumaRegion (
> +  VOID
> +  )
> +{
> +  UINT8 SubNumaMode;
> +  UINT8 NumberOfSubNumaRegion;
> +
> +  SubNumaMode = CpuGetSubNumaMode ();
> +  ASSERT (SubNumaMode <= SUBNUMA_MODE_QUADRANT);
> +
> +  switch (SubNumaMode) {
> +  case SUBNUMA_MODE_MONOLITHIC:
> +    NumberOfSubNumaRegion = MONOLITIC_NUM_OF_REGION;
> +    break;
> +
> +  case SUBNUMA_MODE_HEMISPHERE:
> +    NumberOfSubNumaRegion = HEMISPHERE_NUM_OF_REGION;
> +    break;
> +
> +  case SUBNUMA_MODE_QUADRANT:
> +    NumberOfSubNumaRegion = QUADRANT_NUM_OF_REGION;
> +    break;
> +
> +  default:
> +    // Should never reach there.
> +    ASSERT (FALSE);
> +    break;
> +  }
> +
> +  return NumberOfSubNumaRegion;
> +}
> +
> +/**
> +  Get the SubNUMA node of a CPM.
> +
> +  @param    SocketId    Socket index.
> +  @param    Cpm         CPM index.
> +  @return   UINT8       The SubNUMA node of a CPM.
> +
> +**/
> +UINT8
> +EFIAPI
> +CpuGetSubNumNode (
> +  UINT8  SocketId,
> +  UINT16 Cpm
> +  )
> +{
> +  BOOLEAN IsAsymMesh;
> +  UINT8   SubNumaNode;
> +  UINT16  MaxNumberOfCPM;
> +  UINT8   MiddleRow;
> +  UINT8   QuadrantHigherRowNodeNumber[NUM_OF_CPM_PER_MESH_ROW] = {1, 1, 1, 1, 3, 3, 3, 3};
> +  UINT8   QuadrantLowerRowNodeNumber[NUM_OF_CPM_PER_MESH_ROW]  = {0, 0, 0, 0, 2, 2, 2, 2};
> +  UINT8   QuadrantMiddleRowNodeNumber[NUM_OF_CPM_PER_MESH_ROW] = {0, 0, 1, 1, 3, 3, 2, 2};
> +  UINT8   SubNumaMode;
> +
> +  MaxNumberOfCPM = GetMaximumNumberOfCPMs ();
> +  SubNumaMode = CpuGetSubNumaMode ();
> +  ASSERT (SubNumaMode <= SUBNUMA_MODE_QUADRANT);
> +
> +  switch (SubNumaMode) {
> +  case SUBNUMA_MODE_MONOLITHIC:
> +    SubNumaNode = (SocketId == 0) ? 0 : 1;
> +    break;
> +
> +  case SUBNUMA_MODE_HEMISPHERE:
> +    if (CPM_PER_ROW_OFFSET (Cpm) >= SUBNUMA_CPM_REGION_SIZE) {
> +      SubNumaNode = 1;
> +    } else {
> +      SubNumaNode = 0;
> +    }
> +
> +    if (SocketId == 1) {
> +      SubNumaNode += HEMISPHERE_NUM_OF_REGION;
> +    }
> +    break;
> +
> +  case SUBNUMA_MODE_QUADRANT:
> +    //
> +    // CPM Mesh Rows
> +    //
> +    // |---------------------------------------|
> +    // | 00 ----------- 03 | 04 ----------- 07 | Row 0
> +    // |-------------------|-------------------|
> +    // | 08 ----------- 11 | 12 ----------- 15 | Row 1
> +    // |-------------------|-------------------|
> +    // | 16 - 17 | 18 - 19 | 20 - 21 | 22 - 23 | Middle Row
> +    // |-------------------|-------------------|
> +    // | 24 ----------- 27 | 28 ----------- 31 | Row 3
> +    // |-------------------|-------------------|
> +    // | 32 ----------- 35 | 36 ----------- 39 | Row 4
> +    // |---------------------------------------|
> +    //
> +
> +    IsAsymMesh = (BOOLEAN)(CPM_ROW_NUMBER (MaxNumberOfCPM) % 2 != 0);
> +    MiddleRow = CPM_ROW_NUMBER (MaxNumberOfCPM) / 2;
> +    if (IsAsymMesh
> +        && CPM_ROW_NUMBER (Cpm) == MiddleRow)
> +    {
> +      SubNumaNode = QuadrantMiddleRowNodeNumber[CPM_PER_ROW_OFFSET (Cpm)];
> +
> +    } else if (CPM_ROW_NUMBER (Cpm) >= MiddleRow) {
> +      SubNumaNode = QuadrantHigherRowNodeNumber[CPM_PER_ROW_OFFSET (Cpm)];
> +
> +    } else {
> +      SubNumaNode = QuadrantLowerRowNodeNumber[CPM_PER_ROW_OFFSET (Cpm)];
> +    }
> +
> +    if (SocketId == 1) {
> +      SubNumaNode += QUADRANT_NUM_OF_REGION;
> +    }
> +    break;
> +
> +  default:
> +    // Should never reach there.
> +    ASSERT (FALSE);
> +    break;
> +  }
> +
> +  return SubNumaNode;
> +}
> +
> +/**
> +  Get the associativity of cache.
> +
> +  @param    Level       Cache level.
> +  @return   UINT32      Associativity of cache.
> +
> +**/
> +UINT32
> +EFIAPI
> +CpuGetAssociativity (
> +  UINT32 Level
> +  )
> +{

I do feel some of this stuff could be replaced by the common functions
we now have in ArmLib (that we didn't when v1 went out).

> +  UINT64 CacheCCSIDR;
> +  UINT64 CacheCLIDR;
> +  UINT32 Value = 0x2; /* Unknown Set-Associativity */
> +
> +  CacheCLIDR = ReadCLIDR ();
> +  if (!CLIDR_CTYPE (CacheCLIDR, Level)) {
> +    return Value;
> +  }
> +
> +  CacheCCSIDR = ReadCCSIDR (Level);
> +  switch (CCSIDR_ASSOCIATIVITY (CacheCCSIDR)) {
> +  case 0:
> +    /* Direct mapped */
> +    Value = 0x3;
> +    break;
> +
> +  case 1:
> +    /* 2-way Set-Associativity */
> +    Value = 0x4;
> +    break;
> +
> +  case 3:
> +    /* 4-way Set-Associativity */
> +    Value = 0x5;
> +    break;
> +
> +  case 7:
> +    /* 8-way Set-Associativity */
> +    Value = 0x7;
> +    break;
> +
> +  case 15:
> +    /* 16-way Set-Associativity */
> +    Value = 0x8;
> +    break;
> +
> +  case 11:
> +    /* 12-way Set-Associativity */
> +    Value = 0x9;
> +    break;
> +
> +  case 23:
> +    /* 24-way Set-Associativity */
> +    Value = 0xA;
> +    break;
> +
> +  case 31:
> +    /* 32-way Set-Associativity */
> +    Value = 0xB;
> +    break;
> +
> +  case 47:
> +    /* 48-way Set-Associativity */
> +    Value = 0xC;
> +    break;
> +
> +  case 63:
> +    /* 64-way Set-Associativity */
> +    Value = 0xD;
> +    break;
> +
> +  case 19:
> +    /* 20-way Set-Associativity */
> +    Value = 0xE;
> +    break;
> +  }
> +
> +  return Value;
> +}
> +
> +/**
> +  Get the cache size.
> +
> +  @param    Level       Cache level.
> +  @return   UINT32      Cache size.
> +
> +**/
> +UINT32
> +EFIAPI
> +CpuGetCacheSize (
> +  UINT32 Level
> +  )
> +{
> +  UINT32 CacheLineSize;
> +  UINT32 Count;
> +  UINT64 CacheCCSIDR;
> +  UINT64 CacheCLIDR;
> +
> +  CacheCLIDR = ReadCLIDR ();
> +  if (!CLIDR_CTYPE (CacheCLIDR, Level)) {
> +    return 0;
> +  }
> +
> +  CacheCCSIDR = ReadCCSIDR (Level);
> +  CacheLineSize = 1;
> +  Count = CCSIDR_LINE_SIZE (CacheCCSIDR) + 4;
> +  while (Count-- > 0) {
> +    CacheLineSize *= 2;
> +  }
> +
> +  return ((CCSIDR_NUMSETS (CacheCCSIDR) + 1) *
> +          (CCSIDR_ASSOCIATIVITY (CacheCCSIDR) + 1) *
> +          CacheLineSize);
> +}
> +
> +/**
> +  Get the number of supported socket.
> +
> +  @return   UINT8      Number of supported socket.
> +
> +**/
> +UINT8
> +EFIAPI
> +GetNumberOfSupportedSockets (
> +  VOID
> +  )
> +{
> +  PLATFORM_INFO_HOB  *PlatformHob;
> +
> +  PlatformHob = GetPlatformHob ();
> +  if (PlatformHob == NULL) {
> +    //
> +    // By default, the number of supported sockets is 1.
> +    //
> +    return 1;
> +  }
> +
> +  return (sizeof (PlatformHob->ClusterEn) / sizeof (PLATFORM_CLUSTER_EN));
> +}
> +
> +/**
> +  Get the number of active socket.
> +
> +  @return   UINT8      Number of active socket.
> +
> +**/
> +UINT8
> +EFIAPI
> +GetNumberOfActiveSockets (
> +  VOID
> +  )
> +{
> +  UINT8               NumberOfActiveSockets, Count, Index, Index1;
> +  PLATFORM_CLUSTER_EN *Socket;
> +  PLATFORM_INFO_HOB   *PlatformHob;
> +
> +  PlatformHob = GetPlatformHob ();
> +  if (PlatformHob == NULL) {
> +    //
> +    // By default, the number of active sockets is 1.
> +    //
> +    return 1;
> +  }
> +
> +  NumberOfActiveSockets = 0;
> +
> +  for (Index = 0; Index < GetNumberOfSupportedSockets (); Index++) {
> +    Socket = &PlatformHob->ClusterEn[Index];
> +    Count = ARRAY_SIZE (Socket->EnableMask);
> +    for (Index1 = 0; Index1 < Count; Index1++) {
> +      if (Socket->EnableMask[Index1] != 0) {
> +        NumberOfActiveSockets++;
> +        break;
> +      }
> +    }
> +  }
> +
> +  return NumberOfActiveSockets;
> +}
> +
> +/**
> +  Get the number of active CPM per socket.
> +
> +  @param    SocketId    Socket index.
> +  @return   UINT16      Number of CPM.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfActiveCPMsPerSocket (
> +  UINT8 SocketId
> +  )
> +{
> +  UINT16              NumberOfCPMs, Count, Index;
> +  UINT32              Val32;
> +  PLATFORM_CLUSTER_EN *Socket;
> +  PLATFORM_INFO_HOB   *PlatformHob;
> +
> +  PlatformHob = GetPlatformHob ();
> +  if (PlatformHob == NULL) {
> +    return 0;
> +  }
> +
> +  if (SocketId >= GetNumberOfActiveSockets ()) {
> +    return 0;
> +  }
> +
> +  NumberOfCPMs = 0;
> +  Socket = &PlatformHob->ClusterEn[SocketId];
> +  Count = ARRAY_SIZE (Socket->EnableMask);
> +  for (Index = 0; Index < Count; Index++) {
> +    Val32 = Socket->EnableMask[Index];
> +    while (Val32 > 0) {
> +      if ((Val32 & 0x1) != 0) {
> +        NumberOfCPMs++;
> +      }
> +      Val32 >>= 1;
> +    }
> +  }
> +
> +  return NumberOfCPMs;
> +}
> +
> +/**
> +  Get the number of configured CPM per socket. This number
> +  should be the same for all sockets.
> +
> +  @param    SocketId    Socket index.
> +  @return   UINT8       Number of configured CPM.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfConfiguredCPMs (
> +  UINT8 SocketId
> +  )
> +{
> +  EFI_STATUS Status;
> +  UINT32     Value;
> +  UINT32     Param, ParamStart, ParamEnd;
> +  UINT16     Count;
> +
> +  Count = 0;
> +  ParamStart = NV_SI_S0_PCP_ACTIVECPM_0_31 + SocketId * NV_PARAM_ENTRYSIZE * (PLATFORM_CPU_MAX_CPM / 32);
> +  ParamEnd = ParamStart + NV_PARAM_ENTRYSIZE * (PLATFORM_CPU_MAX_CPM / 32);
> +  for (Param = ParamStart; Param < ParamEnd; Param += NV_PARAM_ENTRYSIZE) {
> +    Status = NVParamGet (
> +               Param,
> +               NV_PERM_ATF | NV_PERM_BIOS | NV_PERM_MANU | NV_PERM_BMC,
> +               &Value
> +               );
> +    if (EFI_ERROR (Status)) {
> +      break;
> +    }
> +    while (Value != 0) {
> +      if ((Value & 0x01) != 0) {
> +        Count++;
> +      }
> +      Value >>= 1;
> +    }
> +  }
> +
> +  return Count;
> +}
> +
> +/**
> +  Set the number of configured CPM per socket.
> +
> +  @param    SocketId        Socket index.
> +  @param    NumberOfCPMs    Number of CPM to be configured.
> +  @return   EFI_SUCCESS     Operation succeeded.
> +  @return   Others          An error has occurred.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +SetNumberOfConfiguredCPMs (
> +  UINT8  SocketId,
> +  UINT16 NumberOfCPMs
> +  )
> +{
> +  EFI_STATUS Status;
> +  UINT32     Value;
> +  UINT32     Param, ParamStart, ParamEnd;
> +  BOOLEAN    IsClear;
> +
> +  IsClear = FALSE;
> +  if (NumberOfCPMs == 0) {
> +    IsClear = TRUE;
> +  }
> +
> +  Status = EFI_SUCCESS;
> +
> +  ParamStart = NV_SI_S0_PCP_ACTIVECPM_0_31 + SocketId * NV_PARAM_ENTRYSIZE * (PLATFORM_CPU_MAX_CPM / 32);
> +  ParamEnd = ParamStart + NV_PARAM_ENTRYSIZE * (PLATFORM_CPU_MAX_CPM / 32);
> +  for (Param = ParamStart; Param < ParamEnd; Param += NV_PARAM_ENTRYSIZE) {
> +    if (NumberOfCPMs >= 32) {
> +      Value = 0xffffffff;
> +      NumberOfCPMs -= 32;
> +    } else {
> +      Value = 0;
> +      while (NumberOfCPMs > 0) {
> +        Value |= (1 << (--NumberOfCPMs));
> +      }
> +    }
> +    if (IsClear) {
> +      /* Clear this param */
> +      Status = NVParamClr (
> +                 Param,
> +                 NV_PERM_BIOS | NV_PERM_MANU
> +                 );
> +    } else {
> +      Status = NVParamSet (
> +                 Param,
> +                 NV_PERM_ATF | NV_PERM_BIOS | NV_PERM_MANU | NV_PERM_BMC,
> +                 NV_PERM_BIOS | NV_PERM_MANU,
> +                 Value
> +                 );
> +    }
> +  }
> +
> +  return Status;
> +}
> +
> +/**
> +  Get the maximum number of core per socket.
> +
> +  @return   UINT16      Maximum number of core.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetMaximumNumberOfCores (
> +  VOID
> +  )
> +{
> +  PLATFORM_INFO_HOB  *PlatformHob;
> +
> +  PlatformHob = GetPlatformHob ();
> +  if (PlatformHob == NULL) {
> +    return 0;
> +  }
> +
> +  return PlatformHob->MaxNumOfCore[0];
> +}
> +
> +/**
> +  Get the maximum number of CPM per socket. This number
> +  should be the same for all sockets.
> +
> +  @return   UINT16      Maximum number of CPM.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetMaximumNumberOfCPMs (
> +  VOID
> +  )
> +{
> +  return GetMaximumNumberOfCores () / PLATFORM_CPU_NUM_CORES_PER_CPM;
> +}
> +
> +/**
> +  Get the number of active cores of a sockets.
> +
> +  @param    SocketId    Socket Index.
> +  @return   UINT16      Number of active core.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfActiveCoresPerSocket (
> +  UINT8 SocketId
> +  )
> +{
> +  return GetNumberOfActiveCPMsPerSocket (SocketId) * PLATFORM_CPU_NUM_CORES_PER_CPM;
> +}
> +
> +/**
> +  Get the number of active cores of all sockets.
> +
> +  @return   UINT16      Number of active core.
> +
> +**/
> +UINT16
> +EFIAPI
> +GetNumberOfActiveCores (
> +  VOID
> +  )
> +{
> +  UINT16 NumberOfActiveCores;
> +  UINT8  Index;
> +
> +  NumberOfActiveCores = 0;
> +
> +  for (Index = 0; Index < GetNumberOfActiveSockets (); Index++) {
> +    NumberOfActiveCores += GetNumberOfActiveCoresPerSocket (Index);
> +  }
> +
> +  return NumberOfActiveCores;
> +}
> +
> +/**
> +  Check if the logical CPU is enabled or not.
> +
> +  @param    CpuId       The logical Cpu ID. Started from 0.
> +  @return   BOOLEAN     TRUE if the Cpu enabled
> +                        FALSE if the Cpu disabled.
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsCpuEnabled (
> +  UINT16 CpuId
> +  )
> +{
> +  PLATFORM_CLUSTER_EN *Socket;
> +  PLATFORM_INFO_HOB   *PlatformHob;
> +  UINT8               SocketId;
> +  UINT16              ClusterId;
> +
> +  SocketId = SOCKET_ID (CpuId);
> +  ClusterId = CLUSTER_ID (CpuId);
> +
> +  PlatformHob = GetPlatformHob ();
> +  if (PlatformHob == NULL) {
> +    return FALSE;
> +  }
> +
> +  if (SocketId >= GetNumberOfActiveSockets ()) {
> +    return FALSE;
> +  }
> +
> +  Socket = &PlatformHob->ClusterEn[SocketId];
> +  if ((Socket->EnableMask[ClusterId / 32] & (1 << (ClusterId % 32))) != 0) {
> +    return TRUE;
> +  }
> +
> +  return FALSE;
> +}
> +
> +/**
> +  Check if the slave socket is present
> +
> +  @return   BOOLEAN     TRUE if the Slave Cpu is present
> +                        FALSE if the Slave Cpu is not present
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsSlaveSocketPresent (
> +  VOID
> +  )
> +{
> +  UINT32 Value;
> +
> +  Value = MmioRead32 (SMPRO_EFUSE_SHADOW0 + CFG2P_OFFSET);
> +
> +  return ((Value & SLAVE_PRESENT_N) != 0) ? FALSE : TRUE;
> +}
> +
> +/**
> +  Check if the slave socket is active
> +
> +  @return   BOOLEAN     TRUE if the Slave CPU Socket is active.
> +                        FALSE if the Slave CPU Socket is not active.
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsSlaveSocketActive (
> +  VOID
> +  )
> +{
> +  return (GetNumberOfActiveSockets () > 1) ? TRUE : FALSE;
> +}
> +
> +/**
> +  Check if the CPU product ID is Ac01
> +  @return   BOOLEAN     TRUE if the Product ID is Ac01
> +                        FALSE otherwise.
> +
> +**/
> +BOOLEAN
> +EFIAPI
> +IsAc01Processor (
> +  VOID
> +  )
> +{
> +  PLATFORM_INFO_HOB  *PlatformHob;
> +
> +  PlatformHob = GetPlatformHob ();
> +  ASSERT (PlatformHob != NULL);
> +
> +  if (PlatformHob != NULL) {
> +    if ((PlatformHob->ScuProductId[0] & 0xFF) == 0x01) {
> +      return TRUE;
> +    }
> +  }
> +
> +  return FALSE;
> +}



> diff --git a/Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLib.c b/Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLib.c
> new file mode 100644
> index 000000000000..8c1eb93f00fd
> --- /dev/null
> +++ b/Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLib.c
> @@ -0,0 +1,169 @@
> +/** @file
> +
> +  Copyright (c) 2020 - 2021, Ampere Computing LLC. All rights reserved.<BR>
> +
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <Uefi.h>
> +
> +#include <Guid/PlatformInfoHobGuid.h>
> +#include <Library/AmpereCpuLib.h>
> +#include <Library/ArmLib.h>
> +#include <Library/ArmPlatformLib.h>
> +#include <Library/BaseMemoryLib.h>
> +#include <Library/DebugLib.h>
> +#include <Library/HobLib.h>
> +#include <Library/IoLib.h>
> +#include <Library/MemoryAllocationLib.h>
> +#include <Library/PcdLib.h>
> +#include <Library/PL011UartLib.h>
> +#include <Library/SerialPortLib.h>
> +#include <Platform/Ac01.h>
> +#include <PlatformInfoHob.h>
> +#include <Ppi/ArmMpCoreInfo.h>
> +#include <Ppi/TemporaryRamSupport.h>
> +
> +ARM_CORE_INFO mArmPlatformMpCoreInfoTable[PLATFORM_CPU_MAX_NUM_CORES];
> +
> +/**
> +  Return the current Boot Mode
> +
> +  This function returns the boot reason on the platform
> +
> +  @return   Return the current Boot Mode of the platform
> +
> +**/
> +EFI_BOOT_MODE
> +ArmPlatformGetBootMode (
> +  VOID
> +  )
> +{
> +  return BOOT_WITH_FULL_CONFIGURATION;
> +}
> +
> +/**
> +  Initialize controllers that must setup in the normal world
> +
> +  This function is called by the ArmPlatformPkg/PrePi or ArmPlatformPkg/PlatformPei
> +  in the PEI phase.
> +
> +**/
> +EFI_STATUS
> +ArmPlatformInitialize (
> +  IN UINTN MpId
> +  )
> +{
> +  RETURN_STATUS      Status;
> +  UINT64             BaudRate;
> +  UINT32             ReceiveFifoDepth;
> +  EFI_PARITY_TYPE    Parity;
> +  UINT8              DataBits;
> +  EFI_STOP_BITS_TYPE StopBits;
> +
> +  Status = EFI_SUCCESS;
> +
> +  if (FixedPcdGet64 (PcdSerialRegisterBase) != 0) {
> +    /* Debug port should use the same parameters with console */
> +    BaudRate = FixedPcdGet64 (PcdUartDefaultBaudRate);
> +    ReceiveFifoDepth = FixedPcdGet32 (PcdUartDefaultReceiveFifoDepth);
> +    Parity = (EFI_PARITY_TYPE)FixedPcdGet8 (PcdUartDefaultParity);
> +    DataBits = FixedPcdGet8 (PcdUartDefaultDataBits);
> +    StopBits = (EFI_STOP_BITS_TYPE)FixedPcdGet8 (PcdUartDefaultStopBits);
> +
> +    /* Initialize uart debug port */
> +    Status = PL011UartInitializePort (
> +               (UINTN)FixedPcdGet64 (PcdSerialRegisterBase),
> +               FixedPcdGet32 (PL011UartClkInHz),
> +               &BaudRate,
> +               &ReceiveFifoDepth,
> +               &Parity,
> +               &DataBits,
> +               &StopBits
> +               );
> +  }
> +
> +  return Status;
> +}
> +
> +EFI_STATUS
> +PrePeiCoreGetMpCoreInfo (
> +  OUT UINTN         *CoreCount,
> +  OUT ARM_CORE_INFO **ArmCoreTable
> +  )
> +{
> +  UINTN              mArmPlatformCoreCount;
> +  UINTN              ClusterId;
> +  UINTN              SocketId;
> +  UINTN              Index;
> +
> +  ASSERT (CoreCount != NULL);
> +  ASSERT (ArmCoreTable != NULL);
> +  ASSERT (*ArmCoreTable != NULL);
> +
> +  mArmPlatformCoreCount = 0;
> +  for  (Index = 0; Index < PLATFORM_CPU_MAX_NUM_CORES; Index++) {
> +    if (!IsCpuEnabled (Index)) {
> +      continue;
> +    }
> +    SocketId = SOCKET_ID (Index);
> +    ClusterId = CLUSTER_ID (Index);
> +    mArmPlatformMpCoreInfoTable[mArmPlatformCoreCount].ClusterId = SocketId;
> +    mArmPlatformMpCoreInfoTable[mArmPlatformCoreCount].CoreId =
> +      (ClusterId << 8) | (Index % PLATFORM_CPU_NUM_CORES_PER_CPM);
> +    mArmPlatformMpCoreInfoTable[mArmPlatformCoreCount].MailboxClearAddress = 0;
> +    mArmPlatformMpCoreInfoTable[mArmPlatformCoreCount].MailboxClearValue = 0;
> +    mArmPlatformMpCoreInfoTable[mArmPlatformCoreCount].MailboxGetAddress = 0;
> +    mArmPlatformMpCoreInfoTable[mArmPlatformCoreCount].MailboxSetAddress = 0;
> +    mArmPlatformCoreCount++;
> +  }
> +
> +  *CoreCount = mArmPlatformCoreCount;
> +
> +  *ArmCoreTable = mArmPlatformMpCoreInfoTable;
> +  ASSERT (*ArmCoreTable != NULL);
> +
> +  return EFI_SUCCESS;
> +}
> +
> +// Needs to be declared in the file. Otherwise gArmMpCoreInfoPpiGuid is undefined in the contect of PrePeiCore
> +EFI_GUID             mArmMpCoreInfoPpiGuid = ARM_MP_CORE_INFO_PPI_GUID;
> +ARM_MP_CORE_INFO_PPI mMpCoreInfoPpi = { PrePeiCoreGetMpCoreInfo };
> +
> +EFI_PEI_PPI_DESCRIPTOR gPlatformPpiTable[] = {
> +  {
> +    EFI_PEI_PPI_DESCRIPTOR_PPI,
> +    &mArmMpCoreInfoPpiGuid,
> +    &mMpCoreInfoPpi
> +  },
> +};
> +
> +/**
> +  Return the Platform specific PPIs
> +
> +  This function exposes the Platform Specific PPIs. They can be used by any PrePi modules or passed
> +  to the PeiCore by PrePeiCore.
> +
> +  @param[out]   PpiListSize         Size in Bytes of the Platform PPI List
> +  @param[out]   PpiList             Platform PPI List
> +
> +**/
> +VOID
> +ArmPlatformGetPlatformPpiList (
> +  OUT UINTN                  *PpiListSize,
> +  OUT EFI_PEI_PPI_DESCRIPTOR **PpiList
> +  )
> +{
> +  ASSERT (PpiListSize != NULL);
> +  ASSERT (PpiList != NULL);
> +  ASSERT (*PpiList != NULL);
> +
> +  if (ArmIsMpCore ()) {
> +    *PpiListSize = sizeof (gPlatformPpiTable);
> +    *PpiList = gPlatformPpiTable;
> +  } else {
> +    *PpiListSize = 0;
> +    *PpiList = NULL;
> +  }
> +}
> diff --git a/Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLibMemory.c b/Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLibMemory.c
> new file mode 100644
> index 000000000000..117c9cc56ac2
> --- /dev/null
> +++ b/Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLibMemory.c
> @@ -0,0 +1,399 @@
> +/** @file
> +
> +  Copyright (c) 2020 - 2021, Ampere Computing LLC. All rights reserved.<BR>
> +
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <Uefi.h>
> +
> +#include <Guid/PlatformInfoHobGuid.h>
> +#include <Library/AmpereCpuLib.h>
> +#include <Library/ArmPlatformLib.h>
> +#include <Library/DebugLib.h>
> +#include <Library/HobLib.h>
> +#include <Library/MemoryAllocationLib.h>
> +#include <Library/PcdLib.h>
> +#include <PlatformInfoHob.h>
> +
> +/* Number of Virtual Memory Map Descriptors */
> +#define MAX_VIRTUAL_MEMORY_MAP_DESCRIPTORS          50
> +
> +/* DDR attributes */
> +#define DDR_ATTRIBUTES_CACHED           ARM_MEMORY_REGION_ATTRIBUTE_WRITE_BACK
> +#define DDR_ATTRIBUTES_UNCACHED         ARM_MEMORY_REGION_ATTRIBUTE_UNCACHED_UNBUFFERED
> +
> +/**
> +  Return the Virtual Memory Map of your platform
> +
> +  This Virtual Memory Map is used by MemoryInitPei Module to initialize the MMU on your platform.
> +
> +  @param[out]   VirtualMemoryMap    Array of ARM_MEMORY_REGION_DESCRIPTOR describing a Physical-to-
> +                                    Virtual Memory mapping. This array must be ended by a zero-filled
> +                                    entry
> +
> +**/
> +VOID
> +ArmPlatformGetVirtualMemoryMap (
> +  OUT ARM_MEMORY_REGION_DESCRIPTOR **VirtualMemoryMap
> +  )
> +{
> +  UINTN                        Index = 0;
> +  ARM_MEMORY_REGION_DESCRIPTOR *VirtualMemoryTable;
> +  UINT32                       NumRegion;
> +  UINTN                        Count;
> +  VOID                         *Hob;
> +  PLATFORM_INFO_HOB            *PlatformHob;
> +
> +  Hob = GetFirstGuidHob (&gPlatformHobGuid);
> +  ASSERT (Hob != NULL);
> +  if (Hob == NULL) {
> +    return;
> +  }
> +
> +  PlatformHob = (PLATFORM_INFO_HOB *)GET_GUID_HOB_DATA (Hob);
> +
> +  ASSERT (VirtualMemoryMap != NULL);
> +
> +  VirtualMemoryTable = (ARM_MEMORY_REGION_DESCRIPTOR *)AllocatePages (EFI_SIZE_TO_PAGES (sizeof (ARM_MEMORY_REGION_DESCRIPTOR) * MAX_VIRTUAL_MEMORY_MAP_DESCRIPTORS));
> +  if (VirtualMemoryTable == NULL) {
> +    return;
> +  }
> +
> +  /* For Address space 0x1000_0000_0000 to 0x1001_00FF_FFFF
> +   *  - Device memory
> +   */
> +  VirtualMemoryTable[Index].PhysicalBase = 0x100000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x100000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x102000000ULL;

Please move all of these live-coded addresses/sizes to a private .h
and use symbolic names here.

> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /* For Address space 0x5000_0000_0000 to 0x5001_00FF_FFFF
> +   *  - Device memory
> +   */
> +  if (IsSlaveSocketActive ())
> +  {
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x500000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x500000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x101000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +  }
> +
> +  /*
> +   *  - PCIe RCA0 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x300000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x300000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCA0 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB2 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x20000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x20000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - PCIe RCA1 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x340000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x340000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCA1 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB2 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x28000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x28000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - PCIe RCA2 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x380000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x380000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCA2 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB3 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x30000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x30000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - PCIe RCA3 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x3C0000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x3C0000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCA3 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB3 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x38000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x38000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - PCIe RCB0 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x200000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x200000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCB0 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB0 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x00000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x00000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - PCIe RCB1 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x240000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x240000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCB1 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB0 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x08000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x08000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - PCIe RCB2 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x280000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x280000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCB2 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB1 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x10000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x10000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - PCIe RCB3 Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x2C0000000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x2C0000000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket0 RCB3 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCB1 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x18000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x18000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  if (IsSlaveSocketActive ()) {
> +    // Slave socket exist
> +    /*
> +     *  - PCIe RCA0 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x700000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x700000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +    /*
> +     *  - PCIe RCA1 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x740000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x740000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +    /*
> +     *  - PCIe RCA2 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x780000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x780000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +    /*
> +     *  - PCIe RCA3 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x7C0000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x7C0000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +    /*
> +     *  - PCIe RCB0 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x600000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x600000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +    /*
> +     *  - PCIe RCB1 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x640000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x640000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +    /*
> +     *  - PCIe RCB2 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x680000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x680000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +    /*
> +     *  - PCIe RCB3 Device memory
> +     */
> +    VirtualMemoryTable[++Index].PhysicalBase = 0x6C0000000000ULL;
> +    VirtualMemoryTable[Index].VirtualBase  = 0x6C0000000000ULL;
> +    VirtualMemoryTable[Index].Length       = 0x40000000000ULL;
> +    VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +  }
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCA0 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA2 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x60000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x60000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCA1 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA2 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x68000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x68000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCA2 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA3 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x70000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x70000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCA3 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA3 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x78000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x78000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCB0 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA0 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x40000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x40000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCB1 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA0 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x48000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x48000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCB2 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA1 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x50000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x50000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - 2P/PCIe Socket1 RCB3 32-bit Device memory
> +   *  - 1P/PCIe consolidated to RCA1 32-bit Device memory
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x58000000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x58000000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x8000000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - BERT memory region
> +   */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0x88230000ULL;
> +  VirtualMemoryTable[Index].VirtualBase  = 0x88230000ULL;
> +  VirtualMemoryTable[Index].Length       = 0x50000ULL;
> +  VirtualMemoryTable[Index].Attributes   = ARM_MEMORY_REGION_ATTRIBUTE_DEVICE;
> +
> +  /*
> +   *  - DDR memory region
> +   */
> +  NumRegion = PlatformHob->DramInfo.NumRegion;
> +  Count = 0;
> +  while (NumRegion-- > 0) {
> +    if (PlatformHob->DramInfo.NvdRegion[Count]) { /* Skip NVDIMM Region */
> +      Count++;
> +      continue;
> +    }
> +
> +    VirtualMemoryTable[++Index].PhysicalBase = PlatformHob->DramInfo.Base[Count];
> +    VirtualMemoryTable[Index].VirtualBase  = PlatformHob->DramInfo.Base[Count];
> +    VirtualMemoryTable[Index].Length       = PlatformHob->DramInfo.Size[Count];
> +    VirtualMemoryTable[Index].Attributes   = DDR_ATTRIBUTES_CACHED;
> +    Count++;
> +  }
> +
> +  /* SPM MM NS Buffer for MmCommunicateDxe */
> +  VirtualMemoryTable[++Index].PhysicalBase = PcdGet64 (PcdMmBufferBase);
> +  VirtualMemoryTable[Index].VirtualBase  = PcdGet64 (PcdMmBufferBase);
> +  VirtualMemoryTable[Index].Length       = PcdGet64 (PcdMmBufferSize);
> +  VirtualMemoryTable[Index].Attributes   = DDR_ATTRIBUTES_CACHED;
> +
> +  /* End of Table */
> +  VirtualMemoryTable[++Index].PhysicalBase = 0;
> +  VirtualMemoryTable[Index].VirtualBase  = 0;
> +  VirtualMemoryTable[Index].Length       = 0;
> +  VirtualMemoryTable[Index].Attributes   = (ARM_MEMORY_REGION_ATTRIBUTES)0;
> +
> +  ASSERT ((Index + 1) <= MAX_VIRTUAL_MEMORY_MAP_DESCRIPTORS);
> +
> +  *VirtualMemoryMap = VirtualMemoryTable;
> +}



> diff --git a/Silicon/Ampere/AmpereAltraPkg/Library/MailboxInterfaceLib/MailboxInterfaceLib.c b/Silicon/Ampere/AmpereAltraPkg/Library/MailboxInterfaceLib/MailboxInterfaceLib.c
> new file mode 100644
> index 000000000000..0da1f90699f6
> --- /dev/null
> +++ b/Silicon/Ampere/AmpereAltraPkg/Library/MailboxInterfaceLib/MailboxInterfaceLib.c
> @@ -0,0 +1,282 @@
> +/** @file
> +  The library implements the hardware Mailbox (Doorbell) interface for communication
> +  between the Application Processor (ARMv8) and the System Control Processors (SMpro/PMpro).
> +
> +  Copyright (c) 2021, Ampere Computing LLC. All rights reserved.<BR>
> +
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <Uefi.h>
> +
> +#include <Library/AmpereCpuLib.h>
> +#include <Library/BaseMemoryLib.h>
> +#include <Library/DebugLib.h>
> +#include <Library/MailboxInterfaceLib.h>
> +#include <Library/TimerLib.h>
> +#include <Library/IoLib.h>
> +
> +//
> +// Hardware Doorbells
> +//
> +#define SMPRO_DB0_IRQ_OFST               40
> +#define SMPRO_DB0_BASE_ADDRESS           (FixedPcdGet64 (PcdSmproDbBaseReg))
> +
> +#define PMPRO_DB0_IRQ_OFST               56
> +#define PMPRO_DB0_BASE_ADDRESS           (FixedPcdGet64 (PcdPmproDbBaseReg))
> +
> +#define SLAVE_SOCKET_BASE_ADDRESS_OFFSET 0x400000000000
> +
> +//
> +// The base SPI interrupt number of the Slave socket
> +//
> +#define SLAVE_SOCKET_SPI_INTERRUPT 352
> +
> +#define SLAVE_SOCKET_DOORBELL_INTERRUPT_BASE(Socket) ((Socket) * SLAVE_SOCKET_SPI_INTERRUPT - 32)
> +
> +//
> +// Doorbell base register stride size
> +//
> +#define DB_BASE_REG_STRIDE 0x00001000
> +
> +#define SMPRO_DBx_ADDRESS(socket, db) \
> +        ((socket) * SLAVE_SOCKET_BASE_ADDRESS_OFFSET + SMPRO_DB0_BASE_ADDRESS + DB_BASE_REG_STRIDE * (db))
> +
> +#define PMPRO_DBx_ADDRESS(socket, db) \
> +        ((socket) * SLAVE_SOCKET_BASE_ADDRESS_OFFSET + PMPRO_DB0_BASE_ADDRESS + DB_BASE_REG_STRIDE * (db))
> +
> +//
> +// Doorbell Status Bits
> +//
> +#define DB_STATUS_AVAIL_BIT       BIT16
> +#define DB_STATUS_ACK_BIT         BIT0
> +
> +/**
> +  Get the base address of a doorbell.
> +
> +  @param[in]  Socket            Active socket index.
> +  @param[in]  Doorbell          Doorbell channel for communication with the SMpro/PMpro.
> +
> +  @retval UINT32                The base address of the doorbell.
> +                                The returned value is 0 indicate that the input parameters are invalid.
> +
> +**/
> +UINTN
> +EFIAPI
> +MailboxGetDoorbellAddress (
> +  IN UINT8             Socket,
> +  IN DOORBELL_CHANNELS Doorbell
> +  )
> +{
> +  UINTN DoorbellAddress;
> +
> +  if (Socket >= GetNumberOfActiveSockets ()
> +      || Doorbell >= NUMBER_OF_DOORBELLS_PER_SOCKET)
> +  {
> +    return 0;
> +  }

Coding style is
 if () {
 }

This file gets it consistently wrong.

> +
> +  if (Doorbell >= SMproDoorbellChannel0) {
> +    DoorbellAddress = SMPRO_DBx_ADDRESS (Socket, (UINT8)(Doorbell - SMproDoorbellChannel0));
> +  } else {
> +    DoorbellAddress = PMPRO_DBx_ADDRESS (Socket, (UINT8)Doorbell);
> +  }
> +
> +  return DoorbellAddress;
> +}
> +
> +/**
> +  Get the interrupt number of a doorbell.
> +
> +  @param[in]  Socket            Active socket index.
> +  @param[in]  Doorbell          Doorbell channel for communication with the SMpro/PMpro.
> +
> +  @retval UINT32                The interrupt number.
> +                                The returned value is 0 indicate that the input parameters are invalid.
> +
> +**/
> +UINT32
> +EFIAPI
> +MailboxGetDoorbellInterruptNumber (
> +  IN UINT8             Socket,
> +  IN DOORBELL_CHANNELS Doorbell
> +  )
> +{
> +  UINT32 DoorbellInterruptNumber;
> +
> +  if (Socket >= GetNumberOfActiveSockets ()
> +      || Doorbell >= NUMBER_OF_DOORBELLS_PER_SOCKET)
> +  {
> +    return 0;
> +  }
> +
> +  DoorbellInterruptNumber = 0;
> +
> +  if (Socket > 0) {
> +    DoorbellInterruptNumber = SLAVE_SOCKET_DOORBELL_INTERRUPT_BASE (Socket);
> +  }
> +
> +  if (Doorbell >= SMproDoorbellChannel0) {
> +    DoorbellInterruptNumber += SMPRO_DB0_IRQ_OFST + (UINT8)(Doorbell - SMproDoorbellChannel0);
> +  } else {
> +    DoorbellInterruptNumber += PMPRO_DB0_IRQ_OFST + (UINT8)Doorbell;
> +  }
> +
> +  return DoorbellInterruptNumber;
> +}
> +
> +/**
> +  Read a message via the hardware Doorbell interface.
> +
> +  @param[in]  Socket            Active socket index.
> +  @param[in]  Doorbell          Doorbell channel for communication with the SMpro/PMpro.
> +  @param[out] Message           Pointer to the Mailbox message.
> +
> +  @retval EFI_SUCCESS           Read the message successfully.
> +  @retval EFI_TIMEOUT           Timeout occurred when waiting for available message in the mailbox.
> +  @retval EFI_INVALID_PARAMETER A parameter is invalid.
> +**/
> +EFI_STATUS
> +EFIAPI
> +MailboxRead (
> +  IN  UINT8                Socket,
> +  IN  DOORBELL_CHANNELS    Doorbell,
> +  OUT MAILBOX_MESSAGE_DATA *Message
> +  )
> +{
> +  UINTN TimeoutCount;
> +  UINTN DoorbellAddress;
> +
> +  if (Socket >= GetNumberOfActiveSockets ()
> +      || Doorbell >= NUMBER_OF_DOORBELLS_PER_SOCKET
> +      || Message == NULL)
> +  {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  TimeoutCount = MAILBOX_POLL_COUNT;
> +
> +  DoorbellAddress = MailboxGetDoorbellAddress (Socket, Doorbell);
> +  ASSERT (DoorbellAddress != 0);
> +
> +  //
> +  // Polling Doorbell status
> +  //
> +  while ((MmioRead32 ((DoorbellAddress + DB_STATUS_REG_OFST)) & DB_STATUS_AVAIL_BIT) == 0) {
> +    MicroSecondDelay (MAILBOX_POLL_INTERVAL_US);
> +    if (--TimeoutCount == 0) {
> +      return EFI_TIMEOUT;
> +    }
> +  }
> +
> +  Message->ExtendedData[0] = MmioRead32 (DoorbellAddress + DB_DIN0_REG_OFST);
> +  Message->ExtendedData[1] = MmioRead32 (DoorbellAddress + DB_DIN1_REG_OFST);
> +  Message->Data = MmioRead32 (DoorbellAddress + DB_IN_REG_OFST);
> +
> +  //
> +  // Write 1 to clear the AVAIL status
> +  //
> +  MmioWrite32 (DoorbellAddress + DB_STATUS_REG_OFST, DB_STATUS_AVAIL_BIT);
> +
> +  return EFI_SUCCESS;
> +}
> +
> +/**
> +  Write a message via the hardware Doorbell interface.
> +
> +  @param[in]  Socket            Active socket index.
> +  @param[in]  Doorbell          Doorbel channel for communication with the SMpro/PMpro.
> +  @param[in]  Message           Pointer to the Mailbox message.
> +
> +  @retval EFI_SUCCESS           Write the message successfully.
> +  @retval EFI_TIMEOUT           Timeout occurred when waiting for acknowledge signal from the mailbox.
> +  @retval EFI_INVALID_PARAMETER A parameter is invalid.
> +**/
> +EFI_STATUS
> +EFIAPI
> +MailboxWrite (
> +  IN UINT8                Socket,
> +  IN DOORBELL_CHANNELS    Doorbell,
> +  IN MAILBOX_MESSAGE_DATA *Message
> +  )
> +{
> +  UINTN TimeoutCount;
> +  UINTN DoorbellAddress;
> +
> +  if (Socket >= GetNumberOfActiveSockets ()
> +      || Doorbell >= NUMBER_OF_DOORBELLS_PER_SOCKET
> +      || Message == NULL)
> +  {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  TimeoutCount = MAILBOX_POLL_COUNT;
> +
> +  DoorbellAddress = MailboxGetDoorbellAddress (Socket, Doorbell);
> +  ASSERT (DoorbellAddress != 0);
> +
> +  //
> +  // Clear previous pending ack if any
> +  //
> +  if ((MmioRead32 (DoorbellAddress + DB_STATUS_REG_OFST) & DB_STATUS_ACK_BIT) != 0) {
> +    MmioWrite32 (DoorbellAddress + DB_STATUS_REG_OFST, DB_STATUS_ACK_BIT);
> +  }
> +
> +  //
> +  // Send message
> +  //
> +  MmioWrite32 (DoorbellAddress + DB_DOUT0_REG_OFST, Message->ExtendedData[0]);
> +  MmioWrite32 (DoorbellAddress + DB_DOUT1_REG_OFST, Message->ExtendedData[1]);
> +  MmioWrite32 (DoorbellAddress + DB_OUT_REG_OFST, Message->Data);
> +
> +  //
> +  // Wait for ACK
> +  //
> +  while ((MmioRead32 (DoorbellAddress + DB_STATUS_REG_OFST) & DB_STATUS_ACK_BIT) == 0) {
> +    MicroSecondDelay (MAILBOX_POLL_INTERVAL_US);
> +    if (--TimeoutCount == 0) {
> +      return EFI_TIMEOUT;
> +    }
> +  }
> +
> +  //
> +  // Write 1 to clear the ACK status
> +  //
> +  MmioWrite32 (DoorbellAddress + DB_STATUS_REG_OFST, DB_STATUS_ACK_BIT);
> +
> +  return EFI_SUCCESS;
> +}
> +
> +/**
> +  Unmask the Doorbell interrupt status.
> +
> +  @param  Socket    Active socket index.
> +  @param  Doorbell  Doorbel channel for communication with the SMpro/PMpro.
> +
> +  @retval EFI_SUCCESS            Unmask the Doorbell interrupt successfully.
> +  @retval EFI_INVALID_PARAMETER  A parameter is invalid.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +MailboxUnmaskInterrupt (
> +  IN UINT8  Socket,
> +  IN UINT16 Doorbell
> +  )
> +{
> +  UINTN DoorbellAddress;
> +
> +  if (Socket >= GetNumberOfActiveSockets ()
> +      || Doorbell >= NUMBER_OF_DOORBELLS_PER_SOCKET)
> +  {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  DoorbellAddress = MailboxGetDoorbellAddress (Socket, Doorbell);
> +  ASSERT (DoorbellAddress != 0);
> +
> +  MmioWrite32 (DoorbellAddress + DB_STATUS_MASK_REG_OFST, ~DB_STATUS_AVAIL_BIT);
> +
> +  return EFI_SUCCESS;
> +}


> diff --git a/Silicon/Ampere/AmpereAltraPkg/Library/MmCommunicationLib/MmCommunicationLib.c b/Silicon/Ampere/AmpereAltraPkg/Library/MmCommunicationLib/MmCommunicationLib.c
> new file mode 100644
> index 000000000000..bf400ec0a835
> --- /dev/null
> +++ b/Silicon/Ampere/AmpereAltraPkg/Library/MmCommunicationLib/MmCommunicationLib.c
> @@ -0,0 +1,184 @@
> +/** @file
> +
> +  Copyright (c) 2020 - 2021, Ampere Computing LLC. All rights reserved.<BR>
> +
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <IndustryStandard/ArmStdSmc.h>
> +#include <Library/ArmLib.h>
> +#include <Library/ArmSmcLib.h>
> +#include <Library/BaseMemoryLib.h>
> +#include <Library/DebugLib.h>
> +#include <Library/MmCommunicationLib.h>
> +#include <Library/PcdLib.h>
> +#include <Protocol/MmCommunication.h>
> +
> +//
> +// Address, Length of the pre-allocated buffer for communication with the secure
> +// world.
> +//
> +STATIC ARM_MEMORY_REGION_DESCRIPTOR mNsCommBuffMemRegion;
> +
> +EFI_STATUS
> +EFIAPI
> +MmCommunicationLibConstructor (
> +  VOID
> +  )
> +{
> +  mNsCommBuffMemRegion.PhysicalBase = PcdGet64 (PcdMmBufferBase);
> +  // During boot , Virtual and Physical are same

Ideally, use UEFI-defined terms. "During boot" is quite ambiguous.


> +  mNsCommBuffMemRegion.VirtualBase = mNsCommBuffMemRegion.PhysicalBase;
> +  mNsCommBuffMemRegion.Length = PcdGet64 (PcdMmBufferSize);
> +
> +  return EFI_SUCCESS;
> +}
> +
> +/**
> +  Communicates with a registered handler.
> +
> +  This function provides an interface to send and receive messages to the
> +  Standalone MM environment in UEFI PEI phase.
> +
> +  @param[in, out] CommBuffer          A pointer to the buffer to convey
> +                                      into MMRAM.
> +  @param[in, out] CommSize            The size of the data buffer being
> +                                      passed in. This is optional.
> +
> +  @retval EFI_SUCCESS                 The message was successfully posted.
> +  @retval EFI_INVALID_PARAMETER       The CommBuffer was NULL.
> +  @retval EFI_BAD_BUFFER_SIZE         The buffer size is incorrect for the MM
> +                                      implementation. If this error is
> +                                      returned, the MessageLength field in
> +                                      the CommBuffer header or the integer
> +                                      pointed by CommSize are updated to reflect
> +                                      the maximum payload size the
> +                                      implementation can accommodate.
> +  @retval EFI_ACCESS_DENIED           The CommunicateBuffer parameter
> +                                      or CommSize parameter, if not omitted,
> +                                      are in address range that cannot be
> +                                      accessed by the MM environment
> +**/
> +EFI_STATUS
> +EFIAPI
> +MmCommunicationCommunicate (
> +  IN OUT VOID  *CommBuffer,
> +  IN OUT UINTN *CommSize OPTIONAL
> +  )
> +{
> +  EFI_MM_COMMUNICATE_HEADER *CommunicateHeader;
> +  ARM_SMC_ARGS              CommunicateSmcArgs;
> +  EFI_STATUS                Status;
> +  UINTN                     BufferSize;
> +
> +  Status = EFI_ACCESS_DENIED;
> +  BufferSize = 0;
> +
> +  ZeroMem (&CommunicateSmcArgs, sizeof (ARM_SMC_ARGS));
> +
> +  //
> +  // Check parameters
> +  //
> +  if (CommBuffer == NULL) {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  CommunicateHeader = CommBuffer;
> +  // CommBuffer is a mandatory parameter. Hence, Rely on
> +  // MessageLength + Header to ascertain the
> +  // total size of the communication payload rather than
> +  // rely on optional CommSize parameter
> +  BufferSize = CommunicateHeader->MessageLength +
> +               sizeof (CommunicateHeader->HeaderGuid) +
> +               sizeof (CommunicateHeader->MessageLength);
> +
> +  // If the length of the CommBuffer is 0 then return the expected length.
> +  if (CommSize != NULL) {
> +    // This case can be used by the consumer of this driver to find out the
> +    // max size that can be used for allocating CommBuffer.
> +    if ((*CommSize == 0) ||
> +        (*CommSize > mNsCommBuffMemRegion.Length))
> +    {

{ at end of preceding line.
Please address throughout this file.

> +      *CommSize = mNsCommBuffMemRegion.Length;
> +      return EFI_BAD_BUFFER_SIZE;
> +    }
> +    //
> +    // CommSize must match MessageLength + sizeof (EFI_MM_COMMUNICATE_HEADER);
> +    //
> +    if (*CommSize != BufferSize) {
> +      return EFI_INVALID_PARAMETER;
> +    }
> +  }
> +
> +  //
> +  // If the buffer size is 0 or greater than what can be tolerated by the MM
> +  // environment then return the expected size.
> +  //
> +  if ((BufferSize == 0) ||
> +      (BufferSize > mNsCommBuffMemRegion.Length))
> +  {
> +    CommunicateHeader->MessageLength = mNsCommBuffMemRegion.Length -
> +                                       sizeof (CommunicateHeader->HeaderGuid) -
> +                                       sizeof (CommunicateHeader->MessageLength);
> +    return EFI_BAD_BUFFER_SIZE;
> +  }
> +
> +  // SMC Function ID
> +  CommunicateSmcArgs.Arg0 = ARM_SMC_ID_MM_COMMUNICATE_AARCH64;
> +
> +  // Cookie
> +  CommunicateSmcArgs.Arg1 = 0;
> +
> +  // Copy Communication Payload
> +  CopyMem ((VOID *)mNsCommBuffMemRegion.VirtualBase, CommBuffer, BufferSize);
> +
> +  // comm_buffer_address (64-bit physical address)
> +  CommunicateSmcArgs.Arg2 = (UINTN)mNsCommBuffMemRegion.PhysicalBase;
> +
> +  // comm_size_address (not used, indicated by setting to zero)
> +  CommunicateSmcArgs.Arg3 = 0;
> +
> +  // Call the Standalone MM environment.
> +  ArmCallSmc (&CommunicateSmcArgs);
> +
> +  switch (CommunicateSmcArgs.Arg0) {
> +  case ARM_SMC_MM_RET_SUCCESS:
> +    ZeroMem (CommBuffer, BufferSize);
> +    // On successful return, the size of data being returned is inferred from
> +    // MessageLength + Header.
> +    CommunicateHeader = (EFI_MM_COMMUNICATE_HEADER *)mNsCommBuffMemRegion.VirtualBase;
> +    BufferSize = CommunicateHeader->MessageLength +
> +                 sizeof (CommunicateHeader->HeaderGuid) +
> +                 sizeof (CommunicateHeader->MessageLength);
> +
> +    CopyMem (
> +      CommBuffer,
> +      (VOID *)mNsCommBuffMemRegion.VirtualBase,
> +      BufferSize
> +      );
> +    Status = EFI_SUCCESS;
> +    break;
> +
> +  case ARM_SMC_MM_RET_INVALID_PARAMS:
> +    Status = EFI_INVALID_PARAMETER;
> +    break;
> +
> +  case ARM_SMC_MM_RET_DENIED:
> +    Status = EFI_ACCESS_DENIED;
> +    break;
> +
> +  case ARM_SMC_MM_RET_NO_MEMORY:
> +    // Unexpected error since the CommSize was checked for zero length
> +    // prior to issuing the SMC
> +    Status = EFI_OUT_OF_RESOURCES;
> +    ASSERT (0);
> +    break;
> +
> +  default:
> +    Status = EFI_ACCESS_DENIED;
> +    ASSERT (0);
> +  }
> +
> +  return Status;
> +}

> diff --git a/Platform/Ampere/JadePkg/JadeBoardSetting.cfg b/Platform/Ampere/JadePkg/JadeBoardSetting.cfg
> new file mode 100644
> index 000000000000..5a67e8fc6a75
> --- /dev/null
> +++ b/Platform/Ampere/JadePkg/JadeBoardSetting.cfg
> @@ -0,0 +1,209 @@
> +# Sample board setting
> +#
> +# This is a sample board setting as used for the
> +# Ampere Altra reference design.

What is a board setting?

> +#
> +# Name, offset (hex), value
> +# value can be hex or decimal
> +#
> +
> +NV_SI_RO_BOARD_VENDOR, 0x0000, 0x0000CD3A
> +NV_SI_RO_BOARD_TYPE, 0x0008, 0x00000000
> +NV_SI_RO_BOARD_REV, 0x0010, 0x00000000
> +NV_SI_RO_BOARD_CFG, 0x0018, 0x00000000
> +NV_SI_RO_BOARD_S0_DIMM_AVAIL, 0x0020, 0x0000FFFF
> +NV_SI_RO_BOARD_S1_DIMM_AVAIL, 0x0028, 0x0000FFFF
> +NV_SI_RO_BOARD_SPI0CS0_FREQ_KHZ, 0x0030, 0x000080E8
> +NV_SI_RO_BOARD_SPI0CS1_FREQ_KHZ, 0x0038, 0x000080E8
> +NV_SI_RO_BOARD_SPI1CS0_FREQ_KHZ, 0x0040, 0x00002710
> +NV_SI_RO_BOARD_SPI1CS1_FREQ_KHZ, 0x0048, 0x00002710
> +NV_SI_RO_BOARD_TPM_LOC, 0x0050, 0x00000000
> +NV_SI_RO_BOARD_I2C0_FREQ_KHZ, 0x0058, 0x00000190
> +NV_SI_RO_BOARD_I2C1_FREQ_KHZ, 0x0060, 0x00000190
> +NV_SI_RO_BOARD_I2C2_10_FREQ_KHZ, 0x0068, 0x00000190
> +NV_SI_RO_BOARD_I2C3_FREQ_KHZ, 0x0070, 0x00000190
> +NV_SI_RO_BOARD_I2C9_FREQ_KHZ, 0x0078, 0x00000190
> +NV_SI_RO_BOARD_2P_CFG, 0x0080, 0xFFFFFF01
> +NV_SI_RO_BOARD_S0_RCA0_CFG, 0x0088, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA1_CFG, 0x0090, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA2_CFG, 0x0098, 0x00000004
> +NV_SI_RO_BOARD_S0_RCA3_CFG, 0x00A0, 0x00000004
> +NV_SI_RO_BOARD_S0_RCB0_LO_CFG, 0x00A8, 0x00020002
> +NV_SI_RO_BOARD_S0_RCB0_HI_CFG, 0x00B0, 0x00020002
> +NV_SI_RO_BOARD_S0_RCB1_LO_CFG, 0x00B8, 0x00020002
> +NV_SI_RO_BOARD_S0_RCB1_HI_CFG, 0x00C0, 0x00020002
> +NV_SI_RO_BOARD_S0_RCB2_LO_CFG, 0x00C8, 0x00020002
> +NV_SI_RO_BOARD_S0_RCB2_HI_CFG, 0x00D0, 0x00000003
> +NV_SI_RO_BOARD_S0_RCB3_LO_CFG, 0x00D8, 0x00000003
> +NV_SI_RO_BOARD_S0_RCB3_HI_CFG, 0x00E0, 0x00020002
> +NV_SI_RO_BOARD_S1_RCA0_CFG, 0x00E8, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA1_CFG, 0x00F0, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA2_CFG, 0x00F8, 0x02020202
> +NV_SI_RO_BOARD_S1_RCA3_CFG, 0x0100, 0x00030003
> +NV_SI_RO_BOARD_S1_RCB0_LO_CFG, 0x0108, 0x00000003
> +NV_SI_RO_BOARD_S1_RCB0_HI_CFG, 0x0110, 0x00020002
> +NV_SI_RO_BOARD_S1_RCB1_LO_CFG, 0x0118, 0x00020002
> +NV_SI_RO_BOARD_S1_RCB1_HI_CFG, 0x0120, 0x00000003
> +NV_SI_RO_BOARD_S1_RCB2_LO_CFG, 0x0128, 0x00020002
> +NV_SI_RO_BOARD_S1_RCB2_HI_CFG, 0x0130, 0x00020002
> +NV_SI_RO_BOARD_S1_RCB3_LO_CFG, 0x0138, 0x00020002
> +NV_SI_RO_BOARD_S1_RCB3_HI_CFG, 0x0140, 0x00020002
> +NV_SI_RO_BOARD_T_LTLM_DELTA_P0, 0x0148, 0x00000001
> +NV_SI_RO_BOARD_T_LTLM_DELTA_P1, 0x0150, 0x00000002
> +NV_SI_RO_BOARD_T_LTLM_DELTA_P2, 0x0158, 0x00000003
> +NV_SI_RO_BOARD_T_LTLM_DELTA_P3, 0x0160, 0x00000004
> +NV_SI_RO_BOARD_T_LTLM_DELTA_M1, 0x0168, 0xFFFFFFFF
> +NV_SI_RO_BOARD_T_LTLM_DELTA_M2, 0x0170, 0xFFFFFFFE
> +NV_SI_RO_BOARD_T_LTLM_DELTA_M3, 0x0178, 0xFFFFFFFD
> +NV_SI_RO_BOARD_P_LM_PID_P, 0x0180, 0x00000000
> +NV_SI_RO_BOARD_P_LM_PID_I, 0x0188, 0x00000000
> +NV_SI_RO_BOARD_P_LM_PID_I_L_THOLD, 0x0190, 0x00000000
> +NV_SI_RO_BOARD_P_LM_PID_I_H_THOLD, 0x0198, 0x00000000
> +NV_SI_RO_BOARD_P_LM_PID_D, 0x01A0, 0x00000000
> +NV_SI_RO_BOARD_P_LM_EXP_SMOOTH_CONST, 0x01A8, 0x00000000
> +NV_SI_RO_BOARD_TPM_ALG_ID, 0x01B0, 0x00000002
> +NV_SI_RO_BOARD_DDR_SPEED_GRADE, 0x01B8, 0x00000C80
> +NV_SI_RO_BOARD_DDR_S0_RTT_WR, 0x01C0, 0x00020000
> +NV_SI_RO_BOARD_DDR_S1_RTT_WR, 0x01C8, 0x00020000
> +NV_SI_RO_BOARD_DDR_S0_RTT_NOM, 0x01D0, 0xFF060177
> +NV_SI_RO_BOARD_DDR_S1_RTT_NOM, 0x01D8, 0xFF060177
> +NV_SI_RO_BOARD_DDR_S0_RTT_PARK, 0x01E0, 0x00060070
> +NV_SI_RO_BOARD_DDR_S1_RTT_PARK, 0x01E8, 0x00060070
> +NV_SI_RO_BOARD_DDR_CS0_RDODT_MASK_1DPC, 0x01F0, 0x00000000
> +NV_SI_RO_BOARD_DDR_CS1_RDODT_MASK_1DPC, 0x01F8, 0x00000000
> +NV_SI_RO_BOARD_DDR_CS2_RDODT_MASK_1DPC, 0x0200, 0x00000000
> +NV_SI_RO_BOARD_DDR_CS3_RDODT_MASK_1DPC, 0x0208, 0x00000000
> +NV_SI_RO_BOARD_DDR_CS0_RDODT_MASK_2DPC, 0x0210, 0x000C0CCC
> +NV_SI_RO_BOARD_DDR_CS1_RDODT_MASK_2DPC, 0x0218, 0x000C0CCC
> +NV_SI_RO_BOARD_DDR_CS2_RDODT_MASK_2DPC, 0x0220, 0x00030333
> +NV_SI_RO_BOARD_DDR_CS3_RDODT_MASK_2DPC, 0x0228, 0x00030333
> +NV_SI_RO_BOARD_DDR_CS0_WRODT_MASK_1DPC, 0x0230, 0x00030333
> +NV_SI_RO_BOARD_DDR_CS1_WRODT_MASK_1DPC, 0x0238, 0x00030333
> +NV_SI_RO_BOARD_DDR_CS2_WRODT_MASK_1DPC, 0x0240, 0x00030333
> +NV_SI_RO_BOARD_DDR_CS3_WRODT_MASK_1DPC, 0x0248, 0x00030333
> +NV_SI_RO_BOARD_DDR_CS0_WRODT_MASK_2DPC, 0x0250, 0x000EDEED
> +NV_SI_RO_BOARD_DDR_CS1_WRODT_MASK_2DPC, 0x0258, 0x000DEDDE
> +NV_SI_RO_BOARD_DDR_CS2_WRODT_MASK_2DPC, 0x0260, 0x000B7BB7
> +NV_SI_RO_BOARD_DDR_CS3_WRODT_MASK_2DPC, 0x0268, 0x0007B77B
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQ_CTRL_1DPC, 0x0270, 0x00000005
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQ_VAL_1DPC, 0x0278, 0x0090DD90
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQS_CTRL_1DPC, 0x0280, 0x00000005
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQS_VAL_1DPC, 0x0288, 0x0090DD90
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQ_CTRL_2DPC, 0x0290, 0x00000005
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQ_VAL_2DPC, 0x0298, 0x0090DD90
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQS_CTRL_2DPC, 0x02A0, 0x00000005
> +NV_SI_RO_BOARD_DDR_PHY_TERM_DQS_VAL_2DPC, 0x02A8, 0x0090DD90
> +NV_SI_RO_BOARD_DDR_PHY_VREFDQ_RANGE_VAL_1DPC, 0x02B0, 0x00000024
> +NV_SI_RO_BOARD_DDR_DRAM_VREFDQ_RANGE_VAL_1DPC, 0x02B8, 0x0000001A
> +NV_SI_RO_BOARD_DDR_PHY_VREFDQ_RANGE_VAL_2DPC, 0x02C0, 0x00000050
> +NV_SI_RO_BOARD_DDR_DRAM_VREFDQ_RANGE_VAL_2DPC, 0x02C8, 0x00000020
> +NV_SI_RO_BOARD_DDR_CLK_WRDQ_DLY_DEFAULT, 0x02D0, 0x02800280
> +NV_SI_RO_BOARD_DDR_RDDQS_DQ_DLY_DEFAULT, 0x02D8, 0x90909090
> +NV_SI_RO_BOARD_DDR_WRDQS_SHIFT_DEFAULT, 0x02E0, 0x00000000
> +NV_SI_RO_BOARD_DDR_ADCMD_DLY_DEFAULT, 0x02E8, 0x00C000C0
> +NV_SI_RO_BOARD_DDR_CLK_WRDQ_DLY_ADJ, 0x02F0, 0x00000000
> +NV_SI_RO_BOARD_DDR_RDDQS_DQ_DLY_ADJ, 0x02F8, 0x00000000
> +NV_SI_RO_BOARD_DDR_PHY_VREF_ADJ, 0x0300, 0x00000000
> +NV_SI_RO_BOARD_DDR_DRAM_VREF_ADJ, 0x0308, 0x00000000
> +NV_SI_RO_BOARD_DDR_WR_PREAMBLE_CYCLE, 0x0310, 0x02010201
> +NV_SI_RO_BOARD_DDR_ADCMD_2T_MODE, 0x0318, 0x00000000
> +NV_SI_RO_BOARD_I2C_VRD_CONFIG_INFO, 0x0320, 0x00000000
> +NV_SI_RO_BOARD_DDR_PHY_FEATURE_CTRL, 0x0328, 0x00000000
> +NV_SI_RO_BOARD_BMC_HANDSHAKE_SPI_ACCESS, 0x0330, 0x01050106
> +NV_SI_RO_BOARD_DIMM_TEMP_THRESHOLD, 0x0338, 0x000005F4
> +NV_SI_RO_BOARD_DIMM_SPD_COMPARE_DISABLE, 0x0340, 0x00000000
> +NV_SI_RO_BOARD_S0_PCIE_CLK_CFG, 0x0348, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA4_CFG, 0x0350, 0x02020202
> +NV_SI_RO_BOARD_S0_RCA5_CFG, 0x0358, 0x02020202
> +NV_SI_RO_BOARD_S0_RCA6_CFG, 0x0360, 0x02020202
> +NV_SI_RO_BOARD_S0_RCA7_CFG, 0x0368, 0x02020003
> +NV_SI_RO_BOARD_S0_RCA0_TXRX_G3PRESET, 0x0370, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA1_TXRX_G3PRESET, 0x0378, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA2_TXRX_G3PRESET, 0x0380, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA3_TXRX_G3PRESET, 0x0388, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB0A_TXRX_G3PRESET, 0x0390, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB0B_TXRX_G3PRESET, 0x0398, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB1A_TXRX_G3PRESET, 0x03A0, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB1B_TXRX_G3PRESET, 0x03A8, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB2A_TXRX_G3PRESET, 0x03B0, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB2B_TXRX_G3PRESET, 0x03B8, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB3A_TXRX_G3PRESET, 0x03C0, 0x00000000
> +NV_SI_RO_BOARD_S0_RCB3B_TXRX_G3PRESET, 0x03C8, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA4_TXRX_G3PRESET, 0x03D0, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA5_TXRX_G3PRESET, 0x03D8, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA6_TXRX_G3PRESET, 0x03E0, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA7_TXRX_G3PRESET, 0x03E8, 0x00000000
> +NV_SI_RO_BOARD_S0_RCA0_TXRX_G4PRESET, 0x03F0, 0x57575757
> +NV_SI_RO_BOARD_S0_RCA1_TXRX_G4PRESET, 0x03F8, 0x57575757
> +NV_SI_RO_BOARD_S0_RCA2_TXRX_G4PRESET, 0x0400, 0x57575757
> +NV_SI_RO_BOARD_S0_RCA3_TXRX_G4PRESET, 0x0408, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB0A_TXRX_G4PRESET, 0x0410, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB0B_TXRX_G4PRESET, 0x0418, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB1A_TXRX_G4PRESET, 0x0420, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB1B_TXRX_G4PRESET, 0x0428, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB2A_TXRX_G4PRESET, 0x0430, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB2B_TXRX_G4PRESET, 0x0438, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB3A_TXRX_G4PRESET, 0x0440, 0x57575757
> +NV_SI_RO_BOARD_S0_RCB3B_TXRX_G4PRESET, 0x0448, 0x57575757
> +NV_SI_RO_BOARD_S0_RCA4_TXRX_G4PRESET, 0x0450, 0x57575757
> +NV_SI_RO_BOARD_S0_RCA5_TXRX_G4PRESET, 0x0458, 0x57575757
> +NV_SI_RO_BOARD_S0_RCA6_TXRX_G4PRESET, 0x0460, 0x57575757
> +NV_SI_RO_BOARD_S0_RCA7_TXRX_G4PRESET, 0x0468, 0x57575757
> +NV_SI_RO_BOARD_S1_PCIE_CLK_CFG, 0x0470, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA4_CFG, 0x0478, 0x02020202
> +NV_SI_RO_BOARD_S1_RCA5_CFG, 0x0480, 0x02020202
> +NV_SI_RO_BOARD_S1_RCA6_CFG, 0x0488, 0x02020202
> +NV_SI_RO_BOARD_S1_RCA7_CFG, 0x0490, 0x02020003
> +NV_SI_RO_BOARD_S1_RCA2_TXRX_G3PRESET, 0x0498, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA3_TXRX_G3PRESET, 0x04A0, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB0A_TXRX_G3PRESET, 0x04A8, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB0B_TXRX_G3PRESET, 0x04B0, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB1A_TXRX_G3PRESET, 0x04B8, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB1B_TXRX_G3PRESET, 0x04C0, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB2A_TXRX_G3PRESET, 0x04C8, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB2B_TXRX_G3PRESET, 0x04D0, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB3A_TXRX_G3PRESET, 0x04D8, 0x00000000
> +NV_SI_RO_BOARD_S1_RCB3B_TXRX_G3PRESET, 0x04E0, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA4_TXRX_G3PRESET, 0x04E8, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA5_TXRX_G3PRESET, 0x04F0, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA6_TXRX_G3PRESET, 0x04F8, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA7_TXRX_G3PRESET, 0x0500, 0x00000000
> +NV_SI_RO_BOARD_S1_RCA2_TXRX_G4PRESET, 0x0508, 0x57575757
> +NV_SI_RO_BOARD_S1_RCA3_TXRX_G4PRESET, 0x0510, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB0A_TXRX_G4PRESET, 0x0518, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB0B_TXRX_G4PRESET, 0x0520, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB1A_TXRX_G4PRESET, 0x0528, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB1B_TXRX_G4PRESET, 0x0530, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB2A_TXRX_G4PRESET, 0x0538, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB2B_TXRX_G4PRESET, 0x0540, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB3A_TXRX_G4PRESET, 0x0548, 0x57575757
> +NV_SI_RO_BOARD_S1_RCB3B_TXRX_G4PRESET, 0x0550, 0x57575757
> +NV_SI_RO_BOARD_S1_RCA4_TXRX_G4PRESET, 0x0558, 0x57575757
> +NV_SI_RO_BOARD_S1_RCA5_TXRX_G4PRESET, 0x0560, 0x57575757
> +NV_SI_RO_BOARD_S1_RCA6_TXRX_G4PRESET, 0x0568, 0x57575757
> +NV_SI_RO_BOARD_S1_RCA7_TXRX_G4PRESET, 0x0570, 0x57575757
> +NV_SI_RO_BOARD_2P_CE_MASK_THRESHOLD, 0x0578, 0x00000003
> +NV_SI_RO_BOARD_2P_CE_MASK_INTERVAL, 0x0580, 0x000001A4
> +NV_SI_RO_BOARD_SX_PHY_CFG_SETTING, 0x0588, 0x00000000
> +NV_SI_RO_BOARD_DDR_PHY_DC_CLK, 0x0590, 0x00018000
> +NV_SI_RO_BOARD_DDR_PHY_DC_DATA, 0x0598, 0x80018000
> +NV_SI_RO_BOARD_SX_RCA0_TXRX_20GPRESET, 0x05A0, 0x00000000
> +NV_SI_RO_BOARD_SX_RCA1_TXRX_20GPRESET, 0x05A8, 0x00000000
> +NV_SI_RO_BOARD_SX_RCA2_TXRX_20GPRESET, 0x05B0, 0x00000000
> +NV_SI_RO_BOARD_SX_RCA3_TXRX_20GPRESET, 0x05B8, 0x00000000
> +NV_SI_RO_BOARD_SX_RCA0_TXRX_25GPRESET, 0x05C0, 0x00000000
> +NV_SI_RO_BOARD_SX_RCA1_TXRX_25GPRESET, 0x05C8, 0x00000000
> +NV_SI_RO_BOARD_SX_RCA2_TXRX_25GPRESET, 0x05D0, 0x00000000
> +NV_SI_RO_BOARD_SX_RCA3_TXRX_25GPRESET, 0x05D8, 0x00000000
> +NV_SI_RO_BOARD_DDR_2X_REFRESH_TEMP_THRESHOLD, 0x05E0, 0x00550055
> +NV_SI_RO_BOARD_PCP_VRD_VOUT_WAIT_US, 0x05E8, 0x00000064
> +NV_SI_RO_BOARD_PCP_VRD_VOUT_RESOLUTION_MV, 0x05F0, 0x00000005
> +NV_SI_RO_BOARD_DVFS_VOLT_READ_BACK_EN, 0x05F8, 0x00000001
> +NV_SI_RO_BOARD_DVFS_VOLT_READ_BACK_TIME, 0x0600, 0x00000002
> +NV_SI_RO_BOARD_DVFS_VOUT_20MV_RAMP_TIME_US, 0x0608, 0x00000005
> +NV_SI_RO_BOARD_PCIE_AER_FW_FIRST, 0x0610, 0x00000000
> +NV_SI_RO_BOARD_RTC_GPI_LOCK_BYPASS, 0x0618, 0x00000000
> +NV_SI_RO_BOARD_TPM_DISABLE, 0x0620, 0x00000000
> +NV_SI_RO_BOARD_MESH_S0_CXG_RC_STRONG_ORDERING_EN, 0x0628, 0x00000000
> +NV_SI_RO_BOARD_MESH_S1_CXG_RC_STRONG_ORDERING_EN, 0x0630, 0x00000000
> +NV_SI_RO_BOARD_GPIO_SW_WATCHDOG_EN, 0x0638, 0x00000000

There was also a few things in this patch where I felt names had
insufficient namespace - such as starting with TRNG_ or NV_.
I'm not going to insist on adding prefixes to those, I'm just going to
say I have warned you, and those might get you in trouble in the
future :)

Best Regards,

Leif

  reply	other threads:[~2021-06-04 23:04 UTC|newest]

Thread overview: 87+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-26 10:06 [edk2-platforms][PATCH v2 00/32] Add new Ampere Mt. Jade platform Nhi Pham
2021-05-26 10:06 ` [edk2-platforms][PATCH v2 01/32] Ampere: Initial support for Ampere Altra processor and " Nhi Pham
2021-06-04 23:04   ` Leif Lindholm [this message]
2021-06-09  4:50     ` Nhi Pham
2021-06-09 12:40       ` Leif Lindholm
2021-06-15 16:46     ` Nhi Pham
2021-05-26 10:06 ` [PATCH 1/1] UsbCdcNetDxe: Remove reading connection status in SNP GetStatus Nhi Pham
2021-05-26 10:23   ` [edk2-devel] " Nhi Pham
2021-05-26 10:06 ` [edk2-platforms][PATCH v2 02/32] AmpereAltraPkg: Add MmCommunication modules Nhi Pham
2021-06-04 23:05   ` Leif Lindholm
2021-05-26 10:06 ` [edk2-platforms][PATCH v2 03/32] AmperePlatformPkg: Implement FailSafe library Nhi Pham
2021-06-04 23:07   ` Leif Lindholm
2021-05-26 10:06 ` [edk2-platforms][PATCH v2 04/32] AmperePlatformPkg: Add FailSafe and WDT support Nhi Pham
2021-06-04 23:12   ` Leif Lindholm
2021-06-15 16:47     ` Nhi Pham
2021-05-26 10:06 ` [edk2-platforms][PATCH v2 05/32] AmpereAltraPkg: Add DwI2cLib library Nhi Pham
2021-06-04 23:21   ` Leif Lindholm
2021-06-15 16:47     ` Nhi Pham
2021-05-26 10:06 ` [edk2-platforms][PATCH v2 06/32] AmpereAltraPkg: Add DwGpioLib library Nhi Pham
2021-06-04 23:22   ` Leif Lindholm
2021-05-26 10:06 ` [edk2-platforms][PATCH v2 07/32] JadePkg: Implement RealTimeClockLib for PCF85063 Nhi Pham
2021-06-04 23:26   ` Leif Lindholm
2021-06-15 16:48     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 08/32] AmpereAltraPkg: Add BootProgress support Nhi Pham
2021-06-04 23:27   ` Leif Lindholm
2021-06-15 16:48     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 09/32] AmpereAltraPkg: Support non-volatile variables Nhi Pham
2021-06-04 23:36   ` Leif Lindholm
2021-06-15 16:48     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 10/32] AmpereSiliconPkg: Add PlatformManagerUiLib library instance Nhi Pham
2021-06-04 23:37   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 11/32] AmperePlatformPkg: Add AcpiPccLib to support ACPI PCCT Table Nhi Pham
2021-06-04 23:44   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 12/32] AmperePlatformPkg: Add AcpiHelperLib to update ACPI DSDT table Nhi Pham
2021-06-04 23:47   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 13/32] AmpereAltraPkg, JadePkg: Add ACPI support Nhi Pham
2021-06-04 23:50   ` Leif Lindholm
2021-06-15 16:49     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 14/32] AmpereAltraPkg: Add PcieCoreLib library instance Nhi Pham
2021-06-05  0:05   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 15/32] JadePkg: Add PcieBoardLib " Nhi Pham
2021-06-07 22:45   ` Leif Lindholm
2021-06-15 16:50     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 16/32] AmpereAltraPkg: Add PciHostBridge driver Nhi Pham
2021-06-08 22:26   ` Leif Lindholm
2021-06-09  5:29   ` Ard Biesheuvel
2021-06-15 15:54     ` Nhi Pham
2021-06-16 14:20       ` Ard Biesheuvel
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 17/32] JadePkg: Enable PCIe-related libraries and device drivers Nhi Pham
2021-06-07 22:51   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 18/32] JadePkg: Add ASpeed GOP driver Nhi Pham
2021-06-07 22:51   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 19/32] AmpereAltraPkg: Add Random Number Generator Support Nhi Pham
2021-06-08 11:13   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 20/32] JadePkg: Add SMBIOS tables support Nhi Pham
2021-06-07 23:00   ` Leif Lindholm
2021-06-15 16:51     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 21/32] AmpereAltraPkg: Add DebugInfoPei module Nhi Pham
2021-06-07 23:08   ` Leif Lindholm
2021-06-15 16:51     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 22/32] AmpereAltraPkg: Add platform info screen Nhi Pham
2021-06-07 23:10   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 23/32] AmpereAltraPkg: Add configuration screen for memory Nhi Pham
2021-06-07 23:14   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 24/32] AmpereAltraPkg: Add configuration screen for CPU Nhi Pham
2021-06-07 23:15   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 25/32] AmpereAltraPkg: Add configuration screen for ACPI Nhi Pham
2021-06-07 23:20   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 26/32] AmpereAltraPkg: Add configuration screen for RAS Nhi Pham
2021-06-07 23:22   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 27/32] AmpereAltraPkg: Add configuration screen for Watchdog timer Nhi Pham
2021-06-07 23:24   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 28/32] AmpereAltraPkg: Add configuration screen for Pcie Devices Nhi Pham
2021-06-07 23:34   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 29/32] JadePkg: Recover boot options when NVRAM cleared Nhi Pham
2021-06-07 23:46   ` Leif Lindholm
2021-06-15 16:52     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 30/32] AmpereSiliconPkg: Implement PlatformBootManagerLib for LinuxBoot Nhi Pham
2021-06-07 23:50   ` Leif Lindholm
2021-06-09 15:21     ` Nhi Pham
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 31/32] Platform/Ampere: Introduce the LinuxBootPkg Nhi Pham
2021-06-07 23:51   ` Leif Lindholm
2021-05-26 10:07 ` [edk2-platforms][PATCH v2 32/32] AmpereAltraPkg,JadePkg: Support LinuxBoot DSC/FDF build for Jade platform Nhi Pham
2021-06-07 23:58   ` Leif Lindholm
2021-06-09 15:20     ` Nhi Pham
2021-05-27 12:56 ` [edk2-platforms][PATCH v2 00/32] Add new Ampere Mt. " Leif Lindholm
2021-06-04 13:54 ` Leif Lindholm

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=20210604230410.s7vuarxir2fqkqtw@leviathan \
    --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