[coreboot-gerrit] Patch set updated for coreboot: nb/amd/mct_ddr3: Enhance debugging around MEMCLK frequency change

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Thu Apr 21 08:54:08 CEST 2016


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

-gerrit

commit 9702cc3f9fd0260f62561889b7f06984002ce2bf
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Thu Apr 21 01:34:03 2016 -0500

    nb/amd/mct_ddr3: Enhance debugging around MEMCLK frequency change
    
    Change-Id: I5056cf885b7063a97c095bfaaf01dd8da777a425
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdmct/mct_ddr3/mctwl.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mctwl.c b/src/northbridge/amd/amdmct/mct_ddr3/mctwl.c
index b354d92..44ea6e8 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mctwl.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mctwl.c
@@ -2,7 +2,7 @@
  * This file is part of the coreboot project.
  *
  * Copyright (C) 2010 Advanced Micro Devices, Inc.
- * Copyright (C) 2015 Timothy Pearson <tpearson at raptorengineeringinc.com>, Raptor Engineering
+ * Copyright (C) 2015 - 2016 Raptor Engineering, LLC
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -358,9 +358,13 @@ void SetTargetFreq(struct MCTStatStruc *pMCTstat,
 	uint32_t dword;
 	uint8_t package_type = mctGet_NVbits(NV_PACK_TYPE);
 
+	printk(BIOS_DEBUG, "%s: Start\n", __func__);
+
 	struct DCTStatStruc *pDCTstat;
 	pDCTstat = pDCTstatA + Node;
 
+	printk(BIOS_DEBUG, "%s: Node %d: New frequency code: %04x\n", __func__, Node, pDCTstat->TargetFreq);
+
 	if (is_fam15h()) {
 		/* Program F2x[1, 0]90[DisDllShutDownSR]=1. */
 		if (pDCTstat->DIMMValidDCT[0]) {
@@ -448,6 +452,8 @@ void SetTargetFreq(struct MCTStatStruc *pMCTstat,
 			FreqChgCtrlWrd(pMCTstat, pDCTstat, 1);
 		}
 	}
+
+	printk(BIOS_DEBUG, "%s: Done\n", __func__);
 }
 
 static void Modify_OnDimmMirror(struct DCTStatStruc *pDCTstat, u8 dct, u8 set)



More information about the coreboot-gerrit mailing list