[coreboot-gerrit] Change in coreboot[master]: nb/intel/x4x: Add the option for stacked channel map settings
Arthur Heymans (Code Review)
gerrit at coreboot.org
Sat May 26 15:12:53 CEST 2018
Arthur Heymans has uploaded a new patch set (#2). ( https://review.coreboot.org/26563 )
Change subject: nb/intel/x4x: Add the option for stacked channel map settings
......................................................................
nb/intel/x4x: Add the option for stacked channel map settings
There seems to be a hardware bug where the combination of non-stacked
channel settings, both channels populated and 533MHz dram speed cause
the display to be unusable.
The code to actually select stacked mode based on hardware
configuration will be add in a followup patch.
This patch does the following:
* Add option to the sysinfo struct for stacked mode
* Fix programming channel 1 DRB which needs special care for the last
populated rank in stacked mode
TESTED on Intel dg41wv (with stacked mode hardcoded and dram at 533MHz)
Change-Id: I95965bfea129b37f64163159fefa1c8f16331b62
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/northbridge/intel/x4x/raminit_ddr23.c
M src/northbridge/intel/x4x/x4x.h
2 files changed, 37 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/63/26563/2
--
To view, visit https://review.coreboot.org/26563
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I95965bfea129b37f64163159fefa1c8f16331b62
Gerrit-Change-Number: 26563
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180526/667e305b/attachment.html>
More information about the coreboot-gerrit
mailing list