Commit 99fef587 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Greg Kroah-Hartman

driver core: platform: Respect return code of platform_device_register_full()

The platform_device_register_full() might return an error pointer. If we
instantiate platform device which is optional we may simplify the routine at
removal stage by simply calling platform_device_unregister(). For now it
requires to check parameter for being an error pointer in each caller.

To make users' life easier, check for an error pointer inside driver core.
Reported-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 186bddb2
...@@ -234,7 +234,7 @@ struct platform_object { ...@@ -234,7 +234,7 @@ struct platform_object {
*/ */
void platform_device_put(struct platform_device *pdev) void platform_device_put(struct platform_device *pdev)
{ {
if (pdev) if (!IS_ERR_OR_NULL(pdev))
put_device(&pdev->dev); put_device(&pdev->dev);
} }
EXPORT_SYMBOL_GPL(platform_device_put); EXPORT_SYMBOL_GPL(platform_device_put);
...@@ -447,7 +447,7 @@ void platform_device_del(struct platform_device *pdev) ...@@ -447,7 +447,7 @@ void platform_device_del(struct platform_device *pdev)
{ {
int i; int i;
if (pdev) { if (!IS_ERR_OR_NULL(pdev)) {
device_remove_properties(&pdev->dev); device_remove_properties(&pdev->dev);
device_del(&pdev->dev); device_del(&pdev->dev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment