diff options
| author | Wolfram Sang <w.sang@pengutronix.de> | 2013-01-24 11:27:46 +0100 |
|---|---|---|
| committer | Wolfram Sang <w.sang@pengutronix.de> | 2013-01-28 05:26:44 +0100 |
| commit | 876ae85c8b811668c2c115ea78c2241f562977fb (patch) | |
| tree | 592f6fd8698811d88db00bbdf2956f646e8041c4 /drivers/i2c/busses/i2c-nomadik.c | |
| parent | 24e9e157d5197e469a414d0f52ce04e8b539a715 (diff) | |
| download | linux-876ae85c8b811668c2c115ea78c2241f562977fb.tar.bz2 | |
i2c: nomadik: drop superfluous variable initialization
cppcheck rightfully reports those as "reassigned a value before the old
one has been used."
Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
Cc: Linus Walleij <linus.walleij@stericsson.com>
Diffstat (limited to 'drivers/i2c/busses/i2c-nomadik.c')
| -rw-r--r-- | drivers/i2c/busses/i2c-nomadik.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/i2c/busses/i2c-nomadik.c b/drivers/i2c/busses/i2c-nomadik.c index dd6a6913bd73..5b1b1948128e 100644 --- a/drivers/i2c/busses/i2c-nomadik.c +++ b/drivers/i2c/busses/i2c-nomadik.c @@ -412,8 +412,7 @@ static void setup_i2c_controller(struct nmk_i2c_dev *dev) static int read_i2c(struct nmk_i2c_dev *dev, u16 flags) { u32 status = 0; - u32 mcr; - u32 irq_mask = 0; + u32 mcr, irq_mask; int timeout; mcr = load_i2c_mcr_reg(dev, flags); @@ -482,8 +481,7 @@ static void fill_tx_fifo(struct nmk_i2c_dev *dev, int no_bytes) static int write_i2c(struct nmk_i2c_dev *dev, u16 flags) { u32 status = 0; - u32 mcr; - u32 irq_mask = 0; + u32 mcr, irq_mask; int timeout; mcr = load_i2c_mcr_reg(dev, flags); @@ -731,8 +729,7 @@ static irqreturn_t i2c_irq_handler(int irq, void *arg) struct nmk_i2c_dev *dev = arg; u32 tft, rft; u32 count; - u32 misr; - u32 src = 0; + u32 misr, src; /* load Tx FIFO and Rx FIFO threshold values */ tft = readl(dev->virtbase + I2C_TFTR); |