ddr: altera: sdram: Clean up sdram_calculate_size() part 2
authorMarek Vasut <marex@denx.de>
Sat, 1 Aug 2015 19:47:16 +0000 (21:47 +0200)
committerMarek Vasut <marex@denx.de>
Sat, 8 Aug 2015 12:14:27 +0000 (14:14 +0200)
Clean up coding style, mostly clean up comments, add kerneldoc.
Also, zap assignment of the "cs" variable, which is outright
dead code, so just remove it.

Signed-off-by: Marek Vasut <marex@denx.de>
drivers/ddr/altera/sdram.c

index b540c78..45ae690 100644 (file)
@@ -451,14 +451,11 @@ int sdram_mmr_init_full(unsigned int sdr_phy_reg)
        return status;
 }
 
-/*
- * To calculate SDRAM device size based on SDRAM controller parameters.
- * Size is specified in bytes.
+/**
+ * sdram_calculate_size() - Calculate SDRAM size
  *
- * NOTE:
- * This function is compiled and linked into the preloader and
- * Uboot (there may be others). So if this function changes, the Preloader
- * and UBoot must be updated simultaneously.
+ * Calculate SDRAM device size based on SDRAM controller parameters.
+ * Size is specified in bytes.
  */
 unsigned long sdram_calculate_size(void)
 {
@@ -476,23 +473,17 @@ unsigned long sdram_calculate_size(void)
        col = (temp & SDR_CTRLGRP_DRAMADDRW_COLBITS_MASK) >>
                SDR_CTRLGRP_DRAMADDRW_COLBITS_LSB;
 
-       /* SDRAM Failure When Accessing Non-Existent Memory
+       /*
+        * SDRAM Failure When Accessing Non-Existent Memory
         * Use ROWBITS from Quartus/QSys to calculate SDRAM size
         * since the FB specifies we modify ROWBITs to work around SDRAM
         * controller issue.
-        *
-        * If the stored handoff value for rows is 0, it probably means
-        * the preloader is older than UBoot. Use the
-        * #define from the SOCEDS Tools per Crucible review
-        * uboot-socfpga-204. Note that this is not a supported
-        * configuration and is not tested. The customer
-        * should be using preloader and uboot built from the
-        * same tag.
         */
        row = readl(&sysmgr_regs->iswgrp_handoff[4]);
        if (row == 0)
                row = rowbits;
-       /* If the stored handoff value for rows is greater than
+       /*
+        * If the stored handoff value for rows is greater than
         * the field width in the sdr.dramaddrw register then
         * something is very wrong. Revert to using the the #define
         * value handed off by the SOCEDS tool chain instead of
@@ -504,18 +495,16 @@ unsigned long sdram_calculate_size(void)
        bank = (temp & SDR_CTRLGRP_DRAMADDRW_BANKBITS_MASK) >>
                SDR_CTRLGRP_DRAMADDRW_BANKBITS_LSB;
 
-       /* SDRAM Failure When Accessing Non-Existent Memory
+       /*
+        * SDRAM Failure When Accessing Non-Existent Memory
         * Use CSBITs from Quartus/QSys to calculate SDRAM size
         * since the FB specifies we modify CSBITs to work around SDRAM
         * controller issue.
         */
-       cs = (temp & SDR_CTRLGRP_DRAMADDRW_CSBITS_MASK) >>
-             SDR_CTRLGRP_DRAMADDRW_CSBITS_LSB;
-       cs += 1;
-
        cs = csbits;
 
        width = readl(&sdr_ctrl->dram_if_width);
+
        /* ECC would not be calculated as its not addressible */
        if (width == SDRAM_WIDTH_32BIT_WITH_ECC)
                width = 32;
@@ -526,7 +515,7 @@ unsigned long sdram_calculate_size(void)
        temp = 1 << (row + bank + col);
        temp = temp * cs * (width  / 8);
 
-       debug("sdram_calculate_memory returns %ld\n", temp);
+       debug("%s returns %ld\n", __func__, temp);
 
        return temp;
 }