Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: abc16e2531dc62d40fef40e62ac65a55604ddb54
      
https://github.com/tianocore/edk2/commit/abc16e2531dc62d40fef40e62ac65a55604ddb54
  Author: Ray Ni <[email protected]>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M UefiCpuPkg/Include/Library/MtrrLib.h
    M UefiCpuPkg/Library/MtrrLib/MtrrLib.c

  Log Message:
  -----------
  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 <[email protected]>
Reviewed-by: Eric Dong <[email protected]>
Reviewed-by: Laszlo Ersek <[email protected]>
Cc: Rahul Kumar <[email protected]>


  Commit: 3c9291288540f48a6812857fd59cf635aa294aa1
      
https://github.com/tianocore/edk2/commit/3c9291288540f48a6812857fd59cf635aa294aa1
  Author: Ray Ni <[email protected]>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M UefiCpuPkg/Include/Library/MtrrLib.h
    M UefiCpuPkg/Library/MtrrLib/MtrrLib.c

  Log Message:
  -----------
  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 <[email protected]>
Reviewed-by: Eric Dong <[email protected]>
Reviewed-by: Laszlo Ersek <[email protected]>
Cc: Rahul Kumar <[email protected]>


  Commit: 9c6f3545aee0808b78a0ad4480b6eb9d24989dc1
      
https://github.com/tianocore/edk2/commit/9c6f3545aee0808b78a0ad4480b6eb9d24989dc1
  Author: Ray Ni <[email protected]>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M UefiCpuPkg/Include/Library/MtrrLib.h
    M UefiCpuPkg/Library/MtrrLib/MtrrLib.c

  Log Message:
  -----------
  UefiCpuPkg/MtrrLib: Remove unnecessary API MtrrSetFixedMtrr()

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

MtrrSetFixedMtrr() sets all the fixed 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 MtrrSetFixedMtrr()
to simplify the MtrrLib API.

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

Signed-off-by: Ray Ni <[email protected]>
Reviewed-by: Eric Dong <[email protected]>
Reviewed-by: Laszlo Ersek <[email protected]>
Cc: Rahul Kumar <[email protected]>


Compare: https://github.com/tianocore/edk2/compare/d9a408454413...9c6f3545aee0


_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to