[coreboot-gerrit] New patch to review for coreboot: northbridge/amd/amdht: Fix poor performance on Family 15h CPUs

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Fri Oct 16 21:46:44 CEST 2015


Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12030

-gerrit

commit e3af706b1868225c9889a9ab45abfe434394aa30
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Fri Aug 7 19:05:45 2015 -0500

    northbridge/amd/amdht: Fix poor performance on Family 15h CPUs
    
    Change-Id: I37db191c144c81aba5d4a1e6291db5669a35a31a
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdht/h3ncmn.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/northbridge/amd/amdht/h3ncmn.c b/src/northbridge/amd/amdht/h3ncmn.c
index e03e5eb..e377ff2 100644
--- a/src/northbridge/amd/amdht/h3ncmn.c
+++ b/src/northbridge/amd/amdht/h3ncmn.c
@@ -1559,6 +1559,10 @@ static void setLinkData(sMainData *pDat, cNorthBridge *nb)
 			} else {
 				temp2 = 0x0;
 			}
+			/* NOTE
+			 * The Family 15h BKDG Rev. 3.14 is wrong
+			 * Freq[4] must be set before Freq[3:0], otherwise the register writes will be ignored!
+			 */
 			if (is_gt_rev_d())
 				AmdPCIWriteBits(linkBase + HTHOST_FREQ_REV_REG_2, 0, 0, &temp2);
 			AmdPCIWriteBits(linkBase + HTHOST_FREQ_REV_REG, 11, 8, &temp);



More information about the coreboot-gerrit mailing list