GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

nb/intel/sandybridge/raminit: Find CMD rate per channel 

No need to find the same CMD rate for all channels.
Allow different CMD rates for every channel.

Tested on Lenovo T420 with different modules on each channel.
No regressions found.

Change-Id: I7036275ae89335dd3549ec392fa64824355b3cbf
Signed-off-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-on: https://review.coreboot.org/17472
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Patrick Rudolph 2016-11-16 19:21:31 +01:00
parent 069018d69a
commit 45d6a5588a
1 changed files with 56 additions and 56 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
src/northbridge/intel/sandybridge/raminit.c
View File

@ -2971,22 +2971,23 @@ static void reprogram_320c(ramctr_timing * ctrl)
#define MIN_C320C_LEN 13
static int try_cmd_stretch(ramctr_timing *ctrl, int cmd_stretch)
static int try_cmd_stretch(ramctr_timing *ctrl, int channel, int cmd_stretch)
{
struct ram_rank_timings saved_timings[NUM_CHANNELS][NUM_SLOTRANKS];
int channel, slotrank;
int slotrank;
int c320c;
int stat[NUM_SLOTRANKS][256];
int delta = 0;
FOR_ALL_CHANNELS FOR_ALL_POPULATED_RANKS {
saved_timings[channel][slotrank] = ctrl->timings[channel][slotrank];
printram("Trying cmd_stretch %d on channel %d\n", cmd_stretch, channel);
FOR_ALL_POPULATED_RANKS {
saved_timings[channel][slotrank] =
ctrl->timings[channel][slotrank];
}
FOR_ALL_POPULATED_CHANNELS {
ctrl->cmd_stretch[channel] = cmd_stretch;
}
FOR_ALL_POPULATED_CHANNELS
MCHBAR32(0x4004 + 0x400 * channel) =
ctrl->tRRD
| (ctrl->tRTP << 4)
@ -2996,9 +2997,6 @@ static int try_cmd_stretch(ramctr_timing *ctrl, int cmd_stretch)
| (ctrl->tWR << 24)
| (ctrl->cmd_stretch[channel] << 30);
FOR_ALL_CHANNELS {
int delta = 0;
if (ctrl->cmd_stretch[channel] == 2)
delta = 2;
else if (ctrl->cmd_stretch[channel] == 0)
@ -3007,9 +3005,7 @@ static int try_cmd_stretch(ramctr_timing *ctrl, int cmd_stretch)
FOR_ALL_POPULATED_RANKS {
ctrl->timings[channel][slotrank].val_4024 -= delta;
}
}
FOR_ALL_POPULATED_CHANNELS {
for (c320c = -127; c320c <= 127; c320c++) {
FOR_ALL_POPULATED_RANKS {
ctrl->timings[channel][slotrank].val_320c = c320c;
@ -3033,13 +3029,14 @@ static int try_cmd_stretch(ramctr_timing *ctrl, int cmd_stretch)
slotrank,
ctrl->timings[channel][slotrank].val_320c);
if (rn.all || rn.length < MIN_C320C_LEN) {
FOR_ALL_CHANNELS FOR_ALL_POPULATED_RANKS {
ctrl->timings[channel][slotrank] = saved_timings[channel][slotrank];
FOR_ALL_POPULATED_RANKS {
ctrl->timings[channel][slotrank] =
saved_timings[channel][slotrank];
}
return MAKE_ERR;
}
}
}
return 0;
}
@ -3056,19 +3053,22 @@ static int command_training(ramctr_timing *ctrl)
write32(DEFAULT_MCHBAR + 0x4288 + 0x400 * channel, 0x1f);
}
FOR_ALL_POPULATED_CHANNELS {
/* try command rate 1T and 2T */
err = try_cmd_stretch(ctrl, 0);
err = try_cmd_stretch(ctrl, channel, 0);
if (err) {
err = try_cmd_stretch(ctrl, 2);
err = try_cmd_stretch(ctrl, channel, 2);
if (err) {
printk(BIOS_EMERG, "c320c discovery failed\n");
return err;
}
printram("Using CMD rate 2T on channel %u\n", channel);
} else
printram("Using CMD rate 1T on channel %u\n", channel);
}
FOR_ALL_POPULATED_CHANNELS {
FOR_ALL_POPULATED_CHANNELS
program_timings(ctrl, channel);
}
reprogram_320c(ctrl);
return 0;