diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2022-05-30 18:51:45 +0200 |
|---|---|---|
| committer | Arthur Heymans <arthur@aheymans.xyz> | 2022-06-01 09:48:44 +0000 |
| commit | 29aa1e156771c7e24cefa52077f8de30f56b9d5d (patch) | |
| tree | db23743457c4b2551f972b2d7916cf64bdf289a4 /src/cpu | |
| parent | 4ed22601369d074425a43d2077ff84ab4b328d51 (diff) | |
Revert "cpu/x86: Add function to set `put_back_original_solution` variable"
Now that mtrr_use_temp_range() can deal with multiple ranges there is no
need to expose this to restore the MTRR solution.
This reverts commit 00aaffaf470adfbaa0fbfa0ec3cc67311763810b.
Change-Id: Ib77a0f52228cd2f19f3227824f704ac690be4aba
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64803
Reviewed-by: Subrata Banik <subratabanik@google.com>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/cpu')
| -rw-r--r-- | src/cpu/x86/mtrr/mtrr.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c index d1098d988f..6d4aa5730b 100644 --- a/src/cpu/x86/mtrr/mtrr.c +++ b/src/cpu/x86/mtrr/mtrr.c @@ -861,11 +861,6 @@ void x86_mtrr_check(void) static bool put_back_original_solution; -void need_restore_mtrr(void) -{ - put_back_original_solution = true; -} - void mtrr_use_temp_range(uintptr_t begin, size_t size, int type) { const struct range_entry *r; @@ -935,7 +930,7 @@ void mtrr_use_temp_range(uintptr_t begin, size_t size, int type) (long long)begin, (long long)begin + size - 1, (long long)size, type); else - need_restore_mtrr(); + put_back_original_solution = true; memranges_teardown(&addr_space); } |