[coreboot-gerrit] Patch merged into coreboot/master: 582b2ae FSP & CBMEM: Fix broken cbmem CAR transition.

gerrit at coreboot.org gerrit at coreboot.org
Fri Feb 6 00:53:16 CET 2015


the following patch was just integrated into master:
commit 582b2aee0f5ac84339853551cca1f4d21e1e3fca
Author: Martin Roth <gaumless at gmail.com>
Date:   Sun Jan 11 14:29:29 2015 -0700

    FSP & CBMEM: Fix broken cbmem CAR transition.
    
    1) Save the pointer to the FSP HOB list to low memory at address 0x614.
    
    This is the same location as CBMEM_RESUME_BACKUP - the two aren't used
    in the same platform, so overlapping should be OK.  I didn't see any
    documentation that actually said that this location was free to use, and
    didn't need to be restored after use in S3 resume, but it looks like
    the DOS boot vector gets loaded juat above this location, so it SHOULD
    be ok.  The alternative is to copy the memory out and store it in cbmem
    until we're ready to restore it.
    
    2) When a request for the pointer to a CAR variable comes in, pass back
    the location inside the FSP hob structure.
    
    3) Skip the memcopy of the CAR Data.   The CAR variables do not
    get transitioned back into cbmem, but used out of the HOB structure.
    
    4) Remove the BROKEN_CAR_MIGRATE Kconfig option from the FSP platform.
    
    Change-Id: Iaf566dce1b41a3bcb17e4134877f68262b5e113f
    Signed-off-by: Martin Roth <gaumless at gmail.com>
    Reviewed-on: http://review.coreboot.org/8196
    Reviewed-by: Aaron Durbin <adurbin at google.com>
    Tested-by: build bot (Jenkins)
    Reviewed-by: Kyösti Mälkki <kyosti.malkki at gmail.com>


See http://review.coreboot.org/8196 for details.

-gerrit



More information about the coreboot-gerrit mailing list