[AMD Official Use Only - General] Hi Isaac, I am not quite understand, this patch set should be independent of edk2. What do you mean the edk2 changes were pushed first? Or IpmiFeaturePkg is good now?
Abner > -----Original Message----- > From: Oram, Isaac W <[email protected]> > Sent: Friday, January 6, 2023 1:42 AM > To: [email protected]; Oram, Isaac W <[email protected]>; Chang, > Abner <[email protected]> > Cc: Desimone, Nathaniel L <[email protected]>; Kinney, > Michael D <[email protected]>; Gao, Liming > <[email protected]>; Liu, Zhiguang <[email protected]>; > Nickle Wang <[email protected]>; Igor Kulchytskyy <[email protected]> > Subject: RE: [edk2-devel] [edk2-platforms][PATCH V2] IPMI changes for > Redfish > > Caution: This message originated from an External Source. Use proper > caution when opening attachments, clicking links, or responding. > > > Series pushed as: 28f3ce4a6a251de327b8d19c3316a654887f4f13.. > f5da8af6befbfc655fd3e4cb899eda158dd9ede5 > > My apologies, I was sloppy and didn't ensure the edk2 changes were pushed > first, so IpmiFeaturePkg build is currently broken pending > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fedk > 2.groups.io%2Fg%2Fdevel%2Fmessage%2F97789being&data=05%7C01%7Ca > bner.chang%40amd.com%7C6301c092067c4f19a1b208daef441af4%7C3dd896 > 1fe4884e608e11a82d994e183d%7C0%7C0%7C638085373047816357%7CUnkn > own%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik > 1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=tBhHMIQhVL8AI3bof > YIPHcTIzobpQ7lgZgwFJa0z0Gc%3D&reserved=0 being pushed. > > Regards, > Isaac > > > -----Original Message----- > From: [email protected] <[email protected]> On Behalf Of Isaac > Oram > Sent: Thursday, January 5, 2023 9:05 AM > To: [email protected]; [email protected] > Cc: Desimone, Nathaniel L <[email protected]>; Kinney, > Michael D <[email protected]>; Gao, Liming > <[email protected]>; Liu, Zhiguang <[email protected]>; > Nickle Wang <[email protected]>; Igor Kulchytskyy <[email protected]> > Subject: Re: [edk2-devel] [edk2-platforms][PATCH V2] IPMI changes for > Redfish > > Series Reviewed-by: Isaac Oram <[email protected]> > > -----Original Message----- > From: [email protected] <[email protected]> > Sent: Thursday, January 5, 2023 8:01 AM > To: [email protected] > Cc: Oram, Isaac W <[email protected]>; Desimone, Nathaniel L > <[email protected]>; Kinney, Michael D > <[email protected]>; Gao, Liming <[email protected]>; > Liu, Zhiguang <[email protected]>; Nickle Wang <[email protected]>; > Igor Kulchytskyy <[email protected]> > Subject: [edk2-platforms][PATCH V2] IPMI changes for Redfish > > From: Abner Chang <[email protected]> > > In V2: Addressed the comments from Isaac. > > BZ# 4214 > This patch adds IPMI functions for building up SMBIOS 42 record for the USB > NIC host interface exposed by BMC. > The corresponding updates on edk2 is in the separate pacth. > > This is the part I of upstream, the follow up patche sets are: > - RedfishPlatformCredentialLib contributed by NVidia. > - Relocate IpmiCommandLib.h from edk2-platforms > to edk2/MdePkg. A NULL instance of IpmiCommandLib is created > and located under MdePkg. With above, RedfishPkg doesn't have the > dependence with edk2-platforms. The implementation of IpmiCommandLib > stays in edk2-platforms without relocation, which can be pulled in > to platform DSC as needed. > - PlatformHostInterfaceBmcUsbNic would be upstream to edk2-staging > temporarily because some requirements are necessray to be spec out > in the Redfish section in UEFI spec. > > Signed-off-by: Abner Chang <[email protected]> > Cc: Isaac Oram <[email protected]> > Cc: Nate DeSimone <[email protected]> > Cc: Michael D Kinney <[email protected]> > Cc: Liming Gao <[email protected]> > Cc: Zhiguang Liu <[email protected]> > Cc: Nickle Wang <[email protected]> > Cc: Igor Kulchytskyy <[email protected]> > > Abner Chang (3): > Features/IpmiFeaturePkg: Add IPMI functions > IpmiFeaturePkg/IpmiCommandLib: Add IPMI functions > IpmiFeaturePkg: Add reference of IpmiBaseLib > > .../IpmiFeaturePkg/IpmiFeaturePkg.dec | 5 ++ > .../Include/Library/IpmiCommandLib.h | 62 +++++++++++++ > .../IpmiCommandLib/IpmiCommandLibNetFnApp.c | 88 > +++++++++++++++++++ > .../IpmiCommandLibNetFnTransport.c | 43 +++++++++ > 4 files changed, 198 insertions(+) > > -- > 2.37.1.windows.1 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#98048): https://edk2.groups.io/g/devel/message/98048 Mute This Topic: https://groups.io/mt/96074589/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
