diff options
author | Tushar Behera <tushar.behera@linaro.org> | 2012-11-23 07:18:43 -0500 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2012-11-30 05:51:30 -0500 |
commit | 4e1328be4d4e4967d54bf7ae3b0805ec53fc4f91 (patch) | |
tree | 195c32c1474fa064383d3c221c7082d0805f07b7 /drivers/mfd | |
parent | 13ea5813238239930d12d28bd7504a3ac6ba14d1 (diff) |
mfd: ab8500-core: Fix invalid free of devm_ allocated data
The objects allocated by devm_* APIs are managed by devres and are
freed when the device is detached. Hence there is no need to remove
them explicitly in remove function.
Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'drivers/mfd')
-rw-r--r-- | drivers/mfd/ab8500-core.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/mfd/ab8500-core.c b/drivers/mfd/ab8500-core.c index 5b8a4de4328b..660a09bf4564 100644 --- a/drivers/mfd/ab8500-core.c +++ b/drivers/mfd/ab8500-core.c | |||
@@ -1437,11 +1437,6 @@ static int __devexit ab8500_remove(struct platform_device *pdev) | |||
1437 | sysfs_remove_group(&ab8500->dev->kobj, &ab8500_attr_group); | 1437 | sysfs_remove_group(&ab8500->dev->kobj, &ab8500_attr_group); |
1438 | 1438 | ||
1439 | mfd_remove_devices(ab8500->dev); | 1439 | mfd_remove_devices(ab8500->dev); |
1440 | free_irq(ab8500->irq, ab8500); | ||
1441 | |||
1442 | kfree(ab8500->oldmask); | ||
1443 | kfree(ab8500->mask); | ||
1444 | kfree(ab8500); | ||
1445 | 1440 | ||
1446 | return 0; | 1441 | return 0; |
1447 | } | 1442 | } |