public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Zhang, Shenglei" <shenglei.zhang@intel.com>
To: "Wu, Hao A" <hao.a.wu@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Ni, Ruiyu" <ruiyu.ni@intel.com>, "Gao, Liming" <liming.gao@intel.com>
Subject: Re: [PATCH v2 0/2] Remove DuetPkg and unused tools
Date: Wed, 28 Nov 2018 08:13:10 +0000	[thread overview]
Message-ID: <C0706E73DB8C124D9B9C38AA364E5D5E04BB6977@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <B80AF82E9BFB8E4FBD8C89DA810C6A093C8500B0@SHSMSX104.ccr.corp.intel.com>

Hi Hao,

I thinks it's a good advice. Thanks for your comments and I'll take relevant actions.

Thanks,
Shenglei

> -----Original Message-----
> From: Wu, Hao A
> Sent: Wednesday, November 28, 2018 4:06 PM
> To: Wu, Hao A <hao.a.wu@intel.com>; Zhang, Shenglei
> <shenglei.zhang@intel.com>; edk2-devel@lists.01.org
> Cc: Ni, Ruiyu <ruiyu.ni@intel.com>; Gao, Liming <liming.gao@intel.com>
> Subject: RE: [PATCH v2 0/2] Remove DuetPkg and unused tools
> 
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> > Wu, Hao A
> > Sent: Wednesday, November 28, 2018 3:39 PM
> > To: Zhang, Shenglei; edk2-devel@lists.01.org
> > Cc: Ni, Ruiyu; Gao, Liming
> > Subject: Re: [edk2] [PATCH v2 0/2] Remove DuetPkg and unused tools
> >
> > > -----Original Message-----
> > > From: Zhang, Shenglei
> > > Sent: Tuesday, November 27, 2018 10:42 AM
> > > To: edk2-devel@lists.01.org
> > > Cc: Ni, Ruiyu; Wu, Hao A; Zhu, Yonghong; Gao, Liming
> > > Subject: [PATCH v2 0/2] Remove DuetPkg and unused tools
> > >
> > > DuetPkg depends on Legacy BIOS to provide a UEFI environment.
> > > It was invented in the era when UEFI environment is hard to find.
> > > Since now UEFI is very popular in PC area, we could stop the
> > > official support of this package and remove it from the master.
> > > And moreover, the tools only used by DuetPkg can also be removed.
> > > https://bugzilla.tianocore.org/show_bug.cgi?id=1322
> > >
> > > The changes are placed in
> > > https://github.com/shenglei10/edk2/tree/duet
> > >
> > > v2:Remove these tools in Makefile and GNUmakefile.
> >
> > Hi,
> >
> > For DuetPkg patch (2/2), I am wondering we can move the Makefile
> changes
> > in the
> > BaseTools to either A) or B) below:
> >
> > A) To the 1/2 patch
> > B) To a separate patch
> 
> Another thing is that, it makes more sense to me that the deletion of the
> DuetPkg should be the first patch of the series. Then, it comes the removal
> of
> the tools within BaseTools that are only used by DuetPkg.
> 
> Best Regards,
> Hao Wu
> 
> >
> >
> > Best Regards,
> > Hao Wu
> >
> > >
> > > Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> > > Cc: Hao Wu <hao.a.wu@intel.com>
> > > Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> > > Cc: Liming Gao <liming.gao@intel.com>
> > > Shenglei Zhang (2):
> > >   BaseTools: Remove tools only used by DuetPkg
> > >   DuetPkg: Remove DuetPkg
> > >
> > >  BaseTools/Source/BinaryFiles.txt              |    4 -
> > >  BaseTools/Source/C/BootSectImage/GNUmakefile  |   21 -
> > >  BaseTools/Source/C/BootSectImage/Makefile     |   22 -
> > >  .../Source/C/BootSectImage/bootsectimage.c    |  955 ------
> > >  BaseTools/Source/C/BootSectImage/fat.h        |  152 -
> > >  BaseTools/Source/C/BootSectImage/mbr.h        |   58 -
> > >  BaseTools/Source/C/EfiLdrImage/EfiLdrImage.c  |  319 --
> > >  BaseTools/Source/C/EfiLdrImage/GNUmakefile    |   21 -
> > >  BaseTools/Source/C/EfiLdrImage/Makefile       |   22 -
> > >  BaseTools/Source/C/GNUmakefile                |    5 -
> > >  BaseTools/Source/C/GenBootSector/FatFormat.h  |  152 -
> > >  .../Source/C/GenBootSector/GenBootSector.c    |  823 -----
> > >  .../Source/C/GenBootSector/GetDrvNumOffset.c  |   73 -
> > >  BaseTools/Source/C/GenBootSector/Makefile     |   22 -
> > >  BaseTools/Source/C/GenPage/GNUmakefile        |   21 -
> > >  BaseTools/Source/C/GenPage/GenPage.c          |  441 ---
> > >  BaseTools/Source/C/GenPage/Makefile           |   22 -
> > >  BaseTools/Source/C/GenPage/VirtualMemory.h    |  122 -
> > >  .../Source/C/GnuGenBootSector/FatFormat.h     |  152 -
> > >  .../Source/C/GnuGenBootSector/GNUmakefile     |   21 -
> > >  .../C/GnuGenBootSector/GnuGenBootSector.c     |  455 ---
> > >  BaseTools/Source/C/Makefile                   |    4 -
> > >  BaseTools/toolsetup.bat                       |    4 -
> > >  DuetPkg/AcpiResetDxe/Reset.c                  |  212 --
> > >  DuetPkg/AcpiResetDxe/Reset.inf                |   47 -
> > >  DuetPkg/BiosVideoThunkDxe/BiosVideo.c         | 2822 -----------------
> > >  DuetPkg/BiosVideoThunkDxe/BiosVideo.h         |  504 ---
> > >  DuetPkg/BiosVideoThunkDxe/BiosVideo.inf       |   50 -
> > >  DuetPkg/BiosVideoThunkDxe/ComponentName.c     |  166 -
> > >  DuetPkg/BiosVideoThunkDxe/LegacyBiosThunk.c   |  220 --
> > >  .../BiosVideoThunkDxe/VesaBiosExtensions.h    |  457 ---
> > >  DuetPkg/BootSector/BootSector.inf             |   79 -
> > >  DuetPkg/BootSector/FILE.LST                   |   39 -
> > >  DuetPkg/BootSector/GNUmakefile                |  140 -
> > >  DuetPkg/BootSector/Gpt.S                      |  297 --
> > >  DuetPkg/BootSector/Gpt.asm                    |  294 --
> > >  DuetPkg/BootSector/Makefile                   |  173 -
> > >  DuetPkg/BootSector/Mbr.S                      |  262 --
> > >  DuetPkg/BootSector/Mbr.asm                    |  261 --
> > >  DuetPkg/BootSector/bin/Gpt.com                |  Bin 512 -> 0 bytes
> > >  DuetPkg/BootSector/bin/Mbr.com                |  Bin 512 -> 0 bytes
> > >  DuetPkg/BootSector/bin/Readme.txt             |    8 -
> > >  DuetPkg/BootSector/bin/St16_64.com            |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bin/St32_64.com            |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bin/Start.com              |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bin/Start16.com            |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bin/Start32.com            |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bin/Start64.com            |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bin/bootsect.com           |  Bin 512 -> 0 bytes
> > >  DuetPkg/BootSector/bin/bs16.com               |  Bin 512 -> 0 bytes
> > >  DuetPkg/BootSector/bin/bs32.com               |  Bin 512 -> 0 bytes
> > >  DuetPkg/BootSector/bin/efi32.com              |  Bin 139264 -> 0 bytes
> > >  DuetPkg/BootSector/bin/efi32.com2             |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bin/efi64.com              |  Bin 139264 -> 0 bytes
> > >  DuetPkg/BootSector/bin/efi64.com2             |  Bin 4096 -> 0 bytes
> > >  DuetPkg/BootSector/bootsect.S                 |  303 --
> > >  DuetPkg/BootSector/bootsect.asm               |  301 --
> > >  DuetPkg/BootSector/bs16.S                     |  291 --
> > >  DuetPkg/BootSector/bs16.asm                   |  288 --
> > >  DuetPkg/BootSector/bs32.S                     |  312 --
> > >  DuetPkg/BootSector/bs32.asm                   |  310 --
> > >  DuetPkg/BootSector/efi32.S                    | 1176 -------
> > >  DuetPkg/BootSector/efi32.asm                  |  582 ----
> > >  DuetPkg/BootSector/efi64.S                    | 1385 --------
> > >  DuetPkg/BootSector/efi64.asm                  |  787 -----
> > >  DuetPkg/BootSector/st16_64.S                  | 1142 -------
> > >  DuetPkg/BootSector/st16_64.asm                | 1140 -------
> > >  DuetPkg/BootSector/st32_64.S                  | 1157 -------
> > >  DuetPkg/BootSector/st32_64.asm                | 1156 -------
> > >  DuetPkg/BootSector/start.S                    |  919 ------
> > >  DuetPkg/BootSector/start.asm                  |  916 ------
> > >  DuetPkg/BootSector/start16.S                  |  910 ------
> > >  DuetPkg/BootSector/start16.asm                |  909 ------
> > >  DuetPkg/BootSector/start32.S                  |  927 ------
> > >  DuetPkg/BootSector/start32.asm                |  924 ------
> > >  DuetPkg/BootSector/start64.S                  | 1149 -------
> > >  DuetPkg/BootSector/start64.asm                | 1147 -------
> > >  DuetPkg/CreateBootDisk.bat                    |  145 -
> > >  DuetPkg/CreateBootDisk.sh                     |  212 --
> > >  DuetPkg/DuetPkg.dec                           |   42 -
> > >  DuetPkg/DuetPkg.fdf                           |  150 -
> > >  DuetPkg/DuetPkgIa32.dsc                       |  283 --
> > >  DuetPkg/DuetPkgX64.dsc                        |  284 --
> > >  DuetPkg/DxeIpl/Debug.c                        |   83 -
> > >  DuetPkg/DxeIpl/Debug.h                        |   41 -
> > >  DuetPkg/DxeIpl/DxeInit.c                      |  278 --
> > >  DuetPkg/DxeIpl/DxeIpl.h                       |   49 -
> > >  DuetPkg/DxeIpl/DxeIpl.inf                     |   70 -
> > >  DuetPkg/DxeIpl/HobGeneration.c                | 1000 ------
> > >  DuetPkg/DxeIpl/HobGeneration.h                |  171 -
> > >  DuetPkg/DxeIpl/Ia32/EnterDxeCore.c            |   31 -
> > >  DuetPkg/DxeIpl/Ia32/Paging.c                  |  172 -
> > >  DuetPkg/DxeIpl/Ia32/VirtualMemory.h           |   88 -
> > >  DuetPkg/DxeIpl/LegacyTable.c                  |  357 ---
> > >  DuetPkg/DxeIpl/LegacyTable.h                  |   31 -
> > >  DuetPkg/DxeIpl/PpisNeededByDxeCore.c          |   59 -
> > >  DuetPkg/DxeIpl/PpisNeededByDxeCore.h          |   64 -
> > >  DuetPkg/DxeIpl/SerialStatusCode.c             |  760 -----
> > >  DuetPkg/DxeIpl/SerialStatusCode.h             |   39 -
> > >  DuetPkg/DxeIpl/X64/EnterDxeCore.c             |   32 -
> > >  DuetPkg/DxeIpl/X64/Paging.c                   |  238 --
> > >  DuetPkg/DxeIpl/X64/VirtualMemory.h            |  117 -
> > >  DuetPkg/EfiLdr/Debug.c                        |   80 -
> > >  DuetPkg/EfiLdr/Debug.h                        |   41 -
> > >  DuetPkg/EfiLdr/EfiLdr.h                       |   89 -
> > >  DuetPkg/EfiLdr/EfiLdr.inf                     |   55 -
> > >  DuetPkg/EfiLdr/EfiLdrHandoff.h                |   56 -
> > >  DuetPkg/EfiLdr/EfiLoader.c                    |  294 --
> > >  DuetPkg/EfiLdr/LzmaDecompress.h               |   70 -
> > >  DuetPkg/EfiLdr/PeLoader.c                     |  635 ----
> > >  DuetPkg/EfiLdr/PeLoader.h                     |   42 -
> > >  DuetPkg/EfiLdr/Support.c                      |  237 --
> > >  DuetPkg/EfiLdr/Support.h                      |   50 -
> > >  DuetPkg/EfiLdr/TianoDecompress.c              |  990 ------
> > >  DuetPkg/EfiLdr/TianoDecompress.h              |   90 -
> > >  DuetPkg/EfiLdr/efildr.c                       |   28 -
> > >  DuetPkg/FSVariable/FSVariable.c               | 1992 ------------
> > >  DuetPkg/FSVariable/FSVariable.h               |  162 -
> > >  DuetPkg/FSVariable/FSVariable.inf             |   80 -
> > >  DuetPkg/FSVariable/FileStorage.c              |  437 ---
> > >  DuetPkg/FSVariable/MemStorage.c               |  135 -
> > >  DuetPkg/FSVariable/VariableStorage.h          |  118 -
> > >  DuetPkg/FvbRuntimeService/DUETFwh.inf         |   58 -
> > >  DuetPkg/FvbRuntimeService/FWBlockService.c    | 1659 ----------
> > >  DuetPkg/FvbRuntimeService/FileIo.c            |  210 --
> > >  DuetPkg/FvbRuntimeService/FileIo.h            |   58 -
> > >  DuetPkg/FvbRuntimeService/FvbInfo.c           |  121 -
> > >  DuetPkg/FvbRuntimeService/FwBlockService.h    |  298 --
> > >  DuetPkg/Include/EfiFlashMap.h                 |  153 -
> > >  DuetPkg/Include/EfiLdrHandoff.h               |   59 -
> > >  DuetPkg/Include/FlashLayout.h                 |   44 -
> > >  DuetPkg/Include/Guid/AcpiDescription.h        |  109 -
> > >  DuetPkg/Include/Guid/ConsoleOutConfig.h       |   33 -
> > >  DuetPkg/Include/Guid/DxeCoreFileName.h        |   23 -
> > >  DuetPkg/Include/Guid/FlashMapHob.h            |   33 -
> > >  DuetPkg/Include/Guid/LdrMemoryDescriptor.h    |   33 -
> > >  DuetPkg/Include/Guid/PciExpressBaseAddress.h  |   46 -
> > >  DuetPkg/Include/Guid/PciOptionRomTable.h      |   41 -
> > >  DuetPkg/LegacyMetronome/Metronome.c           |  125 -
> > >  DuetPkg/LegacyMetronome/Metronome.h           |   75 -
> > >  DuetPkg/LegacyMetronome/Metronome.inf         |   43 -
> > >  DuetPkg/Library/DuetBdsLib/BdsPlatform.c      | 1731 ----------
> > >  DuetPkg/Library/DuetBdsLib/BdsPlatform.h      |  288 --
> > >  DuetPkg/Library/DuetBdsLib/PlatformBds.inf    |   65 -
> > >  DuetPkg/Library/DuetBdsLib/PlatformData.c     |  161 -
> > >  DuetPkg/Library/DuetTimerLib/DuetTimerLib.inf |   54 -
> > >  DuetPkg/Library/DuetTimerLib/X86TimerLib.c    |  281 --
> > >  .../DxeCoreReportStatusCodeLibFromHob.inf     |   63 -
> > >  .../DxeSupport.c                              |  177 --
> > >  .../ReportStatusCodeLib.c                     |  535 ----
> > >  .../ReportStatusCodeLibInternal.h             |  118 -
> > >  .../PciBusNoEnumerationDxe/ComponentName.c    |  161 -
> > >  DuetPkg/PciBusNoEnumerationDxe/PciBus.c       |  346 --
> > >  DuetPkg/PciBusNoEnumerationDxe/PciBus.h       |  225 --
> > >  .../PciBusNoEnumeration.inf                   |   72 -
> > >  DuetPkg/PciBusNoEnumerationDxe/PciCommand.c   |  453 ---
> > >  DuetPkg/PciBusNoEnumerationDxe/PciCommand.h   |  167 -
> > >  .../PciBusNoEnumerationDxe/PciDeviceSupport.c |  973 ------
> > >  .../PciBusNoEnumerationDxe/PciDeviceSupport.h |  324 --
> > >  .../PciDriverOverride.c                       |  176 -
> > >  .../PciDriverOverride.h                       |  110 -
> > >  .../PciBusNoEnumerationDxe/PciEnumerator.c    |   57 -
> > >  .../PciBusNoEnumerationDxe/PciEnumerator.h    |   47 -
> > >  .../PciEnumeratorSupport.c                    | 1385 --------
> > >  .../PciEnumeratorSupport.h                    |  108 -
> > >  DuetPkg/PciBusNoEnumerationDxe/PciIo.c        | 1852 -----------
> > >  DuetPkg/PciBusNoEnumerationDxe/PciIo.h        |   48 -
> > >  .../PciOptionRomSupport.c                     |  557 ----
> > >  .../PciOptionRomSupport.h                     |   92 -
> > >  .../PciPowerManagement.c                      |  100 -
> > >  .../PciPowerManagement.h                      |   49 -
> > >  DuetPkg/PciBusNoEnumerationDxe/PciRomTable.c  |  393 ---
> > >  DuetPkg/PciBusNoEnumerationDxe/PciRomTable.h  |   58 -
> > >  .../PciRootBridgeNoEnumerationDxe/DeviceIo.c  |  845 -----
> > >  .../PciRootBridgeNoEnumerationDxe/DeviceIo.h  |  449 ---
> > >  .../Ia32/PcatIo.c                             |  738 -----
> > >  .../PcatPciRootBridge.c                       | 1009 ------
> > >  .../PcatPciRootBridge.h                       |  244 --
> > >  .../PcatPciRootBridgeDevicePath.c             |   93 -
> > >  .../PcatPciRootBridgeIo.c                     | 1036 ------
> > >  .../PciRootBridgeNoEnumeration.inf            |   66 -
> > >  .../X64/PcatIo.c                              |  738 -----
> > >  DuetPkg/PostBuild.bat                         |   83 -
> > >  DuetPkg/PostBuild.sh                          |  103 -
> > >  DuetPkg/ReadMe.txt                            |   84 -
> > >  DuetPkg/SataControllerDxe/ComponentName.c     |  176 -
> > >  DuetPkg/SataControllerDxe/SataController.c    | 1049 ------
> > >  DuetPkg/SataControllerDxe/SataController.h    |  542 ----
> > >  .../SataControllerDxe/SataControllerDxe.inf   |   49 -
> > >  DuetPkg/SmbiosGenDxe/SmbiosGen.c              |  330 --
> > >  DuetPkg/SmbiosGenDxe/SmbiosGen.h              |   80 -
> > >  DuetPkg/SmbiosGenDxe/SmbiosGen.inf            |   61 -
> > >  DuetPkg/SmbiosGenDxe/SmbiosGenStrings.uni     |   32 -
> > >  DuetPkg/build32.sh                            |  133 -
> > >  DuetPkg/build64.sh                            |  132 -
> > >  195 files changed, 61049 deletions(-)
> > >  delete mode 100644 BaseTools/Source/C/BootSectImage/GNUmakefile
> > >  delete mode 100644 BaseTools/Source/C/BootSectImage/Makefile
> > >  delete mode 100644
> BaseTools/Source/C/BootSectImage/bootsectimage.c
> > >  delete mode 100644 BaseTools/Source/C/BootSectImage/fat.h
> > >  delete mode 100644 BaseTools/Source/C/BootSectImage/mbr.h
> > >  delete mode 100644 BaseTools/Source/C/EfiLdrImage/EfiLdrImage.c
> > >  delete mode 100644 BaseTools/Source/C/EfiLdrImage/GNUmakefile
> > >  delete mode 100644 BaseTools/Source/C/EfiLdrImage/Makefile
> > >  delete mode 100644 BaseTools/Source/C/GenBootSector/FatFormat.h
> > >  delete mode 100644
> > BaseTools/Source/C/GenBootSector/GenBootSector.c
> > >  delete mode 100644
> > > BaseTools/Source/C/GenBootSector/GetDrvNumOffset.c
> > >  delete mode 100644 BaseTools/Source/C/GenBootSector/Makefile
> > >  delete mode 100644 BaseTools/Source/C/GenPage/GNUmakefile
> > >  delete mode 100644 BaseTools/Source/C/GenPage/GenPage.c
> > >  delete mode 100644 BaseTools/Source/C/GenPage/Makefile
> > >  delete mode 100644 BaseTools/Source/C/GenPage/VirtualMemory.h
> > >  delete mode 100644
> > BaseTools/Source/C/GnuGenBootSector/FatFormat.h
> > >  delete mode 100644
> > BaseTools/Source/C/GnuGenBootSector/GNUmakefile
> > >  delete mode 100644
> > > BaseTools/Source/C/GnuGenBootSector/GnuGenBootSector.c
> > >  delete mode 100644 DuetPkg/AcpiResetDxe/Reset.c
> > >  delete mode 100644 DuetPkg/AcpiResetDxe/Reset.inf
> > >  delete mode 100644 DuetPkg/BiosVideoThunkDxe/BiosVideo.c
> > >  delete mode 100644 DuetPkg/BiosVideoThunkDxe/BiosVideo.h
> > >  delete mode 100644 DuetPkg/BiosVideoThunkDxe/BiosVideo.inf
> > >  delete mode 100644 DuetPkg/BiosVideoThunkDxe/ComponentName.c
> > >  delete mode 100644 DuetPkg/BiosVideoThunkDxe/LegacyBiosThunk.c
> > >  delete mode 100644
> DuetPkg/BiosVideoThunkDxe/VesaBiosExtensions.h
> > >  delete mode 100644 DuetPkg/BootSector/BootSector.inf
> > >  delete mode 100644 DuetPkg/BootSector/FILE.LST
> > >  delete mode 100644 DuetPkg/BootSector/GNUmakefile
> > >  delete mode 100644 DuetPkg/BootSector/Gpt.S
> > >  delete mode 100644 DuetPkg/BootSector/Gpt.asm
> > >  delete mode 100644 DuetPkg/BootSector/Makefile
> > >  delete mode 100644 DuetPkg/BootSector/Mbr.S
> > >  delete mode 100644 DuetPkg/BootSector/Mbr.asm
> > >  delete mode 100644 DuetPkg/BootSector/bin/Gpt.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/Mbr.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/Readme.txt
> > >  delete mode 100644 DuetPkg/BootSector/bin/St16_64.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/St32_64.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/Start.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/Start16.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/Start32.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/Start64.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/bootsect.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/bs16.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/bs32.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/efi32.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/efi32.com2
> > >  delete mode 100644 DuetPkg/BootSector/bin/efi64.com
> > >  delete mode 100644 DuetPkg/BootSector/bin/efi64.com2
> > >  delete mode 100644 DuetPkg/BootSector/bootsect.S
> > >  delete mode 100644 DuetPkg/BootSector/bootsect.asm
> > >  delete mode 100644 DuetPkg/BootSector/bs16.S
> > >  delete mode 100644 DuetPkg/BootSector/bs16.asm
> > >  delete mode 100644 DuetPkg/BootSector/bs32.S
> > >  delete mode 100644 DuetPkg/BootSector/bs32.asm
> > >  delete mode 100644 DuetPkg/BootSector/efi32.S
> > >  delete mode 100644 DuetPkg/BootSector/efi32.asm
> > >  delete mode 100644 DuetPkg/BootSector/efi64.S
> > >  delete mode 100644 DuetPkg/BootSector/efi64.asm
> > >  delete mode 100644 DuetPkg/BootSector/st16_64.S
> > >  delete mode 100644 DuetPkg/BootSector/st16_64.asm
> > >  delete mode 100644 DuetPkg/BootSector/st32_64.S
> > >  delete mode 100644 DuetPkg/BootSector/st32_64.asm
> > >  delete mode 100644 DuetPkg/BootSector/start.S
> > >  delete mode 100644 DuetPkg/BootSector/start.asm
> > >  delete mode 100644 DuetPkg/BootSector/start16.S
> > >  delete mode 100644 DuetPkg/BootSector/start16.asm
> > >  delete mode 100644 DuetPkg/BootSector/start32.S
> > >  delete mode 100644 DuetPkg/BootSector/start32.asm
> > >  delete mode 100644 DuetPkg/BootSector/start64.S
> > >  delete mode 100644 DuetPkg/BootSector/start64.asm
> > >  delete mode 100644 DuetPkg/CreateBootDisk.bat
> > >  delete mode 100755 DuetPkg/CreateBootDisk.sh
> > >  delete mode 100644 DuetPkg/DuetPkg.dec
> > >  delete mode 100644 DuetPkg/DuetPkg.fdf
> > >  delete mode 100644 DuetPkg/DuetPkgIa32.dsc
> > >  delete mode 100644 DuetPkg/DuetPkgX64.dsc
> > >  delete mode 100644 DuetPkg/DxeIpl/Debug.c
> > >  delete mode 100644 DuetPkg/DxeIpl/Debug.h
> > >  delete mode 100644 DuetPkg/DxeIpl/DxeInit.c
> > >  delete mode 100644 DuetPkg/DxeIpl/DxeIpl.h
> > >  delete mode 100644 DuetPkg/DxeIpl/DxeIpl.inf
> > >  delete mode 100644 DuetPkg/DxeIpl/HobGeneration.c
> > >  delete mode 100644 DuetPkg/DxeIpl/HobGeneration.h
> > >  delete mode 100644 DuetPkg/DxeIpl/Ia32/EnterDxeCore.c
> > >  delete mode 100644 DuetPkg/DxeIpl/Ia32/Paging.c
> > >  delete mode 100644 DuetPkg/DxeIpl/Ia32/VirtualMemory.h
> > >  delete mode 100644 DuetPkg/DxeIpl/LegacyTable.c
> > >  delete mode 100644 DuetPkg/DxeIpl/LegacyTable.h
> > >  delete mode 100644 DuetPkg/DxeIpl/PpisNeededByDxeCore.c
> > >  delete mode 100644 DuetPkg/DxeIpl/PpisNeededByDxeCore.h
> > >  delete mode 100644 DuetPkg/DxeIpl/SerialStatusCode.c
> > >  delete mode 100644 DuetPkg/DxeIpl/SerialStatusCode.h
> > >  delete mode 100644 DuetPkg/DxeIpl/X64/EnterDxeCore.c
> > >  delete mode 100644 DuetPkg/DxeIpl/X64/Paging.c
> > >  delete mode 100644 DuetPkg/DxeIpl/X64/VirtualMemory.h
> > >  delete mode 100644 DuetPkg/EfiLdr/Debug.c
> > >  delete mode 100644 DuetPkg/EfiLdr/Debug.h
> > >  delete mode 100644 DuetPkg/EfiLdr/EfiLdr.h
> > >  delete mode 100644 DuetPkg/EfiLdr/EfiLdr.inf
> > >  delete mode 100644 DuetPkg/EfiLdr/EfiLdrHandoff.h
> > >  delete mode 100644 DuetPkg/EfiLdr/EfiLoader.c
> > >  delete mode 100644 DuetPkg/EfiLdr/LzmaDecompress.h
> > >  delete mode 100644 DuetPkg/EfiLdr/PeLoader.c
> > >  delete mode 100644 DuetPkg/EfiLdr/PeLoader.h
> > >  delete mode 100644 DuetPkg/EfiLdr/Support.c
> > >  delete mode 100644 DuetPkg/EfiLdr/Support.h
> > >  delete mode 100644 DuetPkg/EfiLdr/TianoDecompress.c
> > >  delete mode 100644 DuetPkg/EfiLdr/TianoDecompress.h
> > >  delete mode 100644 DuetPkg/EfiLdr/efildr.c
> > >  delete mode 100644 DuetPkg/FSVariable/FSVariable.c
> > >  delete mode 100644 DuetPkg/FSVariable/FSVariable.h
> > >  delete mode 100644 DuetPkg/FSVariable/FSVariable.inf
> > >  delete mode 100644 DuetPkg/FSVariable/FileStorage.c
> > >  delete mode 100644 DuetPkg/FSVariable/MemStorage.c
> > >  delete mode 100644 DuetPkg/FSVariable/VariableStorage.h
> > >  delete mode 100644 DuetPkg/FvbRuntimeService/DUETFwh.inf
> > >  delete mode 100644 DuetPkg/FvbRuntimeService/FWBlockService.c
> > >  delete mode 100644 DuetPkg/FvbRuntimeService/FileIo.c
> > >  delete mode 100644 DuetPkg/FvbRuntimeService/FileIo.h
> > >  delete mode 100644 DuetPkg/FvbRuntimeService/FvbInfo.c
> > >  delete mode 100644 DuetPkg/FvbRuntimeService/FwBlockService.h
> > >  delete mode 100644 DuetPkg/Include/EfiFlashMap.h
> > >  delete mode 100644 DuetPkg/Include/EfiLdrHandoff.h
> > >  delete mode 100644 DuetPkg/Include/FlashLayout.h
> > >  delete mode 100644 DuetPkg/Include/Guid/AcpiDescription.h
> > >  delete mode 100644 DuetPkg/Include/Guid/ConsoleOutConfig.h
> > >  delete mode 100644 DuetPkg/Include/Guid/DxeCoreFileName.h
> > >  delete mode 100644 DuetPkg/Include/Guid/FlashMapHob.h
> > >  delete mode 100644 DuetPkg/Include/Guid/LdrMemoryDescriptor.h
> > >  delete mode 100644 DuetPkg/Include/Guid/PciExpressBaseAddress.h
> > >  delete mode 100644 DuetPkg/Include/Guid/PciOptionRomTable.h
> > >  delete mode 100644 DuetPkg/LegacyMetronome/Metronome.c
> > >  delete mode 100644 DuetPkg/LegacyMetronome/Metronome.h
> > >  delete mode 100644 DuetPkg/LegacyMetronome/Metronome.inf
> > >  delete mode 100644 DuetPkg/Library/DuetBdsLib/BdsPlatform.c
> > >  delete mode 100644 DuetPkg/Library/DuetBdsLib/BdsPlatform.h
> > >  delete mode 100644 DuetPkg/Library/DuetBdsLib/PlatformBds.inf
> > >  delete mode 100644 DuetPkg/Library/DuetBdsLib/PlatformData.c
> > >  delete mode 100644 DuetPkg/Library/DuetTimerLib/DuetTimerLib.inf
> > >  delete mode 100644 DuetPkg/Library/DuetTimerLib/X86TimerLib.c
> > >  delete mode 100644
> > >
> >
> DuetPkg/Library/DxeCoreReportStatusCodeLibFromHob/DxeCoreReportStat
> > > usCodeLibFromHob.inf
> > >  delete mode 100644
> > > DuetPkg/Library/DxeCoreReportStatusCodeLibFromHob/DxeSupport.c
> > >  delete mode 100644
> > >
> >
> DuetPkg/Library/DxeCoreReportStatusCodeLibFromHob/ReportStatusCodeL
> > > ib.c
> > >  delete mode 100644
> > >
> >
> DuetPkg/Library/DxeCoreReportStatusCodeLibFromHob/ReportStatusCodeL
> > > ibInternal.h
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/ComponentName.c
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciBus.c
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciBus.h
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciBusNoEnumeration.inf
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciCommand.c
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciCommand.h
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciDeviceSupport.c
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciDeviceSupport.h
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciDriverOverride.c
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciDriverOverride.h
> > >  delete mode 100644
> DuetPkg/PciBusNoEnumerationDxe/PciEnumerator.c
> > >  delete mode 100644
> DuetPkg/PciBusNoEnumerationDxe/PciEnumerator.h
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciEnumeratorSupport.c
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciEnumeratorSupport.h
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciIo.c
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciIo.h
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciOptionRomSupport.c
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciOptionRomSupport.h
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciPowerManagement.c
> > >  delete mode 100644
> > > DuetPkg/PciBusNoEnumerationDxe/PciPowerManagement.h
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciRomTable.c
> > >  delete mode 100644 DuetPkg/PciBusNoEnumerationDxe/PciRomTable.h
> > >  delete mode 100644
> > DuetPkg/PciRootBridgeNoEnumerationDxe/DeviceIo.c
> > >  delete mode 100644
> > DuetPkg/PciRootBridgeNoEnumerationDxe/DeviceIo.h
> > >  delete mode 100644
> > > DuetPkg/PciRootBridgeNoEnumerationDxe/Ia32/PcatIo.c
> > >  delete mode 100644
> > > DuetPkg/PciRootBridgeNoEnumerationDxe/PcatPciRootBridge.c
> > >  delete mode 100644
> > > DuetPkg/PciRootBridgeNoEnumerationDxe/PcatPciRootBridge.h
> > >  delete mode 100644
> > >
> >
> DuetPkg/PciRootBridgeNoEnumerationDxe/PcatPciRootBridgeDevicePath.c
> > >  delete mode 100644
> > > DuetPkg/PciRootBridgeNoEnumerationDxe/PcatPciRootBridgeIo.c
> > >  delete mode 100644
> > >
> >
> DuetPkg/PciRootBridgeNoEnumerationDxe/PciRootBridgeNoEnumeration.in
> > > f
> > >  delete mode 100644
> > > DuetPkg/PciRootBridgeNoEnumerationDxe/X64/PcatIo.c
> > >  delete mode 100644 DuetPkg/PostBuild.bat
> > >  delete mode 100755 DuetPkg/PostBuild.sh
> > >  delete mode 100644 DuetPkg/ReadMe.txt
> > >  delete mode 100644 DuetPkg/SataControllerDxe/ComponentName.c
> > >  delete mode 100644 DuetPkg/SataControllerDxe/SataController.c
> > >  delete mode 100644 DuetPkg/SataControllerDxe/SataController.h
> > >  delete mode 100644 DuetPkg/SataControllerDxe/SataControllerDxe.inf
> > >  delete mode 100644 DuetPkg/SmbiosGenDxe/SmbiosGen.c
> > >  delete mode 100644 DuetPkg/SmbiosGenDxe/SmbiosGen.h
> > >  delete mode 100644 DuetPkg/SmbiosGenDxe/SmbiosGen.inf
> > >  delete mode 100644 DuetPkg/SmbiosGenDxe/SmbiosGenStrings.uni
> > >  delete mode 100755 DuetPkg/build32.sh
> > >  delete mode 100755 DuetPkg/build64.sh
> > >
> > > --
> > > 2.18.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel


      reply	other threads:[~2018-11-28  8:13 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-27  2:42 [PATCH v2 0/2] Remove DuetPkg and unused tools Shenglei Zhang
2018-11-28  7:38 ` Wu, Hao A
2018-11-28  8:05   ` Wu, Hao A
2018-11-28  8:13     ` Zhang, Shenglei [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=C0706E73DB8C124D9B9C38AA364E5D5E04BB6977@shsmsx102.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox