[coreboot-gerrit] New patch to review for coreboot: 2a59b1d AMD CAR: Fix issue with gcc 4.8.x

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Wed Nov 20 01:11:43 CET 2013


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4205

-gerrit

commit 2a59b1d72552c9ec2df04a1775d095e4ba5c384e
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Tue Nov 19 14:26:50 2013 -0800

    AMD CAR: Fix issue with gcc 4.8.x
    
    Starting with gcc 4.8.x we can't add ebp to the clobber list
    anymore. Since we're not actually using the register here, just
    removing it does not cause issues.
    
    Change-Id: I226c14b7e651919ec2d61cbcb18be50cf7f2f7a8
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 src/cpu/amd/car/post_cache_as_ram.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/cpu/amd/car/post_cache_as_ram.c b/src/cpu/amd/car/post_cache_as_ram.c
index eca7673..bad7535 100644
--- a/src/cpu/amd/car/post_cache_as_ram.c
+++ b/src/cpu/amd/car/post_cache_as_ram.c
@@ -119,7 +119,7 @@ static void post_cache_as_ram(void)
 		::"a"( (CONFIG_DCACHE_RAM_BASE + CONFIG_DCACHE_RAM_SIZE)- (CONFIG_RAMTOP) )
 		/* discard all registers (eax is used for %0), so gcc redoes everything
 		   after the stack is moved */
-		: "cc", "memory", "%ebx", "%ecx", "%edx", "%esi", "%edi", "%ebp"
+		: "cc", "memory", "%ebx", "%ecx", "%edx", "%esi", "%edi"
 	);
 
 	/* We can put data to stack again */



More information about the coreboot-gerrit mailing list