From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.93, mailfrom: michael.a.kubacki@intel.com) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by groups.io with SMTP; Thu, 02 May 2019 14:16:01 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 02 May 2019 14:15:54 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,423,1549958400"; d="scan'208";a="136354813" Received: from orsmsx105.amr.corp.intel.com ([10.22.225.132]) by orsmga007.jf.intel.com with ESMTP; 02 May 2019 14:15:54 -0700 Received: from orsmsx157.amr.corp.intel.com (10.22.240.23) by ORSMSX105.amr.corp.intel.com (10.22.225.132) with Microsoft SMTP Server (TLS) id 14.3.408.0; Thu, 2 May 2019 14:15:53 -0700 Received: from orsmsx122.amr.corp.intel.com ([169.254.11.68]) by ORSMSX157.amr.corp.intel.com ([169.254.9.192]) with mapi id 14.03.0415.000; Thu, 2 May 2019 14:15:53 -0700 From: "Kubacki, Michael A" To: Leif Lindholm , "devel@edk2.groups.io" CC: "Gao, Liming" , Andrew Fish , Laszlo Ersek , "Kinney, Michael D" , Ard Biesheuvel Subject: Re: [edk2-devel] [edk2-platforms] [RFC] Migrate devel-MinPlatform branch to master branch Thread-Topic: [edk2-devel] [edk2-platforms] [RFC] Migrate devel-MinPlatform branch to master branch Thread-Index: AdT2KouLyR+4PxT0TviHqeHXbgAoHQLCHa2AAARUs8A= Date: Thu, 2 May 2019 21:15:53 +0000 Message-ID: <49AB4ACB9627B8468F29D589A27B7455888E47BF@ORSMSX122.amr.corp.intel.com> References: <49AB4ACB9627B8468F29D589A27B7455888CA32F@ORSMSX122.amr.corp.intel.com> <20190502150427.ihzom3w4fvjpxopk@bivouac.eciton.net> In-Reply-To: <20190502150427.ihzom3w4fvjpxopk@bivouac.eciton.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMDQ1MDVlMmQtY2JlYS00MDk1LWIxOGYtZjg0ZTcxMGEzZGU3IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoicHVHNVY1ZFRrdFZGT3dteDMzOERkeCtLRjFjanB3M1p0c0krdUc3YnY0T1RXdmVJMXdWTm15ZHNKWE9XRDV4WCJ9 x-ctpclassification: CTP_NT 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: michael.a.kubacki@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Leif, > -----Original Message----- > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > Sent: Thursday, May 2, 2019 8:04 AM > To: devel@edk2.groups.io; Kubacki, Michael A > > Cc: Gao, Liming ; Andrew Fish ; > Laszlo Ersek ; Kinney, Michael D > ; Ard Biesheuvel > Subject: Re: [edk2-devel] [edk2-platforms] [RFC] Migrate devel-MinPlatfo= rm > branch to master branch >=20 > Hi Michael, >=20 > On Thu, Apr 18, 2019 at 09:11:38PM +0000, Kubacki, Michael A wrote: > > Hello, > > > > This RFC proposes moving the content on the devel-MinPlatform branch > > in the edk2-platforms repository to the master branch in the > > edk2-platforms repository. > > > > The devel-MinPlatform branch has been used for the initial development > > of an EDK II based platform design referred to as "Minimum Platform". > > This design is intended to provide a structured approach to > > introducing Intel platform code into open source in a consistent manne= r. > > > > For more information about the EDK II Minimum Platform, please refer > > to the Readme.md in devel-MinPlatform. > > https://github.com/tianocore/edk2-platforms/blob/devel- > MinPlatform/Rea > > dMe.md > > > > The following packages would be added in Platform/Intel: > > * Generic packages: > > * AdvancedFeaturePkg > > * MinPlatformPkg >=20 > First a generic comment with some examples: > Both of these include non-architecture-specific components that could be > useful to have more generally available. >=20 > Certainly AdvancedFeaturePkg/Smbios/ and AcpiDebug look like something > of a generic nature rather than Intel-platform specific. >=20 The intention is for advanced features to be generic and architecture agnostic when possible. We ultimately want to simplify the process to enable open source edk2 platforms and cross-architecture compatibility certainly aids in that goal. We're starting with the code in the Intel dir= ectory=20 and we are open to moving features elsewhere based on interest and usefulness. I suspect we'll also evolve some elements of how these features are organized and designed based on feedback over time. For example, while we're starting with one AdvancedFeaturePkg, it may be too monolithic. More cohesive packages are likely easier to integrate and maintain. So we may propose breaking this into something like DebugFeaturePkg, IoFeaturePkg, PowerManagementFeaturePkg, or to some other degree. I=20 expect the definition to be a fluid process based on actual demand. > And Platform/Intel/MinPlatformPkg/Library/CompressLib/CompressLib.c > appears to have nearly only whitespace differences compared to edk2 > ShellPkg/Library/UefiShellDebug1CommandsLib/Compress.c. >=20 > (With edk2 already having 3 additional quite similar files in > BaseTools/Source/C/Common/EfiCompress.c > BaseTools/Source/C/Common/TianoCompress.c > BaseTools/Source/C/TianoCompress/TianoCompress.c) >=20 >=20 That's a good point. In particular, it would be nice to consolidate the co= de usage between ShellPkg and MinPlatformPkg. Perhaps the compression code could be moved somewhere like MdePkg where it could be used by both packages. > Secondly - edk2 has recently transitioned to bsd+patents license, and it > would make sense if edk2-platforms did the same. Do we want to do that > before or after this addition? >=20 I will defer this to Mike Kinney. >=20 > Finally, what should we do for Maintainers.txt? >=20 I believe we need to have per-package maintainers for the packages being a= dded. For example, merge what is in https://github.com/lgao4/edk2-platforms/blob/master/Platform/Intel/Maintai= ners.txt I can update the following Maintainers.txt with a proposal. https://github.com/lgao4/edk2-platforms/blob/master/Maintainers.txt Regards, Michael > Best Regards, >=20 > Leif >=20 > > * Board-specific packages: > > * ClevoOpenBoardPkg > > * KabylakeOpenBoardPkg > > * PurleyOpenBoardPkg > > > > The following packages would be added in Silicon/Intel: > > * KabylakeSiliconPkg > > * LewisburgPkg > > * PurleyRcPkg > > * PurleySktPkg > > > > The following growth is expected over time: > > * Platform/Intel - Additional board packages for Intel reference boar= ds > > including support for some pre-existing product releases > > * AdvancedFeaturePkg - Additional modular features capable of being > used > > in board packages > > * Silicon/Intel - Additional silicon packages roughly keeping 1:1 par= ity > > with board packages > > > > We hope the content will enable others to add new board packages and > > advanced features over time. > > > > The result of the change is available here for reference: > > https://github.com/lgao4/edk2-platforms > > > > Regards, > > Michael > > > >=20 > >