Re: [edk2-devel] [PATCH V3 0/9] Refine BMC USB NIC discovery and Redfish service enablement

["Mike Maslenkin" <mike.maslenkin@gmail.com>]

On Mon, Nov 27, 2023 at 8:31 AM <abner.chang@amd.com> wrote:
>
> From: Abner Chang <abner.chang@amd.com>
>
> In V3: Address Mike's comments to V2.
>
> In V2: Send additional two patches those are missed
>        in V1.
>
> This patch set updates the algorithm of BMC USB
> NIC discovery and fixes some bugs.
>
> - Add a new protocol to trigger the notification
>   of Redfish Host Interface readiness.
>   This fixes the issue Redfish config handler driver
>   acquires Redfish service before Redfish Host
>   Interface is published.
> - Add more error debug messages.
> - Address some minor issues
>
> Signed-off-by: Abner Chang <abner.chang@amd.com>
> Cc: Nickle Wang <nicklew@nvidia.com>
> Cc: Igor Kulchytskyy <igork@ami.com>
> Cc: Mike Maslenkin <mike.maslenkin@gmail.com>
>
> Abner Chang (9):
>   RedfishPkg/BmcUsbNicLib: Update BMC USB NIC searching algorithm
>   RedfishPkg/RedfishHostInterfaceDxe: Add Redfish HI readiness
>     notification
>   RedfishPkg/RedfishConfigHandler: Use Redfish HI readiness notification
>   RedfishPkg/RedfishConfigHandler: Correct the prototype of callback
>     function
>   RedfishPkg/RedfishDiscovery: Add more debug message
>   RedfishPkg/RedfishDiscovery: Refine SMBIOS 42h code
>   RedfishPkg/HostInterfaceBmcUsbNic: Fix incorrect reference of MAC
>     address pointer
>   RedfishPkg/HostInterfaceBmcUsbNic: Fix incorrect HI protocol record
>     size
>   RedfishPkg/HostInterfaceBmcUsbNic: Fix potential memory corruption
>     issue
>
>  RedfishPkg/RedfishPkg.dec                     |   3 +
>  .../RedfishConfigHandlerDriver.inf            |   9 +-
>  .../RedfishHostInterfaceDxe.inf               |   3 +-
>  .../RedfishDiscoverInternal.h                 |   2 +
>  .../PlatformHostInterfaceBmcUsbNicLib.c       | 196 ++++++++++++------
>  .../RedfishConfigHandlerDriver.c              | 170 +++++++++------
>  .../RedfishDiscoverDxe/RedfishDiscoverDxe.c   |  23 ++
>  .../RedfishSmbiosHostInterface.c              |  20 +-
>  .../RedfishHostInterfaceDxe.c                 |  18 ++
>  9 files changed, 315 insertions(+), 129 deletions(-)
>
> --
> 2.37.1.windows.1
>

Looks good to me.
Thank you.

Regards,
Mike


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112012): https://edk2.groups.io/g/devel/message/112012
Mute This Topic: https://groups.io/mt/102824306/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-