* 回复: [edk2-devel] [edk2-platforms][PATCH] Features/IpmiFeaturePkg: Remove IpmiCommandLib from IpmiFeaturePkg
2023-11-08 3:22 ` [edk2-devel] [edk2-platforms][PATCH] Features/IpmiFeaturePkg: Remove IpmiCommandLib from IpmiFeaturePkg Chang, Abner via groups.io
@ 2023-11-09 14:20 ` gaoliming via groups.io
0 siblings, 0 replies; 2+ messages in thread
From: gaoliming via groups.io @ 2023-11-09 14:20 UTC (permalink / raw)
To: 'Chang, Abner', devel
Cc: 'Nate DeSimone', 'Chasel Chiu',
'Li-Xia Huang',
'Attar, AbdulLateef (Abdul Lateef)',
'Nickle Wang'
Abner:
I agree this change. Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
Thanks
Liming
> -----邮件原件-----
> 发件人: Chang, Abner <Abner.Chang@amd.com>
> 发送时间: 2023年11月8日 11:22
> 收件人: devel@edk2.groups.io
> 抄送: Nate DeSimone <nathaniel.l.desimone@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Chasel Chiu <chasel.chiu@intel.com>; Li-Xia
> Huang <lisa.huang@intel.com>; Attar, AbdulLateef (Abdul Lateef)
> <AbdulLateef.Attar@amd.com>; Nickle Wang <nicklew@nvidia.com>
> 主题: RE: [edk2-devel] [edk2-platforms][PATCH] Features/IpmiFeaturePkg:
> Remove IpmiCommandLib from IpmiFeaturePkg
>
> [AMD Official Use Only - General]
>
> Hi Nate and Liming,
> Please check this patch. The one under IpmiFeaturePkg is an old one, we
> should remove it to avoid the confusions.
>
> Thanks
> Abner
>
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chang,
> > Abner via groups.io
> > Sent: Monday, October 30, 2023 2:59 PM
> > To: devel@edk2.groups.io
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>; Liming Gao
> > <gaoliming@byosoft.com.cn>; Chasel Chiu <chasel.chiu@intel.com>; Li-Xia
> > Huang <lisa.huang@intel.com>; Attar, AbdulLateef (Abdul Lateef)
> > <AbdulLateef.Attar@amd.com>; Nickle Wang <nicklew@nvidia.com>
> > Subject: [edk2-devel] [edk2-platforms][PATCH] Features/IpmiFeaturePkg:
> > Remove IpmiCommandLib from IpmiFeaturePkg
> >
> > Caution: This message originated from an External Source. Use proper
> caution
> > when opening attachments, clicking links, or responding.
> >
> >
> > From: Abner Chang <abner.chang@amd.com>
> >
> > Remove IpmiCommandLib from IpmiFeaturePkg as ManageabilityPkg
> > already had one and is newer.
> >
> > Signed-off-by: Abner Chang <abner.chang@amd.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Li-Xia Huang <lisa.huang@intel.com>
> > Cc: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
> > Cc: Nickle Wang <nicklew@nvidia.com>
> > ---
> > .../IpmiFeaturePkg/IpmiFeaturePkg.dec | 4 -
> > .../IpmiFeaturePkg/Include/IpmiFeature.dsc | 2 +-
> > .../Library/IpmiCommandLib/IpmiCommandLib.inf | 32 --
> > .../IpmiCommandLib/IpmiCommandLibNetFnApp.c | 335
> ------------------
> > .../IpmiCommandLibNetFnChassis.c | 144 --------
> > .../IpmiCommandLibNetFnStorage.c | 274 --------------
> > .../IpmiCommandLibNetFnTransport.c | 123 -------
> > 7 files changed, 1 insertion(+), 913 deletions(-)
> > delete mode 100644
> >
> Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiComma
> > ndLib/IpmiCommandLib.inf
> > delete mode 100644
> >
> Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiComma
> > ndLib/IpmiCommandLibNetFnApp.c
> > delete mode 100644
> >
> Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiComma
> > ndLib/IpmiCommandLibNetFnChassis.c
> > delete mode 100644
> >
> Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiComma
> > ndLib/IpmiCommandLibNetFnStorage.c
> > delete mode 100644
> >
> Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiComma
> > ndLib/IpmiCommandLibNetFnTransport.c
> >
> > diff --git
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/IpmiFeaturePkg.d
> > ec
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/IpmiFeaturePkg.d
> > ec
> > index 5df71300cb..f26741e244 100644
> > ---
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/IpmiFeaturePkg.d
> > ec
> > +++
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/IpmiFeaturePkg.d
> > ec
> > @@ -32,10 +32,6 @@
> > OsWdt
> >
> > [LibraryClasses]
> > - ## @libraryclass Provides services to send IPMI commands.
> > - #
> > - IpmiCommandLib|Include/Library/IpmiCommandLib.inf
> > -
> > ## @libraryclass Provides an API for platform-specific IPMI hooks.
> > #
> > IpmiCommandLib|Include/Library/IpmiPlatformHookLib.h
> > diff --git
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeat
> > ure.dsc
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeat
> > ure.dsc
> > index 0401974b82..063638ce56 100644
> > ---
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeat
> > ure.dsc
> > +++
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeat
> > ure.dsc
> > @@ -37,7 +37,7 @@
> > [LibraryClasses]
> > IpmiLib|MdeModulePkg/Library/BaseIpmiLibNull/BaseIpmiLibNull.inf
> >
> > -
> >
> IpmiCommandLib|IpmiFeaturePkg/Library/IpmiCommandLib/IpmiCommand
> > Lib.inf
> > +
> >
> IpmiCommandLib|ManageabilityPkg/Library/IpmiCommandLib/IpmiComman
> > dLib.inf
> >
> >
> IpmiPlatformHookLib|IpmiFeaturePkg/Library/IpmiPlatformHookLibNull/Ipmi
> > PlatformHookLibNull.inf
> >
> > [LibraryClasses.common.PEI_CORE,LibraryClasses.common.PEIM]
> > diff --git
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLib.inf
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLib.inf
> > deleted file mode 100644
> > index d5c14ff2a4..0000000000
> > ---
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLib.inf
> > +++ /dev/null
> > @@ -1,32 +0,0 @@
> > -### @file
> > -# Component description file for IPMI Command Library.
> > -#
> > -# Copyright (c) 2018 - 2021, Intel Corporation. All rights
reserved.<BR>
> > -#
> > -# SPDX-License-Identifier: BSD-2-Clause-Patent
> > -#
> > -###
> > -
> > -[Defines]
> > - INF_VERSION = 0x00010005
> > - BASE_NAME = IpmiCommandLib
> > - FILE_GUID =
> E599C9C7-5913-40A0-8669-67282E2BEC53
> > - MODULE_TYPE = UEFI_DRIVER
> > - VERSION_STRING = 1.0
> > - LIBRARY_CLASS = IpmiCommandLib
> > -
> > -[sources]
> > - IpmiCommandLibNetFnApp.c
> > - IpmiCommandLibNetFnTransport.c
> > - IpmiCommandLibNetFnChassis.c
> > - IpmiCommandLibNetFnStorage.c
> > -
> > -[Packages]
> > - MdePkg/MdePkg.dec
> > - MdeModulePkg/MdeModulePkg.dec
> > - IpmiFeaturePkg/IpmiFeaturePkg.dec
> > -
> > -[LibraryClasses]
> > - BaseMemoryLib
> > - DebugLib
> > - IpmiBaseLib
> > diff --git
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnApp.c
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnApp.c
> > deleted file mode 100644
> > index 2e34909f3e..0000000000
> > ---
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnApp.c
> > +++ /dev/null
> > @@ -1,335 +0,0 @@
> > -/** @file
> > - IPMI Command - NetFnApp.
> > -
> > - Copyright (c) 2018 - 2021, Intel Corporation. All rights
reserved.<BR>
> > - Copyright (C) 2023 Advanced Micro Devices, Inc. All rights
> reserved.<BR>
> > -
> > - SPDX-License-Identifier: BSD-2-Clause-Patent
> > -**/
> > -
> > -#include <PiPei.h>
> > -#include <Library/BaseMemoryLib.h>
> > -#include <Library/DebugLib.h>
> > -#include <Library/IpmiBaseLib.h>
> > -
> > -#include <IndustryStandard/Ipmi.h>
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetDeviceId (
> > - OUT IPMI_GET_DEVICE_ID_RESPONSE *DeviceId
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*DeviceId);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_DEVICE_ID,
> > - NULL,
> > - 0,
> > - (VOID *)DeviceId,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSelfTestResult (
> > - OUT IPMI_SELF_TEST_RESULT_RESPONSE *SelfTestResult
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*SelfTestResult);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_SELFTEST_RESULTS,
> > - NULL,
> > - 0,
> > - (VOID *)SelfTestResult,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiResetWatchdogTimer (
> > - OUT UINT8 *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_RESET_WATCHDOG_TIMER,
> > - NULL,
> > - 0,
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSetWatchdogTimer (
> > - IN IPMI_SET_WATCHDOG_TIMER_REQUEST *SetWatchdogTimer,
> > - OUT UINT8 *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_SET_WATCHDOG_TIMER,
> > - (VOID *)SetWatchdogTimer,
> > - sizeof(*SetWatchdogTimer),
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetWatchdogTimer (
> > - OUT IPMI_GET_WATCHDOG_TIMER_RESPONSE *GetWatchdogTimer
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetWatchdogTimer);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_WATCHDOG_TIMER,
> > - NULL,
> > - 0,
> > - (VOID *)GetWatchdogTimer,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSetBmcGlobalEnables (
> > - IN IPMI_SET_BMC_GLOBAL_ENABLES_REQUEST
> *SetBmcGlobalEnables,
> > - OUT UINT8 *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_SET_BMC_GLOBAL_ENABLES,
> > - (VOID *)SetBmcGlobalEnables,
> > - sizeof(*SetBmcGlobalEnables),
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetBmcGlobalEnables (
> > - OUT IPMI_GET_BMC_GLOBAL_ENABLES_RESPONSE
> *GetBmcGlobalEnables
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetBmcGlobalEnables);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_BMC_GLOBAL_ENABLES,
> > - NULL,
> > - 0,
> > - (VOID *)GetBmcGlobalEnables,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiClearMessageFlags (
> > - IN IPMI_CLEAR_MESSAGE_FLAGS_REQUEST
> *ClearMessageFlagsRequest,
> > - OUT UINT8 *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_CLEAR_MESSAGE_FLAGS,
> > - (VOID *)ClearMessageFlagsRequest,
> > - sizeof(*ClearMessageFlagsRequest),
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetMessageFlags (
> > - OUT IPMI_GET_MESSAGE_FLAGS_RESPONSE
> *GetMessageFlagsResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetMessageFlagsResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_MESSAGE_FLAGS,
> > - NULL,
> > - 0,
> > - (VOID *)GetMessageFlagsResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetMessage (
> > - OUT IPMI_GET_MESSAGE_RESPONSE *GetMessageResponse,
> > - IN OUT UINT32 *GetMessageResponseSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_MESSAGE,
> > - NULL,
> > - 0,
> > - (VOID *)GetMessageResponse,
> > - GetMessageResponseSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSendMessage (
> > - IN IPMI_SEND_MESSAGE_REQUEST *SendMessageRequest,
> > - IN UINT32 SendMessageRequestSize,
> > - OUT IPMI_SEND_MESSAGE_RESPONSE *SendMessageResponse,
> > - IN OUT UINT32 *SendMessageResponseSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_SEND_MESSAGE,
> > - (VOID *)SendMessageRequest,
> > - SendMessageRequestSize,
> > - (VOID *)SendMessageResponse,
> > - SendMessageResponseSize
> > - );
> > - return Status;
> > -}
> > -
> > -/**
> > - This function gets the system UUID.
> > -
> > - @param[out] SystemGuid The pointer to retrieve system UUID.
> > -
> > - @retval EFI_SUCCESS UUID is returned.
> > - @retval EFI_INVALID_PARAMETER SystemGuid is a NULL pointer.
> > - @retval Others Other errors.
> > -
> > -**/
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSystemUuid (
> > - OUT EFI_GUID *SystemGuid
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 RequestSize;
> > - UINT32 ResponseSize;
> > - IPMI_GET_SYSTEM_UUID_RESPONSE GetSystemUuidResponse;
> > -
> > - if (SystemGuid == NULL) {
> > - return EFI_INVALID_PARAMETER;
> > - }
> > - RequestSize = 0;
> > - ResponseSize = sizeof (IPMI_GET_SYSTEM_UUID_RESPONSE);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_SYSTEM_GUID,
> > - (VOID *)NULL,
> > - RequestSize,
> > - (VOID *)&GetSystemUuidResponse,
> > - &ResponseSize
> > - );
> > - if (!EFI_ERROR (Status) && GetSystemUuidResponse.CompletionCode ==
> > IPMI_COMP_CODE_NORMAL) {
> > - CopyMem (
> > - (VOID *)SystemGuid,
> > - (VOID *)&GetSystemUuidResponse.SystemUuid,
> > - sizeof (EFI_GUID)
> > - );
> > - }
> > - return Status;
> > -}
> > -
> > -/**
> > - This function gets the channel information.
> > -
> > - @param[in] GetChannelInfoRequest The get channel
> information
> > request.
> > - @param[out] GetChannelInfoResponse The get channel
> information
> > response.
> > - @param[out] GetChannelInfoResponseSize When input, the
> expected size
> > of response.
> > - When output, the
> exact size of the returned
> > - response.
> > -
> > - @retval EFI_SUCCESS Get channel information
> successfully.
> > - @retval EFI_INVALID_PARAMETER One of the given input parameters
> is
> > invalid.
> > - @retval Others Other errors.
> > -
> > -**/
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetChannelInfo (
> > - IN IPMI_GET_CHANNEL_INFO_REQUEST *GetChannelInfoRequest,
> > - OUT IPMI_GET_CHANNEL_INFO_RESPONSE *GetChannelInfoResponse,
> > - OUT UINT32
> *GetChannelInfoResponseSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - if (GetChannelInfoRequest == NULL ||
> > - GetChannelInfoResponse == NULL ||
> > - GetChannelInfoResponseSize == NULL) {
> > - return EFI_INVALID_PARAMETER;
> > - }
> > -
> > - *GetChannelInfoResponseSize = sizeof
> > (IPMI_GET_CHANNEL_INFO_RESPONSE);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_APP,
> > - IPMI_APP_GET_CHANNEL_INFO,
> > - (UINT8 *)GetChannelInfoRequest,
> > - sizeof (IPMI_GET_CHANNEL_INFO_REQUEST),
> > - (UINT8 *)GetChannelInfoResponse,
> > - GetChannelInfoResponseSize
> > - );
> > - return Status;
> > -}
> > diff --git
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnChassis.c
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnChassis.c
> > deleted file mode 100644
> > index 9c19f52ce4..0000000000
> > ---
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnChassis.c
> > +++ /dev/null
> > @@ -1,144 +0,0 @@
> > -/** @file
> > - IPMI Command - NetFnChassis.
> > -
> > - Copyright (c) 2018 - 2021, Intel Corporation. All rights
reserved.<BR>
> > - SPDX-License-Identifier: BSD-2-Clause-Patent
> > -**/
> > -
> > -#include <PiPei.h>
> > -#include <Library/BaseMemoryLib.h>
> > -#include <Library/DebugLib.h>
> > -#include <Library/IpmiBaseLib.h>
> > -
> > -#include <IndustryStandard/Ipmi.h>
> > -
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetChassisCapabilities (
> > - OUT IPMI_GET_CHASSIS_CAPABILITIES_RESPONSE
> > *GetChassisCapabilitiesResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetChassisCapabilitiesResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_CHASSIS,
> > - IPMI_CHASSIS_GET_CAPABILITIES,
> > - NULL,
> > - 0,
> > - (VOID *)GetChassisCapabilitiesResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetChassisStatus (
> > - OUT IPMI_GET_CHASSIS_STATUS_RESPONSE
> *GetChassisStatusResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetChassisStatusResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_CHASSIS,
> > - IPMI_CHASSIS_GET_STATUS,
> > - NULL,
> > - 0,
> > - (VOID *)GetChassisStatusResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiChassisControl (
> > - IN IPMI_CHASSIS_CONTROL_REQUEST *ChassisControlRequest,
> > - OUT UINT8 *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_CHASSIS,
> > - IPMI_CHASSIS_CONTROL,
> > - (VOID *)ChassisControlRequest,
> > - sizeof(*ChassisControlRequest),
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSetPowerRestorePolicy (
> > - IN IPMI_SET_POWER_RESTORE_POLICY_REQUEST
> *ChassisControlRequest,
> > - OUT IPMI_SET_POWER_RESTORE_POLICY_RESPONSE
> > *ChassisControlResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*ChassisControlResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_CHASSIS,
> > - IPMI_CHASSIS_SET_POWER_RESTORE_POLICY,
> > - (VOID *)ChassisControlRequest,
> > - sizeof(*ChassisControlRequest),
> > - (VOID *)ChassisControlResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSetSystemBootOptions (
> > - IN IPMI_SET_BOOT_OPTIONS_REQUEST *BootOptionsRequest,
> > - OUT IPMI_SET_BOOT_OPTIONS_RESPONSE *BootOptionsResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*BootOptionsResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_CHASSIS,
> > - IPMI_CHASSIS_SET_SYSTEM_BOOT_OPTIONS,
> > - (VOID *)BootOptionsRequest,
> > - sizeof(*BootOptionsRequest),
> > - (VOID *)BootOptionsResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSystemBootOptions (
> > - IN IPMI_GET_BOOT_OPTIONS_REQUEST *BootOptionsRequest,
> > - OUT IPMI_GET_BOOT_OPTIONS_RESPONSE *BootOptionsResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*BootOptionsResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_CHASSIS,
> > - IPMI_CHASSIS_GET_SYSTEM_BOOT_OPTIONS,
> > - (VOID *)BootOptionsRequest,
> > - sizeof(*BootOptionsRequest),
> > - (VOID *)BootOptionsResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > diff --git
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnStorage.c
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnStorage.c
> > deleted file mode 100644
> > index 2215028089..0000000000
> > ---
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnStorage.c
> > +++ /dev/null
> > @@ -1,274 +0,0 @@
> > -/** @file
> > - IPMI Command - NetFnStorage.
> > -
> > - Copyright (c) 2018 - 2021, Intel Corporation. All rights
reserved.<BR>
> > - SPDX-License-Identifier: BSD-2-Clause-Patent
> > -**/
> > -
> > -#include <PiPei.h>
> > -#include <Library/BaseMemoryLib.h>
> > -#include <Library/DebugLib.h>
> > -#include <Library/IpmiBaseLib.h>
> > -
> > -#include <IndustryStandard/Ipmi.h>
> > -
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetFruInventoryAreaInfo (
> > - IN IPMI_GET_FRU_INVENTORY_AREA_INFO_REQUEST
> > *GetFruInventoryAreaInfoRequest,
> > - OUT IPMI_GET_FRU_INVENTORY_AREA_INFO_RESPONSE
> > *GetFruInventoryAreaInfoResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetFruInventoryAreaInfoResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_GET_FRU_INVENTORY_AREAINFO,
> > - (VOID *)GetFruInventoryAreaInfoRequest,
> > - sizeof(*GetFruInventoryAreaInfoRequest),
> > - (VOID *)GetFruInventoryAreaInfoResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiReadFruData (
> > - IN IPMI_READ_FRU_DATA_REQUEST *ReadFruDataRequest,
> > - OUT IPMI_READ_FRU_DATA_RESPONSE
> *ReadFruDataResponse,
> > - IN OUT UINT32
> *ReadFruDataResponseSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_READ_FRU_DATA,
> > - (VOID *)ReadFruDataRequest,
> > - sizeof(*ReadFruDataRequest),
> > - (VOID *)ReadFruDataResponse,
> > - ReadFruDataResponseSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiWriteFruData (
> > - IN IPMI_WRITE_FRU_DATA_REQUEST *WriteFruDataRequest,
> > - IN UINT32
> WriteFruDataRequestSize,
> > - OUT IPMI_WRITE_FRU_DATA_RESPONSE
> *WriteFruDataResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*WriteFruDataResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_WRITE_FRU_DATA,
> > - (VOID *)WriteFruDataRequest,
> > - WriteFruDataRequestSize,
> > - (VOID *)WriteFruDataResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSelInfo (
> > - OUT IPMI_GET_SEL_INFO_RESPONSE *GetSelInfoResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetSelInfoResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_GET_SEL_INFO,
> > - NULL,
> > - 0,
> > - (VOID *)GetSelInfoResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSelEntry (
> > - IN IPMI_GET_SEL_ENTRY_REQUEST *GetSelEntryRequest,
> > - OUT IPMI_GET_SEL_ENTRY_RESPONSE *GetSelEntryResponse,
> > - IN OUT UINT32 *GetSelEntryResponseSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_GET_SEL_ENTRY,
> > - (VOID *)GetSelEntryRequest,
> > - sizeof(*GetSelEntryRequest),
> > - (VOID *)GetSelEntryResponse,
> > - GetSelEntryResponseSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiAddSelEntry (
> > - IN IPMI_ADD_SEL_ENTRY_REQUEST *AddSelEntryRequest,
> > - OUT IPMI_ADD_SEL_ENTRY_RESPONSE *AddSelEntryResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*AddSelEntryResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_ADD_SEL_ENTRY,
> > - (VOID *)AddSelEntryRequest,
> > - sizeof(*AddSelEntryRequest),
> > - (VOID *)AddSelEntryResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiPartialAddSelEntry (
> > - IN IPMI_PARTIAL_ADD_SEL_ENTRY_REQUEST
> *PartialAddSelEntryRequest,
> > - IN UINT32
> PartialAddSelEntryRequestSize,
> > - OUT IPMI_PARTIAL_ADD_SEL_ENTRY_RESPONSE
> > *PartialAddSelEntryResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*PartialAddSelEntryResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_PARTIAL_ADD_SEL_ENTRY,
> > - (VOID *)PartialAddSelEntryRequest,
> > - PartialAddSelEntryRequestSize,
> > - (VOID *)PartialAddSelEntryResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiClearSel (
> > - IN IPMI_CLEAR_SEL_REQUEST *ClearSelRequest,
> > - OUT IPMI_CLEAR_SEL_RESPONSE *ClearSelResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*ClearSelResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_CLEAR_SEL,
> > - (VOID *)ClearSelRequest,
> > - sizeof(*ClearSelRequest),
> > - (VOID *)ClearSelResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSelTime (
> > - OUT IPMI_GET_SEL_TIME_RESPONSE *GetSelTimeResponse
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetSelTimeResponse);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_GET_SEL_TIME,
> > - NULL,
> > - 0,
> > - (VOID *)GetSelTimeResponse,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSetSelTime (
> > - IN IPMI_SET_SEL_TIME_REQUEST *SetSelTimeRequest,
> > - OUT UINT8 *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_SET_SEL_TIME,
> > - (VOID *)SetSelTimeRequest,
> > - sizeof(*SetSelTimeRequest),
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSdrRepositoryInfo (
> > - OUT IPMI_GET_SDR_REPOSITORY_INFO_RESPONSE
> > *GetSdrRepositoryInfoResp
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*GetSdrRepositoryInfoResp);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_GET_SDR_REPOSITORY_INFO,
> > - NULL,
> > - 0,
> > - (VOID *)GetSdrRepositoryInfoResp,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSdr (
> > - IN IPMI_GET_SDR_REQUEST *GetSdrRequest,
> > - OUT IPMI_GET_SDR_RESPONSE *GetSdrResponse,
> > - IN OUT UINT32 *GetSdrResponseSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_STORAGE,
> > - IPMI_STORAGE_GET_SDR,
> > - (VOID *)GetSdrRequest,
> > - sizeof(*GetSdrRequest),
> > - (VOID *)GetSdrResponse,
> > - GetSdrResponseSize
> > - );
> > - return Status;
> > -}
> > diff --git
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnTransport.c
> >
> b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnTransport.c
> > deleted file mode 100644
> > index 30ea84c04b..0000000000
> > ---
> >
> a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiCom
> > mandLib/IpmiCommandLibNetFnTransport.c
> > +++ /dev/null
> > @@ -1,123 +0,0 @@
> > -/** @file
> > - IPMI Command - NetFnTransport.
> > -
> > - Copyright (c) 2018 - 2021, Intel Corporation. All rights
reserved.<BR>
> > - Copyright (C) 2023 Advanced Micro Devices, Inc. All rights
> reserved.<BR>
> > -
> > - SPDX-License-Identifier: BSD-2-Clause-Patent
> > -**/
> > -
> > -#include <PiPei.h>
> > -#include <Library/BaseMemoryLib.h>
> > -#include <Library/DebugLib.h>
> > -#include <Library/IpmiBaseLib.h>
> > -
> > -#include <IndustryStandard/Ipmi.h>
> > -
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSolActivating (
> > - IN IPMI_SOL_ACTIVATING_REQUEST *SolActivatingRequest,
> > - OUT UINT8 *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_TRANSPORT,
> > - IPMI_TRANSPORT_SOL_ACTIVATING,
> > - (VOID *)SolActivatingRequest,
> > - sizeof(*SolActivatingRequest),
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiSetSolConfigurationParameters (
> > - IN IPMI_SET_SOL_CONFIGURATION_PARAMETERS_REQUEST
> > *SetConfigurationParametersRequest,
> > - IN UINT32
> SetConfigurationParametersRequestSize,
> > - OUT UINT8
> *CompletionCode
> > - )
> > -{
> > - EFI_STATUS Status;
> > - UINT32 DataSize;
> > -
> > - DataSize = sizeof(*CompletionCode);
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_TRANSPORT,
> > - IPMI_TRANSPORT_SET_SOL_CONFIG_PARAM,
> > - (VOID *)SetConfigurationParametersRequest,
> > - SetConfigurationParametersRequestSize,
> > - (VOID *)CompletionCode,
> > - &DataSize
> > - );
> > - return Status;
> > -}
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetSolConfigurationParameters (
> > - IN IPMI_GET_SOL_CONFIGURATION_PARAMETERS_REQUEST
> > *GetConfigurationParametersRequest,
> > - OUT IPMI_GET_SOL_CONFIGURATION_PARAMETERS_RESPONSE
> > *GetConfigurationParametersResponse,
> > - IN OUT UINT32
> > *GetConfigurationParametersResponseSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_TRANSPORT,
> > - IPMI_TRANSPORT_GET_SOL_CONFIG_PARAM,
> > - (VOID *)GetConfigurationParametersRequest,
> > - sizeof(*GetConfigurationParametersRequest),
> > - (VOID *)GetConfigurationParametersResponse,
> > - GetConfigurationParametersResponseSize
> > - );
> > - return Status;
> > -}
> > -
> > -/**
> > - This function gets the LAN configuration parameter.
> > -
> > - @param[in] GetLanConfigurationParametersRequest Request
> data
> > - @param[out] GetLanConfigurationParametersResponse Response
> data
> > - @param[in,out] GetLanConfigurationParametersSize When input,
> the
> > expected size of response data.
> > -
> When out, the exact size of response data.
> > -
> > - @retval EFI_SUCCESS Lan configuration parameter is
> returned in the
> > response.
> > - @retval EFI_INVALID_PARAMETER One of the given input parameters
> is
> > invalid.
> > - @retval Others Other errors.
> > -
> > -**/
> > -
> > -EFI_STATUS
> > -EFIAPI
> > -IpmiGetLanConfigurationParameters (
> > - IN IPMI_GET_LAN_CONFIGURATION_PARAMETERS_REQUEST
> > *GetLanConfigurationParametersRequest,
> > - OUT IPMI_GET_LAN_CONFIGURATION_PARAMETERS_RESPONSE
> > *GetLanConfigurationParametersResponse,
> > - IN OUT UINT32
> *GetLanConfigurationParametersSize
> > - )
> > -{
> > - EFI_STATUS Status;
> > -
> > - if (GetLanConfigurationParametersRequest == NULL ||
> > - GetLanConfigurationParametersResponse == NULL ||
> > - GetLanConfigurationParametersSize == NULL) {
> > - return EFI_INVALID_PARAMETER;
> > - }
> > -
> > - Status = IpmiSubmitCommand (
> > - IPMI_NETFN_TRANSPORT,
> > - IPMI_TRANSPORT_GET_LAN_CONFIG_PARAMETERS,
> > - (UINT8 *)GetLanConfigurationParametersRequest,
> > - sizeof(*GetLanConfigurationParametersRequest),
> > - (UINT8 *)GetLanConfigurationParametersResponse,
> > - GetLanConfigurationParametersSize
> > - );
> > - return Status;
> > -}
> > --
> > 2.37.1.windows.1
> >
> >
> >
> >
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#110986): https://edk2.groups.io/g/devel/message/110986
Mute This Topic: https://groups.io/mt/102486123/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 2+ messages in thread