[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Remove unnecessary comments

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/48401 )

Change subject: nb/intel/sandybridge: Remove unnecessary comments
......................................................................

nb/intel/sandybridge: Remove unnecessary comments

These comments were helpful before the massive IOSAV refactoring, but
they are no longer needed since the function names are clear enough.

Change-Id: Ieb9bdf3f7fc72f63a8978f2b98e0bc8228c55868
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48401
Reviewed-by: Felix Held 
Tested-by: build bot (Jenkins) 
---
M src/northbridge/intel/sandybridge/raminit_common.c
1 file changed, 1 insertion(+), 29 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Held: Looks good to me, approved

diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index c705f2a..289398a 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -546,11 +546,7 @@

iosav_write_zqcs_sequence(channel, slotrank, 3, 8, 0);

-	/*
-	 * Execute command queue - why is bit 22 set here?!
-	 *
-	 * This is actually using the IOSAV state machine as a timer, so refresh is allowed.
-	 */
+	/* This is actually using the IOSAV state machine as a timer */
    iosav_run_queue(channel, 1, 1);

wait_for_iosav(channel);
@@ -684,7 +680,6 @@
    };
    iosav_write_sequence(channel, sequence, ARRAY_SIZE(sequence));

-	/* Execute command queue */
    iosav_run_once(channel);
 }

@@ -895,11 +890,9 @@
    };
    iosav_write_sequence(BROADCAST_CH, zqcl_sequence, ARRAY_SIZE(zqcl_sequence));

-	/* Execute command queue on all channels. Do it four times. */
    iosav_run_queue(BROADCAST_CH, 4, 0);

FOR_ALL_CHANNELS {
-		/* Wait for ref drained */
    	wait_for_iosav(channel);
    }

@@ -913,15 +906,12 @@

slotrank = (ctrl->rankmap[channel] & 1) ? 0 : 2;

-		/* Drain */
    	wait_for_iosav(channel);

iosav_write_zqcs_sequence(channel, slotrank, 4, 101, 31);

-		/* Execute command queue */
    	iosav_run_once(channel);

-		/* Drain */
    	wait_for_iosav(channel);
    }
 }
@@ -1062,7 +1052,6 @@
    /* Send a burst of 16 back-to-back read commands (4 DCLK apart) */
    iosav_write_read_mpr_sequence(channel, slotrank, ctrl->tMOD, 1, 3, 15, ctrl->CAS + 36);

-	/* Execute command queue */
    iosav_run_once(channel);

wait_for_iosav(channel);
@@ -1347,7 +1336,6 @@

iosav_write_prea_sequence(channel, slotrank, ctrl->tRP, 0);

-		/* Execute command queue */
    	iosav_run_once(channel);

const union gdcr_training_mod_reg training_mod = {
@@ -1440,14 +1428,12 @@
    iosav_write_misc_write_sequence(ctrl, channel, slotrank,
    	MAX(ctrl->tRRD, (ctrl->tFAW >> 2) + 1), 4, 4, 500, 18);

-	/* Execute command queue */
    iosav_run_once(channel);

wait_for_iosav(channel);

iosav_write_prea_act_read_sequence(ctrl, channel, slotrank);

-	/* Execute command queue */
    iosav_run_once(channel);

wait_for_iosav(channel);
@@ -1482,7 +1468,6 @@

iosav_write_prea_sequence(channel, slotrank, ctrl->tRP, 18);

-	/* Execute command queue */
    iosav_run_once(channel);

for (tx_dq = 0; tx_dq <= MAX_TX_DQ; tx_dq++) {
@@ -1605,7 +1590,6 @@
    	}
    	program_timings(ctrl, channel);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -1686,7 +1670,6 @@

iosav_write_misc_write_sequence(ctrl, channel, slotrank, 3, 1, 3, 3, 31);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -1757,7 +1740,6 @@
    	};
    	iosav_write_sequence(channel, rd_sequence, ARRAY_SIZE(rd_sequence));

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -1788,7 +1770,6 @@

iosav_write_zqcs_sequence(channel, slotrank, 4, 4, 31);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -1869,7 +1850,6 @@

iosav_write_zqcs_sequence(channel, 0, 4, 101, 31);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -1951,7 +1931,6 @@
    	MCHBAR32(IOSAV_n_ADDRESS_LFSR_ch(channel, 1)) = 0x389abcd;
    	MCHBAR32(IOSAV_n_ADDRESS_LFSR_ch(channel, 2)) = 0x389abcd;

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -2165,7 +2144,6 @@
    	iosav_write_read_mpr_sequence(
    		channel, slotrank, ctrl->tMOD, 500, 4, 1, ctrl->CAS + 8);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -2212,7 +2190,6 @@
    	iosav_write_read_mpr_sequence(
    		channel, slotrank, ctrl->tMOD, 3, 4, 1, ctrl->CAS + 8);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -2233,7 +2210,6 @@
    	iosav_write_read_mpr_sequence(
    		channel, slotrank, ctrl->tMOD, 3, 4, 1, ctrl->CAS + 8);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -2349,7 +2325,6 @@

iosav_write_data_write_sequence(ctrl, channel, slotrank);

-				/* Execute command queue */
    			iosav_run_once(channel);

wait_for_iosav(channel);
@@ -2447,7 +2422,6 @@

iosav_write_aggressive_write_read_sequence(ctrl, channel, slotrank);

-	/* Execute command queue */
    iosav_run_once(channel);

wait_for_iosav(channel);
@@ -2629,7 +2603,6 @@

iosav_write_memory_test_sequence(ctrl, channel, slotrank);

-		/* Execute command queue */
    	iosav_run_once(channel);

wait_for_iosav(channel);
@@ -2752,7 +2725,6 @@
    			};
    			iosav_write_sequence(channel, sequence, ARRAY_SIZE(sequence));

-				/* Execute command queue */
    			iosav_run_queue(channel, 16, 0);

wait_for_iosav(channel);