diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2020-11-12 13:51:46 +0100 |
|---|---|---|
| committer | Angel Pons <th3fanbus@gmail.com> | 2020-11-19 23:02:33 +0000 |
| commit | 1a9b5aa46288ba25f033d4e89fd2817ab198dbb2 (patch) | |
| tree | 8f45415fdf5721c2896c7c2700b10a7838edc1c6 /src | |
| parent | 2bf28ed63219a6bb76c6e39c8de0e58f4e35e28c (diff) | |
| download | coreboot-1a9b5aa46288ba25f033d4e89fd2817ab198dbb2.tar.xz | |
nb/intel/sandybridge: Relocate `get_ODT` function
This function is only used in two places, so move its definition closer.
Change-Id: I21d3e04de45f58cef0603b6b75119cae4b1a7aae
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47517
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Frans Hendriks <fhendriks@eltan.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/northbridge/intel/sandybridge/raminit_common.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c index 59dd5be357..83fea80dc6 100644 --- a/src/northbridge/intel/sandybridge/raminit_common.c +++ b/src/northbridge/intel/sandybridge/raminit_common.c @@ -597,18 +597,6 @@ void dram_jedecreset(ramctr_timing *ctrl) } } -static odtmap get_ODT(ramctr_timing *ctrl, u8 rank, int channel) -{ - /* Get ODT based on rankmap */ - int dimms_per_ch = (ctrl->rankmap[channel] & 1) + ((ctrl->rankmap[channel] >> 2) & 1); - - if (dimms_per_ch == 1) { - return (const odtmap){60, 60}; - } else { - return (const odtmap){120, 30}; - } -} - static void write_mrreg(ramctr_timing *ctrl, int channel, int slotrank, int reg, u32 val) { wait_for_iosav(channel); @@ -718,6 +706,18 @@ static void dram_mr0(ramctr_timing *ctrl, u8 rank, int channel) write_mrreg(ctrl, channel, rank, 0, make_mr0(ctrl, rank)); } +static odtmap get_ODT(ramctr_timing *ctrl, u8 rank, int channel) +{ + /* Get ODT based on rankmap */ + int dimms_per_ch = (ctrl->rankmap[channel] & 1) + ((ctrl->rankmap[channel] >> 2) & 1); + + if (dimms_per_ch == 1) { + return (const odtmap){60, 60}; + } else { + return (const odtmap){120, 30}; + } +} + static u32 encode_odt(u32 odt) { switch (odt) { |