nb/intel/sandybridge: add and use more MCHBAR register defines

Change-Id: Ie0a9be0899830a2bf9a994d10c417b0968d1cd47
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38010
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Felix Held 2019-12-30 17:30:16 +01:00
parent 85e1491eba
commit dee167ee39
4 changed files with 30 additions and 22 deletions

View File

@ -101,9 +101,9 @@ static void report_memory_config(void)
u32 addr_decoder_common, addr_decode_ch[NUM_CHANNELS];
int i, refclk;
addr_decoder_common = MCHBAR32(0x5000);
addr_decode_ch[0] = MCHBAR32(0x5004);
addr_decode_ch[1] = MCHBAR32(0x5008);
addr_decoder_common = MCHBAR32(MAD_CHNL);
addr_decode_ch[0] = MCHBAR32(MAD_DIMM_CH0);
addr_decode_ch[1] = MCHBAR32(MAD_DIMM_CH1);
refclk = MCHBAR32(MC_BIOS_REQ) & 0x100 ? 100 : 133;

View File

@ -229,8 +229,8 @@ void dram_timing_regs(ramctr_timing *ctrl)
reg |= (ctrl->CAS << 8);
reg |= (ctrl->CWL << 12);
reg |= (ctrl->tRAS << 16);
printram("DBP [%x] = %x\n", 0x400 * channel + 0x4000, reg);
MCHBAR32(0x400 * channel + 0x4000) = reg;
printram("DBP [%x] = %x\n", TC_DBP_C0 + 0x400 * channel, reg);
MCHBAR32(TC_DBP_C0 + 0x400 * channel) = reg;
// RAP
reg = 0;
@ -241,8 +241,8 @@ void dram_timing_regs(ramctr_timing *ctrl)
reg |= (ctrl->tFAW << 16);
reg |= (ctrl->tWR << 24);
reg |= (3 << 30);
printram("RAP [%x] = %x\n", 0x400 * channel + 0x4004, reg);
MCHBAR32(0x400 * channel + 0x4004) = reg;
printram("RAP [%x] = %x\n", TC_RAP_C0 + 0x400 * channel, reg);
MCHBAR32(TC_RAP_C0 + 0x400 * channel) = reg;
// OTHP
addr = 0x400 * channel + 0x400c;
@ -271,10 +271,10 @@ void dram_timing_regs(ramctr_timing *ctrl)
reg = ((ctrl->tREFI & 0xffff) << 0) |
((ctrl->tRFC & 0x1ff) << 16) |
(((val32 / 1024) & 0x7f) << 25);
printram("REFI [%x] = %x\n", 0x400 * channel + 0x4298, reg);
MCHBAR32(0x400 * channel + 0x4298) = reg;
printram("REFI [%x] = %x\n", TC_RFTP_C0 + 0x400 * channel, reg);
MCHBAR32(TC_RFTP_C0 + 0x400 * channel) = reg;
MCHBAR32_OR(0x400 * channel + 0x4294, 0xff);
MCHBAR32_OR(TC_RFP_C0 + 0x400 * channel, 0xff);
// SRFTP
reg = 0;
@ -340,7 +340,7 @@ void dram_dimm_set_mapping(ramctr_timing * ctrl)
{
int channel;
FOR_ALL_CHANNELS {
MCHBAR32(0x5004 + channel * 4) = ctrl->mad_dimm[channel];
MCHBAR32(MAD_DIMM_CH0 + channel * 4) = ctrl->mad_dimm[channel];
}
}
@ -364,14 +364,14 @@ void dram_zones(ramctr_timing * ctrl, int training)
reg = (reg & ~0xff000000) | val << 24;
reg = (reg & ~0xff0000) | (2 * val) << 16;
MCHBAR32(0x5014) = reg;
MCHBAR32(0x5000) = 0x24;
MCHBAR32(MAD_CHNL) = 0x24;
} else {
reg = MCHBAR32(0x5014);
val = ch0size / 256;
reg = (reg & ~0xff000000) | val << 24;
reg = (reg & ~0xff0000) | (2 * val) << 16;
MCHBAR32(0x5014) = reg;
MCHBAR32(0x5000) = 0x21;
MCHBAR32(MAD_CHNL) = 0x21;
}
}
@ -926,7 +926,7 @@ static const u32 lane_registers[] = {
void program_timings(ramctr_timing * ctrl, int channel)
{
u32 reg32, reg_4024, reg_c14, reg_c18, reg_4028;
u32 reg32, reg_4024, reg_c14, reg_c18, reg_io_latency;
int lane;
int slotrank, slot;
int full_shift = 0;
@ -988,8 +988,8 @@ void program_timings(ramctr_timing * ctrl, int channel)
MCHBAR32(0xc14 + channel * 0x100) = reg_c14;
MCHBAR32(0xc18 + channel * 0x100) = reg_c18;
reg_4028 = MCHBAR32(0x4028 + 0x400 * channel);
reg_4028 &= 0xffff0000;
reg_io_latency = MCHBAR32(SC_IO_LATENCY_C0 + 0x400 * channel);
reg_io_latency &= 0xffff0000;
reg_4024 = 0;
@ -1025,7 +1025,7 @@ void program_timings(ramctr_timing * ctrl, int channel)
post_timA_max_high - post_timA_min_high)
shift_402x = -1;
reg_4028 |=
reg_io_latency |=
(ctrl->timings[channel][slotrank].val_4028 + shift_402x -
post_timA_min_high) << (4 * slotrank);
reg_4024 |=
@ -1065,7 +1065,7 @@ void program_timings(ramctr_timing * ctrl, int channel)
}
}
MCHBAR32(0x4024 + 0x400 * channel) = reg_4024;
MCHBAR32(0x4028 + 0x400 * channel) = reg_4028;
MCHBAR32(SC_IO_LATENCY_C0 + 0x400 * channel) = reg_io_latency;
}
static void test_timA(ramctr_timing * ctrl, int channel, int slotrank)
@ -3169,7 +3169,7 @@ void final_registers(ramctr_timing * ctrl)
MCHBAR32_AND(0x58a8, ~0x1f);
FOR_ALL_CHANNELS
MCHBAR32_AND_OR(0x4294 + 0x400 * channel, ~0x30000, 1 << 16);
MCHBAR32_AND_OR(TC_RFP_C0 + 0x400 * channel, ~0x30000, 1 << 16);
MCHBAR32_OR(0x5030, 1);
MCHBAR32_OR(0x5030, 0x80);

