summaryrefslogtreecommitdiffstats
path: root/drivers/mmc
diff options
context:
space:
mode:
authorSteven Noonan <steven@uplinklabs.net>2008-10-01 01:50:25 -0700
committerPierre Ossman <drzeus@drzeus.cx>2008-10-12 11:04:36 +0200
commit7244b85bd17313d7d300ee93ec7bfbca1f4ccf3d (patch)
tree96ba4ef056e3c351ac9a01e4deca359aa1e10c82 /drivers/mmc
parent57105737f6a0b39305a85ac176cc9fd4a236d8c2 (diff)
downloadop-kernel-dev-7244b85bd17313d7d300ee93ec7bfbca1f4ccf3d.zip
op-kernel-dev-7244b85bd17313d7d300ee93ec7bfbca1f4ccf3d.tar.gz
sdhci: 'scratch' may be used uninitialized
The variable 'scratch' is always initialized before it's used. The conditional which is responsible for initialization of 'scratch' will always evaluate 'true' when the first loop iteration occurs, and thus, it's properly initialized. GCC doesn't see this, of course, so using the uninitialized_var() macro seems to work for silencing this case. Signed-off-by: Steven Noonan <steven@uplinklabs.net> Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
Diffstat (limited to 'drivers/mmc')
-rw-r--r--drivers/mmc/host/sdhci.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
index 3c808d2..30f64b1 100644
--- a/drivers/mmc/host/sdhci.c
+++ b/drivers/mmc/host/sdhci.c
@@ -177,7 +177,7 @@ static void sdhci_read_block_pio(struct sdhci_host *host)
{
unsigned long flags;
size_t blksize, len, chunk;
- u32 scratch;
+ u32 uninitialized_var(scratch);
u8 *buf;
DBG("PIO reading\n");
OpenPOWER on IntegriCloud