Commit 056693a3 authored by David S. Miller's avatar David S. Miller

caif: Fix set-but-unused variables.

The variable 'caifdef' is set but unused in modemcmd().

Similarly for 'net' in receive(), and 'res' in
caif_device_notify() and caif_exit_net().

Just kill them off.
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent b8ee8328
...@@ -133,9 +133,7 @@ static int transmit(struct cflayer *layer, struct cfpkt *pkt) ...@@ -133,9 +133,7 @@ static int transmit(struct cflayer *layer, struct cfpkt *pkt)
static int modemcmd(struct cflayer *layr, enum caif_modemcmd ctrl) static int modemcmd(struct cflayer *layr, enum caif_modemcmd ctrl)
{ {
struct caif_device_entry *caifd; struct caif_device_entry *caifd;
struct caif_dev_common *caifdev;
caifd = container_of(layr, struct caif_device_entry, layer); caifd = container_of(layr, struct caif_device_entry, layer);
caifdev = netdev_priv(caifd->netdev);
if (ctrl == _CAIF_MODEMCMD_PHYIF_USEFULL) { if (ctrl == _CAIF_MODEMCMD_PHYIF_USEFULL) {
atomic_set(&caifd->in_use, 1); atomic_set(&caifd->in_use, 1);
wake_up_interruptible(&caifd->event); wake_up_interruptible(&caifd->event);
...@@ -154,10 +152,8 @@ static int modemcmd(struct cflayer *layr, enum caif_modemcmd ctrl) ...@@ -154,10 +152,8 @@ static int modemcmd(struct cflayer *layr, enum caif_modemcmd ctrl)
static int receive(struct sk_buff *skb, struct net_device *dev, static int receive(struct sk_buff *skb, struct net_device *dev,
struct packet_type *pkttype, struct net_device *orig_dev) struct packet_type *pkttype, struct net_device *orig_dev)
{ {
struct net *net;
struct cfpkt *pkt; struct cfpkt *pkt;
struct caif_device_entry *caifd; struct caif_device_entry *caifd;
net = dev_net(dev);
pkt = cfpkt_fromnative(CAIF_DIR_IN, skb); pkt = cfpkt_fromnative(CAIF_DIR_IN, skb);
caifd = caif_get(dev); caifd = caif_get(dev);
if (!caifd || !caifd->layer.up || !caifd->layer.up->receive) if (!caifd || !caifd->layer.up || !caifd->layer.up->receive)
...@@ -195,7 +191,6 @@ static int caif_device_notify(struct notifier_block *me, unsigned long what, ...@@ -195,7 +191,6 @@ static int caif_device_notify(struct notifier_block *me, unsigned long what,
struct caif_device_entry *caifd = NULL; struct caif_device_entry *caifd = NULL;
struct caif_dev_common *caifdev; struct caif_dev_common *caifdev;
enum cfcnfg_phy_preference pref; enum cfcnfg_phy_preference pref;
int res = -EINVAL;
enum cfcnfg_phy_type phy_type; enum cfcnfg_phy_type phy_type;
if (dev->type != ARPHRD_CAIF) if (dev->type != ARPHRD_CAIF)
...@@ -210,7 +205,6 @@ static int caif_device_notify(struct notifier_block *me, unsigned long what, ...@@ -210,7 +205,6 @@ static int caif_device_notify(struct notifier_block *me, unsigned long what,
caifdev = netdev_priv(dev); caifdev = netdev_priv(dev);
caifdev->flowctrl = dev_flowctrl; caifdev->flowctrl = dev_flowctrl;
atomic_set(&caifd->state, what); atomic_set(&caifd->state, what);
res = 0;
break; break;
case NETDEV_UP: case NETDEV_UP:
...@@ -274,7 +268,7 @@ static int caif_device_notify(struct notifier_block *me, unsigned long what, ...@@ -274,7 +268,7 @@ static int caif_device_notify(struct notifier_block *me, unsigned long what,
_CAIF_CTRLCMD_PHYIF_DOWN_IND, _CAIF_CTRLCMD_PHYIF_DOWN_IND,
caifd->layer.id); caifd->layer.id);
might_sleep(); might_sleep();
res = wait_event_interruptible_timeout(caifd->event, wait_event_interruptible_timeout(caifd->event,
atomic_read(&caifd->in_use) == 0, atomic_read(&caifd->in_use) == 0,
TIMEOUT); TIMEOUT);
break; break;
...@@ -344,12 +338,11 @@ static int caif_init_net(struct net *net) ...@@ -344,12 +338,11 @@ static int caif_init_net(struct net *net)
static void caif_exit_net(struct net *net) static void caif_exit_net(struct net *net)
{ {
struct net_device *dev; struct net_device *dev;
int res;
rtnl_lock(); rtnl_lock();
for_each_netdev(net, dev) { for_each_netdev(net, dev) {
if (dev->type != ARPHRD_CAIF) if (dev->type != ARPHRD_CAIF)
continue; continue;
res = dev_close(dev); dev_close(dev);
caif_device_destroy(dev); caif_device_destroy(dev);
} }
rtnl_unlock(); rtnl_unlock();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment