For the series...

Reviewed-by: Nate DeSimone <nathaniel.l.desim...@intel.com>

-----Original Message-----
From: Oram, Isaac W <isaac.w.o...@intel.com> 
Sent: Wednesday, September 14, 2022 11:40 AM
To: devel@edk2.groups.io
Cc: Oram, Isaac W <isaac.w.o...@intel.com>; Chaganty, Rangasai V 
<rangasai.v.chaga...@intel.com>; Desimone, Nathaniel L 
<nathaniel.l.desim...@intel.com>; Gao, Liming <gaolim...@byosoft.com.cn>; Chiu, 
Chasel <chasel.c...@intel.com>; Dong, Eric <eric.d...@intel.com>; Benjamin 
Doron <benjamin.doro...@gmail.com>
Subject: [edk2-devel][edk2-platforms][PATCH V1 0/2] Platforms/Intel: Build fixes

The S3FeaturePkg changes in
  [edk2-platforms][PATCH v3 2/4] S3FeaturePkg: Implement working S3 resume

Introduces some build issues with standalone package build for S3FeaturePkg and 
AdvancedFeaturePkg.  There are also some type cast related compiler warnings.

We do not currently have continuous integration testing.
We do not currently have documented build testing configuration requirements.
Therefore I am just fixing the minor issues and intend to merge both patch 
series together to maintain git bisect to the best of my ability.
I do plan to document required and recommended board port and feature pkg 
builds.

Note that the use of UINTN for intermediate data instead of 
EFI_PHYSICAL_ADDRESS is only to be consistent with other ACPI implementations 
of similar functionality.

Cc: Sai Chaganty <rangasai.v.chaga...@intel.com>
Cc: Nate DeSimone <nathaniel.l.desim...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Chasel Chiu <chasel.c...@intel.com>
Cc: Eric Dong <eric.d...@intel.com>
Cc: Benjamin Doron <benjamin.doro...@gmail.com>
Signed-off-by: Isaac Oram <isaac.w.o...@intel.com>

Isaac Oram (2):
  S3FeaturePkg/Build: Add libraries needed by S3FeaturePkg
  MinPlatformPkg/S3: Use EFI_PHYSICAL_ADDRESS for address

 .../Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc    |  3 +++
 .../Intel/PowerManagement/S3FeaturePkg/S3Dxe/S3Dxe.c   | 10 +++++-----
 .../PowerManagement/S3FeaturePkg/S3FeaturePkg.dsc      |  3 +++
 .../Intel/PowerManagement/S3FeaturePkg/S3Pei/S3Pei.c   |  2 +-
 .../Intel/MinPlatformPkg/Include/AcpiS3MemoryNvData.h  |  4 ++--
 5 files changed, 14 insertions(+), 8 deletions(-)

--
2.36.1.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95015): https://edk2.groups.io/g/devel/message/95015
Mute This Topic: https://groups.io/mt/93684404/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to