Re: [edk2-devel] [edk2-rfc] [RFC] Request to move MinPlatformPkg out of the Intel folder

["Hot Tian" <hot.tian@intel.com>]

Why not move to edk2 repo?

Thanks,
Hot

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Leif Lindholm
Sent: Friday, October 02, 2020 7:26
To: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>
Cc: rfc@edk2.groups.io; devel@edk2.groups.io; Laszlo Ersek <lersek@redhat.com>; Ard Biesheuvel <ard.biesheuvel@arm.com>; Kirkendall, Garrett <garrett.kirkendall@amd.com>; Lendacky, Thomas <thomas.lendacky@amd.com>; thomas.abraham@arm.com; Frank.Orr@dell.com; Jim Dailey <jim.dailey@dell.com>; Abner Chang <abner.chang@hpe.com>; Wang, Sunny (HPS SW) <sunnywang@hpe.com>; Daniel Schaefer <daniel.schaefer@hpe.com>; Liming Gao <gaoliming@byosoft.com.cn>; Tim Lewis <tim.lewis@insyde.com>; Kevin@Insyde <kevin.davis@insyde.com>; Felix Polyudov <felixp@ami.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Oram, Isaac W <isaac.w.oram@intel.com>; Ni, Ray <ray.ni@intel.com>
Subject: Re: [edk2-devel] [edk2-rfc] [RFC] Request to move MinPlatformPkg out of the Intel folder

On Thu, Oct 01, 2020 at 23:20:01 +0000, Desimone, Nathaniel L wrote:
> > One thing I think is needed for this to move out of Platform/Intel 
> > would be a ReadMe.rst describing the intended real-world use.
> > Some of this lives in Platform/Intel/Readme.md today.
> > Could we start with a set moving this text into a separate file 
> > under MinPlatformPkg and rework that into something that stands on its own?
> 
> Yeah agreed, given that the feedback on this RFC has been pretty 
> positive thus far, I'm planning on making a patch series for this. 
> I'll make these changes part of said patch series.

Sounds good.

> > If I was to bikeshed, I'd probably suggest 
> > Platform/TianoCore/MinPlatformPkg. Oops, guess I did.
> 
> Seems a little superfluous to me... isn't edk2-platforms part of 
> TianoCore by definition? 😊

It is. It's just the pattern for everything else in edk2-platforms (apart from OptionRomPkg, which was just transplanted from edk2) exists in the form of <section>/$VENDOR/. Platform/MinPlatformPkg would be the only bare package in Platform/.

> > I think we should see this as a separate step, regardless.
> > 
> > I do think we should be able to do better on sharing some of this 
> > platform image tooling cross-architecture (even moreso now Risc-V is 
> > involved), but I would prefer for us to spend the effort of 
> > identifying the intersections upfront before we start uploading 
> > tools to places where they look generic and may confuse people.
> > 
> > Maybe we should raise this part as a BZ?
> 
> Yup, sounds like good next steps to me. I've filed the BZ: 
> https://bugzilla.tianocore.org/show_bug.cgi?id=2992

Top man.

Best Regards,

Leif