Commit c1b6d380 authored by Grant Likely's avatar Grant Likely

of/device: Protect against binding of_platform_drivers to non-OF devices

There is an unlikely chance of this situation is occurring, but it is
easy to protect against.  If a matching entry cannot be found in the
of_match_table, then don't bind the driver.
Signed-off-by: default avatarGrant Likely <grant.likely@secretlab.ca>
parent 29596042
...@@ -30,8 +30,13 @@ static int platform_driver_probe_shim(struct platform_device *pdev) ...@@ -30,8 +30,13 @@ static int platform_driver_probe_shim(struct platform_device *pdev)
pdrv = container_of(pdev->dev.driver, struct platform_driver, driver); pdrv = container_of(pdev->dev.driver, struct platform_driver, driver);
ofpdrv = container_of(pdrv, struct of_platform_driver, platform_driver); ofpdrv = container_of(pdrv, struct of_platform_driver, platform_driver);
/* There is an unlikely chance that an of_platform driver might match
* on a non-OF platform device. If so, then of_match_device() will
* come up empty. Return -EINVAL in this case so other drivers get
* the chance to bind. */
match = of_match_device(pdev->dev.driver->of_match_table, &pdev->dev); match = of_match_device(pdev->dev.driver->of_match_table, &pdev->dev);
return ofpdrv->probe(pdev, match); return match ? ofpdrv->probe(pdev, match) : -EINVAL;
} }
static void platform_driver_shutdown_shim(struct platform_device *pdev) static void platform_driver_shutdown_shim(struct platform_device *pdev)
......
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