diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2017-10-12 11:32:20 -0600 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2017-10-16 16:56:19 +0000 |
| commit | c06a3f72f82f46eb2b56c7b84a605001f7c38049 (patch) | |
| tree | 3f1b76e5fbed82981ae847a6a97531b4e61922a7 | |
| parent | c714137d9519c12d2b951abf3c335fb1d6912029 (diff) | |
arch/x86: initialize EBDA in S3 and S0/S5 path
It's more consistent to re-initialize EBDA in all boot paths.
That way, the data living in EBDA is cleared prior to be
accessed (assuming it's after setup_ebda()).
Change-Id: I05ff84f869f7b6a463e52b4cb954acc5566475cd
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/21997
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subrata.banik@intel.com>
| -rw-r--r-- | src/arch/x86/ebda.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/arch/x86/ebda.c b/src/arch/x86/ebda.c index ba02e50401..f6726cf1af 100644 --- a/src/arch/x86/ebda.c +++ b/src/arch/x86/ebda.c @@ -18,7 +18,6 @@ #include <string.h> #include <arch/io.h> #include <arch/ebda.h> -#include <arch/acpi.h> #include <commonlib/endian.h> #include <console/console.h> @@ -84,10 +83,6 @@ void setup_ebda(u32 low_memory_size, u16 ebda_segment, u16 ebda_size) u16 ebda_kb; void *ebda; - /* Skip in S3 resume path */ - if (acpi_is_wakeup_s3()) - return; - if (!low_memory_size || !ebda_segment || !ebda_size) return; |