public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Pedro Falcato" <pedro.falcato@gmail.com>
To: "Kirkendall, Garrett" <Garrett.Kirkendall@amd.com>
Cc: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"Oram, Isaac W" <isaac.w.oram@intel.com>,
	 "Chiu, Chasel" <chasel.chiu@intel.com>,
	 "Desimone, Nathaniel L" <nathaniel.l.desimone@intel.com>,
	"Gao, Liming" <gaoliming@byosoft.com.cn>,
	 "Dong, Eric" <eric.dong@intel.com>,
	"Bobroff, Zachary" <zacharyb@ami.com>,
	 "Zimmer, Vincent" <vincent.zimmer@intel.com>
Subject: Re: [edk2-devel] MinPlatformPkg question
Date: Wed, 1 Feb 2023 01:18:11 +0000	[thread overview]
Message-ID: <CAKbZUD2eKNDJPHj4tVAOHeY1RX4K=_5MDWBPgqi_f9nqTDC=Yw@mail.gmail.com> (raw)
In-Reply-To: <PH7PR12MB64412EDC7A46F389FD0295F985D09@PH7PR12MB6441.namprd12.prod.outlook.com>


[-- Attachment #1.1: Type: text/plain, Size: 3018 bytes --]

On Tue, Jan 31, 2023 at 7:28 PM Kirkendall, Garrett <
Garrett.Kirkendall@amd.com> wrote:

> [AMD Official Use Only - General]
>
>
>
> While I can work with Fsp named items in the MinPlatformPkg specification,
> I assumed the UEFI/edk2 team and maintainers might be amenable to making
> the specification more generic.  One of my concerns with Fsp named FVs is
> that critical core edk2 components are specified in them like PeiCore is
> specified in FvFspM.fv, etc.  There is only one guaranteed vendor
> implementing FSP and therefore it might be better to have more generic
> names which could attract more adopters more easily and reduce confusion.
> Maybe there could be specified alternate names for non-FSP implementations?
>

Hi,

To be clear, you are right that renaming things from FspX (and other
possible examples) is a good idea. It's just not really a hindrance
(despite silly politics), the docs are. You do not need the FSP to use
MinPlatform.

If you want to talk about real problems, we can start by mentioning that
MinPlatform docs are not really all that great. They:
 1) Mention Intel and Intel platforms all that time (why? this is supposed
to be vendor agnostic) as well, almost assuming you're an Intel platform at
points
 2) Do not go in much depth about creating your own MinPlatform (and
subsequently OpenBoardPkg) from scratch, and assume you already have an
OpenBoardPkg to simply copy from
 3) Lack simplicity and conciseness in many points

All of this may very well slow adoption. As far as I know, on the
QemuOpenBoard GSoC project the spec was mostly very underutilized except
some bits like the list of required modules.


> Having FSP in the name would imply that the product supports FSP when it
> does not.
>
>
>
> I'm looking forward in time as much as possible where this specification
> could encompass ARM, RISCV, etc. and provide similar useful items
> MinPlatformPkg can provide to x86 platforms.
>

MinPlatform is AIUI at the moment pretty x86 PC centric. It requires
components from PcAtChipsetPkg, UefiCpuPkg (which as you may know, is
x86-only/mostly despite the name). Also requires a bunch of components like
SATA and USB support which may not be wanted or needed in e.g a smaller
riscv64 platform. It assumes a bog-standard PC environment where you need
all sorts of features and components and drivers.

Something I would personally love would be all sorts of reduced booting in
MinPlatform (as in boot-to-OS-but-load-it-from-flash, etc. Many options),
but it's probably far away still.

I look forward to the next level of unified flow/structure that Minimum
> Platform can provide to the industry.
>
>
>
> *GARRETT KIRKENDALL*
>
>
> *----------------------------------------------------------------------------------------------------------------------------------*
>
> Facebook <https://www.facebook.com/AMD> |  Twitter
> <https://twitter.com/AMD> |  amd.com <http://www.amd.com/>
>
>
>
>
>
> Words to live by: "Slow is Smooth.  Smooth is Fast."
>
>
>


-- 
Pedro

[-- Attachment #1.2: Type: text/html, Size: 6230 bytes --]

[-- Attachment #2: image001.png --]
[-- Type: image/png, Size: 2614 bytes --]

      parent reply	other threads:[~2023-02-01  1:18 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-12 13:33 MinPlatformPkg question Kirkendall, Garrett
2023-01-12 21:44 ` Isaac Oram
2023-01-31 16:54   ` Kirkendall, Garrett
2023-01-31 18:57     ` [edk2-devel] " Pedro Falcato
2023-01-31 19:28       ` Kirkendall, Garrett
2023-01-31 20:43         ` Isaac Oram
2023-02-01  1:18         ` Pedro Falcato [this message]

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='CAKbZUD2eKNDJPHj4tVAOHeY1RX4K=_5MDWBPgqi_f9nqTDC=Yw@mail.gmail.com' \
    --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