diff options
| author | Miod Vallat <miod@cvs.openbsd.org> | 2009-08-02 10:38:35 +0000 |
|---|---|---|
| committer | Miod Vallat <miod@cvs.openbsd.org> | 2009-08-02 10:38:35 +0000 |
| commit | ce067087ed3da82f031e1e2c4a115f2fa3e1a016 (patch) | |
| tree | ee6d514a3c1e53f8ec32065e94788df80a3d266a /sys/dev/usb/uchcom.c | |
| parent | fbc38f0c7c7513e4cc3723a070fbbcc0a25af328 (diff) | |
Never return nonzero in a device activate method invoked with DVACT_ACTIVATE,
for this prevents it to be invoked with DVACT_DEACTIVATE later. This had
been sweeped some time ago already, but bad constructs crept in again.
Diffstat (limited to 'sys/dev/usb/uchcom.c')
| -rw-r--r-- | sys/dev/usb/uchcom.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/dev/usb/uchcom.c b/sys/dev/usb/uchcom.c index 9159158fa8f..9ff5eb8320f 100644 --- a/sys/dev/usb/uchcom.c +++ b/sys/dev/usb/uchcom.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uchcom.c,v 1.7 2008/06/26 05:42:18 ray Exp $ */ +/* $OpenBSD: uchcom.c,v 1.8 2009/08/02 10:38:34 miod Exp $ */ /* $NetBSD: uchcom.c,v 1.1 2007/09/03 17:57:37 tshiozak Exp $ */ /* @@ -347,7 +347,6 @@ uchcom_activate(struct device *self, enum devact act) switch (act) { case DVACT_ACTIVATE: - rv = EOPNOTSUPP; break; case DVACT_DEACTIVATE: uchcom_close_intr_pipe(sc); |