diff options
| author | Jean Delvare <khali@linux-fr.org> | 2005-07-28 00:14:49 +0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-09-05 20:14:14 +0400 |
| commit | 7bef559455fc71f66f8573cc1aafe1dd33966c1c (patch) | |
| tree | 6c342574710f730d39bf5383f81eaa9f9bb10a25 /drivers/i2c/chips/pcf8591.c | |
| parent | 1ff4e3065b41847b6fe5908b1214cb508dd49e61 (diff) | |
| download | linux-7bef559455fc71f66f8573cc1aafe1dd33966c1c.tar.xz | |
[PATCH] I2C: refactor message in i2c_detach_client
We could refactor the error message 34 different i2c drivers print if
i2c_detach_client() fails in this function itself. Saves quite a few
lines of code. Documentation is updated to reflect that change.
Note that this patch should be applied after Rudolf Marek's w83792d
patches.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/i2c/chips/pcf8591.c')
| -rw-r--r-- | drivers/i2c/chips/pcf8591.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/i2c/chips/pcf8591.c b/drivers/i2c/chips/pcf8591.c index 225b512dd4ad..7fce0fc048d9 100644 --- a/drivers/i2c/chips/pcf8591.c +++ b/drivers/i2c/chips/pcf8591.c @@ -240,11 +240,8 @@ static int pcf8591_detach_client(struct i2c_client *client) { int err; - if ((err = i2c_detach_client(client))) { - dev_err(&client->dev, - "Client deregistration failed, client not detached.\n"); + if ((err = i2c_detach_client(client))) return err; - } kfree(i2c_get_clientdata(client)); return 0; |