diff options
| author | Jean Delvare <khali@linux-fr.org> | 2007-05-01 23:26:32 +0200 |
|---|---|---|
| committer | Jean Delvare <khali@hyperion.delvare> | 2007-05-01 23:26:32 +0200 |
| commit | b3e820968ad47219f7d559117a30e85cf96b4e4e (patch) | |
| tree | 84e5ae3275cea6fa5074b0c9b08eaada55f02bea /Documentation | |
| parent | a97f1ed090fc01a5876a7caf2cbdf93470436201 (diff) | |
| download | lwn-b3e820968ad47219f7d559117a30e85cf96b4e4e.tar.gz lwn-b3e820968ad47219f7d559117a30e85cf96b4e4e.zip | |
i2c: Make i2c_del_driver a void function
Make i2c_del_driver a void function, like all other driver removal
functions. It always returned 0 even when errors occured, and nobody
ever actually checked the return value anyway. And we cannot fail
a module removal anyway.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Diffstat (limited to 'Documentation')
| -rw-r--r-- | Documentation/i2c/writing-clients | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/Documentation/i2c/writing-clients b/Documentation/i2c/writing-clients index e62fbfa1282d..f7e04ec849b1 100644 --- a/Documentation/i2c/writing-clients +++ b/Documentation/i2c/writing-clients @@ -586,10 +586,7 @@ the driver module is usually enough. void foo_cleanup(void) { if (foo_initialized == 1) { - if ((res = i2c_del_driver(&foo_driver))) { - printk("foo: Driver registration failed, module not removed.\n"); - return; - } + i2c_del_driver(&foo_driver); foo_initialized --; } } |