staging: ft1000: ft1000-pcmcia: removed unused variable in ft1000_hw.c
authorJiayi Ye <yejiayily@gmail.com>
Mon, 20 Oct 2014 11:13:56 +0000 (19:13 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Oct 2014 04:31:18 +0000 (12:31 +0800)
Variable whose value is initialized but never used is unnecessary. The following Coccinelle semantic patch removed the unused variable.

@e@
identifier i;
position p;
type T;
@@

extern T i@p;

@@
type T;
identifier i;
constant C;
position p != e.p;
@@

- T i@p;
  <+... when != i
- i = C;
  ...+>

Signed-off-by: Jiayi Ye <yejiayily@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c

index 9574775ba3b507d175adfd440caacb819120fb81..a9bcb9e1af78e5eafed9f192963e9b6e2ee4c805 100644 (file)
@@ -1270,7 +1270,6 @@ static int ft1000_parse_dpram_msg(struct net_device *dev)
        u16 nxtph;
        u16 total_len;
        int i = 0;
-       int cnt;
        unsigned long flags;
 
        doorbell = ft1000_read_reg(dev, FT1000_REG_DOORBELL);
@@ -1336,7 +1335,6 @@ static int ft1000_parse_dpram_msg(struct net_device *dev)
                          total_len);
                if ((total_len < MAX_CMD_SQSIZE) && (total_len > sizeof(struct pseudo_hdr))) {
             total_len += nxtph;
-            cnt = 0;
                        /*
                         * ft1000_read_reg will return a value that needs to be byteswap
                         * in order to get DSP_QID_OFFSET.