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>
diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index 59dd5be..83fea80 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -597,18 +597,6 @@
}
}
-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 @@
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) {