diff options
author | Wolfram Sang <wsa@the-dreams.de> | 2013-10-13 12:06:12 -0400 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2013-10-24 06:08:23 -0400 |
commit | 501d609abe43894430eecee7f5e498607718b26b (patch) | |
tree | 890a473df56aaaedc6a9026c12ced3012d13b6b4 /drivers/mfd/twl6040.c | |
parent | 34c5c735a89bc0ad5fab1c29b9f425b77fbb7ae7 (diff) |
mfd: twl6040: Remove obsolete cleanup for i2c clientdata
A few new i2c-drivers came into the kernel which clear the clientdata-pointer
on exit or error. This is obsolete meanwhile, the core will do it.
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'drivers/mfd/twl6040.c')
-rw-r--r-- | drivers/mfd/twl6040.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/mfd/twl6040.c b/drivers/mfd/twl6040.c index daf66942071c..9d948af420db 100644 --- a/drivers/mfd/twl6040.c +++ b/drivers/mfd/twl6040.c | |||
@@ -565,13 +565,13 @@ static int twl6040_probe(struct i2c_client *client, | |||
565 | twl6040->supplies); | 565 | twl6040->supplies); |
566 | if (ret != 0) { | 566 | if (ret != 0) { |
567 | dev_err(&client->dev, "Failed to get supplies: %d\n", ret); | 567 | dev_err(&client->dev, "Failed to get supplies: %d\n", ret); |
568 | goto regulator_get_err; | 568 | return ret; |
569 | } | 569 | } |
570 | 570 | ||
571 | ret = regulator_bulk_enable(TWL6040_NUM_SUPPLIES, twl6040->supplies); | 571 | ret = regulator_bulk_enable(TWL6040_NUM_SUPPLIES, twl6040->supplies); |
572 | if (ret != 0) { | 572 | if (ret != 0) { |
573 | dev_err(&client->dev, "Failed to enable supplies: %d\n", ret); | 573 | dev_err(&client->dev, "Failed to enable supplies: %d\n", ret); |
574 | goto regulator_get_err; | 574 | return ret; |
575 | } | 575 | } |
576 | 576 | ||
577 | twl6040->dev = &client->dev; | 577 | twl6040->dev = &client->dev; |
@@ -671,9 +671,6 @@ readyirq_err: | |||
671 | regmap_del_irq_chip(twl6040->irq, twl6040->irq_data); | 671 | regmap_del_irq_chip(twl6040->irq, twl6040->irq_data); |
672 | gpio_err: | 672 | gpio_err: |
673 | regulator_bulk_disable(TWL6040_NUM_SUPPLIES, twl6040->supplies); | 673 | regulator_bulk_disable(TWL6040_NUM_SUPPLIES, twl6040->supplies); |
674 | regulator_get_err: | ||
675 | i2c_set_clientdata(client, NULL); | ||
676 | |||
677 | return ret; | 674 | return ret; |
678 | } | 675 | } |
679 | 676 | ||
@@ -689,7 +686,6 @@ static int twl6040_remove(struct i2c_client *client) | |||
689 | regmap_del_irq_chip(twl6040->irq, twl6040->irq_data); | 686 | regmap_del_irq_chip(twl6040->irq, twl6040->irq_data); |
690 | 687 | ||
691 | mfd_remove_devices(&client->dev); | 688 | mfd_remove_devices(&client->dev); |
692 | i2c_set_clientdata(client, NULL); | ||
693 | 689 | ||
694 | regulator_bulk_disable(TWL6040_NUM_SUPPLIES, twl6040->supplies); | 690 | regulator_bulk_disable(TWL6040_NUM_SUPPLIES, twl6040->supplies); |
695 | 691 | ||