From: Abner Chang <abner.chang@amd.com>
Add Readme file of edk2 platform ManageabilityPkg.
Signed-off-by: Abner Chang <abner.chang@amd.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Isaac Oram <isaac.w.oram@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Nickle Wang <nicklew@nvidia.com>
Cc: Igor Kulchytskyy <igork@ami.com>
Cc: Abdul Lateef Attar <abdattar@amd.com>
---
Features/ManageabilityPkg/Readme.md | 37 +++++++++++++++++++++++++++++
1 file changed, 37 insertions(+)
create mode 100644 Features/ManageabilityPkg/Readme.md
diff --git a/Features/ManageabilityPkg/Readme.md b/Features/ManageabilityPkg/Readme.md
new file mode 100644
index 0000000000..880aa4aa48
--- /dev/null
+++ b/Features/ManageabilityPkg/Readme.md
@@ -0,0 +1,37 @@
+# EDK2 Manageability Package
+
+The edk2 Manageability package is introduced to provide edk2 drivers and
+libraries for the industry platform management standards, such as PLDM (Platform
+Level Data Model), MCTP (Management Component Transfer Protocol) and IPMI
+(Intelligent Platform Management Interface). The framework of edk2 Manageability
+package is designed to flexibly support transports for the above industry standards, the transports such as KCS or I2C for IPMI, PCI VDM (Vendor Defined
+Message), I2C or KCS for MCTP, or the OEM proprietary transports.
+
+## EDK2 Manageability Package Driver Stack
+
+Below figure shows the driver stacks which are abstracted to support disparate
+transports for the platform management.
+
+
+## Build the Manageability Package
+In order to use the modules provided by ManageabilityPkg, **PACKAGES_PATH** must
+contains the path to point to [edk2-platform Features](https://github.com/tianocore/edk2-platforms/tree/master/Features):
+
+```
+$ export PACKAGES_PATH=$PWD/edk2:$PWD/edk2-platforms:$PWD/edk2-platforms/Features
+```
+
+Due to the [IPMI Migration Work](#migration-of-ipmi-driver-stack-from-outofbandmanagement) is in progress, an
+additional package path is required to add to **PACKAGES_PATH** for the reference
+of IPMI modules those are currently located under
+[OutOfBandManagement](https://github.com/tianocore/edk2-platforms/tree/master/Features/Intel/OutOfBandManagement) folder.
+
+```
+$PWD/edk2-platforms/Features/Intel/OutOfBandManagement
+```
+
+## Migration of IPMI Driver Stack from OutOfBandManagement
+
+This is the ongoing task that relocates IPMI modules from OutOfBandManagement to
+under ManageabilityPkg. This task also requires to abstract the transport layer
+from IPMI driver in order to flexibly support different transports adopted on the platform.
--
2.37.1.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#99747): https://edk2.groups.io/g/devel/message/99747
Mute This Topic: https://groups.io/mt/96810543/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Nits, not blocking (I don't know how pedantic the community is and if we care about these insignificant details, obviously lots of code doesn't): - I don't find that saying it is the "EDK2" Manageability Package adds much value/clarity. I would strip out "EDK2" from the discussion, except where it is clarifying. My reasoning is that we may expand with content that isn't really edk2 content in some senses. We might want to put more content in here and I wouldn't want people to thing it doesn't belong because of the name. - "for the industry platform management standards". I stumbled on this sentance, maybe removing "the" would be clearer. - "PACKAGES_PATH must contains the path". "contains" should probably be the singular "contain". Regards, Isaac -----Original Message----- From: abner.chang@amd.com <abner.chang@amd.com> Sent: Tuesday, February 7, 2023 8:23 AM To: devel@edk2.groups.io Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Oram, Isaac W <isaac.w.oram@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Nickle Wang <nicklew@nvidia.com>; Igor Kulchytskyy <igork@ami.com>; Abdul Lateef Attar <abdattar@amd.com> Subject: [edk2-platforms][PATCH 3/7] ManageabilityPkg: Add Readme file From: Abner Chang <abner.chang@amd.com> Add Readme file of edk2 platform ManageabilityPkg. Signed-off-by: Abner Chang <abner.chang@amd.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Isaac Oram <isaac.w.oram@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Nickle Wang <nicklew@nvidia.com> Cc: Igor Kulchytskyy <igork@ami.com> Cc: Abdul Lateef Attar <abdattar@amd.com> --- Features/ManageabilityPkg/Readme.md | 37 +++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) create mode 100644 Features/ManageabilityPkg/Readme.md diff --git a/Features/ManageabilityPkg/Readme.md b/Features/ManageabilityPkg/Readme.md new file mode 100644 index 0000000000..880aa4aa48 --- /dev/null +++ b/Features/ManageabilityPkg/Readme.md @@ -0,0 +1,37 @@ +# EDK2 Manageability Package + +The edk2 Manageability package is introduced to provide edk2 drivers +and libraries for the industry platform management standards, such as +PLDM (Platform Level Data Model), MCTP (Management Component Transfer +Protocol) and IPMI (Intelligent Platform Management Interface). The +framework of edk2 Manageability package is designed to flexibly support +transports for the above industry standards, the transports such as KCS or I2C for IPMI, PCI VDM (Vendor Defined Message), I2C or KCS for MCTP, or the OEM proprietary transports. + +## EDK2 Manageability Package Driver Stack + +Below figure shows the driver stacks which are abstracted to support +disparate transports for the platform management. + + +## Build the Manageability Package +In order to use the modules provided by ManageabilityPkg, +**PACKAGES_PATH** must contains the path to point to [edk2-platform Features](https://github.com/tianocore/edk2-platforms/tree/master/Features): + +``` +$ export +PACKAGES_PATH=$PWD/edk2:$PWD/edk2-platforms:$PWD/edk2-platforms/Feature +s +``` + +Due to the [IPMI Migration +Work](#migration-of-ipmi-driver-stack-from-outofbandmanagement) is in +progress, an additional package path is required to add to +**PACKAGES_PATH** for the reference of IPMI modules those are currently +located under +[OutOfBandManagement](https://github.com/tianocore/edk2-platforms/tree/master/Features/Intel/OutOfBandManagement) folder. + +``` +$PWD/edk2-platforms/Features/Intel/OutOfBandManagement +``` + +## Migration of IPMI Driver Stack from OutOfBandManagement + +This is the ongoing task that relocates IPMI modules from +OutOfBandManagement to under ManageabilityPkg. This task also requires +to abstract the transport layer from IPMI driver in order to flexibly support different transports adopted on the platform. -- 2.37.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#100255): https://edk2.groups.io/g/devel/message/100255 Mute This Topic: https://groups.io/mt/96810543/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
[AMD Official Use Only - General] > -----Original Message----- > From: Oram, Isaac W <isaac.w.oram@intel.com> > Sent: Thursday, February 16, 2023 9:44 AM > To: Chang, Abner <Abner.Chang@amd.com>; devel@edk2.groups.io > Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Desimone, Nathaniel L > <nathaniel.l.desimone@intel.com>; Nickle Wang <nicklew@nvidia.com>; > Igor Kulchytskyy <igork@ami.com>; Attar, AbdulLateef (Abdul Lateef) > <AbdulLateef.Attar@amd.com> > Subject: RE: [edk2-platforms][PATCH 3/7] ManageabilityPkg: Add Readme > file > > Caution: This message originated from an External Source. Use proper > caution when opening attachments, clicking links, or responding. > > > Nits, not blocking (I don't know how pedantic the community is and if we > care about these insignificant details, obviously lots of code doesn't): So what do you mean here as I am not the English native person and not quite understand the meaning of this sentence. Do you think we don't need the readme? > - I don't find that saying it is the "EDK2" Manageability Package adds much > value/clarity. Not sure what do you mean here. > I would strip out "EDK2" from the discussion, except where it is > clarifying. My reasoning is that we may expand with content that isn't really > edk2 content in some senses. We might want to put more content in here > and I wouldn't want people to thing it doesn't belong because of the name. This is the initial version and we will add more content to describe the edk2 protocols for the manageability drivers such as protocol for PLDM and MCTP. The design of library API and etc. Thanks Abner > - "for the industry platform management standards". I stumbled on this > sentance, maybe removing "the" would be clearer. > - "PACKAGES_PATH must contains the path". "contains" should probably be > the singular "contain". > > Regards, > Isaac > > -----Original Message----- > From: abner.chang@amd.com <abner.chang@amd.com> > Sent: Tuesday, February 7, 2023 8:23 AM > To: devel@edk2.groups.io > Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Oram, Isaac W > <isaac.w.oram@intel.com>; Desimone, Nathaniel L > <nathaniel.l.desimone@intel.com>; Nickle Wang <nicklew@nvidia.com>; > Igor Kulchytskyy <igork@ami.com>; Abdul Lateef Attar > <abdattar@amd.com> > Subject: [edk2-platforms][PATCH 3/7] ManageabilityPkg: Add Readme file > > From: Abner Chang <abner.chang@amd.com> > > Add Readme file of edk2 platform ManageabilityPkg. > > Signed-off-by: Abner Chang <abner.chang@amd.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Cc: Isaac Oram <isaac.w.oram@intel.com> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> > Cc: Nickle Wang <nicklew@nvidia.com> > Cc: Igor Kulchytskyy <igork@ami.com> > Cc: Abdul Lateef Attar <abdattar@amd.com> > --- > Features/ManageabilityPkg/Readme.md | 37 > +++++++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 Features/ManageabilityPkg/Readme.md > > diff --git a/Features/ManageabilityPkg/Readme.md > b/Features/ManageabilityPkg/Readme.md > new file mode 100644 > index 0000000000..880aa4aa48 > --- /dev/null > +++ b/Features/ManageabilityPkg/Readme.md > @@ -0,0 +1,37 @@ > +# EDK2 Manageability Package > + > +The edk2 Manageability package is introduced to provide edk2 drivers > +and libraries for the industry platform management standards, such as > +PLDM (Platform Level Data Model), MCTP (Management Component > Transfer > +Protocol) and IPMI (Intelligent Platform Management Interface). The > +framework of edk2 Manageability package is designed to flexibly support > +transports for the above industry standards, the transports such as KCS or > I2C for IPMI, PCI VDM (Vendor Defined Message), I2C or KCS for MCTP, or > the OEM proprietary transports. > + > +## EDK2 Manageability Package Driver Stack > + > +Below figure shows the driver stacks which are abstracted to support > +disparate transports for the platform management. > + > + > +## Build the Manageability Package > +In order to use the modules provided by ManageabilityPkg, > +**PACKAGES_PATH** must contains the path to point to [edk2-platform > Features](https://github.com/tianocore/edk2- > platforms/tree/master/Features): > + > +``` > +$ export > +PACKAGES_PATH=$PWD/edk2:$PWD/edk2-platforms:$PWD/edk2- > platforms/Feature > +s > +``` > + > +Due to the [IPMI Migration > +Work](#migration-of-ipmi-driver-stack-from-outofbandmanagement) is in > +progress, an additional package path is required to add to > +**PACKAGES_PATH** for the reference of IPMI modules those are > currently > +located under > +[OutOfBandManagement](https://github.com/tianocore/edk2- > platforms/tree/master/Features/Intel/OutOfBandManagement) folder. > + > +``` > +$PWD/edk2-platforms/Features/Intel/OutOfBandManagement > +``` > + > +## Migration of IPMI Driver Stack from OutOfBandManagement > + > +This is the ongoing task that relocates IPMI modules from > +OutOfBandManagement to under ManageabilityPkg. This task also requires > +to abstract the transport layer from IPMI driver in order to flexibly support > different transports adopted on the platform. > -- > 2.37.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#100303): https://edk2.groups.io/g/devel/message/100303 Mute This Topic: https://groups.io/mt/96810543/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2026 Red Hat, Inc.