Zhichao: The change is good. What functionality test is done? Thanks Liming >-----Original Message----- >From: Gao, Zhichao >Sent: Thursday, December 12, 2019 10:09 AM >To: devel@edk2.groups.io >Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming ><liming....@intel.com>; Vitaly Cheptsov <vit9...@protonmail.com> >Subject: [PATCH V2 0/2] MdePkg/UefiDevicePathLib: Separate the lib >instances > >REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2298 > >The UefiDevicePathLibOptionalDevicePathProtocolConstructor's >implementation >isn't match with its instance name. >Remove the ASSERT and depex of the gEfiDevicePathUtilitiesProtocolGuid >because of "Optional". > >Add a mandatory instance to force using the DevicePathUtilities, >DevicePathToText and DevicePathFromText protocol with the ASSERT >and depex. > >V2: >The optional lib instance's construction should return success all the >time. >Change the desciption of the optional lib uni file. >Change the copyright date of the mandatory one's uni file. > >Cc: Michael D Kinney <michael.d.kin...@intel.com> >Cc: Liming Gao <liming....@intel.com> >Cc: Vitaly Cheptsov <vit9...@protonmail.com> >Signed-off-by: Zhichao Gao <zhichao....@intel.com> > >Zhichao Gao (2): > MdePkg/UefiDevicePathLib: Separate the device path lib > MdePkg/dsc: Add UefiDevicePathLibMandatoryDevicePathProtocol for build > > ...DevicePathLibMandatoryDevicePathProtocol.c | 469 >++++++++++++++++++ > ...vicePathLibMandatoryDevicePathProtocol.inf | 86 ++++ > ...vicePathLibMandatoryDevicePathProtocol.uni | 18 + > ...iDevicePathLibOptionalDevicePathProtocol.c | 9 +- > ...evicePathLibOptionalDevicePathProtocol.inf | 5 +- > ...evicePathLibOptionalDevicePathProtocol.uni | 6 +- > MdePkg/MdePkg.dsc | 3 +- > 7 files changed, 582 insertions(+), 14 deletions(-) > create mode 100644 >MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibMandatoryDevicePath >Protocol.c > create mode 100644 >MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibMandatoryDevicePath >Protocol.inf > create mode 100644 >MdePkg/Library/UefiDevicePathLib/UefiDevicePathLibMandatoryDevicePath >Protocol.uni > >-- >2.21.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52143): https://edk2.groups.io/g/devel/message/52143 Mute This Topic: https://groups.io/mt/68227874/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-