lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

mmc: dw_mmc: Remove set-but-unused variable.

Browse Source 
count is only ever used by assigning to old_len if count == 0, and
then old_len isn't ever used at all.  So, both are redundant.  Fixes:

drivers/mmc/host/dw_mmc.c: In function ‘dw_mci_read_data_pio’:
drivers/mmc/host/dw_mmc.c:1034:32: warning: variable ‘old_len’ set but
not used [-Wunused-but-set-variable]

Signed-off-by: Chris Ball <cjb@laptop.org>
Acked-by: Will Newton <will.newton@imgtec.com>
This commit is contained in:
Chris Ball
2011-02-28 16:45:10 -05:00
parent 7e29c30603
commit ba6a902d9d
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/mmc/host/dw_mmc.c
View File

@@ -1031,13 +1031,10 @@ static void dw_mci_read_data_pio(struct dw_mci *host)
struct mmc_data *data = host->data; struct mmc_data *data = host->data;
int shift = host->data_shift; int shift = host->data_shift;
u32 status; u32 status;
unsigned int nbytes = 0, len, old_len, count = 0; unsigned int nbytes = 0, len;
do { do {
len = SDMMC_GET_FCNT(mci_readl(host, STATUS)) << shift; len = SDMMC_GET_FCNT(mci_readl(host, STATUS)) << shift;
if (count == 0)
old_len = len;
if (offset + len <= sg->length) { if (offset + len <= sg->length) {
host->pull_data(host, (void *)(buf + offset), len); host->pull_data(host, (void *)(buf + offset), len);
@@ -1082,7 +1079,6 @@ static void dw_mci_read_data_pio(struct dw_mci *host)
tasklet_schedule(&host->tasklet); tasklet_schedule(&host->tasklet);
return; return;
} }
count++;
} while (status & SDMMC_INT_RXDR); /*if the RXDR is ready read again*/ } while (status & SDMMC_INT_RXDR); /*if the RXDR is ready read again*/
len = SDMMC_GET_FCNT(mci_readl(host, STATUS)); len = SDMMC_GET_FCNT(mci_readl(host, STATUS));
host->pio_offset = offset; host->pio_offset = offset;