[U-Boot] [PATCH 171/172] ddr: altera: Clean up of delay_for_n_mem_clocks() part 4

Marek Vasut marex at denx.de
Mon Jul 27 22:52:14 CEST 2015


Simplify the loop code, optimizing compiler can deal with this.
No functional change.

Signed-off-by: Marek Vasut <marex at denx.de>
---
 drivers/ddr/altera/sequencer.c | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/drivers/ddr/altera/sequencer.c b/drivers/ddr/altera/sequencer.c
index ed4d791..d4e720f 100644
--- a/drivers/ddr/altera/sequencer.c
+++ b/drivers/ddr/altera/sequencer.c
@@ -822,18 +822,11 @@ static void delay_for_n_mem_clocks(const u32 clocks)
 		writel(RW_MGR_IDLE_LOOP2,
 			&sdr_rw_load_jump_mgr_regs->load_jump_add1);
 
-		/* hack to get around compiler not being smart enough */
-		if (afi_clocks <= 0x10000) {
-			/* only need to run once */
-			writel(RW_MGR_IDLE_LOOP2, SDR_PHYGRP_RWMGRGRP_ADDRESS |
-						  RW_MGR_RUN_SINGLE_GROUP_OFFSET);
-		} else {
-			do {
-				writel(RW_MGR_IDLE_LOOP2,
-					SDR_PHYGRP_RWMGRGRP_ADDRESS |
-					RW_MGR_RUN_SINGLE_GROUP_OFFSET);
-			} while (c_loop-- != 0);
-		}
+		do {
+			writel(RW_MGR_IDLE_LOOP2,
+				SDR_PHYGRP_RWMGRGRP_ADDRESS |
+				RW_MGR_RUN_SINGLE_GROUP_OFFSET);
+		} while (c_loop-- != 0);
 	}
 	debug("%s:%d clocks=%u ... end\n", __func__, __LINE__, clocks);
 }
-- 
2.1.4



More information about the U-Boot mailing list