Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/50756 )
Change subject: mb/purism/librem_mini: Enable DRAM Refresh2X ......................................................................
mb/purism/librem_mini: Enable DRAM Refresh2X
Enable Refresh2X to mitigate RAM corruption during long (> 1hr) periods of S3/suspend, which leads to failure to successfully resume from S3. Unknown if an issue with all DRAM types, but tested w/Kingston KVR24S17D8 16GiB DDR4 SODIMMs.
Test: Build/boot Librem Mini v1/v2, put device in suspend, wait > 1hr, ensure resume from S3 successful 100% of the time.
Change-Id: Ie8e3ebbb1ebdcd98813b5f36f580a235712d2f97 Signed-off-by: Matt DeVillier matt.devillier@puri.sm Reviewed-on: https://review.coreboot.org/c/coreboot/+/50756 Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/purism/librem_cnl/variants/librem_mini/variant.c 1 file changed, 3 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Angel Pons: Looks good to me, approved
diff --git a/src/mainboard/purism/librem_cnl/variants/librem_mini/variant.c b/src/mainboard/purism/librem_cnl/variants/librem_mini/variant.c index 9d7e27f..4f415c4 100644 --- a/src/mainboard/purism/librem_cnl/variants/librem_mini/variant.c +++ b/src/mainboard/purism/librem_cnl/variants/librem_mini/variant.c @@ -9,4 +9,7 @@ mem_cfg->PchSataHsioRxGen3EqBoostMagEnable[2] = 1; mem_cfg->PchSataHsioRxGen3EqBoostMag[0] = 2; mem_cfg->PchSataHsioRxGen3EqBoostMag[2] = 1; + + /* Enable Refresh2X to mitigate S3 resume issues */ + mem_cfg->Refresh2X = 1; }