From: Chuong Tran OS <chuong@amperemail.onmicrosoft.com>
Sent: Thursday, August 22, 2024 1:58 PM
To: Nhi Pham OS <nhi@os.amperecomputing.com>; devel@edk2.groups.io <devel@edk2.groups.io>
Cc: quic_llindhol@quicinc.com <quic_llindhol@quicinc.com>; Chuong Tran OS <chuong@os.amperecomputing.com>; Rebecca Cran OS <rebecca@os.amperecomputing.com>
Subject: Re: [edk2-platforms][PATCH 0/3] AmpereAltraPkg: Remove Max Frequency config

For this series:
Reviewed-by: Chuong Tran <chuong@os.amperecomputing.com>

Thanks,
Chuong

On 8/15/2024 2:12 PM, Nhi Pham wrote:
> This patch series aims to remove the maximum (turbo) frequency since the
> CPU nominal frequency is equivalient to the max frequency.
>
> Nhi Pham (3):
>    AmpereAltraPkg/AcpiConfigDxe: Remove Max Frequency config option
>    AmpereAltraPkg/PlatformInfoDxe: Correct CPU frequency info
>    AmpereAltraPkg/SystemFirmwareInterfaceLib: Remove unused
>      MailboxMsgTurboConfig()
>
>   Silicon/Ampere/AmpereAltraPkg/Include/AcpiConfigNVDataStruct.h                                | 4 +-
>   Silicon/Ampere/AmpereAltraPkg/Include/Library/SystemFirmwareInterfaceLib.h                    | 25 ------
>   Silicon/Ampere/AmpereAltraPkg/Drivers/AcpiConfigDxe/AcpiConfigVfr.vfr                         | 13 ---
>   Silicon/Ampere/AmpereAltraPkg/Drivers/AcpiConfigDxe/AcpiConfigDxe.c                           | 83 --------------------
>   Silicon/Ampere/AmpereAltraPkg/Drivers/PlatformInfoDxe/PlatformInfoDxe.c                       | 2 +-
>   Silicon/Ampere/AmpereAltraPkg/Library/SystemFirmwareInterfaceLib/SystemFirmwareInterfaceLib.c | 43 ----------
>   Silicon/Ampere/AmpereAltraPkg/Drivers/AcpiConfigDxe/AcpiConfigStrings.uni                     | 3 -
>   7 files changed, 2 insertions(+), 171 deletions(-)
>