Signed-off-by: Russell King <rmk+ker...@arm.linux.org.uk>
---
 drivers/mmc/host/sdhci.c | 29 +++++++++--------------------
 1 file changed, 9 insertions(+), 20 deletions(-)

diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
index 8a4eb1c41f3e..a1853671a591 100644
--- a/drivers/mmc/host/sdhci.c
+++ b/drivers/mmc/host/sdhci.c
@@ -465,16 +465,12 @@ static void sdhci_adma_mark_end(void *desc)
 static int sdhci_adma_table_pre(struct sdhci_host *host,
        struct mmc_data *data)
 {
-       void *desc;
-       void *align;
-       dma_addr_t addr;
-       dma_addr_t align_addr;
-       int len, offset;
-
        struct scatterlist *sg;
-       int i;
-       char *buffer;
        unsigned long flags;
+       dma_addr_t addr, align_addr;
+       void *desc, *align;
+       char *buffer;
+       int len, offset, i;
 
        /*
         * The spec does not specify endianness of descriptor table.
@@ -495,10 +491,9 @@ static int sdhci_adma_table_pre(struct sdhci_host *host,
                len = sg_dma_len(sg);
 
                /*
-                * The SDHCI specification states that ADMA
-                * addresses must be 32-bit aligned. If they
-                * aren't, then we use a bounce buffer for
-                * the (up to three) bytes that screw up the
+                * The SDHCI specification states that ADMA addresses must
+                * be 32-bit aligned. If they aren't, then we use a bounce
+                * buffer for the (up to three) bytes that screw up the
                 * alignment.
                 */
                offset = (host->align_sz - (addr & host->align_mask)) &
@@ -539,19 +534,13 @@ static int sdhci_adma_table_pre(struct sdhci_host *host,
        }
 
        if (host->quirks & SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC) {
-               /*
-               * Mark the last descriptor as the terminating descriptor
-               */
+               /* Mark the last descriptor as the terminating descriptor */
                if (desc != host->adma_table) {
                        desc -= host->desc_sz;
                        sdhci_adma_mark_end(desc);
                }
        } else {
-               /*
-               * Add a terminating entry.
-               */
-
-               /* nop, end, valid */
+               /* Add a terminating entry - nop, end, valid */
                sdhci_adma_write_desc(host, desc, 0, 0, ADMA2_NOP_END_VALID);
        }
        return 0;
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to