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.88, mailfrom: eric.jin@intel.com) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by groups.io with SMTP; Thu, 08 Aug 2019 00:59:17 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Aug 2019 00:59:15 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,360,1559545200"; d="scan'208";a="193103278" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by fmsmga001.fm.intel.com with ESMTP; 08 Aug 2019 00:59:16 -0700 Received: from fmsmsx112.amr.corp.intel.com (10.18.116.6) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 8 Aug 2019 00:59:16 -0700 Received: from shsmsx107.ccr.corp.intel.com (10.239.4.96) by FMSMSX112.amr.corp.intel.com (10.18.116.6) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 8 Aug 2019 00:59:15 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.139]) by SHSMSX107.ccr.corp.intel.com ([169.254.9.65]) with mapi id 14.03.0439.000; Thu, 8 Aug 2019 15:59:14 +0800 From: "Eric Jin" To: "Gao, Liming" , "devel@edk2.groups.io" , "Bi, Dandan" 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+6Vqbty9wwgAKEM4CAAJ1l0A== Date: Thu, 8 Aug 2019 07:59:14 +0000 Message-ID: References: <20190801083847.14524-1-eric.jin@intel.com> <3C0D5C461C9E904E8F62152F6274C0BB40C3BBF3@SHSMSX104.ccr.corp.intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4CC979@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4CC979@SHSMSX104.ccr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYzkzNmY0MzgtZDU1Ny00OGM0LWEzZTItZDY0ZGYzY2JhNmIyIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiWW54aFZ1WlNZeU1RcXppMFVqVU5OdGd6cW81Szg2RUkyXC85ZDQyQ0s0TWc4cWswNjRkOGRnUU1uenZ0N3N2a3QifQ== x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: eric.jin@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Liming and Dandan, Thank you for the comments. Will follow up.=20 Best Regards Eric -----Original Message----- From: Gao, Liming=20 Sent: Thursday, August 8, 2019 2:35 PM To: devel@edk2.groups.io; Bi, Dandan ; Jin, Eric Subject: RE: [edk2-devel] [PATCH 00/14] Multiple Controllers Support solut= ion Eric: I review this patch set. I think patch 14 can be merged into patch 4, p= atch 13 can be merged into patch 5.=20 And, patch 2 introduces new APIs in FmpDeviceLib. I see edk2-platform Pl= atform\Intel\Vlv2TbltDevicePkg\Feature\Capsule\Library\FmpDeviceLib is the FmpDeviceLib library instance. Have you the patch to update it to= support new APIs? Last, ECC reports some issues. Please make sure they be fixed in next ve= rsion.=20 "All include file contents should be guarded by a #ifndef statement.",FmpD= evicePkg\CapsuleUpdatePolicyDxe\CapsuleUpdatePolicyDxe.h,1,"" FmpDevicePkg\Library\FmpDeviceLibNull\FmpDeviceLib.c,71,"in Comment, <@par= am[in, out]> does NOT have [out] " FmpDevicePkg\Include\Library\FmpDeviceLib.h,107,"in Comment, <@param[in, o= ut]> does NOT have [out] " FmpDevicePkg\FmpDxe\FmpDxe.c,202,"No doxygen tags in comment " FmpDevicePkg\FmpDxe\FmpDxe.c,261,"No doxygen tags in comment " Thanks Liming >-----Original Message----- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of=20 >Dandan Bi >Sent: Tuesday, August 06, 2019 4:11 PM >To: devel@edk2.groups.io; Jin, Eric >Subject: Re: [edk2-devel] [PATCH 00/14] Multiple Controllers Support=20 >solution > >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= =20 >> Eric Jin >> Sent: Thursday, August 1, 2019 4:39 PM >> To: devel@edk2.groups.io >> Subject: [edk2-devel] [PATCH 00/14] Multiple Controllers Support=20 >> solution >> >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1525 >> >> The patch set is to support drivers that manage multiple controllers=20 >> and also provide a firmware update capability to each managed controlle= r. >> >> The following modules are related to Multiple Controllers Support=20 >> solution >> >> FmpDevicePkg\FmpDxe\FmpDxe.inf - Driver to manage multiple=20 >> controllers and provide the firmware update capability to each managed = controller. >> FmpDevicePkg\CapsuleUpdatePolicyDxe\CapsuleUpdatePolicyDxe.inf -=20 >> Driver to produce the Capsule Update Policy Protocol using the=20 >> services of the CapsuleUpdatePolicyLib class. The protocol is a=20 >> private interface to the FmpDevicePkg >> >FmpDevicePkg\Library\CapsuleUpdatePolicyLibOnProtocol\CapsuleUpdateP >> olicyLibOnProtocol.inf - CapsuleUpdatePolicyLib instance that uses=20 >> the services of the Capsule Update Policy Protocol produced by=20 >> CapsuleUpdatePolicyDxe=20 >> FmpDevicePkg\Library\CapsuleUpdatePolicyLibNull\CapsuleUpdatePolicyLi >> b Null.inf - Null CapsuleUpdatePolicyLib instance and the template=20 >> for >platform >> specific implementation >> FmpDevicePkg\Library\FmpDeviceLibNull\FmpDeviceLibNull.inf - Null=20 >> FmpDeviceLib instance and the template for platform specific=20 >> implementation >> >> >> 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] >> >> .../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=20 >> 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=20 >> 100644 FmpDevicePkg/PrivateInclude/Protocol/CapsuleUpdatePolicy.h >> >> -- >> 2.20.1.windows.1 >> >> >> > > >