From f0fc74e811b853755eed57847afe65517557f1ae Mon Sep 17 00:00:00 2001 From: Tony Lindgren Date: Mon, 7 May 2007 18:17:26 -0700 Subject: [PATCH] I2C: Don't check return value for a void function Don't check return value for a void function Signed-off-by: Tony Lindgren --- drivers/i2c/chips/menelaus.c | 3 +-- drivers/i2c/chips/tlv320aic23.c | 3 +-- drivers/i2c/chips/twl4030_core.c | 4 +--- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/i2c/chips/menelaus.c b/drivers/i2c/chips/menelaus.c index c12ff208321..50049155765 100644 --- a/drivers/i2c/chips/menelaus.c +++ b/drivers/i2c/chips/menelaus.c @@ -940,8 +940,7 @@ static int __init menelaus_init(void) static void __exit menelaus_exit(void) { - if (i2c_del_driver(&menelaus_i2c_driver) < 0) - pr_err("driver remove failed\n"); + i2c_del_driver(&menelaus_i2c_driver); /* FIXME: Shutdown menelaus parts that can be shut down */ } diff --git a/drivers/i2c/chips/tlv320aic23.c b/drivers/i2c/chips/tlv320aic23.c index 65a8e6246c0..aaf9d1c46f1 100644 --- a/drivers/i2c/chips/tlv320aic23.c +++ b/drivers/i2c/chips/tlv320aic23.c @@ -655,8 +655,7 @@ static void __exit aic23_exit(void) int res; aic23_power_down(); - if ((res = i2c_del_driver(&aic23_driver))) - printk("aic23 i2c: Driver remove failed, module not removed.\n"); + i2c_del_driver(&aic23_driver); platform_device_unregister(&audio_i2c_device); platform_driver_unregister(&audio_i2c_driver); diff --git a/drivers/i2c/chips/twl4030_core.c b/drivers/i2c/chips/twl4030_core.c index 31340642ce1..d64ae65f010 100644 --- a/drivers/i2c/chips/twl4030_core.c +++ b/drivers/i2c/chips/twl4030_core.c @@ -891,9 +891,7 @@ static int __init twl4030_init(void) static void __exit twl4030_exit(void) { - if (i2c_del_driver(&twl4030_driver)) - printk(KERN_ERR - "TWL4030: Driver remove failed, module not removed\n"); + i2c_del_driver(&twl4030_driver); twl_irq_used = FREE; } -- 2.41.1