Re: [edk2-devel] [PATCH V2 00/10] Multiple Controllers Support solution

["Eric Jin" <eric.jin@intel.com>]

Liming,

The differences between V2 and V1 are listed below.

1) The series is composed of 10 patches in V2 (14 in V1). patch 14 is merged into patch 4, and patch 11/12/13 are merged into patch 5.
2) Try to fix the issue exposed by ECC. 

Btw, the patch of edk2-platform Platform\Intel\Vlv2TbltDevicePkg\Feature\Capsule\Library\FmpDeviceLib to support new APIs is provided in separated patch https://edk2.groups.io/g/devel/message/45328

Best Regards
Eric

-----Original Message-----
From: Gao, Liming 
Sent: Monday, August 12, 2019 1:19 PM
To: devel@edk2.groups.io; Jin, Eric <eric.jin@intel.com>
Subject: RE: [edk2-devel] [PATCH V2 00/10] Multiple Controllers Support solution

Eric:
  Can you list the difference compared to version 1?

Thanks
Liming
>-----Original Message-----
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of 
>Eric Jin
>Sent: Monday, August 12, 2019 9:46 AM
>To: devel@edk2.groups.io
>Subject: [edk2-devel] [PATCH V2 00/10] Multiple Controllers Support 
>solution
>
>REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1525
>
>The patch set is to support drivers that manage multiple controllers 
>and also provide a firmware update capability to each managed controller.
>
>The following modules are related to Multiple Controllers Support 
>solution
>
>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\CapsuleUpdatePo
>licyLibOnProtocol.inf -
>CapsuleUpdatePolicyLib instance that uses the services of the Capsule 
>Update Policy Protocol produced by CapsuleUpdatePolicyDxe 
>FmpDevicePkg\Library\CapsuleUpdatePolicyLibNull\CapsuleUpdatePolicyLibN
>ull.inf -
>Null CapsuleUpdatePolicyLib instance and the template for platform 
>specific implementation 
>FmpDevicePkg\Library\FmpDeviceLibNull\FmpDeviceLibNull.inf - Null 
>FmpDeviceLib instance and the template for platform specific 
>implementation
>
>Eric Jin (10):
>  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()
>
> .../CapsuleUpdatePolicyDxe.c                  | 173 ++++
> .../CapsuleUpdatePolicyDxe.h                  | 140 +++
> .../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                  | 792 ++++++++++------
> 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, 2644 insertions(+), 587 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/CapsuleUpdatePo
>licyLibOnProtocol.c
> create mode 100644
>FmpDevicePkg/Library/CapsuleUpdatePolicyLibOnProtocol/CapsuleUpdatePo
>licyLibOnProtocol.inf
> create mode 100644
>FmpDevicePkg/Library/CapsuleUpdatePolicyLibOnProtocol/CapsuleUpdatePo
>licyLibOnProtocol.uni
> rename FmpDevicePkg/{Include =>
>PrivateInclude}/Library/FmpPayloadHeaderLib.h (100%)  create mode 
>100644 FmpDevicePkg/PrivateInclude/Protocol/CapsuleUpdatePolicy.h
>
>--
>2.20.1.windows.1
>
>
>