提交 107ce6d2 编写于 作者: E Eric Sesterhenn 提交者: Jeff Garzik

[PATCH] Remove unnecessary check in drivers/net/depca.c

This was spotted by coverity (cid #793).  All callers dereference dev
before calling this functions, and we dereference it earlier in the
function, when initializing lp.
Signed-off-by: NEric Sesterhenn <snakebyte@gmx.de>
Cc: Jeff Garzik <jeff@garzik.org>
Signed-off-by: NAndrew Morton <akpm@osdl.org>
Signed-off-by: NJeff Garzik <jeff@garzik.org>
上级 a189317f
...@@ -1252,24 +1252,22 @@ static void set_multicast_list(struct net_device *dev) ...@@ -1252,24 +1252,22 @@ static void set_multicast_list(struct net_device *dev)
struct depca_private *lp = (struct depca_private *) dev->priv; struct depca_private *lp = (struct depca_private *) dev->priv;
u_long ioaddr = dev->base_addr; u_long ioaddr = dev->base_addr;
if (dev) { netif_stop_queue(dev);
netif_stop_queue(dev); while (lp->tx_old != lp->tx_new); /* Wait for the ring to empty */
while (lp->tx_old != lp->tx_new); /* Wait for the ring to empty */
STOP_DEPCA; /* Temporarily stop the depca. */
depca_init_ring(dev); /* Initialize the descriptor rings */
if (dev->flags & IFF_PROMISC) { /* Set promiscuous mode */ STOP_DEPCA; /* Temporarily stop the depca. */
lp->init_block.mode |= PROM; depca_init_ring(dev); /* Initialize the descriptor rings */
} else {
SetMulticastFilter(dev);
lp->init_block.mode &= ~PROM; /* Unset promiscuous mode */
}
LoadCSRs(dev); /* Reload CSR3 */ if (dev->flags & IFF_PROMISC) { /* Set promiscuous mode */
InitRestartDepca(dev); /* Resume normal operation. */ lp->init_block.mode |= PROM;
netif_start_queue(dev); /* Unlock the TX ring */ } else {
SetMulticastFilter(dev);
lp->init_block.mode &= ~PROM; /* Unset promiscuous mode */
} }
LoadCSRs(dev); /* Reload CSR3 */
InitRestartDepca(dev); /* Resume normal operation. */
netif_start_queue(dev); /* Unlock the TX ring */
} }
/* /*
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册