diff options
| author | Sebastian Haas <dev@sebastianhaas.info> | 2011-12-22 17:57:49 -0500 |
|---|---|---|
| committer | Marc Kleine-Budde <mkl@pengutronix.de> | 2012-02-02 18:25:31 -0500 |
| commit | d0a71a7e6de0e0ce9f86c8ba6e13414a9df63e0b (patch) | |
| tree | 650e1a68242a6bc949492359e50ede968ef27ff1 | |
| parent | 44b0052c5cb4e75389ed3eb9e98c29295a7dadfb (diff) | |
can: ems_usb: Removed double netif_device_detach
netif_device_attched is called twice when ems_usb_start fails with -ENODEV
Signed-off-by: Sebastian Haas <dev@sebastianhaas.info>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
| -rw-r--r-- | drivers/net/can/usb/ems_usb.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/drivers/net/can/usb/ems_usb.c b/drivers/net/can/usb/ems_usb.c index 9697c14b8dc6..7dae64d44e83 100644 --- a/drivers/net/can/usb/ems_usb.c +++ b/drivers/net/can/usb/ems_usb.c | |||
| @@ -627,9 +627,6 @@ static int ems_usb_start(struct ems_usb *dev) | |||
| 627 | 627 | ||
| 628 | err = usb_submit_urb(urb, GFP_KERNEL); | 628 | err = usb_submit_urb(urb, GFP_KERNEL); |
| 629 | if (err) { | 629 | if (err) { |
| 630 | if (err == -ENODEV) | ||
| 631 | netif_device_detach(dev->netdev); | ||
| 632 | |||
| 633 | usb_unanchor_urb(urb); | 630 | usb_unanchor_urb(urb); |
| 634 | usb_free_coherent(dev->udev, RX_BUFFER_SIZE, buf, | 631 | usb_free_coherent(dev->udev, RX_BUFFER_SIZE, buf, |
| 635 | urb->transfer_dma); | 632 | urb->transfer_dma); |
| @@ -659,9 +656,6 @@ static int ems_usb_start(struct ems_usb *dev) | |||
| 659 | 656 | ||
| 660 | err = usb_submit_urb(dev->intr_urb, GFP_KERNEL); | 657 | err = usb_submit_urb(dev->intr_urb, GFP_KERNEL); |
| 661 | if (err) { | 658 | if (err) { |
| 662 | if (err == -ENODEV) | ||
| 663 | netif_device_detach(dev->netdev); | ||
| 664 | |||
| 665 | dev_warn(netdev->dev.parent, "intr URB submit failed: %d\n", | 659 | dev_warn(netdev->dev.parent, "intr URB submit failed: %d\n", |
| 666 | err); | 660 | err); |
| 667 | 661 | ||
| @@ -692,9 +686,6 @@ static int ems_usb_start(struct ems_usb *dev) | |||
| 692 | return 0; | 686 | return 0; |
| 693 | 687 | ||
| 694 | failed: | 688 | failed: |
| 695 | if (err == -ENODEV) | ||
| 696 | netif_device_detach(dev->netdev); | ||
| 697 | |||
| 698 | dev_warn(netdev->dev.parent, "couldn't submit control: %d\n", err); | 689 | dev_warn(netdev->dev.parent, "couldn't submit control: %d\n", err); |
| 699 | 690 | ||
| 700 | return err; | 691 | return err; |
