I see the information has been added in BZ. This patch set is OK to me. 
Reviewed-by: Liming Gao <[email protected]>


>-----Original Message-----
>From: [email protected] [mailto:[email protected]] On Behalf Of
>Liming Gao
>Sent: Friday, April 26, 2019 12:41 PM
>To: Zhang, Shenglei <[email protected]>; [email protected]
>Cc: Justen, Jordan L <[email protected]>; Laszlo Ersek
><[email protected]>; Ard Biesheuvel <[email protected]>; Carsey,
>Jaben <[email protected]>; Ni, Ray <[email protected]>; Kinney,
>Michael D <[email protected]>
>Subject: Re: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code
>
>Please add the comments in BZ on why Mps.h can't be removed.
>
>Thanks
>Liming
>>-----Original Message-----
>>From: Zhang, Shenglei
>>Sent: Friday, April 26, 2019 11:02 AM
>>To: [email protected]
>>Cc: Justen, Jordan L <[email protected]>; Laszlo Ersek
>><[email protected]>; Ard Biesheuvel <[email protected]>; Carsey,
>>Jaben <[email protected]>; Ni, Ray <[email protected]>; Kinney,
>>Michael D <[email protected]>; Gao, Liming
><[email protected]>
>>Subject: [PATCH v2 0/4] Remove IPF and related code
>>
>>Itanium arch is not supported any longer. So remove IPF and
>>related code.
>>https://bugzilla.tianocore.org/show_bug.cgi?id=1560
>>
>>v2: Withdraw the removal of Mps.h and related changes.
>>
>>Cc: Jordan Justen <[email protected]>
>>Cc: Laszlo Ersek <[email protected]>
>>Cc: Ard Biesheuvel <[email protected]>
>>Cc: Jaben Carsey <[email protected]>
>>Cc: Ray Ni <[email protected]>
>>Cc: Michael D Kinney <[email protected]>
>>Cc: Liming Gao <[email protected]>
>>Shenglei Zhang (4):
>>  OvmfPkg/PlatformBootManagerLib: Remove dependency on Mps.h
>>  ShellPkg/UefiShellDebug1CommandsLib
>>  ShellPkg/UefiHandleParsingLib: Remove some unused Guids
>>  MdePkg: Removed IPF related code
>>
>> MdePkg/Include/Guid/SalSystemTable.h          |   25 -
>> MdePkg/Include/IndustryStandard/Pal.h         | 3296 -----------------
>> MdePkg/Include/IndustryStandard/Sal.h         |  909 -----
>> MdePkg/Include/Library/ExtendedSalLib.h       |  488 ---
>> MdePkg/Include/Library/PalLib.h               |   57 -
>> MdePkg/Include/Library/SalLib.h               |   53 -
>> .../Include/Protocol/ExtendedSalBootService.h |  211 --
>> .../Protocol/ExtendedSalServiceClasses.h      |  272 --
>> MdePkg/Include/Protocol/McaInitPmi.h          |  201 -
>> MdePkg/MdePkg.dec                             |   26 -
>> .../PlatformBootManagerLib/BdsPlatform.h      |    1 -
>> .../UefiHandleParsingLib.c                    |   17 -
>> .../UefiHandleParsingLib.inf                  |   17 -
>> .../UefiHandleParsingLib.uni                  |   17 -
>> .../Library/UefiShellDebug1CommandsLib/Dmem.c |    5 -
>> .../UefiShellDebug1CommandsLib.inf            |    1 -
>> 16 files changed, 5596 deletions(-)
>> delete mode 100644 MdePkg/Include/Guid/SalSystemTable.h
>> delete mode 100644 MdePkg/Include/IndustryStandard/Pal.h
>> delete mode 100644 MdePkg/Include/IndustryStandard/Sal.h
>> delete mode 100644 MdePkg/Include/Library/ExtendedSalLib.h
>> delete mode 100644 MdePkg/Include/Library/PalLib.h
>> delete mode 100644 MdePkg/Include/Library/SalLib.h
>> delete mode 100644 MdePkg/Include/Protocol/ExtendedSalBootService.h
>> delete mode 100644
>MdePkg/Include/Protocol/ExtendedSalServiceClasses.h
>> delete mode 100644 MdePkg/Include/Protocol/McaInitPmi.h
>>
>>--
>>2.18.0.windows.1
>
>
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#39702): https://edk2.groups.io/g/devel/message/39702
Mute This Topic: https://groups.io/mt/31349515/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to