[coreboot-gerrit] Patch set updated for coreboot: northbridge/amd/amdmct/mct_ddr3: Properly indicate clobbered registers

[Timothy Pearson]

Timothy Pearson ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12012

-gerrit

commit 8dd850ec23fc71d1aa026595154270300c4bb433
Author: Timothy Pearson 
Date:   Sat Jun 27 17:52:18 2015 -0500

northbridge/amd/amdmct/mct_ddr3: Properly indicate clobbered registers

Change-Id: Icb2754143762bd64ee1df5674fa071de1c595eaf
    Signed-off-by: Timothy Pearson 
---
 src/northbridge/amd/amdmct/mct_ddr3/mct_d_gcc.h | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d_gcc.h b/src/northbridge/amd/amdmct/mct_ddr3/mct_d_gcc.h
index c4bfd3f..0d9c886 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d_gcc.h
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d_gcc.h
@@ -119,6 +119,9 @@ static void proc_CLFLUSH(u32 addr_hi)

static void WriteLNTestPattern(u32 addr_lo, u8 *buf_a, u32 line_num)
 {
+	uint32_t step = 16;
+	uint32_t count = line_num * 4;
+
    __asm__ volatile (
    	/*prevent speculative execution of following instructions*/
    	/* FIXME: needed ? */
@@ -131,7 +134,7 @@ static void WriteLNTestPattern(u32 addr_lo, u8 *buf_a, u32 line_num)
    	"loop 1b\n\t"
    	"mfence\n\t"

-		 :: "a" (addr_lo), "d" (16), "c" (line_num * 4), "b"(buf_a)
+		 : "+a" (addr_lo), "+d" (step), "+c" (count), "+b" (buf_a) : :
    );

}
@@ -251,6 +254,10 @@ static void ReadMaxRdLat1CLTestPattern_D(u32 addr)

static void WriteMaxRdLat1CLTestPattern_D(u32 buf, u32 addr)
 {
+	uint32_t addr_phys = addr << 8;
+	uint32_t step = 16;
+	uint32_t count = 3 * 4;
+
    SetUpperFSbase(addr);

__asm__ volatile (
@@ -263,7 +270,7 @@ static void WriteMaxRdLat1CLTestPattern_D(u32 buf, u32 addr)
    	"loop 1b\n\t"
    	"mfence\n\t"

-		 :: "a" (addr<<8), "d" (16), "c" (3 * 4), "b"(buf)
+		 : "+a" (addr_phys), "+d" (step), "+c" (count), "+b" (buf) : :
    );
 }