UefiCpuPkg/MtrrLib: Remove unnecessary API MtrrGetVariableMtrr()

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2849

MtrrGetVariableMtrr() returns all the variable MTRR settings.
But in fact MtrrGetAllMtrrs() and
MtrrGetMemoryAttributeInVariableMtrr() are used by callers to get the
MTRR settings. The former one returns both the fixed and variable
MTRR settings.

The patch removes the unnecessary API MtrrGetVariableMtrr() to
simplify the MtrrLib API.

There is no code in edk2 and edk2-platforms repo that calls
MtrrGetVariableMtrr().

Signed-off-by: Ray Ni <ray.ni@intel.com>
Reviewed-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>

1 files changed, 1 insertions, 16 deletions

diff --git a/UefiCpuPkg/Include/Library/MtrrLib.h b/UefiCpuPkg/Include/Library/MtrrLib.h
index 95ffbc8bf1..cfe3c0ab59 100644
--- a/UefiCpuPkg/Include/Library/MtrrLib.h
+++ b/UefiCpuPkg/Include/Library/MtrrLib.h
@@ -1,7 +1,7 @@
 /** @file

   MTRR setting library

 

-  Copyright (c) 2008 - 2018, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2008 - 2020, Intel Corporation. All rights reserved.<BR>

   SPDX-License-Identifier: BSD-2-Clause-Patent

 

 **/

@@ -186,21 +186,6 @@ MtrrGetMemoryAttribute (
 

 

 /**

-  This function will get the raw value in variable MTRRs

-

-  @param[out]  VariableSettings   A buffer to hold variable MTRRs content.

-

-  @return The buffer point to MTRR_VARIABLE_SETTINGS in which holds the content of the variable MTRR

-

-**/

-MTRR_VARIABLE_SETTINGS*

-EFIAPI

-MtrrGetVariableMtrr (

-  OUT MTRR_VARIABLE_SETTINGS         *VariableSettings

-  );

-

-

-/**

   This function sets variable MTRRs

 

   @param[in]  VariableSettings   A buffer to hold variable MTRRs content.