[AMD Official Use Only - General] Thanks Tinh, It looks all good. Reviewed-by: Abner Chang <abner.ch...@amd.com>
> -----Original Message----- > From: Tinh Nguyen <tinhngu...@os.amperecomputing.com> > Sent: Wednesday, May 3, 2023 7:24 PM > To: devel@edk2.groups.io > Cc: patc...@amperecomputing.com; michael.d.kin...@intel.com; > gaolim...@byosoft.com.cn; zhiguang....@intel.com; Chang, Abner > <abner.ch...@amd.com>; Tinh Nguyen > <tinhngu...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
