diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2014-11-03 12:29:23 -0500 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2014-11-25 11:18:52 -0500 |
commit | bde3e706a63d5c258b3e4f5e327bcf032fb1adfe (patch) | |
tree | 6d6f7c4eba4065f08de84b7f3463d80930ab716b /drivers/mfd | |
parent | 1753b40f5c97e0d0bf2f0a562603cfc592945a5e (diff) |
mfd: lpc_sch: Don't call mfd_remove_devices()
MFD core already cares about failing registration. It will remove successfully
registered devices in case of error. Thus, no need to repeatedly call
mfd_remove_devices().
Fixes: 5829e9b64e65 (mfd: lpc_sch: Accomodate partial population of the MFD devices)
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd')
-rw-r--r-- | drivers/mfd/lpc_sch.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/mfd/lpc_sch.c b/drivers/mfd/lpc_sch.c index c980da479a35..5c38df35a84d 100644 --- a/drivers/mfd/lpc_sch.c +++ b/drivers/mfd/lpc_sch.c | |||
@@ -193,11 +193,7 @@ static int lpc_sch_probe(struct pci_dev *dev, const struct pci_device_id *id) | |||
193 | return -ENODEV; | 193 | return -ENODEV; |
194 | } | 194 | } |
195 | 195 | ||
196 | ret = mfd_add_devices(&dev->dev, 0, lpc_sch_cells, cells, NULL, 0, NULL); | 196 | return mfd_add_devices(&dev->dev, 0, lpc_sch_cells, cells, NULL, 0, NULL); |
197 | if (ret) | ||
198 | mfd_remove_devices(&dev->dev); | ||
199 | |||
200 | return ret; | ||
201 | } | 197 | } |
202 | 198 | ||
203 | static void lpc_sch_remove(struct pci_dev *dev) | 199 | static void lpc_sch_remove(struct pci_dev *dev) |