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 | |
| 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>
| -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 5b8a4de4328..660a09bf456 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 | } |
