[U-Boot] [PATCH 02/28] ddr: altera: sdram: Clean up compute_errata_rows() part 1

[Marek Vasut]

Clean up weird parenthesis and odd type casts from the function.
Fix comment style.

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

diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
index 9e6acfe..c60820c 100644
--- a/drivers/ddr/altera/sdram.c
+++ b/drivers/ddr/altera/sdram.c
@@ -54,13 +54,14 @@ static int compute_errata_rows(unsigned long long memsize, int cs, int width,
 	debug("workaround rows - banks     %d\n", banks);
 	debug("workaround rows - cols      %d\n", cols);
 
-	newrows = lldiv(memsize, (cs * (width / 8)));
+	newrows = lldiv(memsize, cs * (width / 8));
 	debug("rows workaround - term1 %lld\n", newrows);
 
-	newrows = lldiv(newrows, ((1 << banks) * (1 << cols)));
+	newrows = lldiv(newrows, (1 << banks) * (1 << cols));
 	debug("rows workaround - term2 %lld\n", newrows);
 
-	/* Compute the hamming weight - same as number of bits set.
+	/*
+	 * Compute the hamming weight - same as number of bits set.
 	 * Need to see if result is ordinal power of 2 before
 	 * attempting log2 of result.
 	 */
@@ -78,13 +79,12 @@ static int compute_errata_rows(unsigned long long memsize, int cs, int width,
 		return rows;
 	}
 
-	inewrowslog2 = __ilog2((unsigned int)newrows);
+	inewrowslog2 = __ilog2(newrows);
 
-	debug("rows workaround - ilog2 %d, %d\n", inewrowslog2,
-	       (int)newrows);
+	debug("rows workaround - ilog2 %d, %lld\n", inewrowslog2, newrows);
 
 	if (inewrowslog2 == -1) {
-		printf("SDRAM workaround failed, newrows %d\n", (int)newrows);
+		printf("SDRAM workaround failed, newrows %lld\n", newrows);
 		return rows;
 	}
 
-- 
2.1.4