View File

@ -149,9 +149,9 @@ static void report_memory_config(void)
u32 addr_decoder_common, addr_decode_ch[2];
int i;
addr_decoder_common = MCHBAR32(0x5000);
addr_decode_ch[0] = MCHBAR32(0x5004);
addr_decode_ch[1] = MCHBAR32(0x5008);
addr_decoder_common = MCHBAR32(MAD_CHNL);
addr_decode_ch[0] = MCHBAR32(MAD_DIMM_CH0);
addr_decode_ch[1] = MCHBAR32(MAD_DIMM_CH1);
printk(BIOS_DEBUG, "memcfg DDR3 clock %d MHz\n",
(MCHBAR32(0x5e04) * 13333 * 2 + 50)/100);

View File

@ -127,7 +127,15 @@ enum platform_type {
#define MCHBAR32_AND_OR(x, and, or) \
(MCHBAR32(x) = (MCHBAR32(x) & (and)) | (or))
#define TC_DBP_C0 0x4000 /* Timing of DDR - bin parameters */
#define TC_RAP_C0 0x4004 /* Timing of DDR - regular access parameters */
#define SC_IO_LATENCY_C0 0x4028 /* IO Latency Configuration */
#define TC_RFP_C0 0x4294 /* Refresh Parameters */
#define TC_RFTP_C0 0x4298 /* Refresh Timing Parameters */
#define PM_PDWN_Config 0x4cb0
#define MAD_CHNL 0x5000 /* Address Decoder Channel Configuration */
#define MAD_DIMM_CH0 0x5004 /* Address Decode Channel 0 */
#define MAD_DIMM_CH1 0x5008 /* Address Decode Channel 1 */
#define MC_BIOS_REQ 0x5e00
#define MC_BIOS_DATA 0x5e04
#define SSKPD 0x5d14 /* 16bit (scratchpad) */