[edk2-devel] [edk2-platforms][PATCH] edk2-platform: Add Readme under Platform\AMD
Chang, Abner via groups.io
abner.chang=amd.com at groups.io
Thu Jun 15 15:25:11 UTC 2023
[AMD Official Use Only - General]
> -----Original Message-----
> From: Leif Lindholm <quic_llindhol at quicinc.com>
> Sent: Thursday, June 15, 2023 11:18 PM
> To: Chang, Abner <Abner.Chang at amd.com>; devel at edk2.groups.io
> Cc: Michael D Kinney <michael.d.kinney at intel.com>; Attar, AbdulLateef
> (Abdul Lateef) <AbdulLateef.Attar at amd.com>; Xing, Eric
> <Eric.Xing at amd.com>; He, Jiangang <Jiangang.He at amd.com>; Ard
> Biesheuvel <ardb+tianocore at kernel.org>
> Subject: Re: [edk2-devel] [edk2-platforms][PATCH] edk2-platform: Add
> Readme under Platform\AMD
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> On 2023-06-14 18:57, Chang, Abner wrote:
> > [AMD Official Use Only - General]
> >
> >> -----Original Message-----
> >> From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Leif
> >> Lindholm via groups.io
> >> Sent: Thursday, June 15, 2023 12:13 AM
> >> To: devel at edk2.groups.io; Chang, Abner <Abner.Chang at amd.com>
> >> Cc: Michael D Kinney <michael.d.kinney at intel.com>; Attar, AbdulLateef
> >> (Abdul Lateef) <AbdulLateef.Attar at amd.com>; Xing, Eric
> >> <Eric.Xing at amd.com>; He, Jiangang <Jiangang.He at amd.com>
> >> Subject: Re: [edk2-devel] [edk2-platforms][PATCH] edk2-platform: Add
> >> Readme under Platform\AMD
> >>
> >> Caution: This message originated from an External Source. Use proper
> caution
> >> when opening attachments, clicking links, or responding.
> >>
> >>
> >> On 2023-06-14 05:22, Chang, Abner via groups.io wrote:
> >>> From: Abner Chang <abner.chang at amd.com>
> >>>
> >>> For AMD edk2 opensource project and the needs
> >>> of opensource AMD platform common modules those can
> >>> be leverage across AMD business units, AMD server,
> >>> client and S3 (silicon strategic solution) BUs
> >>> have the consensus to propose AMD as the maintainer
> >>> of Platform/AMD folder. The folder and edk2 package
> >>> structure under Platform/AMD is defined in the
> >>> Readme for the further upstream works.
> >>>
> >>> We still leave the existing maintainers for
> >>> AMD OverdriveBoard which is no conflicts with this
> >>> change.
> >>>
> >>> Signed-off-by: Abner Chang <abner.chang at amd.com>
> >>> Cc: Leif Lindholm <quic_llindhol at quicinc.com>
> >>> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> >>> Cc: Abdul Lateef Attar <abdattar at amd.com>
> >>> Cc: Eric Xing <eric.xing at amd.com>
> >>> Cc: Jiangang He <jiangang.he at amd.com>
> >>> ---
> >>> Platform/AMD/Readme.md | 85
> >> ++++++++++++++++++++++++++++++++++++++++++
> >>> Maintainers.txt | 10 ++---
> >>> 2 files changed, 90 insertions(+), 5 deletions(-)
> >>> create mode 100644 Platform/AMD/Readme.md
> >>>
> >>> diff --git a/Platform/AMD/Readme.md b/Platform/AMD/Readme.md
> >>> new file mode 100644
> >>> index 0000000000..f39b8140f5
> >>> --- /dev/null
> >>> +++ b/Platform/AMD/Readme.md
> >>> @@ -0,0 +1,85 @@
> >>> +# AMD EDK2 Platform
> >>> +
> >>> +This is AMD folder that provides the edk2 modules to support AMD edk2
> >> firmware
> >>> +solution for the server, client (e.g., Notebook) and S3 (Strategic Silicon
> >> Solutions)
> >>> +platforms. The board packages under this folder are the firmware
> reference
> >> code for
> >>> +booting certain AMD platforms. The definition of sub-folders is described
> in
> >> below sections.
> >>> +
> >>> +## Term and Definitions
> >>> +
> >>> +* **AMD Platform** (platform in short)
> >>> +
> >>> + AMD platform refers to a platform that supports the particular AMD SoC
> >> (processor), such as AMD EPYC Milan and Genoa processors.
> >>> +
> >>> +* **AMD Board** (board in short)
> >>> +
> >>> + AMD board is a generic terminology refers to a board that is designed
> >> based on a
> >>> + specific AMD SoC architecture (also referred as AMD platform). More
> than
> >> one boards
> >>> + are possibly designed to support an AMD platform with different
> >> configuration, such as 1-processor socket or 2-processor sockets board.
> >>> +
> >>> +* **AMD edk2 Platform Package** (platform package in short)
> >>> +
> >>> + The folder has the AMD edk2 platform common modules.
> >>> +
> >>> +* **AMD edk2 Board Package** (board package in short)
> >>> +
> >>> + The folder has the edk2 meta files to build the necessary edk2 firmware
> >> modules
> >>> + and generate the binary to run on a board.
> >>> +
> >>> +## Package Definition
> >>> +
> >>> +* **AmdPlatformPkg**
> >>> +
> >>> + AMD platform edk2 package under this folder provides the common
> edk2
> >> modules those
> >>> + are leverage by platforms. Usually those modules have no dependencies
> >> with
> >>> + particular platforms. The module under this scope can provides a
> common
> >> implementation
> >>> + for all platforms, or it may just provide a framework but the differences
> of
> >> implementation could be configured through the PCDs declared in
> >> AmdPlatformPkg.dec, or
> >>> + the board level library provided in the \<Board name\>Pkg.
> >>> +
> >>> +* **AmdMinBoardPkg**
> >>> +
> >>> + This package provides the common edk2 modules those can be leverage
> >> across AMD boards those use MinPlatformPkg framework.
> >>> +
> >>> +* **\<SoC name\>Board**
> >>> +
> >>> + This is the folder named by SoC and accommodate one or multiple
> board
> >> packages those
> >>> + are designed base on the same SoC platform. <SoC name>Board folder
> >> may contain edk2
> >>> + package meta files directly or the sub-folders named by \<Board
> >> name\>Pkg for a
> >>> + variety configurations of a platform.
> >>> +
> >>> +* **<Board name\>Pkg**
> >>> +
> >>> + This is the folder that contains edk2 package meta files for a board which
> is
> >> designed base on a platform. Besides the edk2 meta files, <Board
> name\>Pkg
> >> may also provides
> >>> + edk2 modules which are specifically to a board.
> >>> +
> >>> + ```
> >>> + e.g. OverdriveBoard
> >>> + e.g. GenoaBoard
> >>> + |------Board1Pkg
> >>> + |------Board2Pkg
> >>> + ```
> >>> +
> >>> + Below is the outline of folder structure under Platform/AMD
> >>> +
> >>> + ```
> >>> + Platform/AMD
> >>> + |----AmdPlatformPkg
> >>> + |----AmdMinBoardPkg
> >>> + |----OverdriveBoard
> >>> + |----GenoaBoard
> >>> + | |------Common Modules for Genoa boards
> >>> + | |------Board1Pkg
> >>> + | | |-------Board specific modules
> >>> + | |------Board2Pkg
> >>> + |
> >>> + |----NextGenBoard
> >>> + |------Common Modules for the next generation
> >>> + platform boards
> >>> + |------Board1Pkg
> >>> + |------Board2Pkg
> >>> + |-------Board specific modules
> >>> + ```
> >>> +
> >>> +
> >>> +## Board Support
> >>> +Under progress
> >>> \ No newline at end of file
> >>
> >> *cough* ^
> > Lol, come on Leif, You always play fun on me 😊. Please give us some time to
> work on this.
> >
> >>
> >>> diff --git a/Maintainers.txt b/Maintainers.txt
> >>> index 020c72677a..d1d7613ef4 100644
> >>> --- a/Maintainers.txt
> >>> +++ b/Maintainers.txt
> >>> @@ -83,6 +83,11 @@ F: Platform/96Boards/
> >>> M: Ard Biesheuvel <ardb+tianocore at kernel.org>
> >>> M: Leif Lindholm <quic_llindhol at quicinc.com>
> >>>
> >>> +AMD
> >>> +F: Platform/AMD
> >>> +M: Abner Chang <abner.chang at amd.com>
> >>> +M: Abdul Lateef Attar <abdattar at amd.com>
> >>> +
> >>> AMD Seattle
> >>> F: Platform/AMD/OverdriveBoard/
> >>> F: Platform/LeMaker/CelloBoard/
> >>> @@ -91,11 +96,6 @@ F: Silicon/AMD/Styx/
> >>> M: Ard Biesheuvel <ardb+tianocore at kernel.org>
> >>> M: Leif Lindholm <quic_llindhol at quicinc.com>
> >>>
> >>> -AMD MinBoard package for MinPlatformPkg
> >>> -F: Platform/AMD/AmdMinBoardPkg
> >>> -M: Abner Chang <abner.chang at amd.com>
> >>> -M: Abdul Lateef Attar <abdattar at amd.com>
> >>> -
> >>
> >> I think this change is ideal, but can I just confirm that it is your
> >> intention to take on overall maintainership of Platform/AMD while
> >> leaving myself and Ard as additional maintainers for the OverDriveBoard?
> >> (If not, you could add an X: tag. But I'd be happier if you didn't :)
> > Hmm..., actually this is the respect for you guys. But yes, my intention is
> AMD take over the maintainership over Platform\AMD, but still leave the
> maintainership to you and Ard for OverdirveBoard 😊. But we can help on it if
> you encounter some problems.
> > Or you would like AMD to take over everything? We are not familiar with
> OverdriveBaord though.
>
> I like the change as present in this patch.
> So if you add that missing newline at end of file:
Sure I will. Thanks you.
Abner
>
> Reviewed-by: Leif Lindholm <quic_llindhol at quicinc.com>
>
> /
> Leif
>
> > Regards,
> > Abner
> >
> >>
> >> /
> >> Leif
> >>
> >>> Ampere Computing
> >>> F: Platform/Ampere
> >>> F: Silicon/Ampere
> >>
> >>
> >>
> >>
> >>
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#106118): https://edk2.groups.io/g/devel/message/106118
Mute This Topic: https://groups.io/mt/99521710/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list