[AMD Official Use Only - General]
Thanks Tinh,
It looks all good.
Reviewed-by: Abner Chang <abner.chang@amd.com>
> -----Original Message-----
> From: Tinh Nguyen <tinhnguyen@os.amperecomputing.com>
> Sent: Wednesday, May 3, 2023 7:24 PM
> To: devel@edk2.groups.io
> Cc: patches@amperecomputing.com; michael.d.kinney@intel.com;
> gaoliming@byosoft.com.cn; zhiguang.liu@intel.com; Chang, Abner
> <Abner.Chang@amd.com>; Tinh Nguyen
> <tinhnguyen@os.amperecomputing.com>
> Subject: [PATCH v3 0/3] Add IPMI SSIF definitions
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Change from v2:
> - Add the carbon copy (Cc) in the commit message.
> - Add Bugzilla ID in the commit message.
> - Mention why I set the number of retries as 250.
>
> Change from v1:
> - Separate the commit for the IPMI SSIF header and IPMI SSIF PCDs
> - Correct some indent and comment style.
> - Change the PCDs name for a more readable.
> - Support KCS/SMIC Interface Capabilities too.
>
>
> Tinh Nguyen (3):
> MdePkg/IndustryStandard: Add definitions for IPMI SSIF
> MdePkg: Add new PCDs for IPMI SSIF
> MdePkg/IndustryStandard: Add Interface Capabilities definitions
>
> MdePkg/MdePkg.dec | 27 ++++++
> MdePkg/Include/IndustryStandard/IpmiNetFnApp.h | 46 +++++++++
> MdePkg/Include/IndustryStandard/IpmiSsif.h | 98 ++++++++++++++++++++
> 3 files changed, 171 insertions(+)
> create mode 100644 MdePkg/Include/IndustryStandard/IpmiSsif.h
>
> --
> 2.40.0
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#103910): https://edk2.groups.io/g/devel/message/103910
Mute This Topic: https://groups.io/mt/98659401/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-