UefiCpuPkg/MtrrLib: Remove unnecessary API MtrrSetVariableMtrr()

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

MtrrSetVariableMtrr() sets all the variable MTRR settings.
But in fact MtrrSetAllMtrrs() is always used by callers to set all
MTRR settings including the fixed and variable ones.

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

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

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, 0 insertions, 29 deletions

diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
index f4a10edc87..8b54f2c03b 100644
--- a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
+++ b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
@@ -2558,35 +2558,6 @@ MtrrSetVariableMtrrWorker (
   }

 }

 

-

-/**

-  This function sets variable MTRRs

-

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

-

-  @return The pointer of VariableSettings

-

-**/

-MTRR_VARIABLE_SETTINGS*

-EFIAPI

-MtrrSetVariableMtrr (

-  IN MTRR_VARIABLE_SETTINGS         *VariableSettings

-  )

-{

-  MTRR_CONTEXT  MtrrContext;

-

-  if (!IsMtrrSupported ()) {

-    return VariableSettings;

-  }

-

-  MtrrLibPreMtrrChange (&MtrrContext);

-  MtrrSetVariableMtrrWorker (VariableSettings);

-  MtrrLibPostMtrrChange (&MtrrContext);

-  MtrrDebugPrintAllMtrrs ();

-

-  return  VariableSettings;

-}

-

 /**

   Worker function setting fixed MTRRs