[coreboot-gerrit] New patch to review for coreboot: Experiment with circular

[Vladimir Serbinenko (phcoder@gmail.com)]

Vladimir Serbinenko (phcoder at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13527

-gerrit

commit fa266bfd17a2e248c59651f09bd4e1b0fa28d693
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Sat Jan 30 17:50:41 2016 +0100

    Experiment with circular
    
    Change-Id: I67c428524f71db88757ead7fc7b1634c0b40728e
---
 src/northbridge/intel/sandybridge/raminit.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/northbridge/intel/sandybridge/raminit.c b/src/northbridge/intel/sandybridge/raminit.c
index 8a287c1..fdfa7cd 100644
--- a/src/northbridge/intel/sandybridge/raminit.c
+++ b/src/northbridge/intel/sandybridge/raminit.c
@@ -1670,14 +1670,14 @@ struct run {
 	int length;
 };
 
-static struct run get_longest_zero_run(int *seq, int sz)
+static struct run get_longest_zero_run(int *seq, int sz, int circular)
 {
 	int i, ls;
 	int bl = 0, bs = 0;
 	struct run ret;
 
 	ls = 0;
-	for (i = 0; i < 2 * sz; i++)
+	for (i = 0; i < (1 + circular) * sz; i++)
 		if (seq[i % sz]) {
 			if (i - ls > bl) {
 				bl = i - ls;
@@ -1726,7 +1726,7 @@ static void discover_timA_coarse(ramctr_timing * ctrl, int channel,
 		}
 	}
 	FOR_ALL_LANES {
-		struct run rn = get_longest_zero_run(statistics[lane], 128);
+		struct run rn = get_longest_zero_run(statistics[lane], 128. 1);
 		ctrl->timings[channel][slotrank].lanes[lane].timA = rn.middle;
 		upperA[lane] = rn.end;
 		if (upperA[lane] < rn.middle)
@@ -2106,7 +2106,7 @@ static void discover_timC(ramctr_timing * ctrl, int channel, int slotrank)
 	}
 	FOR_ALL_LANES {
 		struct run rn =
-		    get_longest_zero_run(statistics[lane], MAX_TIMC + 1);
+			get_longest_zero_run(statistics[lane], MAX_TIMC + 1, 1);
 		ctrl->timings[channel][slotrank].lanes[lane].timC = rn.middle;
 		if (rn.all)
 			printk(BIOS_CRIT, "timC discovery failed");
@@ -2333,7 +2333,7 @@ static void discover_timB(ramctr_timing * ctrl, int channel, int slotrank)
 		}
 	}
 	FOR_ALL_LANES {
-		struct run rn = get_longest_zero_run(statistics[lane], 128);
+		struct run rn = get_longest_zero_run(statistics[lane], 128, 0);
 		ctrl->timings[channel][slotrank].lanes[lane].timB = rn.start;
 		if (rn.all)
 			die("timB discovery failed");
@@ -2821,7 +2821,7 @@ static int try_cmd_stretch(ramctr_timing * ctrl, int cmd_stretch)
 		}
 		FOR_ALL_POPULATED_RANKS {
 			struct run rn =
-			    get_longest_zero_run(stat[slotrank], 255);
+				get_longest_zero_run(stat[slotrank], 255, 1);
 			ctrl->timings[channel][slotrank].val_320c =
 			    rn.middle - 127;
 			printram("3val: %d, %d: %d\n", channel,
@@ -2933,7 +2933,7 @@ static void discover_edges_real(ramctr_timing * ctrl, int channel, int slotrank,
 	}
 	FOR_ALL_LANES {
 		struct run rn =
-		    get_longest_zero_run(statistics[lane], MAX_EDGE_TIMING + 1);
+			get_longest_zero_run(statistics[lane], MAX_EDGE_TIMING + 1, 1);
 		edges[lane] = rn.middle;
 		if (rn.all)
 			die("edge discovery failed");
@@ -3233,7 +3233,7 @@ static void discover_edges_write_real(ramctr_timing * ctrl, int channel,
 					statistics[edge] =
 						! !(raw_statistics[edge] & (1 << lane));
 				rn = get_longest_zero_run(statistics,
-							  MAX_EDGE_TIMING + 1);
+							  MAX_EDGE_TIMING + 1, 1);
 				printram("edges: %d, %d, %d: 0x%x-0x%x-0x%x, 0x%x-0x%x\n",
 					 channel, slotrank, i, rn.start, rn.middle,
 					 rn.end, rn.start + ctrl->edge_offset[i],
@@ -3391,7 +3391,7 @@ static void discover_timC_write(ramctr_timing * ctrl)
 								!!(raw_statistics[timC] &
 								   (1 << lane));
 						rn = get_longest_zero_run(statistics,
-									  MAX_TIMC + 1);
+									  MAX_TIMC + 1, 1);
 						if (rn.all)
 							die("timC write discovery failed");
 						printram("timC: %d, %d, %d: 0x%x-0x%x-0x%x, 0x%x-0x%x\n",