[AMD Official Use Only - General] Hi Leif, How about adding a separate entry? Currently its generic to adapt the MinPlatformPkg, later will add more specific platforms.
AMD board and platform F: Platform/AMD/BoardPkg F: Platform/AMD/PlatformPkg M: maintainer1 M: maintainer2 AMD Seattle F: Platform/AMD/OverdriveBoard/ F: Platform/LeMaker/CelloBoard/ F: Platform/SoftIron/ F: Silicon/AMD/Styx/ M: Ard Biesheuvel <ardb+tianoc...@kernel.org> M: Leif Lindholm <quic_llind...@quicinc.com> Thanks AbduL -----Original Message----- From: Leif Lindholm <quic_llind...@quicinc.com> Sent: 17 March 2023 21:39 To: Attar, AbdulLateef (Abdul Lateef) <abdullateef.at...@amd.com> Cc: devel@edk2.groups.io; Attar, AbdulLateef (Abdul Lateef) <abdullateef.at...@amd.com>; Ard Biesheuvel <ardb+tianoc...@kernel.org>; Chang, Abner <abner.ch...@amd.com>; Michael D Kinney <michael.d.kin...@intel.com> Subject: Re: [PATCH v2 RESEND 4/4] Maintainers.txt: Adds AMD/BoardPkg and AMD/PlatformPkg maintainers Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. On Fri, Mar 17, 2023 at 12:20:07 +0530, Abdul Lateef Attar wrote: > From: Abdul Lateef Attar <abdullateef.at...@amd.com> > > Adds maintainers for AMD/BoardPkg and AMD/PlatformPkg. > > Signed-off-by: Abdul Lateef Attar <abdat...@amd.com> > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > Cc: Leif Lindholm <quic_llind...@quicinc.com> > Cc: Abner Chang <abner.ch...@amd.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > --- > Maintainers.txt | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Maintainers.txt b/Maintainers.txt index > 747191366070..bb8ab643e090 100644 > --- a/Maintainers.txt > +++ b/Maintainers.txt > @@ -91,6 +91,12 @@ F: Silicon/AMD/Styx/ > M: Ard Biesheuvel <ardb+tianoc...@kernel.org> > M: Leif Lindholm <quic_llind...@quicinc.com> > > +AMD Platform Annoyingly, I'm now going to ask the question I have been avoiding up until now. What does "AMD Platform" mean? I mean, you've placed this straight after the entry for the Seattle platforms. I think the name, and package names, are too generic. / Leif > +F: Platform/AMD/BoardPkg > +F: Platform/AMD/PlatformPkg > +M: Abner Chang <abner.ch...@amd.com> > +M: Abdul Lateef Attar <abdat...@amd.com> > + > Ampere Computing > F: Platform/Ampere > F: Silicon/Ampere > -- > 2.25.1 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#101455): https://edk2.groups.io/g/devel/message/101455 Mute This Topic: https://groups.io/mt/97667946/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-