diff options
author | Samuel Iglesias Gonsalvez <siglesias@igalia.com> | 2012-05-25 08:03:02 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-06-07 05:20:29 (GMT) |
commit | b2bc13c5e07501a0a4427b6a9eff8162aef5560a (patch) | |
tree | c96aac1acd499d14241781138c9acf1ba5a2e511 /drivers/staging/ipack | |
parent | 3b86bb2e11d854d258c5e05d709865ca3d3c8951 (diff) | |
download | linux-fsl-qoriq-b2bc13c5e07501a0a4427b6a9eff8162aef5560a.tar.xz |
Staging: ipack: delete the call to remove() in ipack_driver_register
When a bus driver calls ipack_driver_register(), it should manages the returning
NULL value to undo all the operations it did before this call, and print the
corresponding trace.
It is not a task for the ipack driver to call the remove() function here.
Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/ipack')
-rw-r--r-- | drivers/staging/ipack/ipack.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/staging/ipack/ipack.c b/drivers/staging/ipack/ipack.c index e97be99..a1448e6 100644 --- a/drivers/staging/ipack/ipack.c +++ b/drivers/staging/ipack/ipack.c @@ -141,8 +141,6 @@ struct ipack_device *ipack_device_register(struct ipack_bus_device *bus, ret = device_register(&dev->dev); if (ret < 0) { - pr_err("error registering the device.\n"); - dev->driver->ops->remove(dev); kfree(dev); return NULL; } |