From: "Leif Lindholm" <leif@nuviainc.com>
To: "Desimone, Nathaniel L" <nathaniel.l.desimone@intel.com>
Cc: "rfc@edk2.groups.io" <rfc@edk2.groups.io>,
"devel@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" <thomas.abraham@arm.com>,
"Frank.Orr@dell.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-rfc] [RFC] Request to move MinPlatformPkg out of the Intel folder
Date: Fri, 2 Oct 2020 00:26:21 +0100 [thread overview]
Message-ID: <20201001232621.GL5623@vanye> (raw)
In-Reply-To: <MWHPR1101MB21604636A84AA3E2102DCA08CD300@MWHPR1101MB2160.namprd11.prod.outlook.com>
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
next prev parent reply other threads:[~2020-10-01 23:26 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-30 23:17 [RFC] Request to move MinPlatformPkg out of the Intel folder Nate DeSimone
2020-10-01 4:04 ` Abner Chang
2020-10-01 6:54 ` Laszlo Ersek
2020-10-01 23:27 ` [edk2-devel] " Nate DeSimone
2020-10-01 7:43 ` Wim Vervoorn
2020-10-01 8:00 ` [edk2-devel] " christian.walter
2020-10-01 11:24 ` Kirkendall, Garrett
2020-10-01 11:52 ` Leif Lindholm
2020-10-01 23:20 ` [edk2-rfc] " Nate DeSimone
2020-10-01 23:26 ` Leif Lindholm [this message]
2020-10-02 4:19 ` [edk2-devel] " Hot Tian
2020-10-04 20:46 ` Nate DeSimone
2020-10-04 22:00 ` Leif Lindholm
2020-10-05 20:11 ` Nate DeSimone
2020-10-03 3:36 ` [edk2-devel] " Sean
2020-10-04 21:02 ` Nate DeSimone
2020-10-05 19:35 ` Bret Barkelew
2020-10-05 20:18 ` [edk2-rfc] " Nate DeSimone
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201001232621.GL5623@vanye \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox