[coreboot-gerrit] New patch to review for coreboot: MMA: override SAGV to fixed high for MMA tests

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Sat May 7 07:59:44 CEST 2016


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14697

-gerrit

commit 6133597a4a2685409e1637781ff6be625ff68c35
Author: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
Date:   Mon Apr 11 16:44:15 2016 -0700

    MMA: override SAGV to fixed high for MMA tests
    
    Set SAGV to 2 (Fixed High) so that MMA test would
    stress memory at high freq point. MMA tests does not
    support stressing memory at both high and low points.
    
    BRANCH=glados
    BUG=chrome-os-partner:43731
    TEST=Build and Boot kunimitsu and ran MMA tests.
    
    Change-Id: I0b2f6cf9955076f6146b957c4d40fe24e6c3f0e7
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 4b16b756d9a74c9111c78fce848b059daee65669
    Original-Change-Id: I4c4a59407844e1986fa2cf3a0035aff1d8529cf9
    Original-Signed-off-by: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/339002
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    Original-(cherry picked from commit c43d9880fe4efd1e1bb853d35140424fb7dd7e99)
    Original-Reviewed-on: https://chromium-review.googlesource.com/338847
    Original-Commit-Ready: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
    Original-Tested-by: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
    Original-Reviewed-by: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
---
 src/soc/intel/common/mma.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/soc/intel/common/mma.c b/src/soc/intel/common/mma.c
index b044b00..87d8e5c 100644
--- a/src/soc/intel/common/mma.c
+++ b/src/soc/intel/common/mma.c
@@ -204,6 +204,7 @@ void setup_mma(MEMORY_INIT_UPD *memory_params)
 	memory_params->MmaTestConfigPtr = (uintptr_t) mma_test_param;
 	memory_params->MmaTestConfigSize = mma_test_param_file_len;
 	memory_params->MrcFastBoot = 0x00;
+	memory_params->SaGv = 0x02;
 
 	printk(BIOS_DEBUG, "MMA Test name %s\n", test_filename);
 	printk(BIOS_DEBUG, "MMA Test Config name %s\n", test_param_filename);
@@ -218,6 +219,9 @@ void setup_mma(MEMORY_INIT_UPD *memory_params)
 			memory_params->MmaTestConfigSize);
 	printk(BIOS_DEBUG, "memory_params->MrcFastBoot = %d\n",
 			memory_params->MrcFastBoot);
+	printk(BIOS_DEBUG, "memory_params->SaGv = %d\n",
+			memory_params->SaGv);
+
 	printk(BIOS_DEBUG, "MMA setup successfully\n");
 }
 



More information about the coreboot-gerrit mailing list