diff options
| author | Ralph Loader <suckfish@ihug.co.nz> | 2011-01-21 19:27:53 +1300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-01-21 12:35:50 -0800 |
| commit | 3ec6080e2e5be05e6f7fbbe5ab20b0eab84e166b (patch) | |
| tree | 02cfd126f50296adb378fae12617bcc1fa410494 /drivers/staging/westbridge | |
| parent | 01ebd764dab26b3064b0c73ddcf726dd4b41485a (diff) | |
staging: Fix some incorrect use of positive error codes.
Use -E... instead of just E... in a few places where negative error
codes are expected by a functions callers. These were found by grepping
with coccinelle & then inspecting by hand to determine which were bugs.
The staging/cxt1e1 driver appears to intentionally use positive E...
error codes in some places, and negative -E... error codes in others,
making it hard to know which is intended where - very likely I missed
some problems in that driver.
Signed-off-by: Ralph Loader <suckfish@ihug.co.nz>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/westbridge')
| -rw-r--r-- | drivers/staging/westbridge/astoria/device/cyasdevice.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/westbridge/astoria/device/cyasdevice.c b/drivers/staging/westbridge/astoria/device/cyasdevice.c index c76e38375010..088973076517 100644 --- a/drivers/staging/westbridge/astoria/device/cyasdevice.c +++ b/drivers/staging/westbridge/astoria/device/cyasdevice.c @@ -389,7 +389,7 @@ EXPORT_SYMBOL(cyasdevice_gethaltag); static int __init cyasdevice_init(void) { if (cyasdevice_initialize() != 0) - return ENODEV; + return -ENODEV; return 0; } |