Variable ar assigned a value that is never used.
I have also removed all the code that thereby serves no purpose.

This was found using a static code analysis program called cppcheck

Signed-off-by: Rickard Strandqvist <rickard_strandqv...@spectrumdigital.se>
---
 drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c |    9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c 
b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
index d5475b7..3c42737 100644
--- a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
+++ b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
@@ -1230,7 +1230,6 @@ static int ft1000_parse_dpram_msg(struct net_device *dev)
        struct ft1000_info *info = netdev_priv(dev);
        u16 doorbell;
        u16 portid;
-       u16 nxtph;
        u16 total_len;
        int i = 0;
        unsigned long flags;
@@ -1282,7 +1281,6 @@ static int ft1000_parse_dpram_msg(struct net_device *dev)
 
        if (doorbell & FT1000_DB_DPRAM_RX) {
                pr_debug("Got a slow queue message\n");
-               nxtph = FT1000_DPRAM_RX_BASE + 2;
                if (info->AsicID == ELECTRABUZZ_ID) {
                        total_len =
                                ft1000_read_dpram(dev, FT1000_DPRAM_RX_BASE);
@@ -1294,7 +1292,6 @@ static int ft1000_parse_dpram_msg(struct net_device *dev)
                }
                pr_debug("total length = %d\n", total_len);
                if ((total_len < MAX_CMD_SQSIZE) && (total_len > sizeof(struct 
pseudo_hdr))) {
-                       total_len += nxtph;
                        /*
                         * ft1000_read_reg will return a value that needs to be 
byteswap
                         * in order to get DSP_QID_OFFSET.
@@ -1382,7 +1379,6 @@ static void ft1000_flush_fifo(struct net_device *dev, u16 
DrvErrNum)
        struct ft1000_info *info = netdev_priv(dev);
        struct ft1000_pcmcia *pcmcia = info->priv;
        u16 i;
-       u32 templong;
        u16 tempword;
 
        if (pcmcia->PktIntfErr > MAX_PH_ERR) {
@@ -1422,8 +1418,7 @@ static void ft1000_flush_fifo(struct net_device *dev, u16 
DrvErrNum)
                                tempword =
                                        ft1000_read_reg(dev, 
FT1000_REG_DFIFO_STAT);
                        } else {
-                               templong =
-                                       inl(dev->base_addr + 
FT1000_REG_MAG_DFR);
+                               inl(dev->base_addr + FT1000_REG_MAG_DFR);
                                tempword =
                                        inw(dev->base_addr + 
FT1000_REG_MAG_DFSR);
                        }
@@ -1502,7 +1497,7 @@ static void ft1000_flush_fifo(struct net_device *dev, u16 
DrvErrNum)
                } else {
                        pr_debug("Flushing FIFO complete = %x\n", tempword);
                        /* Flush last word in FIFO */
-                       templong = inl(dev->base_addr + FT1000_REG_MAG_DFR);
+                       inl(dev->base_addr + FT1000_REG_MAG_DFR);
                        tempword = inw(dev->base_addr + FT1000_REG_SUP_STAT);
                        pr_debug("FT1000_REG_SUP_STAT = 0x%x\n", tempword);
                        tempword = inw(dev->base_addr + FT1000_REG_MAG_DFSR);
-- 
1.7.10.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to