diff options
| author | Miod Vallat <miod@cvs.openbsd.org> | 2006-06-23 06:27:13 +0000 |
|---|---|---|
| committer | Miod Vallat <miod@cvs.openbsd.org> | 2006-06-23 06:27:13 +0000 |
| commit | 8fe25da9d7762dba747313181158c50980ec44e0 (patch) | |
| tree | 1549a3c414202ab70b3d2420c536b509f0fa2c81 /sys/dev/gpio/gpioow.c | |
| parent | 3d772db769af639e71686fe04b7b9bafc9aa27de (diff) | |
In a drivers activate() entry point, if on DVACT_DEACTIVATE it does
something, then it MUST return zero on DVACT_ACTIVATE, not EOPNOTSUPP;
this very popular bug has been cut and pasted a lot of times...
ok deraadt@ mickey@
Diffstat (limited to 'sys/dev/gpio/gpioow.c')
| -rw-r--r-- | sys/dev/gpio/gpioow.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/dev/gpio/gpioow.c b/sys/dev/gpio/gpioow.c index 94364133eef..23adb2fc6f8 100644 --- a/sys/dev/gpio/gpioow.c +++ b/sys/dev/gpio/gpioow.c @@ -1,4 +1,4 @@ -/* $OpenBSD: gpioow.c,v 1.1 2006/03/04 16:27:03 grange Exp $ */ +/* $OpenBSD: gpioow.c,v 1.2 2006/06/23 06:27:11 miod Exp $ */ /* * Copyright (c) 2006 Alexander Yurchenko <grange@openbsd.org> @@ -171,7 +171,7 @@ gpioow_activate(struct device *self, enum devact act) switch (act) { case DVACT_ACTIVATE: - return (EOPNOTSUPP); + break; case DVACT_DEACTIVATE: sc->sc_dying = 1; if (sc->sc_ow_dev != NULL) |