diff options
| author | Paul Irofti <pirofti@cvs.openbsd.org> | 2013-11-15 10:17:40 +0000 |
|---|---|---|
| committer | Paul Irofti <pirofti@cvs.openbsd.org> | 2013-11-15 10:17:40 +0000 |
| commit | 28f7f0e0c022cf25c6540bfb311267b7ba09cbfa (patch) | |
| tree | 2473d6a60fdc1896c5a5237afaa36089484f7f9f /sys/dev/usb/if_urndis.c | |
| parent | 3ab67e413a613e17c0a2eb67b3dbf371317b313a (diff) | |
Clean-up usbd_abort_pipe() usage.
This function never fails. So change it's return type to void and
adjust the copy-pasted callers that were checking the return.
"If it compiles, ok" mpi@
Diffstat (limited to 'sys/dev/usb/if_urndis.c')
| -rw-r--r-- | sys/dev/usb/if_urndis.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/sys/dev/usb/if_urndis.c b/sys/dev/usb/if_urndis.c index fb3a1ed3142..6c8fc23e6e6 100644 --- a/sys/dev/usb/if_urndis.c +++ b/sys/dev/usb/if_urndis.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_urndis.c,v 1.42 2013/11/08 10:12:20 pirofti Exp $ */ +/* $OpenBSD: if_urndis.c,v 1.43 2013/11/15 10:17:39 pirofti Exp $ */ /* * Copyright (c) 2010 Jonathan Armani <armani@openbsd.org> @@ -1129,10 +1129,7 @@ urndis_stop(struct urndis_softc *sc) ifp->if_flags &= ~(IFF_RUNNING | IFF_OACTIVE); if (sc->sc_bulkin_pipe != NULL) { - err = usbd_abort_pipe(sc->sc_bulkin_pipe); - if (err) - printf("%s: abort rx pipe failed: %s\n", - DEVNAME(sc), usbd_errstr(err)); + usbd_abort_pipe(sc->sc_bulkin_pipe); err = usbd_close_pipe(sc->sc_bulkin_pipe); if (err) printf("%s: close rx pipe failed: %s\n", @@ -1141,10 +1138,7 @@ urndis_stop(struct urndis_softc *sc) } if (sc->sc_bulkout_pipe != NULL) { - err = usbd_abort_pipe(sc->sc_bulkout_pipe); - if (err) - printf("%s: abort tx pipe failed: %s\n", - DEVNAME(sc), usbd_errstr(err)); + usbd_abort_pipe(sc->sc_bulkout_pipe); err = usbd_close_pipe(sc->sc_bulkout_pipe); if (err) printf("%s: close tx pipe failed: %s\n", |