usbnet:pegasus: remove usb_get/put_dev in .probe and .disconnect
usb_device is parent device of usb_interface in the view of driver model, so its reference count is always held during .probe/.disconnect of usb_interface instance. This patch just removes the unnecessay usb_get/put_dev. Signed-off-by: Ming Lei <tom.leiming@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
ef9d884dad
commit
5c2f05139a
|
@ -1329,8 +1329,6 @@ static int pegasus_probe(struct usb_interface *intf,
|
||||||
}
|
}
|
||||||
pegasus_count++;
|
pegasus_count++;
|
||||||
|
|
||||||
usb_get_dev(dev);
|
|
||||||
|
|
||||||
net = alloc_etherdev(sizeof(struct pegasus));
|
net = alloc_etherdev(sizeof(struct pegasus));
|
||||||
if (!net)
|
if (!net)
|
||||||
goto out;
|
goto out;
|
||||||
|
@ -1407,7 +1405,6 @@ out2:
|
||||||
out1:
|
out1:
|
||||||
free_netdev(net);
|
free_netdev(net);
|
||||||
out:
|
out:
|
||||||
usb_put_dev(dev);
|
|
||||||
pegasus_dec_workqueue();
|
pegasus_dec_workqueue();
|
||||||
return res;
|
return res;
|
||||||
}
|
}
|
||||||
|
@ -1425,7 +1422,6 @@ static void pegasus_disconnect(struct usb_interface *intf)
|
||||||
pegasus->flags |= PEGASUS_UNPLUG;
|
pegasus->flags |= PEGASUS_UNPLUG;
|
||||||
cancel_delayed_work(&pegasus->carrier_check);
|
cancel_delayed_work(&pegasus->carrier_check);
|
||||||
unregister_netdev(pegasus->net);
|
unregister_netdev(pegasus->net);
|
||||||
usb_put_dev(interface_to_usbdev(intf));
|
|
||||||
unlink_all_urbs(pegasus);
|
unlink_all_urbs(pegasus);
|
||||||
free_all_urbs(pegasus);
|
free_all_urbs(pegasus);
|
||||||
free_skb_pool(pegasus);
|
free_skb_pool(pegasus);
|
||||||
|
|
Loading…
Reference in New Issue