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.88, mailfrom: michael.a.kubacki@intel.com) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by groups.io with SMTP; Mon, 12 Aug 2019 09:09:12 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 12 Aug 2019 09:09:07 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,378,1559545200"; d="scan'208";a="177523295" Received: from orsmsx107.amr.corp.intel.com ([10.22.240.5]) by fmsmga007.fm.intel.com with ESMTP; 12 Aug 2019 09:09:07 -0700 Received: from orsmsx122.amr.corp.intel.com ([169.254.11.68]) by ORSMSX107.amr.corp.intel.com ([169.254.1.186]) with mapi id 14.03.0439.000; Mon, 12 Aug 2019 09:09:07 -0700 From: "Kubacki, Michael A" To: "Desimone, Nathaniel L" , Leif Lindholm CC: "devel@edk2.groups.io" , Andrew Fish , Laszlo Ersek , "Kinney, Michael D" , "Sinha, Ankit" , "Chiu, Chasel" Subject: Re: [edk2-platforms] [PATCH] Platform/Intel: Update Readme.md to point to edk2-non-osi master branch Thread-Topic: [edk2-platforms] [PATCH] Platform/Intel: Update Readme.md to point to edk2-non-osi master branch Thread-Index: AQHVUJ/ikXeXVbvN0UiARo9fL0ijv6b3q+qw Date: Mon, 12 Aug 2019 16:09:06 +0000 Message-ID: <49AB4ACB9627B8468F29D589A27B745588A42374@ORSMSX122.amr.corp.intel.com> References: <20190808213048.35628-1-nathaniel.l.desimone@intel.com> <20190809103232.GL25813@bivouac.eciton.net> <02A34F284D1DA44BB705E61F7180EF0AAEDF3EFE@ORSMSX114.amr.corp.intel.com> In-Reply-To: <02A34F284D1DA44BB705E61F7180EF0AAEDF3EFE@ORSMSX114.amr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNTlmZTJiZDAtMTBjNy00MDEwLThlZmUtMTM2MDMyNzRiNWVhIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiQVhqNW1udk9oNzY3RGxaU1VCV1hcL0N6dDhCNHpWaFBZNVpcL0FxZVFjbHorOVJNRDY2U2E3ODVzZW1idFpzaUZiIn0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.22.254.140] MIME-Version: 1.0 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable I also support moving the MinPlatform platforms to a dedicated directory. Leif, please let me know if I can be of any help to prepare the RFC. Thanks, Michael > -----Original Message----- > From: Desimone, Nathaniel L > Sent: Sunday, August 11, 2019 4:53 PM > To: Leif Lindholm > Cc: devel@edk2.groups.io; Andrew Fish ; Laszlo Ersek > ; Kinney, Michael D ; > Kubacki, Michael A ; Sinha, Ankit > ; Chiu, Chasel > Subject: RE: [edk2-platforms] [PATCH] Platform/Intel: Update Readme.md to > point to edk2-non-osi master branch >=20 > Hi Leif, >=20 > > I am well aware that there are platform-specific steps required to > > achieve the final firmware image for some systems. But we need to work > > to address that. There have been discussions about implementing support= for > hooking pre/post-build python scripts into the .dsc format - has there be= en any > progress on that? >=20 > I'm not sure. Liming would know more on that than I. For now, our > MinPlatform... platforms are using the build_bios.py script written by Pr= ince as > a substitute. >=20 > > For this patch, I would prefer if instead of adding these notes we > > changed the ## [Intel](Platform/Intel/Readme.md) ### Minimum Platforms >=20 > Done. Please see patch V2. >=20 > > Arguably, it would make sense to move all of the MinPlatform ... > > platforms ... and the Readme.md to a separate subdirectory and link to > > that directory. Not everything under Platform/Intel is a MinPlatform pl= atform > - soon there'll even be an ARM one. >=20 > I totally agree with you that would make more sense. Could you perhaps se= nd > an RFC and then we can start the reorganization process? >=20 > Thanks, > Nate