Marc Jones ([email protected]) just uploaded a new patch set to gerrit, which 
you can find at http://review.coreboot.org/1034

-gerrit

commit a5a543259f455c6d99cf1a8e69cd12cb4bafee20
Author: Marc Jones <[email protected]>
Date:   Tue May 15 16:08:24 2012 -0600

    Fix Persimmon build without S3.
    
    In the heap function, only check for S3 check when it is built in
    with CONFIG_HAVE_ACPI_RESUME.
    
    Change-Id: I439275a4e1b7b446b499bcf90c925785a14b980d
    Signed-off-by: Marc Jones <[email protected]>
---
 src/mainboard/amd/persimmon/agesawrapper.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/mainboard/amd/persimmon/agesawrapper.c 
b/src/mainboard/amd/persimmon/agesawrapper.c
index 0d63abb..195ff54 100644
--- a/src/mainboard/amd/persimmon/agesawrapper.c
+++ b/src/mainboard/amd/persimmon/agesawrapper.c
@@ -253,10 +253,12 @@ UINT32 GetHeapBase(
 {
        UINT32 heap;
 
+#if CONFIG_HAVE_APCI_RESUME
        /* Both romstage and ramstage has this S3 detect. */
        if (acpi_get_sleep_type() == 3)
                heap = (UINT32)cbmem_find(CBMEM_ID_RESUME_SCRATCH) + 
(CONFIG_HIGH_SCRATCH_MEMORY_SIZE - BIOS_HEAP_SIZE); /* himem_heap_base + 
high_stack_size */
        else
+#endif
                heap = BIOS_HEAP_START_ADDRESS; /* low mem */
 
        return heap;

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to