From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: maurice.ma@intel.com) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by groups.io with SMTP; Wed, 08 May 2019 17:44:06 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 May 2019 17:44:05 -0700 X-ExtLoop1: 1 Received: from orsmsx106.amr.corp.intel.com ([10.22.225.133]) by orsmga003.jf.intel.com with ESMTP; 08 May 2019 17:44:05 -0700 Received: from orsmsx151.amr.corp.intel.com (10.22.226.38) by ORSMSX106.amr.corp.intel.com (10.22.225.133) with Microsoft SMTP Server (TLS) id 14.3.408.0; Wed, 8 May 2019 17:44:05 -0700 Received: from orsmsx105.amr.corp.intel.com ([169.254.2.167]) by ORSMSX151.amr.corp.intel.com ([169.254.7.24]) with mapi id 14.03.0415.000; Wed, 8 May 2019 17:44:05 -0700 From: "Ma, Maurice" To: "Dong, Guo" , "devel@edk2.groups.io" CC: "Agyeman, Prince" , "You, Benjamin" Subject: Re: [edk2-devel] [PATCH 0/2] Retire CorebootModulePkg and CorebootPayloadPkg Thread-Topic: [edk2-devel] [PATCH 0/2] Retire CorebootModulePkg and CorebootPayloadPkg Thread-Index: AQHVBSQQeROdEgBQrEmXIp6Hi/Oa56Zh9XNQ Date: Thu, 9 May 2019 00:44:04 +0000 Message-ID: <7AAC936950815649B5F88FAE785306C2F6F68CD6@ORSMSX105.amr.corp.intel.com> References: <20190507222728.28296-1-guo.dong@intel.com> In-Reply-To: <20190507222728.28296-1-guo.dong@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMjNmMTE3NWMtNjU0OC00ZDA1LWFhNjgtNWJiMjI3OTk1MTM2IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiVGR3OXd4dWVncXV4dWNmM1NEK1p0ZjM5UjVXVEhyZWxJWktIQXU3UjdyMkhpMG9ZbmVLczRnSkNVdm5WU0RGZSJ9 dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [10.22.254.138] MIME-Version: 1.0 Return-Path: maurice.ma@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable This patchset looks good me. Reviewed-by: Maurice Ma Regards, Maurice > -----Original Message----- > From: Dong, Guo > Sent: Tuesday, May 7, 2019 15:27 > To: devel@edk2.groups.io > Cc: Ma, Maurice ; Agyeman, Prince > ; You, Benjamin ; > Dong, Guo > Subject: [edk2-devel] [PATCH 0/2] Retire CorebootModulePkg and > CorebootPayloadPkg >=20 > Since UefiPayloadPkg in EDK2 supports Coreboot and Slim Bootloader, and > no concerns were received for the RFC to remove CorebootModulePkg and > CorebootPayloadPkg from EDK2 > (https://edk2.groups.io/g/devel/message/39126), > here is the action patch to remove CorebootModulePkg and > CorebootPayloadPkg. >=20 > Guo Dong (2): > Retire CorebootModulePkg from EDK2 > Retire CorebootPayloadPkg from EDK2 >=20 > CorebootModulePkg/CbSupportDxe/CbSupportDxe.c | 158 -- > CorebootModulePkg/CbSupportDxe/CbSupportDxe.h | 30 - > CorebootModulePkg/CbSupportDxe/CbSupportDxe.inf | 60 - > CorebootModulePkg/CbSupportPei/CbSupportPei.c | 440 ------ > CorebootModulePkg/CbSupportPei/CbSupportPei.h | 42 - > CorebootModulePkg/CbSupportPei/CbSupportPei.inf | 74 - > CorebootModulePkg/CorebootModulePkg.dec | 59 - > CorebootModulePkg/Include/Coreboot.h | 249 ---- > CorebootModulePkg/Include/Guid/AcpiBoardInfoGuid.h | 26 - > .../Include/Guid/FrameBufferInfoGuid.h | 34 - > .../Include/Guid/SystemTableInfoGuid.h | 24 - > CorebootModulePkg/Include/Library/CbParseLib.h | 186 --- > .../Include/Library/CbPlatformSupportLib.h | 28 - > .../BaseSerialPortLib16550.c | 1089 -------------- > .../BaseSerialPortLib16550.inf | 42 - > .../BaseSerialPortLib16550.uni | 16 - > CorebootModulePkg/Library/CbParseLib/CbParseLib.c | 721 --------- > .../Library/CbParseLib/CbParseLib.inf | 39 - > .../CbPlatformSupportLibNull.c | 29 - > .../CbPlatformSupportLibNull.inf | 29 - > .../CbPlatformSupportLibNull.uni | 14 - > .../SataControllerDxe/ComponentName.c | 170 --- > .../SataControllerDxe/SataController.c | 1043 ------------- > .../SataControllerDxe/SataController.h | 536 ------- > .../SataControllerDxe/SataControllerDxe.inf | 43 - > CorebootModulePkg/SecCore/FindPeiCore.c | 193 --- > CorebootModulePkg/SecCore/Ia32/SecEntry.asm | 72 - > CorebootModulePkg/SecCore/Ia32/SecEntry.nasm | 66 - > CorebootModulePkg/SecCore/Ia32/Stack.asm | 76 - > CorebootModulePkg/SecCore/Ia32/Stack.nasm | 72 - > CorebootModulePkg/SecCore/SecCore.inf | 57 - > CorebootModulePkg/SecCore/SecMain.c | 285 ---- > CorebootModulePkg/SecCore/SecMain.h | 128 -- > .../BuildAndIntegrationInstructions.txt | 82 -- > CorebootPayloadPkg/CorebootPayloadPkg.dec | 48 - > CorebootPayloadPkg/CorebootPayloadPkg.fdf | 285 ---- > CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc | 574 -------- > CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc | 575 -------- > CorebootPayloadPkg/FbGop/ComponentName.c | 306 ---- > CorebootPayloadPkg/FbGop/FbGop.c | 1536 --------------= ------ > CorebootPayloadPkg/FbGop/FbGop.h | 413 ------ > CorebootPayloadPkg/FbGop/FbGop.inf | 63 - > .../Library/AcpiTimerLib/AcpiTimerLib.c | 270 ---- > .../Library/AcpiTimerLib/AcpiTimerLib.inf | 40 - > .../Library/PciHostBridgeLib/PciHostBridge.h | 80 - > .../Library/PciHostBridgeLib/PciHostBridgeLib.c | 222 --- > .../Library/PciHostBridgeLib/PciHostBridgeLib.inf | 41 - > .../PciHostBridgeLib/PciHostBridgeSupport.c | 584 -------- > .../PlatformBootManagerLib/PlatformBootManager.c | 265 ---- > .../PlatformBootManagerLib/PlatformBootManager.h | 121 -- > .../PlatformBootManagerLib.inf | 74 - > .../PlatformBootManagerLib/PlatformConsole.c | 611 -------- > .../PlatformBootManagerLib/PlatformConsole.h | 70 - > .../Library/PlatformBootManagerLib/PlatformData.c | 19 - > .../Library/PlatformHookLib/PlatformHookLib.c | 105 -- > .../Library/PlatformHookLib/PlatformHookLib.inf | 39 - > .../Library/ResetSystemLib/ResetSystemLib.c | 221 --- > .../Library/ResetSystemLib/ResetSystemLib.inf | 38 - > 58 files changed, 12812 deletions(-) > delete mode 100755 CorebootModulePkg/CbSupportDxe/CbSupportDxe.c > delete mode 100644 CorebootModulePkg/CbSupportDxe/CbSupportDxe.h > delete mode 100644 CorebootModulePkg/CbSupportDxe/CbSupportDxe.inf > delete mode 100755 CorebootModulePkg/CbSupportPei/CbSupportPei.c > delete mode 100644 CorebootModulePkg/CbSupportPei/CbSupportPei.h > delete mode 100644 CorebootModulePkg/CbSupportPei/CbSupportPei.inf > delete mode 100644 CorebootModulePkg/CorebootModulePkg.dec > delete mode 100644 CorebootModulePkg/Include/Coreboot.h > delete mode 100644 > CorebootModulePkg/Include/Guid/AcpiBoardInfoGuid.h > delete mode 100644 > CorebootModulePkg/Include/Guid/FrameBufferInfoGuid.h > delete mode 100644 > CorebootModulePkg/Include/Guid/SystemTableInfoGuid.h > delete mode 100644 CorebootModulePkg/Include/Library/CbParseLib.h > delete mode 100644 > CorebootModulePkg/Include/Library/CbPlatformSupportLib.h > delete mode 100644 > CorebootModulePkg/Library/BaseSerialPortLib16550/BaseSerialPortLib16550 > .c > delete mode 100644 > CorebootModulePkg/Library/BaseSerialPortLib16550/BaseSerialPortLib16550 > .inf > delete mode 100644 > CorebootModulePkg/Library/BaseSerialPortLib16550/BaseSerialPortLib16550 > .uni > delete mode 100644 CorebootModulePkg/Library/CbParseLib/CbParseLib.c > delete mode 100644 > CorebootModulePkg/Library/CbParseLib/CbParseLib.inf > delete mode 100644 > CorebootModulePkg/Library/CbPlatformSupportLibNull/CbPlatformSupportL > ibNull.c > delete mode 100644 > CorebootModulePkg/Library/CbPlatformSupportLibNull/CbPlatformSupportL > ibNull.inf > delete mode 100644 > CorebootModulePkg/Library/CbPlatformSupportLibNull/CbPlatformSupportL > ibNull.uni > delete mode 100644 > CorebootModulePkg/SataControllerDxe/ComponentName.c > delete mode 100644 > CorebootModulePkg/SataControllerDxe/SataController.c > delete mode 100644 > CorebootModulePkg/SataControllerDxe/SataController.h > delete mode 100644 > CorebootModulePkg/SataControllerDxe/SataControllerDxe.inf > delete mode 100644 CorebootModulePkg/SecCore/FindPeiCore.c > delete mode 100644 CorebootModulePkg/SecCore/Ia32/SecEntry.asm > delete mode 100644 CorebootModulePkg/SecCore/Ia32/SecEntry.nasm > delete mode 100644 CorebootModulePkg/SecCore/Ia32/Stack.asm > delete mode 100644 CorebootModulePkg/SecCore/Ia32/Stack.nasm > delete mode 100644 CorebootModulePkg/SecCore/SecCore.inf > delete mode 100644 CorebootModulePkg/SecCore/SecMain.c > delete mode 100644 CorebootModulePkg/SecCore/SecMain.h > delete mode 100644 > CorebootPayloadPkg/BuildAndIntegrationInstructions.txt > delete mode 100644 CorebootPayloadPkg/CorebootPayloadPkg.dec > delete mode 100644 CorebootPayloadPkg/CorebootPayloadPkg.fdf > delete mode 100644 CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc > delete mode 100644 CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc > delete mode 100644 CorebootPayloadPkg/FbGop/ComponentName.c > delete mode 100644 CorebootPayloadPkg/FbGop/FbGop.c delete mode > 100644 CorebootPayloadPkg/FbGop/FbGop.h delete mode 100644 > CorebootPayloadPkg/FbGop/FbGop.inf > delete mode 100644 > CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.c > delete mode 100644 > CorebootPayloadPkg/Library/AcpiTimerLib/AcpiTimerLib.inf > delete mode 100644 > CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridge.h > delete mode 100644 > CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.c > delete mode 100644 > CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf > delete mode 100644 > CorebootPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeSupport.c > delete mode 100644 > CorebootPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManag > er.c > delete mode 100644 > CorebootPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManag > er.h > delete mode 100644 > CorebootPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManag > erLib.inf > delete mode 100644 > CorebootPayloadPkg/Library/PlatformBootManagerLib/PlatformConsole.c > delete mode 100644 > CorebootPayloadPkg/Library/PlatformBootManagerLib/PlatformConsole.h > delete mode 100644 > CorebootPayloadPkg/Library/PlatformBootManagerLib/PlatformData.c > delete mode 100644 > CorebootPayloadPkg/Library/PlatformHookLib/PlatformHookLib.c > delete mode 100644 > CorebootPayloadPkg/Library/PlatformHookLib/PlatformHookLib.inf > delete mode 100644 > CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.c > delete mode 100644 > CorebootPayloadPkg/Library/ResetSystemLib/ResetSystemLib.inf >=20 > -- > 2.16.2.windows.1