public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Benjamin Doron" <benjamin.doron00@gmail.com>
To: "Oram, Isaac W" <isaac.w.oram@intel.com>
Cc: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	 "Desimone, Nathaniel L" <nathaniel.l.desimone@intel.com>,
	"Sinha, Ankit" <ankit.sinha@intel.com>,
	 "Ni, Ray" <ray.ni@intel.com>,
	"Chaganty, Rangasai V" <rangasai.v.chaganty@intel.com>
Subject: Re: [edk2-devel][edk2-platforms][PATCH v1 2/5] Silicon/Intel: Port SmmControl protocol to PPI for S3
Date: Mon, 29 Aug 2022 20:57:44 -0400	[thread overview]
Message-ID: <CAONYkK_k7tkt961SC6NY4SoU+RhLaKcNvNH5bJvCUycF+5s5xw@mail.gmail.com> (raw)
In-Reply-To: <SA1PR11MB58016C6ED17E938C667B91CDD0769@SA1PR11MB5801.namprd11.prod.outlook.com>

[-- Attachment #1: Type: text/plain, Size: 19291 bytes --]

Right, but Kabylake has a different implementation that retrieves it from
HW registers - PchAcpiBaseGet(). This is probably optional, there is a PCD,
but it's in a different package scope. I don't know how to handle the
Packages in the INF to keep this silicon package agnostic. For that matter,
it might not really be a CFL *plus* shim, because Tigerlake, etc are
different package DECs too.

Best regards,
Benjamin


On Mon, 29 Aug 2022 at 19:17, Oram, Isaac W <isaac.w.oram@intel.com> wrote:

> I think that the shim lib might be overkill.  PmcGetAcpiBase just resolves
> to PcdGet16 (PcdAcpiBaseAddress);
> I think that you should be able to use that PCD for any Intel
> chipset/silicon for the foreseeable future.
>
> I would prefer to see contents of sections in INF files indented, but it
> is a nit.
>
> Regards,
> Isaac
>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Benjamin
> Doron
> Sent: Monday, August 29, 2022 1:36 PM
> To: devel@edk2.groups.io
> Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Sinha, Ankit <
> ankit.sinha@intel.com>; Ni, Ray <ray.ni@intel.com>; Chaganty, Rangasai V <
> rangasai.v.chaganty@intel.com>; Oram, Isaac W <isaac.w.oram@intel.com>
> Subject: [edk2-devel][edk2-platforms][PATCH v1 2/5] Silicon/Intel: Port
> SmmControl protocol to PPI for S3
>
> S3 resume may require communication with SMM, for which we need the
> SmmControl PPI. Therefore, port the DXE drivers to a library, like there is
> for SMM Access.
>
> As the registers are common across Intel platforms in the tree, while a
> helper function definition is not, implement a new library as a
> compatibility shim.
>
> Tested, working on Kabylake. Further testing required after the refactor
> for compatibility.
>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Ankit Sinha <ankit.sinha@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
> Cc: Isaac Oram <isaac.w.oram@intel.com>
> Signed-off-by: Benjamin Doron <benjamin.doron00@gmail.com>
> ---
>  .../BaseIntelCompatShimLibCfl.c               |  24 ++
>  .../BaseIntelCompatShimLibCfl.inf             |  24 ++
>  .../PeiSmmControlLib/PeiSmmControlLib.c       | 309 ++++++++++++++++++
>  .../PeiSmmControlLib/PeiSmmControlLib.inf     |  36 ++
>  .../Include/Library/IntelCompatShimLib.h      |  20 ++
>  .../Include/Library/SmmControlLib.h           |  26 ++
>  .../Intel/IntelSiliconPkg/IntelSiliconPkg.dec |   4 +
>  .../BaseIntelCompatShimLibKbl.c               |  29 ++
>  .../BaseIntelCompatShimLibKbl.inf             |  24 ++
>  9 files changed, 496 insertions(+)
>  create mode 100644
> Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibCfl/BaseIntelCompatShimLibCfl.c
>  create mode 100644
> Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibCfl/BaseIntelCompatShimLibCfl.inf
>  create mode 100644
> Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmControlLib/PeiSmmControlLib.c
>  create mode 100644
> Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmControlLib/PeiSmmControlLib.inf
>  create mode 100644
> Silicon/Intel/IntelSiliconPkg/Include/Library/IntelCompatShimLib.h
>  create mode 100644
> Silicon/Intel/IntelSiliconPkg/Include/Library/SmmControlLib.h
>  create mode 100644
> Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl/BaseIntelCompatShimLibKbl.c
>  create mode 100644
> Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl/BaseIntelCompatShimLibKbl.inf
>
> diff --git
> a/Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibCfl/BaseIntelCompatShimLibCfl.c
> b/Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibCfl/BaseIntelCompatShimLibCfl.c
> new file mode 100644
> index 000000000000..5353126267e6
> --- /dev/null
> +++ b/Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibC
> +++ fl/BaseIntelCompatShimLibCfl.c
> @@ -0,0 +1,24 @@
> +/** @file
> +  A Coffeelake+ compatibility shim
> +
> +  Copyright (c) 2022, Baruch Binyamin Doron<BR>
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <Uefi.h>
> +#include <Library/PmcLib.h>
> +
> +/**
> +  Get PCH ACPI base address.
> +
> +  @retval Address                   Address of PWRM base address.
> +**/
> +UINT16
> +EFIAPI
> +CompatShimGetAcpiBase (
> +  VOID
> +  )
> +{
> +  return PmcGetAcpiBase ();
> +}
> diff --git
> a/Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibCfl/BaseIntelCompatShimLibCfl.inf
> b/Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibCfl/BaseIntelCompatShimLibCfl.inf
> new file mode 100644
> index 000000000000..48b071ed05ae
> --- /dev/null
> +++ b/Silicon/Intel/CoffeelakeSiliconPkg/Library/BaseIntelCompatShimLibC
> +++ fl/BaseIntelCompatShimLibCfl.inf
> @@ -0,0 +1,24 @@
> +## @file
> +# Library description file for the Coffeelake+ compatibility shim # #
> +Copyright (c) 2022, Baruch Binyamin Doron<BR> #
> +SPDX-License-Identifier: BSD-2-Clause-Patent # ##
> +
> +[Defines]
> +INF_VERSION = 0x00010017
> +BASE_NAME = BaseIntelCompatShimLibCfl
> +FILE_GUID = 3D0BB32E-D328-4615-ADFC-782CECC68D53
> +VERSION_STRING = 1.0
> +MODULE_TYPE = BASE
> +LIBRARY_CLASS = IntelCompatShimLib
> +
> +[LibraryClasses]
> +PmcLib
> +
> +[Packages]
> +CoffeelakeSiliconPkg/SiPkg.dec
> +
> +[Sources]
> +BaseIntelCompatShimLibCfl.c
> diff --git
> a/Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmControlLib/PeiSmmControlLib.c
> b/Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmControlLib/PeiSmmControlLib.c
> new file mode 100644
> index 000000000000..80c2c1be90b1
> --- /dev/null
> +++ b/Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmCon
> +++ trolLib/PeiSmmControlLib.c
> @@ -0,0 +1,309 @@
> +/** @file+  This is to publish the SMM Control Ppi instance.++  Copyright
> (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR>+
> SPDX-License-Identifier: BSD-2-Clause-Patent++**/+#include
> <Uefi/UefiBaseType.h>+#include <Library/DebugLib.h>+#include
> <Library/IntelCompatShimLib.h>+#include <Library/IoLib.h>+#include
> <Library/MemoryAllocationLib.h>+#include
> <Library/PeiServicesLib.h>++#include <Ppi/MmControl.h>+#include
> <IndustryStandard/Pci30.h>++#define SMM_CONTROL_PRIVATE_DATA_SIGNATURE
> SIGNATURE_32 ('i', '4', 's', 'c')++typedef struct {+  UINTN
>            Signature;+  EFI_HANDLE                      Handle;+
> EFI_PEI_MM_CONTROL_PPI          SmmControl;+}
> SMM_CONTROL_PRIVATE_DATA;++#define SMM_CONTROL_PRIVATE_DATA_FROM_THIS(a)
> \+        CR (a, \+          SMM_CONTROL_PRIVATE_DATA, \+
> SmmControl, \+          SMM_CONTROL_DEV_SIGNATURE \+      )++//+// Common
> registers:+//+//+// APM Registers+//+#define R_PCH_APM_CNT
>            0xB2+//+// ACPI and legacy I/O register offsets from
> ACPIBASE+//+#define R_PCH_ACPI_PM1_STS                        0x00+#define
> B_PCH_ACPI_PM1_STS_PRBTNOR                BIT11++#define R_PCH_SMI_EN
>                         0x30++#define R_PCH_SMI_STS
>      0x34+#define B_PCH_SMI_STS_APM                         BIT5+#define
> B_PCH_SMI_EN_APMC                         BIT5+#define B_PCH_SMI_EN_EOS
>                       BIT1+#define B_PCH_SMI_EN_GBL_SMI
>   BIT0++/**+  Trigger the software SMI++  @param[in] Data
>  The value to be set on the software SMI data port++  @retval EFI_SUCCESS
>            Function completes successfully+**/+EFI_STATUS+EFIAPI+SmmTrigger
> (+  UINT8   Data+  )+{+  UINT16  ABase;+  UINT32  OutputData;+  UINT32
> OutputPort;++  ABase = CompatShimGetAcpiBase ();++  ///+  /// Enable the
> APMC SMI+  ///+  OutputPort  = ABase + R_PCH_SMI_EN;+  OutputData  =
> IoRead32 ((UINTN) OutputPort);+  OutputData |= (B_PCH_SMI_EN_APMC |
> B_PCH_SMI_EN_GBL_SMI);+  DEBUG (+    (DEBUG_EVENT,+     "The SMI Control
> Port at address %x will be written to %x.\n",+     OutputPort,+
>  OutputData)+    );+  IoWrite32 (+    (UINTN) OutputPort,+    (UINT32)
> (OutputData)+    );++  OutputPort  = R_PCH_APM_CNT;+  OutputData  =
> Data;++  ///+  /// Generate the APMC SMI+  ///+  IoWrite8 (+    (UINTN)
> OutputPort,+    (UINT8) (OutputData)+    );++  return EFI_SUCCESS;+}++/**+
> Clear the SMI status+++  @retval EFI_SUCCESS             The function
> completes successfully+  @retval EFI_DEVICE_ERROR        Something error
> occurred+**/+EFI_STATUS+EFIAPI+SmmClear (+  VOID+  )+{+  UINT16  ABase;+
> UINT32  OutputData;+  UINT32  OutputPort;++  ABase = CompatShimGetAcpiBase
> ();++  ///+  /// Clear the Power Button Override Status Bit, it gates EOS
> from being set.+  ///+  OutputPort  = ABase + R_PCH_ACPI_PM1_STS;+
> OutputData  = B_PCH_ACPI_PM1_STS_PRBTNOR;+  DEBUG (+    (DEBUG_EVENT,+
>  "The PM1 Status Port at address %x will be written to %x.\n",+
>  OutputPort,+     OutputData)+    );+  IoWrite16 (+    (UINTN)
> OutputPort,+    (UINT16) (OutputData)+    );++  ///+  /// Clear the APM SMI
> Status Bit+  ///+  OutputPort  = ABase + R_PCH_SMI_STS;+  OutputData  =
> B_PCH_SMI_STS_APM;+  DEBUG (+    (DEBUG_EVENT,+     "The SMI Status Port at
> address %x will be written to %x.\n",+     OutputPort,+     OutputData)+
> );+  IoWrite32 (+    (UINTN) OutputPort,+    (UINT32) (OutputData)+
> );++  ///+  /// Set the EOS Bit+  ///+  OutputPort  = ABase +
> R_PCH_SMI_EN;+  OutputData  = IoRead32 ((UINTN) OutputPort);+  OutputData
> |= B_PCH_SMI_EN_EOS;+  DEBUG (+    (DEBUG_EVENT,+     "The SMI Control Port
> at address %x will be written to %x.\n",+     OutputPort,+
>  OutputData)+    );+  IoWrite32 (+    (UINTN) OutputPort,+    (UINT32)
> (OutputData)+    );++  ///+  /// There is no need to read EOS back and
> check if it is set.+  /// This can lead to a reading of zero if an SMI
> occurs right after the SMI_EN port read+  /// but before the data is
> returned to the CPU.+  /// SMM Dispatcher should make sure that EOS is set
> after all SMI sources are processed.+  ///+  return EFI_SUCCESS;+}++/**+
> This routine generates an SMI++  @param[in] This                       The
> EFI SMM Control protocol instance+  @param[in, out] ArgumentBuffer
> The buffer of argument+  @param[in, out] ArgumentBufferSize    The size of
> the argument buffer+  @param[in] Periodic                   Periodic or
> not+  @param[in] ActivationInterval         Interval of periodic SMI++
> @retval EFI Status                    Describing the result of the
> operation+  @retval EFI_INVALID_PARAMETER         Some parameter value
> passed is not supported+**/+EFI_STATUS+EFIAPI+Activate (+  IN
> EFI_PEI_SERVICES        **PeiServices,+  IN EFI_PEI_MM_CONTROL_PPI  *
> This,+  IN OUT INT8                *ArgumentBuffer OPTIONAL,+  IN OUT
> UINTN               *ArgumentBufferSize OPTIONAL,+  IN BOOLEAN
>    Periodic OPTIONAL,+  IN UINTN                   ActivationInterval
> OPTIONAL+  )+{+  EFI_STATUS  Status;+  UINT8       Data;++  if (Periodic)
> {+    DEBUG ((DEBUG_WARN, "Invalid parameter\n"));+    return
> EFI_INVALID_PARAMETER;+  }++  // NOTE: Copied from Quark. Matches the usage
> in PiSmmCommunicationPei+  if (ArgumentBuffer == NULL) {+    Data = 0xFF;+
> } else {+    if (ArgumentBufferSize == NULL || *ArgumentBufferSize != 1)
> {+      return EFI_INVALID_PARAMETER;+    }++    Data = *ArgumentBuffer;+
> }+  ///+  /// Clear any pending the APM SMI+  ///+  Status = SmmClear ();+
> if (EFI_ERROR (Status)) {+    return Status;+  }++  return SmmTrigger
> (Data);+}++/**+  This routine clears an SMI++  @param[in] This
>    The EFI SMM Control protocol instance+  @param[in] Periodic
>  Periodic or not++  @retval EFI Status              Describing the result
> of the operation+  @retval EFI_INVALID_PARAMETER   Some parameter value
> passed is not supported+**/+EFI_STATUS+EFIAPI+Deactivate (+  IN
> EFI_PEI_SERVICES        **PeiServices,+  IN EFI_PEI_MM_CONTROL_PPI  *
> This,+  IN BOOLEAN                 Periodic OPTIONAL+  )+{+  if (Periodic)
> {+    return EFI_INVALID_PARAMETER;+  }++  return SmmClear ();+}++/**+
> This function is to install an SMM Control PPI+  - <b>Introduction</b> \n+
>   An API to install an instance of EFI_PEI_MM_CONTROL_PPI. This PPI
> provides a standard+    way for other modules to trigger software SMIs.++
>   @retval EFI_SUCCESS           - Ppi successfully started and installed.+
>   @retval EFI_NOT_FOUND         - Ppi can't be found.+    @retval
> EFI_OUT_OF_RESOURCES  - Ppi does not have enough resources to initialize
> the driver.+**/+EFI_STATUS+EFIAPI+PeiInstallSmmControlPpi (+  VOID+  )+{+
> EFI_STATUS                      Status;+  EFI_PEI_PPI_DESCRIPTOR
> *PpiList;+  SMM_CONTROL_PRIVATE_DATA        *SmmControlPrivate;++  //+  //
> Initialize private data+  //+  SmmControlPrivate  = AllocateZeroPool
> (sizeof (*SmmControlPrivate));+  ASSERT (SmmControlPrivate != NULL);+  if
> (SmmControlPrivate == NULL) {+    return EFI_OUT_OF_RESOURCES;+  }+
> PpiList           = AllocateZeroPool (sizeof (*PpiList));+  ASSERT (PpiList
> != NULL);+  if (PpiList == NULL) {+    return EFI_OUT_OF_RESOURCES;+  }++
> SmmControlPrivate->Signature = SMM_CONTROL_PRIVATE_DATA_SIGNATURE;+
> SmmControlPrivate->Handle    = NULL;++
> SmmControlPrivate->SmmControl.Trigger  = Activate;+
> SmmControlPrivate->SmmControl.Clear    = Deactivate;++  //+  // Install
> PPI+  //+  PpiList->Flags  = (EFI_PEI_PPI_DESCRIPTOR_PPI |
> EFI_PEI_PPI_DESCRIPTOR_TERMINATE_LIST);+  PpiList->Guid   =
> &gEfiPeiMmControlPpiGuid;+  PpiList->Ppi    =
> &SmmControlPrivate->SmmControl;++  Status          = PeiServicesInstallPpi
> (PpiList);+  ASSERT_EFI_ERROR (Status);++  // Unlike driver, do not disable
> SMIs as S3 resume continues+  return EFI_SUCCESS;+}diff --git
> a/Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmControlLib/PeiSmmControlLib.inf
> b/Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmControlLib/PeiSmmControlLib.inf
> new file mode 100644
> index 000000000000..92f2879d82ab
> --- /dev/null
> +++ b/Silicon/Intel/IntelSiliconPkg/Feature/SmmControl/Library/PeiSmmCon
> +++ trolLib/PeiSmmControlLib.inf
> @@ -0,0 +1,36 @@
> +## @file+# Library description file for the SmmControl PPI+#+#
> +Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>+#
> +SPDX-License-Identifier:
> +BSD-2-Clause-Patent+#+##++[Defines]+INF_VERSION = 0x00010017+BASE_NAME
> += PeiSmmControlLib+FILE_GUID =
> +F45D521A-C0DF-4283-A3CA-65AD01B479E7+VERSION_STRING = 1.0+MODULE_TYPE =
> +PEIM+LIBRARY_CLASS =
> +SmmControlLib+++[LibraryClasses]+IntelCompatShimLib+IoLib+DebugLib+Memo
> +ryAllocationLib+PeiServicesLib+++[Packages]+MdePkg/MdePkg.dec+IntelSili
> +conPkg/IntelSiliconPkg.dec+++[Sources]+PeiSmmControlLib.c+++[Ppis]+gEfi
> +PeiMmControlPpiGuid ## PRODUCESdiff --git
> +a/Silicon/Intel/IntelSiliconPkg/Include/Library/IntelCompatShimLib.h
> +b/Silicon/Intel/IntelSiliconPkg/Include/Library/IntelCompatShimLib.h
> new file mode 100644
> index 000000000000..f8621d92e41a
> --- /dev/null
> +++ b/Silicon/Intel/IntelSiliconPkg/Include/Library/IntelCompatShimLib.h
> @@ -0,0 +1,20 @@
> +/** @file
> +  Library description file for compatibility shim
> +
> +  Copyright (c) 2022, Baruch Binyamin Doron<BR>
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <Uefi.h>
> +
> +/**
> +  Get PCH ACPI base address.
> +
> +  @retval Address                   Address of PWRM base address.
> +**/
> +UINT16
> +EFIAPI
> +CompatShimGetAcpiBase (
> +  VOID
> +  );
> diff --git a/Silicon/Intel/IntelSiliconPkg/Include/Library/SmmControlLib.h
> b/Silicon/Intel/IntelSiliconPkg/Include/Library/SmmControlLib.h
> new file mode 100644
> index 000000000000..b532dd13f373
> --- /dev/null
> +++ b/Silicon/Intel/IntelSiliconPkg/Include/Library/SmmControlLib.h
> @@ -0,0 +1,26 @@
> +/** @file+  This is to publish the SMM Control Ppi instance.++  Copyright
> (c) 2019 - 2020, Intel Corporation. All rights reserved.<BR>+
> SPDX-License-Identifier: BSD-2-Clause-Patent++**/+#ifndef
> _SMM_CONTROL_LIB_H_+#define _SMM_CONTROL_LIB_H_++/**+  This function is to
> install an SMM Control PPI+  - <b>Introduction</b> \n+    An API to install
> an instance of EFI_PEI_MM_CONTROL_PPI. This PPI provides a standard+    way
> for other modules to trigger software SMIs.++    @retval EFI_SUCCESS
>    - Ppi successfully started and installed.+    @retval EFI_NOT_FOUND
>    - Ppi can't be found.+    @retval EFI_OUT_OF_RESOURCES  - Ppi does not
> have enough resources to initialize the
> driver.+**/+EFI_STATUS+EFIAPI+PeiInstallSmmControlPpi (+  VOID+
> );+#endifdiff --git a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
> b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
> index c36d130a0197..fc27b394d267 100644
> --- a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
> +++ b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
> @@ -35,6 +35,10 @@
>    #   SmmAccessLib|Include/Library/SmmAccessLib.h +  ## @libraryclass
> Provides services to trigger SMI+  #+
> SmmControlLib|Include/Library/SmmControlLib.h+   ## @libraryclass Provides
> services to access config block   #
>  ConfigBlockLib|Include/Library/ConfigBlockLib.hdiff --git
> a/Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl/BaseIntelCompatShimLibKbl.c
> b/Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl/BaseIntelCompatShimLibKbl.c
> new file mode 100644
> index 000000000000..573f67555aa3
> --- /dev/null
> +++ b/Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl
> +++ /BaseIntelCompatShimLibKbl.c
> @@ -0,0 +1,29 @@
> +/** @file
> +  A Kabylake compatibility shim
> +
> +  Copyright (c) 2022, Baruch Binyamin Doron<BR>
> +  SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#include <Uefi.h>
> +#include <Library/PchCycleDecodingLib.h>
> +
> +/**
> +  Get PCH ACPI base address.
> +
> +  @retval Address                   Address of PWRM base address.
> +**/
> +UINT16
> +EFIAPI
> +CompatShimGetAcpiBase (
> +  VOID
> +  )
> +{
> +  UINT16  Address;
> +
> +  Address = 0;
> +  PchAcpiBaseGet (&Address);
> +
> +  return Address;
> +}
> diff --git
> a/Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl/BaseIntelCompatShimLibKbl.inf
> b/Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl/BaseIntelCompatShimLibKbl.inf
> new file mode 100644
> index 000000000000..af3e5a4726e6
> --- /dev/null
> +++ b/Silicon/Intel/KabylakeSiliconPkg/Library/BaseIntelCompatShimLibKbl
> +++ /BaseIntelCompatShimLibKbl.inf
> @@ -0,0 +1,24 @@
> +## @file
> +# Library description file for the Kabylake compatibility shim # #
> +Copyright (c) 2022, Baruch Binyamin Doron<BR> #
> +SPDX-License-Identifier: BSD-2-Clause-Patent # ##
> +
> +[Defines]
> +INF_VERSION = 0x00010017
> +BASE_NAME = BaseIntelCompatShimLibKbl
> +FILE_GUID = B4A2193E-CF3E-46E6-8617-49E48143B5AB
> +VERSION_STRING = 1.0
> +MODULE_TYPE = BASE
> +LIBRARY_CLASS = IntelCompatShimLib
> +
> +[LibraryClasses]
> +PchCycleDecodingLib
> +
> +[Packages]
> +KabylakeSiliconPkg/SiPkg.dec
> +
> +[Sources]
> +BaseIntelCompatShimLibKbl.c
> --
> 2.37.2
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#92921): https://edk2.groups.io/g/devel/message/92921
> Mute This Topic: https://groups.io/mt/93335519/1492418
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [isaac.w.oram@intel.com]
> -=-=-=-=-=-=
>
>
>

[-- Attachment #2: Type: text/html, Size: 22831 bytes --]

  reply	other threads:[~2022-08-30  0:57 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-29 20:36 [edk2-devel][edk2-platforms][PATCH v1 0/5] Implement S3 resume Benjamin Doron
2022-08-29 20:36 ` [edk2-devel][edk2-platforms][PATCH v1 1/5] IntelSiliconPkg/Feature/PeiSmmAccessLibSmramc: Implement chipset support Benjamin Doron
2022-08-29 22:27   ` Oram, Isaac W
2022-08-30  0:26     ` Ni, Ray
2022-08-30  1:13       ` Benjamin Doron
2022-08-29 20:36 ` [edk2-devel][edk2-platforms][PATCH v1 2/5] Silicon/Intel: Port SmmControl protocol to PPI for S3 Benjamin Doron
2022-08-29 23:17   ` Oram, Isaac W
2022-08-30  0:57     ` Benjamin Doron [this message]
2022-08-30 18:39       ` Oram, Isaac W
2022-08-30 23:25         ` Chaganty, Rangasai V
2022-08-29 20:36 ` [edk2-devel][edk2-platforms][PATCH v1 3/5] S3FeaturePkg: Implement working S3 resume Benjamin Doron
2022-08-30  0:16   ` Oram, Isaac W
2022-09-02 19:31     ` Benjamin Doron
2022-08-29 20:36 ` [edk2-devel][edk2-platforms][PATCH v1 4/5] MinPlatformPkg: " Benjamin Doron
2022-08-30  0:25   ` Oram, Isaac W
2022-08-29 20:36 ` [edk2-devel][edk2-platforms][PATCH v1 5/5] KabylakeOpenBoardPkg: Example of board S3 Benjamin Doron

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=CAONYkK_k7tkt961SC6NY4SoU+RhLaKcNvNH5bJvCUycF+5s5xw@mail.gmail.com \
    --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