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.136, mailfrom: nathaniel.l.desimone@intel.com)
Received: from mga12.intel.com (mga12.intel.com [192.55.52.136])
 by groups.io with SMTP; Sun, 11 Aug 2019 16:52:51 -0700
X-Amp-Result: SKIPPED(no attachment in message)
X-Amp-File-Uploaded: False
Received: from fmsmga004.fm.intel.com ([10.253.24.48])
  by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Aug 2019 16:52:50 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.64,375,1559545200"; 
   d="scan'208";a="199955670"
Received: from orsmsx108.amr.corp.intel.com ([10.22.240.6])
  by fmsmga004.fm.intel.com with ESMTP; 11 Aug 2019 16:52:50 -0700
Received: from orsmsx114.amr.corp.intel.com ([169.254.8.96]) by
 ORSMSX108.amr.corp.intel.com ([169.254.2.65]) with mapi id 14.03.0439.000;
 Sun, 11 Aug 2019 16:52:50 -0700
From: "Nate DeSimone" <nathaniel.l.desimone@intel.com>
To: Leif Lindholm <leif.lindholm@linaro.org>
CC: "devel@edk2.groups.io" <devel@edk2.groups.io>, Andrew Fish
	<afish@apple.com>, Laszlo Ersek <lersek@redhat.com>, "Kinney, Michael D"
	<michael.d.kinney@intel.com>, "Kubacki, Michael A"
	<michael.a.kubacki@intel.com>, "Sinha, Ankit" <ankit.sinha@intel.com>,
	"Chiu, Chasel" <chasel.chiu@intel.com>
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: AQHVTp3GUVKkpTOjj0W0yrBkNb100qb2oFvw
Date: Sun, 11 Aug 2019 23:52:49 +0000
Message-ID: <02A34F284D1DA44BB705E61F7180EF0AAEDF3EFE@ORSMSX114.amr.corp.intel.com>
References: <20190808213048.35628-1-nathaniel.l.desimone@intel.com>
 <20190809103232.GL25813@bivouac.eciton.net>
In-Reply-To: <20190809103232.GL25813@bivouac.eciton.net>
Accept-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
dlp-product: dlpe-windows
dlp-version: 11.2.0.6
dlp-reaction: no-action
x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNTlmZTJiZDAtMTBjNy00MDEwLThlZmUtMTM2MDMyNzRiNWVhIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiQVhqNW1udk9oNzY3RGxaU1VCV1hcL0N6dDhCNHpWaFBZNVpcL0FxZVFjbHorOVJNRDY2U2E3ODVzZW1idFpzaUZiIn0=
x-ctpclassification: CTP_NT
x-originating-ip: [10.22.254.139]
MIME-Version: 1.0
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

Hi Leif,

> I am well aware that there are platform-specific steps required to achiev=
e 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?

I'm not sure. Liming would know more on that than I. For now, our MinPlatfo=
rm... platforms are using the
build_bios.py script written by Prince as a substitute.

> For this patch, I would prefer if instead of adding these notes we change=
d the ## [Intel](Platform/Intel/Readme.md)
> ### Minimum Platforms

Done. Please see patch V2.

> Arguably, it would make sense to move all of the MinPlatform ... platform=
s ... and the Readme.md to a separate
> subdirectory and link to that directory. Not everything under Platform/In=
tel is a MinPlatform platform - soon
> there'll even be an ARM one.

I totally agree with you that would make more sense. Could you perhaps send=
 an RFC and then we can start
the reorganization process?

Thanks,
Nate