diff options
author | Jean Delvare <khali@linux-fr.org> | 2005-07-27 16:14:49 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-09-05 12:14:14 -0400 |
commit | 7bef559455fc71f66f8573cc1aafe1dd33966c1c (patch) | |
tree | 6c342574710f730d39bf5383f81eaa9f9bb10a25 /drivers/hwmon/lm78.c | |
parent | 1ff4e3065b41847b6fe5908b1214cb508dd49e61 (diff) |
[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/hwmon/lm78.c')
-rw-r--r-- | drivers/hwmon/lm78.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/hwmon/lm78.c b/drivers/hwmon/lm78.c index 784935f77016..21b767a66bbd 100644 --- a/drivers/hwmon/lm78.c +++ b/drivers/hwmon/lm78.c | |||
@@ -683,11 +683,8 @@ static int lm78_detach_client(struct i2c_client *client) | |||
683 | 683 | ||
684 | hwmon_device_unregister(data->class_dev); | 684 | hwmon_device_unregister(data->class_dev); |
685 | 685 | ||
686 | if ((err = i2c_detach_client(client))) { | 686 | if ((err = i2c_detach_client(client))) |
687 | dev_err(&client->dev, | ||
688 | "Client deregistration failed, client not detached.\n"); | ||
689 | return err; | 687 | return err; |
690 | } | ||
691 | 688 | ||
692 | if(i2c_is_isa_client(client)) | 689 | if(i2c_is_isa_client(client)) |
693 | release_region(client->addr, LM78_EXTENT); | 690 | release_region(client->addr, LM78_EXTENT); |