From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by ml01.01.org (Postfix) with ESMTP id 1B1771A1E05 for ; Fri, 19 Aug 2016 06:35:35 -0700 (PDT) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga102.fm.intel.com with ESMTP; 19 Aug 2016 06:35:34 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,544,1464678000"; d="scan'208";a="868234823" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by orsmga003.jf.intel.com with ESMTP; 19 Aug 2016 06:35:34 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.248.2; Fri, 19 Aug 2016 06:35:34 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.147]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.8]) with mapi id 14.03.0248.002; Fri, 19 Aug 2016 21:35:31 +0800 From: "Yao, Jiewen" To: "Zeng, Star" , "edk2-devel@lists.01.org" CC: "Kinney, Michael D" , Chan Amy , Laszlo Ersek , "Steele, Kelly" , "Wei, David" , "Zhang, Chao B" Thread-Topic: [PATCH 0/6] PiDxeS3BootScriptLib: Support multiple PCI segment Thread-Index: AQHR+exUcs48KUpDCEOrLIdOYwkAFqBQSTXQ Date: Fri, 19 Aug 2016 13:35:31 +0000 Message-ID: <74D8A39837DF1E4DA445A8C0B3885C5002803881@shsmsx102.ccr.corp.intel.com> References: <1471592131-143456-1-git-send-email-star.zeng@intel.com> In-Reply-To: <1471592131-143456-1-git-send-email-star.zeng@intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH 0/6] PiDxeS3BootScriptLib: Support multiple PCI segment X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 19 Aug 2016 13:35:35 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable 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 ; Yao, Jiewen ; > Kinney, Michael D ; Chan Amy > ; Laszlo Ersek ; Steele, Kelly > ; Wei, David ; Zhang, Chao B > > Subject: [PATCH 0/6] PiDxeS3BootScriptLib: Support multiple PCI segment >=20 > Support multiple PCI segment for PCI_CONFIG2 opcodes. >=20 > 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. >=20 > PciSegmentLib|MdePkg/Library/BasePciSegmentLibPci/BasePciSegmentLibP > ci.inf >=20 > 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. >=20 > Cc: Jiewen Yao > Cc: Michael D Kinney > Cc: Chan Amy > Cc: Laszlo Ersek > Cc: Kelly Steele > Cc: David Wei > Cc: Chao Zhang >=20 > 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 >=20 > .../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(-) >=20 > -- > 2.7.0.windows.1