Re: [PATCH 0/6] PiDxeS3BootScriptLib: Support multiple PCI segment

["Yao, Jiewen" <jiewen.yao@intel.com>]

Reviewed by: jiewen.yao@intel.com

> -----Original Message-----
> From: Zeng, Star
> Sent: Friday, August 19, 2016 3:35 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>;
> Kinney, Michael D <michael.d.kinney@intel.com>; Chan Amy
> <chan.amy@intel.com>; Laszlo Ersek <lersek@redhat.com>; Steele, Kelly
> <kelly.steele@intel.com>; Wei, David <david.wei@intel.com>; Zhang, Chao B
> <chao.b.zhang@intel.com>
> Subject: [PATCH 0/6] PiDxeS3BootScriptLib: Support multiple PCI segment
> 
> Support multiple PCI segment for PCI_CONFIG2 opcodes.
> 
> PiDxeS3BootScriptLib needs to be updated to consume PciSegmentLib
> instead of PciLib. That means platforms need to add PciSegmentLib
> declaration like below in platform dsc if the PciSegmentLib was
> not declared in platform dsc before.
> 
> PciSegmentLib|MdePkg/Library/BasePciSegmentLibPci/BasePciSegmentLibP
> ci.inf
> 
> For platforms only have one segment,
> MdePkg/Library/BasePciSegmentLibPci/BasePciSegmentLibPci.inf is
> recommended
> to be used and declared in platform dsc for PiDxeS3BootScriptLib to have
> equivalent functionality with before.
> 
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Chan Amy <chan.amy@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Kelly Steele <kelly.steele@intel.com>
> Cc: David Wei <david.wei@intel.com>
> Cc: Chao Zhang <chao.b.zhang@intel.com>
> 
> Star Zeng (6):
>   MdeModulePkg PiDxeS3BootScriptLib: Remove the trailing white spaces
>   MdeModulePkg PiDxeS3BootScriptLib: Support multiple PCI segment
>   Vlv2TbltDevicePkg: Declare PciSegmentLib in platform dsc
>   QuarkPlatformPkg: Declare PciSegmentLib in platform dsc
>   QuarkSocPkg/QuarkSocPkg.dsc: Declare PciSegmentLib
>   SecurityPkg/SecurityPkg.dsc: Declare PciSegmentLib
> 
>  .../PiDxeS3BootScriptLib/BootScriptExecute.c       | 411
> +++++++++----------
>  .../BootScriptInternalFormat.h                     |   2 +-
>  .../Library/PiDxeS3BootScriptLib/BootScriptSave.c  | 451
> ++++++++++-----------
>  .../PiDxeS3BootScriptLib/DxeS3BootScriptLib.inf    |   4 +-
>  .../PiDxeS3BootScriptLib/DxeS3BootScriptLib.uni    |   2 +-
>  .../PiDxeS3BootScriptLib/InternalBootScriptLib.h   |  26 +-
>  QuarkPlatformPkg/Quark.dsc                         |   1 +
>  QuarkPlatformPkg/QuarkMin.dsc                      |   1 +
>  QuarkSocPkg/QuarkSocPkg.dsc                        |   1 +
>  SecurityPkg/SecurityPkg.dsc                        |   1 +
>  Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc            |   1 +
>  Vlv2TbltDevicePkg/PlatformPkgIA32.dsc              |   1 +
>  Vlv2TbltDevicePkg/PlatformPkgX64.dsc               |   1 +
>  13 files changed, 450 insertions(+), 453 deletions(-)
> 
> --
> 2.7.0.windows.1