From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.43, mailfrom: dandan.bi@intel.com) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by groups.io with SMTP; Tue, 06 Aug 2019 01:10:55 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Aug 2019 01:10:55 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,352,1559545200"; d="scan'208";a="373340173" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by fmsmga005.fm.intel.com with ESMTP; 06 Aug 2019 01:10:55 -0700 Received: from fmsmsx113.amr.corp.intel.com (10.18.116.7) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 6 Aug 2019 01:10:55 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by FMSMSX113.amr.corp.intel.com (10.18.116.7) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 6 Aug 2019 01:10:54 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.112]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.80]) with mapi id 14.03.0439.000; Tue, 6 Aug 2019 16:10:52 +0800 From: "Dandan Bi" To: "devel@edk2.groups.io" , "Jin, Eric" Subject: Re: [edk2-devel] [PATCH 00/14] Multiple Controllers Support solution Thread-Topic: [edk2-devel] [PATCH 00/14] Multiple Controllers Support solution Thread-Index: AQHVSESYZv2UsKcgSEauLM+UbL+6Vqbty9ww Date: Tue, 6 Aug 2019 08:10:52 +0000 Message-ID: <3C0D5C461C9E904E8F62152F6274C0BB40C3BBF3@SHSMSX104.ccr.corp.intel.com> References: <20190801083847.14524-1-eric.jin@intel.com> In-Reply-To: <20190801083847.14524-1-eric.jin@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: dandan.bi@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Eric, This patch series also can be consolidated to a smaller patch set. Please help double check. Thanks, Dandan > -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Eric Jin > Sent: Thursday, August 1, 2019 4:39 PM > To: devel@edk2.groups.io > Subject: [edk2-devel] [PATCH 00/14] Multiple Controllers Support solutio= n >=20 > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1525 >=20 > The patch set is to support drivers that manage multiple controllers and= also > provide a firmware update capability to each managed controller. >=20 > The following modules are related to Multiple Controllers Support soluti= on >=20 > FmpDevicePkg\FmpDxe\FmpDxe.inf - Driver to manage multiple controllers > and provide the firmware update capability to each managed controller. > FmpDevicePkg\CapsuleUpdatePolicyDxe\CapsuleUpdatePolicyDxe.inf - > Driver to produce the Capsule Update Policy Protocol using the services = of > the CapsuleUpdatePolicyLib class. The protocol is a private interface to= the > FmpDevicePkg > FmpDevicePkg\Library\CapsuleUpdatePolicyLibOnProtocol\CapsuleUpdateP > olicyLibOnProtocol.inf - CapsuleUpdatePolicyLib instance that uses the > services of the Capsule Update Policy Protocol produced by > CapsuleUpdatePolicyDxe > FmpDevicePkg\Library\CapsuleUpdatePolicyLibNull\CapsuleUpdatePolicyLib > Null.inf - Null CapsuleUpdatePolicyLib instance and the template for pla= tform > specific implementation > FmpDevicePkg\Library\FmpDeviceLibNull\FmpDeviceLibNull.inf - Null > FmpDeviceLib instance and the template for platform specific > implementation >=20 >=20 > Eric Jin (14): > FmpDevicePkg: Add UEFI_DRIVER support > FmpDevicePkg: Add APIs to FmpDeviceLib > FmpDEvicePkg/FmpDeviceLibNull: Implement new APIs > FmpDevicePkg/FmpDxe: Use new FmpDeviceLib APIs > FmpDevicePkg/FmpDxe: Different variable for each FMP Descriptor > FmpDevicePkg: Add Capsule Update Policy Protocol > FmpDevicePkg/FmpDxe: Improve all DEBUG() messages > FmpDevicePkg/FmpDxe: Add PcdFmpDeviceImageTypeIdGuid > FmpDevicePkg/FmpDxe: Add PcdFmpDeviceStorageAccessEnable > FmpDevicePkg/FmpDxe: Remove use of CatSprint() > FmpDevicePkg/FmpDxe: Fix SetLowestSupportedVersionInVariable() > FmpDevicePkg/FmpDxe: Change where FmpControllerState is updated > FmpDevicePkg/FmpDxe: Fix code indent > FmpDevicePkg/FmpDxe: Add FmpDxe.h to [Sources] >=20 > .../CapsuleUpdatePolicyDxe.c | 173 ++++ > .../CapsuleUpdatePolicyDxe.h | 135 +++ > .../CapsuleUpdatePolicyDxe.inf | 48 + > .../CapsuleUpdatePolicyDxe.uni | 14 + > .../CapsuleUpdatePolicyDxeExtra.uni | 14 + > FmpDevicePkg/FmpDevicePkg.dec | 43 +- > FmpDevicePkg/FmpDevicePkg.dsc | 64 +- > FmpDevicePkg/FmpDevicePkg.uni | 16 +- > FmpDevicePkg/FmpDxe/DetectTestKey.c | 16 +- > FmpDevicePkg/FmpDxe/FmpDxe.c | 787 ++++++++++------ > FmpDevicePkg/FmpDxe/FmpDxe.h | 355 ++++++++ > FmpDevicePkg/FmpDxe/FmpDxe.inf | 7 +- > FmpDevicePkg/FmpDxe/FmpDxeLib.inf | 7 +- > FmpDevicePkg/FmpDxe/VariableSupport.c | 844 +++++++++++++----- > FmpDevicePkg/FmpDxe/VariableSupport.h | 135 ++- > FmpDevicePkg/Include/Library/FmpDeviceLib.h | 104 ++- > .../CapsuleUpdatePolicyLibOnProtocol.c | 171 ++++ > .../CapsuleUpdatePolicyLibOnProtocol.inf | 40 + > .../CapsuleUpdatePolicyLibOnProtocol.uni | 15 + > .../Library/FmpDeviceLibNull/FmpDeviceLib.c | 93 +- > .../FmpDeviceLibNull/FmpDeviceLibNull.inf | 4 +- > .../FmpPayloadHeaderLibV1.inf | 4 +- > .../Library/FmpPayloadHeaderLib.h | 0 > .../Protocol/CapsuleUpdatePolicy.h | 132 +++ > 24 files changed, 2635 insertions(+), 586 deletions(-) > create mode 100644 > FmpDevicePkg/CapsuleUpdatePolicyDxe/CapsuleUpdatePolicyDxe.c > create mode 100644 > FmpDevicePkg/CapsuleUpdatePolicyDxe/CapsuleUpdatePolicyDxe.h > create mode 100644 > FmpDevicePkg/CapsuleUpdatePolicyDxe/CapsuleUpdatePolicyDxe.inf > create mode 100644 > FmpDevicePkg/CapsuleUpdatePolicyDxe/CapsuleUpdatePolicyDxe.uni > create mode 100644 > FmpDevicePkg/CapsuleUpdatePolicyDxe/CapsuleUpdatePolicyDxeExtra.uni > create mode 100644 FmpDevicePkg/FmpDxe/FmpDxe.h > create mode 100644 > FmpDevicePkg/Library/CapsuleUpdatePolicyLibOnProtocol/CapsuleUpdateP > olicyLibOnProtocol.c > create mode 100644 > FmpDevicePkg/Library/CapsuleUpdatePolicyLibOnProtocol/CapsuleUpdateP > olicyLibOnProtocol.inf > create mode 100644 > FmpDevicePkg/Library/CapsuleUpdatePolicyLibOnProtocol/CapsuleUpdateP > olicyLibOnProtocol.uni > rename FmpDevicePkg/{Include =3D> > PrivateInclude}/Library/FmpPayloadHeaderLib.h (100%) > create mode 100644 > FmpDevicePkg/PrivateInclude/Protocol/CapsuleUpdatePolicy.h >=20 > -- > 2.20.1.windows.1 >=20 >=20 >=20