]> Pileus Git - ~andy/linux/commitdiff
i2c: nomadik: drop superfluous variable initialization
authorWolfram Sang <w.sang@pengutronix.de>
Thu, 24 Jan 2013 10:27:46 +0000 (11:27 +0100)
committerWolfram Sang <w.sang@pengutronix.de>
Mon, 28 Jan 2013 04:26:44 +0000 (05:26 +0100)
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>
drivers/i2c/busses/i2c-nomadik.c

index dd6a6913bd7393bfa1ab1c0307a8cf2e4ba942f5..5b1b1948128ee3ffeb2bf85f538b6b41163dfc9d 100644 (file)
@@ -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);