On 2023-06-14 18:57, Chang, Abner wrote:
[AMD Official Use Only - General]

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Leif
Lindholm via groups.io
Sent: Thursday, June 15, 2023 12:13 AM
To: devel@edk2.groups.io; Chang, Abner <abner.ch...@amd.com>
Cc: Michael D Kinney <michael.d.kin...@intel.com>; Attar, AbdulLateef
(Abdul Lateef) <abdullateef.at...@amd.com>; Xing, Eric
<eric.x...@amd.com>; He, Jiangang <jiangang...@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.ch...@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.ch...@amd.com>
Cc: Leif Lindholm <quic_llind...@quicinc.com>
Cc: Michael D Kinney <michael.d.kin...@intel.com>
Cc: Abdul Lateef Attar <abdat...@amd.com>
Cc: Eric Xing <eric.x...@amd.com>
Cc: Jiangang He <jiangang...@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+tianoc...@kernel.org>
   M: Leif Lindholm <quic_llind...@quicinc.com>

+AMD
+F: Platform/AMD
+M: Abner Chang <abner.ch...@amd.com>
+M: Abdul Lateef Attar <abdat...@amd.com>
+
   AMD Seattle
   F: Platform/AMD/OverdriveBoard/
   F: Platform/LeMaker/CelloBoard/
@@ -91,11 +96,6 @@ F: Silicon/AMD/Styx/
   M: Ard Biesheuvel <ardb+tianoc...@kernel.org>
   M: Leif Lindholm <quic_llind...@quicinc.com>

-AMD MinBoard package for MinPlatformPkg
-F: Platform/AMD/AmdMinBoardPkg
-M: Abner Chang <abner.ch...@amd.com>
-M: Abdul Lateef Attar <abdat...@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:

Reviewed-by: Leif Lindholm <quic_llind...@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 (#106116): https://edk2.groups.io/g/devel/message/106116
Mute This Topic: https://groups.io/mt/99521710/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: 
https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to